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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 20 Jul 2009 19:11:26 -0700 From: Greg KH <gregkh@...e.de> To: Arnd Bergmann <arnd@...db.de> Cc: virtualization@...ts.linux-foundation.org, linux-kernel@...r.kernel.org, devel@...verdev.osuosl.org, virtualization@...ts.osdl.org, Sam Ramji <sramji@...rosoft.com>, Haiyang Zhang <haiyangz@...rosoft.com>, Hank Janssen <hjanssen@...rosoft.com>, shemminger@...ux-foundation.org Subject: Re: [patch 47/54] Staging: hv: make gVmbusConnection.ChannelLock a real spinlock On Tue, Jul 21, 2009 at 01:39:19AM +0200, Arnd Bergmann wrote: > On Friday 17 July 2009, Greg Kroah-Hartman wrote: > > I don't think we really want to be doing a irqsave for this code, but I > > left it alone to preserve the original codepath. It should be reviewed > > later. > > This patch does not preserve the original code path if any of the > callers has a different irq state from the others. When flags is > shared, releasing the lock actually sets the irq state to whatever > another thread was using when failing to acquire the lock while > it was held. > > The patch of course looks good, but the comment is misleading. Good point, sorry for the misleading comment. I'll let Hank work out the details as to if the irqsave mode is needed or not :) thanks, greg k-h -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists