[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <50291617-0872-4ba9-8ca5-329597a0eff5@intel.com>
Date: Fri, 5 Jul 2024 14:32:23 +0200
From: Alexander Lobakin <aleksander.lobakin@...el.com>
To: Johannes Berg <johannes@...solutions.net>
CC: <netdev@...r.kernel.org>, Jesper Dangaard Brouer <hawk@...nel.org>, "Ilias
Apalodimas" <ilias.apalodimas@...aro.org>, "David S. Miller"
<davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski
<kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>, Johannes Berg
<johannes.berg@...el.com>
Subject: Re: [PATCH net-next] net: page_pool: fix warning code
From: Johannes Berg <johannes@...solutions.net>
Date: Fri, 5 Jul 2024 13:42:06 +0200
> From: Johannes Berg <johannes.berg@...el.com>
>
> WARN_ON_ONCE("string") doesn't really do what appears to
> be intended, so fix that.
>
> Signed-off-by: Johannes Berg <johannes.berg@...el.com>
"Fixes:" tag?
> ---
> net/core/page_pool.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/core/page_pool.c b/net/core/page_pool.c
> index 3927a0a7fa9a..97914a9e2a85 100644
> --- a/net/core/page_pool.c
> +++ b/net/core/page_pool.c
> @@ -445,7 +445,7 @@ static bool page_pool_dma_map(struct page_pool *pool, struct page *page)
> return true;
>
> unmap_failed:
> - WARN_ON_ONCE("unexpected DMA address, please report to netdev@");
> + WARN_ONCE(1, "unexpected DMA address, please report to netdev@");
> dma_unmap_page_attrs(pool->p.dev, dma,
> PAGE_SIZE << pool->p.order, pool->p.dma_dir,
> DMA_ATTR_SKIP_CPU_SYNC | DMA_ATTR_WEAK_ORDERING);
Thanks,
Olek
Powered by blists - more mailing lists