Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1195480

Re: [PATCH 1/2] net: Export __netdev_alloc_frag() to allow gfp_mask flags

From Murali Karicheri <m-karicheri2@ti.com>
Newsgroups linux.kernel
Subject Re: [PATCH 1/2] net: Export __netdev_alloc_frag() to allow gfp_mask flags
Date 2015-07-29 22:30 +0200
Message-ID <pRG9b-1VI-11@gated-at.bofh.it> (permalink)
References <pRBjb-3qg-1@gated-at.bofh.it> <pRCyC-587-23@gated-at.bofh.it>
Organization Texas Instruments

Show all headers | View raw


Eric,

On 07/29/2015 12:31 PM, Eric Dumazet wrote:
> On Wed, 2015-07-29 at 11:10 -0400, WingMan Kwok wrote:
>> This patch makes the function __netdev_alloc_frag() non-static and
>> exports it so that drivers that need to specify additional flags,
>> such as __GFP_DMA, can use it. The currently exported function,
>> netdev_alloc_frag() doesn't allow passing in gfp_mask flags.
>>
>> Signed-off-by: WingMan Kwok <w-kwok2@ti.com>
>> Signed-off-by: Reece R. Pollack <x0183204@ti.com>
>> ---
>>   include/linux/skbuff.h |    1 +
>>   net/core/skbuff.c      |    3 ++-
>>   2 files changed, 3 insertions(+), 1 deletion(-)
>
> You can not do this.
>
> __napi_alloc_frag() uses __alloc_page_frag() using a per cpu reserve.
>
Thanks for your response.

I assume you mean to say __netdev_alloc_frag() which is what the patch 
affects. Right?

> This per cpu reserve would be shared by regular GFP_ATOMIC and your
> __GFP_DMA allocations.

I am trying to understand the issue here. Is there any issue in sharing 
this per CPU reserve between DMA and ATOMIC allocations. Without this 
flag, the assumption is this function can return memory which is not 
DMA-able and this flag assures it is allocated from DMA zone.

Murali
>
>
>
>
>


-- 
Murali Karicheri
Linux Kernel, Keystone
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

Re: [PATCH 1/2] net: Export __netdev_alloc_frag() to allow gfp_mask  flags Murali Karicheri <m-karicheri2@ti.com> - 2015-07-29 22:30 +0200
  Re: [PATCH 1/2] net: Export __netdev_alloc_frag() to allow gfp_mask  flags Eric Dumazet <eric.dumazet@gmail.com> - 2015-07-29 22:40 +0200

csiph-web