lkml.org 
[lkml]   [2014]   [Dec]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 14/37] perf tools: Convert dead thread list into rbtree
On 12/25/14 7:26 PM, Namhyung Kim wrote:
>>> diff --git a/tools/perf/util/thread.h b/tools/perf/util/thread.h
>>> index 0b6dcd70bc8b..413f28cf689b 100644
>>> --- a/tools/perf/util/thread.h
>>> +++ b/tools/perf/util/thread.h
>>> @@ -11,10 +11,8 @@
>>> struct thread_stack;
>>>
>>> struct thread {
>>> - union {
>>> - struct rb_node rb_node;
>>> - struct list_head node;
>>> - };
>>> + struct rb_node rb_node;
>>> + struct list_head node;
>>> struct map_groups *mg;
>>> pid_t pid_; /* Not all tools update this */
>>> pid_t tid;
>>> @@ -22,7 +20,8 @@ struct thread {
>>> int cpu;
>>> char shortname[3];
>>> bool comm_set;
>>> - bool dead; /* if set thread has exited */
>>> + bool exited; /* if set thread has exited */
>>> + bool dead; /* thread is in dead_threads list */
>>
>> looks like this also changes the logic (new exited flag),
>> not just the dead threads storage wheel
>
> AFAICS the 'dead' flag is not used other than thread__exited(). And
> it confused me a dead thread might not be in a dead_threads tree (or
> list). So I changed the name and no logical change intended.

git show 236a3bbd5cb51

David



\
 
 \ /
  Last update: 2014-12-26 18:41    [W:0.128 / U:0.800 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site