[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <298f6a92-7bf0-4cf4-afbf-554ef3e8e7e7@suse.cz>
Date: Fri, 12 Sep 2025 09:03:55 +0200
From: Vlastimil Babka <vbabka@...e.cz>
To: Sergey Senozhatsky <senozhatsky@...omium.org>
Cc: Suren Baghdasaryan <surenb@...gle.com>,
"Liam R. Howlett" <Liam.Howlett@...cle.com>,
Christoph Lameter <cl@...two.org>, David Rientjes <rientjes@...gle.com>,
Roman Gushchin <roman.gushchin@...ux.dev>, Harry Yoo <harry.yoo@...cle.com>,
Uladzislau Rezki <urezki@...il.com>,
Sidhartha Kumar <sidhartha.kumar@...cle.com>, linux-mm@...ck.org,
linux-kernel@...r.kernel.org, rcu@...r.kernel.org,
maple-tree@...ts.infradead.org
Subject: Re: [PATCH v8 04/23] slab: add sheaf support for batching kfree_rcu()
operations
On 9/12/25 02:38, Sergey Senozhatsky wrote:
> Hi Vlastimil,
>
> On (25/09/10 10:01), Vlastimil Babka wrote:
> [..]
>> +
>> + if (rcu_free)
>> + call_rcu(&rcu_free->rcu_head, rcu_free_sheaf_nobarn);
>> +}
>> +
>> +
>> +/* needed for kvfree_rcu_barrier() */
>> +void flush_all_rcu_sheaves()
>> +{
>
> mm/slub.c:3960:27: error: a function declaration without a prototype is deprecated in all versions of C [-Werror,-Wstrict-prototypes]
> 3960 | void flush_all_rcu_sheaves()
> | ^
> | void
>
> ---
Thanks, the bots told me too and it's fixed in -next
> diff --git a/mm/slub.c b/mm/slub.c
> index 11ad4173e2f2..a1eae71a0f8c 100644
> --- a/mm/slub.c
> +++ b/mm/slub.c
> @@ -3955,9 +3955,8 @@ static void flush_rcu_sheaf(struct work_struct *w)
> call_rcu(&rcu_free->rcu_head, rcu_free_sheaf_nobarn);
> }
>
> -
> /* needed for kvfree_rcu_barrier() */
> -void flush_all_rcu_sheaves()
> +void flush_all_rcu_sheaves(void)
> {
> struct slub_percpu_sheaves *pcs;
> struct slub_flush_work *sfw;
Powered by blists - more mailing lists