I reported this one already.

regards,
dan carpenter

On Fri, Nov 14, 2014 at 06:13:28PM +0800, kbuild test robot wrote:
> TO: Oleg Nesterov <[email protected]>
> CC: Johannes Weiner <[email protected]>
> CC: Andrew Morton <[email protected]>
> CC: Linux Memory Management List <[email protected]>
> 
> tree:   git://git.cmpxchg.org/linux-mmotm.git master
> head:   e668fb4c5c5e6de5b9432bd36d83b3a0b4ce78e8
> commit: be946c5700e61ff5b193770e852a1df460ebc508 [173/306] exit: reparent: 
> cleanup the changing of ->parent
> :::::: branch date: 33 hours ago
> :::::: commit date: 33 hours ago
> 
> kernel/exit.c:574 forget_original_parent() warn: add some parenthesis here?
> 
> git remote add mmotm git://git.cmpxchg.org/linux-mmotm.git
> git remote update mmotm
> git checkout be946c5700e61ff5b193770e852a1df460ebc508
> vim +574 kernel/exit.c
> 
> 762a24bee Oleg Nesterov     2007-10-18  558  static void 
> forget_original_parent(struct task_struct *father)
> ^1da177e4 Linus Torvalds    2005-04-16  559  {
> be946c570 Oleg Nesterov     2014-11-13  560   struct task_struct *p, *t, *n, 
> *reaper;
> 5dfc80be7 Oleg Nesterov     2009-04-02  561   LIST_HEAD(dead_children);
> 762a24bee Oleg Nesterov     2007-10-18  562  
> 762a24bee Oleg Nesterov     2007-10-18  563   write_lock_irq(&tasklist_lock);
> c7e49c148 Oleg Nesterov     2010-08-10  564   /*
> c7e49c148 Oleg Nesterov     2010-08-10  565    * Note that exit_ptrace() and 
> find_new_reaper() might
> c7e49c148 Oleg Nesterov     2010-08-10  566    * drop tasklist_lock and 
> reacquire it.
> c7e49c148 Oleg Nesterov     2010-08-10  567    */
> c7e49c148 Oleg Nesterov     2010-08-10  568   exit_ptrace(father);
> 950bbabb5 Oleg Nesterov     2008-09-02  569   reaper = 
> find_new_reaper(father);
> 762a24bee Oleg Nesterov     2007-10-18  570  
> 03ff17979 Matthias Kaehlcke 2007-10-18  571   list_for_each_entry_safe(p, n, 
> &father->children, sibling) {
> be946c570 Oleg Nesterov     2014-11-13  572           for_each_thread(p, t) {
> 9cd80bbb0 Oleg Nesterov     2009-12-17  573                   t->real_parent 
> = reaper;
> be946c570 Oleg Nesterov     2014-11-13 @574                   
> BUG_ON(!t->ptrace != (t->parent == father));
> be946c570 Oleg Nesterov     2014-11-13  575                   if 
> (likely(!t->ptrace))
> 9cd80bbb0 Oleg Nesterov     2009-12-17  576                           
> t->parent = t->real_parent;
> 9cd80bbb0 Oleg Nesterov     2009-12-17  577                   if 
> (t->pdeath_signal)
> 9cd80bbb0 Oleg Nesterov     2009-12-17  578                           
> group_send_sig_info(t->pdeath_signal,
> 9cd80bbb0 Oleg Nesterov     2009-12-17  579                                   
>             SEND_SIG_NOINFO, t);
> be946c570 Oleg Nesterov     2014-11-13  580           }
> 9cd80bbb0 Oleg Nesterov     2009-12-17  581           reparent_leader(father, 
> p, &dead_children);
> ^1da177e4 Linus Torvalds    2005-04-16  582   }
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> http://lists.01.org/mailman/listinfo/kbuild                 Intel Corporation
_______________________________________________
kbuild mailing list
[email protected]
https://lists.01.org/mailman/listinfo/kbuild

Reply via email to