netdev - Re: [PATCH net-2.6] 3c59x: Remove incorrect locking; correct documented lock hierarchy

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <20100901.180210.129766746.davem@davemloft.net>
Date:	Wed, 01 Sep 2010 18:02:10 -0700 (PDT)
From:	David Miller <davem@...emloft.net>
To:	ben@...adent.org.uk
Cc:	netdev@...r.kernel.org
Subject: Re: [PATCH net-2.6] 3c59x: Remove incorrect locking; correct
 documented lock hierarchy

From: Ben Hutchings <ben@...adent.org.uk>
Date: Thu, 02 Sep 2010 01:42:58 +0100

> On Wed, 2010-09-01 at 17:37 -0700, David Miller wrote:
>> From: Ben Hutchings <ben@...adent.org.uk>
>> Date: Wed, 01 Sep 2010 23:08:27 +0100
>> 
>> > I think this is safe after all because ethtool and MII ioctls are all
>> > serialised by the RTNL.
>> 
>> Well since you said that there was a regression that you're unable
>> to reproduce, I was trying to find a probably cause for it :-)
>> 
>> I think you should work to resolve that report, and resend this
>> locking patch once that is sorted out.
> 
> That regression seems to due to my *earlier* changes to locking.  This
> new patch fixes a different regression I found while trying to reproduce
> that, and it has not been applied in Debian yet.  Sorry for the
> ambiguous wording.

Gotcha, patch applied to net-2.6, thanks.
--
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