Sorry, sorry for the delay, On 05/20, Eric W. Biederman wrote:Hmm. could you clarify? I tend to always trust you, just can't understand the text above... However, I think this patch adds another subtle race which I missed before. Let's suppose that the task has two threads, A (== main thread) and B. A has already exited, B does exec. In that case it is possible that (without tasklist_lock) kill_something_info() sends the signal to the old leader (A), but before group_send_sig_info(A) takes ->siglock B switches the leader and does release_task(A). In that group_send_sig_info()->lock_task_sighand() fails and we miss the process. Note the (broken) "p->pid > 1" check, kill_something_info() skips init. Not that it matters though. Oleg. --
| Linus Torvalds | Linux 2.6.27 |
| Mark Lord | Re: 2.6.25-rc8: FTP transfer errors |
| Bart Van Assche | Re: Integration of SCST in the mainstream Linux kernel |
| Greg Kroah-Hartman | [PATCH 029/196] ecryptfs: clean up attribute mess |
git: | |
| Jakub Narebski | Re: [RFC] Git User's Survey 2008 |
| Wincent Colaiuta | Possible to make a totally empty repository for remote access? |
| Michael J Gruber | git over rsync+ssh |
| Johannes Schindelin | cogito and remote#branch, was Re: [PATCH] Git homepage: remove all the references ... |
| Martin Schröder | Re: Real men don't attack straw men |
| david l goodrich | Re: max-src-conn-rate rule question |
| Marcos Laufer | dmesg IBM x3650 OpenBSD 4.3 |
| Parvinder Bhasin | colors in regular openbsd terminal |
| steffen.linux | Bluetooth Headset compatible to Freerunner? |
| Carsten Haitzler | Re: New Rasterman Image... |
| flamma | Re: [qtextended] After 4 days... |
| Craig B. Allen | Re: one day usage of qtextended |
