Re: [Dovetail] v6.14: fix up to oob page pool

[Philippe Gerum <rpm@xenomai.org>]

Florian Bezdeka <florian.bezdeka@siemens.com> writes:

> On Mon, 2025-03-03 at 15:11 +0100, Philippe Gerum wrote:
>> Hi Florian,
>> 
>> For some reason, the hunk below disappeared from the Dovetail v6.13 code
>> when rebasing on v6.14-rc, this causes a crash at boot when
>> CONFIG_PAGE_POOL_OOB is turned on. This fix up should be merged to:
>> 
>> e4ecbcc40e7e6 net: dovetail: add infrastructure for oob device I/O
>> 
>> diff --git a/net/core/page_pool.c b/net/core/page_pool.c
>> index 7860c3a88c5d2..07ad13dd5ed2e 100644
>> --- a/net/core/page_pool.c
>> +++ b/net/core/page_pool.c
>> @@ -563,7 +563,7 @@ static noinline netmem_ref __page_pool_alloc_pages_slow(struct page_pool *pool,
>>  		return pool->alloc.cache[--pool->alloc.count];
>>  
>>  	/* Mark empty alloc.cache slots "empty" for alloc_pages_bulk */
>> -	memset(&pool->alloc.cache, 0, sizeof(void *) * bulk);
>> +	memset(pool->alloc.cache, 0, sizeof(void *) * bulk);
>>  
>>  	nr_pages = alloc_pages_bulk_node(gfp, pool->p.nid, bulk,
>>  					 (struct page **)pool->alloc.cache);
>> 
>
> I will take care of that. I'm currently rebasing on top of 6.14-rc5.

Ok. Meanwhile, I'm preparing a patch to fix the hard irq context
detection (inverted test of IRQS_DEFERRED). We need to have that one as
a separate patch for backporting.

-- 
Philippe.