lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sat, 21 Apr 2007 01:28:35 +0400 From: Oleg Nesterov <oleg@...sign.ru> To: Andrew Morton <akpm@...ux-foundation.org> Cc: David Howells <dhowells@...hat.com>, David Miller <davem@...emloft.net>, ebiederm@...ssion.com, containers@...ts.osdl.org, hch@...radead.org, linux-kernel@...r.kernel.org, netdev@...r.kernel.org Subject: Re: Getting the new RxRPC patches upstream On 04/20, Andrew Morton wrote: > > On Fri, 20 Apr 2007 11:41:46 +0100 > David Howells <dhowells@...hat.com> wrote: > > > There are only two non-net patches that AF_RXRPC depends on: > > > > (1) The key facility changes. That's all my code anyway, and shouldn't be a > > problem to merge unless someone else has put some changes in there that I > > don't know about. > > > > (2) try_to_cancel_delayed_work(). I suppose I could use > > cancel_delayed_work() instead, but that's less efficient as it waits for > > the timer completion function to finish. > > There are significant workqueue changes in -mm and I plan to send them > in for 2.6.22. I doubt if there's anything in there which directly > affects cancel_delayed_work(), but making changes of this nature against > 2.6.21 might lead to grief. I think it is better to use cancel_delayed_work(), but change it to use del_timer(). I belive cancel_delayed_work() doesn't need del_timer_sync(). We only care when del_timer() returns true. In that case, if the timer function still runs (possible for single-threaded wqs), it has already passed __queue_work(). Oleg. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists