netdev
[Top] [All Lists]

Re: [PATCH] NETLINK_UESTABLISHED notifier event

To: Mike Christie <michaelc@xxxxxxxxxxx>
Subject: Re: [PATCH] NETLINK_UESTABLISHED notifier event
From: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
Date: Thu, 7 Apr 2005 07:29:07 +1000
Cc: Dmitry Yusupov <dmitry_yus@xxxxxxxxx>, netdev@xxxxxxxxxxx, davem@xxxxxxxxxxxxx
In-reply-to: <42540CF3.7070501@xxxxxxxxxxx>
References: <E1DJ0YI-0003PR-00@xxxxxxxxxxxxxxxxxxxxxxxx> <42540CF3.7070501@xxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6+20040907i
On Wed, Apr 06, 2005 at 09:23:15AM -0700, Mike Christie wrote:
> 
> It is due to where it is being used. open-iscsi uses netlink
> sockets for communication in a block (scsi specificically)
> driver that has pushed much of its code to usersapce. Forcing
> open-iscsi to use GFP_KERNEL causes a couple of problems. The
> worst would be where a GFP_KERNEL allocation causes a write,
> and that write is to an iscsi disk that open-iscsi is managing.
> The write could then hit the same code path and cause another
> GFP_KERNEL allocation and we could loop like that until the
> system locks up.

In that case it's not enough to just use sk_allocation here.
You'll need a way to actually set it to GFP_ATOMIC.
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt

<Prev in Thread] Current Thread [Next in Thread>