From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965543AbbEMVf7 (ORCPT ); Wed, 13 May 2015 17:35:59 -0400 Received: from mail-pa0-f46.google.com ([209.85.220.46]:34844 "EHLO mail-pa0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965321AbbEMVfz (ORCPT ); Wed, 13 May 2015 17:35:55 -0400 From: Kevin Hilman To: Alexander Duyck Cc: linux-kernel@vger.kernel.org, balbi@ti.com, davem@davemloft.net, tony@atomide.com, netdev@vger.kernel.org, linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [net-next PATCH] net: Reserve skb headroom and set skb->dev even if using __alloc_skb References: <20150513182738.GA28201@saruman.tx.rr.com> <20150513203413.8252.98057.stgit@ahduyck-vm-fedora22> Date: Wed, 13 May 2015 14:35:51 -0700 In-Reply-To: <20150513203413.8252.98057.stgit@ahduyck-vm-fedora22> (Alexander Duyck's message of "Wed, 13 May 2015 13:34:13 -0700") Message-ID: <7h8ucs6tzc.fsf@deeprootsystems.com> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Alexander Duyck writes: > When I had inlined __alloc_rx_skb into __netdev_alloc_skb and > __napi_alloc_skb I had overlooked the fact that there was a return in the > __alloc_rx_skb. As a result we weren't reserving headroom or setting the > skb->dev in certain cases. This change corrects that by adding a couple of > jump labels to jump to depending on __alloc_skb either succeeding or failing. > > Fixes: 9451980a6646 ("net: Use cached copy of pfmemalloc to avoid accessing page") > Reported-by: Felipe Balbi > Signed-off-by: Alexander Duyck Tested this on top of next-20150513 on an ARM/OMAP (am335x-boneblack.dts) an it fixes the boot problem for me. Tested-by: Kevin Hilman Kevin