lkml.org 
[lkml]   [2009]   [Feb]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/4] gcov: add gcov profiling infrastructure
On Thu, 2009-02-26 at 12:46 +0100, Peter Oberparleiter wrote:
> necessary. My proposed fix looks something like this (unless there
> are objections, I'll include this one in the re-post):

I'd prefer a goto. But this looks nice too. Thanks! :)

> ---
> kernel/gcov/fs.c | 21 ++++++++++++++-------
> 1 file changed, 14 insertions(+), 7 deletions(-)
>
> Index: linux-2.6.29-rc6/kernel/gcov/fs.c
> ===================================================================
> --- linux-2.6.29-rc6.orig/kernel/gcov/fs.c
> +++ linux-2.6.29-rc6/kernel/gcov/fs.c
> @@ -453,15 +453,22 @@ static ssize_t reset_write(struct file *
> size_t len, loff_t *pos)
> {
> struct gcov_node *node;
> - struct gcov_node *r;
> + int restart;
>
> mutex_lock(&node_lock);
> - list_for_each_entry_safe(node, r, &all_head, all) {
> - if (node->info)
> - gcov_info_reset(node->info);
> - else
> - remove_node(node);
> - }
> + do {
> + restart = 0;
> + list_for_each_entry(node, &all_head, all) {
> + if (node->info)
> + gcov_info_reset(node->info);
> + else if (list_empty(&node->children)) {
> + remove_node(node);
> + /* Several nodes may have gone - restart. */
> + restart = 1;
> + break;
> + }
> + }
> + } while (restart);
> mutex_unlock(&node_lock);
>
> return len;
>



\
 
 \ /
  Last update: 2009-02-26 13:11    [from the cache]
©2003-2011 Jasper Spaans