From sven.eckelmann@gmx.de Tue Jun 1 13:33:55 2010 From: Sven Eckelmann Date: Sat, 22 May 2010 17:48:47 +0200 Subject: Staging: batman-adv: Don't allocate icmp packet with GFP_KERNEL To: gregkh@suse.de Cc: b.a.t.m.a.n@lists.open-mesh.net, Sven Eckelmann Message-ID: <1274543327-2258-4-git-send-email-sven.eckelmann@gmx.de> A new buffer for a packet is created when a icmp packet is received. This happens in a context with disabled irq. Thus we are not allowed to sleep or call function which might sleep. kmalloc must be called with GFP_ATOMIC instead of GFP_KERNEL to ensure that it does not sleep. Signed-off-by: Sven Eckelmann Signed-off-by: Greg Kroah-Hartman --- drivers/staging/batman-adv/device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/staging/batman-adv/device.c +++ b/drivers/staging/batman-adv/device.c @@ -309,7 +309,7 @@ void bat_device_add_packet(struct device struct device_packet *device_packet; unsigned long flags; - device_packet = kmalloc(sizeof(struct device_packet), GFP_KERNEL); + device_packet = kmalloc(sizeof(struct device_packet), GFP_ATOMIC); if (!device_packet) return;