netdev - Re: [patch 04/13] ppp_generic: fix lockdep warning

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20070514125118.GA3382@ff.dom.local>
Date:	Mon, 14 May 2007 14:51:18 +0200
From:	Jarek Poplawski <jarkao2@...pl>
To:	David Miller <davem@...emloft.net>
Cc:	akpm@...ux-foundation.org, jeff@...zik.org, netdev@...r.kernel.org,
	jura@...ams.com, paulus@...ba.org
Subject: Re: [patch 04/13] ppp_generic: fix lockdep warning

On Mon, May 14, 2007 at 10:08:29AM +0200, Jarek Poplawski wrote:
> On Mon, May 14, 2007 at 09:28:45AM +0200, Jarek Poplawski wrote:
> > On Sun, May 13, 2007 at 11:39:37PM -0700, David Miller wrote:
> ...
> > > For each unique netdev type, use a different locking class.
> > > 
> > > That will fix this forever, anything else is a situation specific
> > > band-aid (but then again isn't that what every lockdep annotation is
> > > :-).
...
> > Yes, this is very good idea, and I wonder, why you didn't try
> > this yourself (after my "ignore"). I thought a little about
> > this, but was afraid of it's wide range. Some things - like
> > in vlans - should be removed then, for this to work. I'll try
> > to send something like this soon (but I'm not so optimistic
> > it will cure all or forever...).
> 
> So, because of this next planned patch (I hope not later than
> tomorrow), my two last patches for vlan and ppp_generic shouldn't 
> be applied - their functionality will be moved to register_netdevice.

...or maybe not so easy... Here is the first problem: vlan does
register_netdevice(new_dev) with new_dev->type == real_dev->type,
so it still couldn't be recognized as something different from
the real ppp or eth. We need some trick... Probably we could
also think about using dev->name (or both), but there always would
be a problem with the size of a static table reserved for this.

So, now it seems something like my patch with lockdep class for ppp
could be settled with this, but more complex situations like vlan
still need special treatment.

Maybe we should also care not to go too far with this: I didn't read
other such reports recently, but here the problems seem to be made
mainly by some virtual devices without queues...

Jarek P.
-
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