On Wed, 16 Apr 2014 09:00:02 -0400 (EDT) David Miller <davem@xxxxxxxxxxxxx> wrote: > From: Eric Dumazet <eric.dumazet@xxxxxxxxx> > Date: Tue, 15 Apr 2014 22:13:46 -0700 > > > For applications handling millions of sockets, this makes a difference. > > Indeed, this really is not acceptable. As you say... I've just discovered that I can get rid of the lockdep message (and hence presumably the deadlock risk) with a well placed: newsock->sk->sk_allocation = GFP_NOFS; which surprised me as it seemed to be an explicit GFP_KERNEL allocation that was mentioned in the lockdep trace. Obviously these traces require quite some sophistication to understand. So - thanks for the feedback, patch can be ignored. Thanks, NeilBrown
Attachment:
signature.asc
Description: PGP signature