[PATCH 1/2] bpf: default log level should be NOTICE

[PATCH 1/2] bpf: default log level should be NOTICE

[Stephen Hemminger]

BPF is used by other services now and the default log
level should match other libraries.

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/bpf/bpf.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/bpf/bpf.c b/lib/bpf/bpf.c
index 740c080591..695454a2d4 100644
--- a/lib/bpf/bpf.c
+++ b/lib/bpf/bpf.c
@@ -78,4 +78,4 @@ __rte_bpf_jit(struct rte_bpf *bpf)
 	return rc;
 }
 
-RTE_LOG_REGISTER_DEFAULT(rte_bpf_logtype, INFO);
+RTE_LOG_REGISTER_DEFAULT(rte_bpf_logtype, NOTICE);
-- 
2.53.0

[PATCH 2/2] bpf: silence noisy message

[Stephen Hemminger]

The recent loader change causes a log message every time BPF
is used. For example, running dpdk-dumpcap cause:
 BPF: rte_bpf_load_ex(): successfully creates ...

This is a debug message and should not normally be needed.

Fixes: 55192fe168b9 ("bpf: introduce extensible load API")

Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
 lib/bpf/bpf_load.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/bpf/bpf_load.c b/lib/bpf/bpf_load.c
index 2fd0ee0720..36f6556fd5 100644
--- a/lib/bpf/bpf_load.c
+++ b/lib/bpf/bpf_load.c
@@ -286,7 +286,7 @@ rte_bpf_load_ex(const struct rte_bpf_prm_ex *prm)
 		return NULL;
 	}
 
-	RTE_BPF_LOG_FUNC_LINE(INFO, "successfully creates %p(jit={.func=%p,.sz=%zu});",
+	RTE_BPF_LOG_FUNC_LINE(DEBUG, "successfully creates %p(jit={.func=%p,.sz=%zu});",
 		load.bpf, load.bpf->jit.raw, load.bpf->jit.sz);
 	return load.bpf;
 }
-- 
2.53.0

RE: [PATCH 2/2] bpf: silence noisy message

[Marat Khalili]

> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: Wednesday 1 July 2026 23:35
> To: dev@dpdk.org
> Cc: Stephen Hemminger <stephen@networkplumber.org>; Konstantin Ananyev <konstantin.ananyev@huawei.com>;
> Marat Khalili <marat.khalili@huawei.com>
> Subject: [PATCH 2/2] bpf: silence noisy message
> 
> The recent loader change causes a log message every time BPF
> is used. For example, running dpdk-dumpcap cause:
>  BPF: rte_bpf_load_ex(): successfully creates ...
> 
> This is a debug message and should not normally be needed.
> 
> Fixes: 55192fe168b9 ("bpf: introduce extensible load API")

The commit message and Fixes tag are not 100% accurate, this message existed before:
https://github.com/DPDK/dpdk/blob/09e4f37ddd0f7d5cc7c696741f31f1582c73a716/lib/bpf/bpf_load_elf.c#L331-L333

(In fact I tried to eliminate it, but had to but back at the insistence of AI reviewer
and even broke the build in the process...)

So naturally I support downgrading this message. With the commit message corrected,

Acked-by: Marat Khalili <marat.khalili@huawei.com>

> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  lib/bpf/bpf_load.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/bpf/bpf_load.c b/lib/bpf/bpf_load.c
> index 2fd0ee0720..36f6556fd5 100644
> --- a/lib/bpf/bpf_load.c
> +++ b/lib/bpf/bpf_load.c
> @@ -286,7 +286,7 @@ rte_bpf_load_ex(const struct rte_bpf_prm_ex *prm)
>  		return NULL;
>  	}
> 
> -	RTE_BPF_LOG_FUNC_LINE(INFO, "successfully creates %p(jit={.func=%p,.sz=%zu});",
> +	RTE_BPF_LOG_FUNC_LINE(DEBUG, "successfully creates %p(jit={.func=%p,.sz=%zu});",
>  		load.bpf, load.bpf->jit.raw, load.bpf->jit.sz);
>  	return load.bpf;
>  }
> --
> 2.53.0

Re: [PATCH 2/2] bpf: silence noisy message

[Stephen Hemminger]

On Thu, 2 Jul 2026 08:36:14 +0000
Marat Khalili <marat.khalili@huawei.com> wrote:

> The commit message and Fixes tag are not 100% accurate, this message existed before:
> https://github.com/DPDK/dpdk/blob/09e4f37ddd0f7d5cc7c696741f31f1582c73a716/lib/bpf/bpf_load_elf.c#L331-L333
> 
> (In fact I tried to eliminate it, but had to but back at the insistence of AI reviewer
> and even broke the build in the process...)
> 
> So naturally I support downgrading this message. With the commit message corrected,
> 
> Acked-by: Marat Khalili <marat.khalili@huawei.com>

Looks like you added that message in:
  55192fe168b9 ("bpf: introduce extensible load API")

RE: [PATCH 2/2] bpf: silence noisy message

[Marat Khalili]

> https://github.com/DPDK/dpdk/blob/09e4f37ddd0f7d5cc7c696741f31f1582c73a716/lib/bpf/bpf_load_elf.c#L331
> -L333
> >
> > (In fact I tried to eliminate it, but had to but back at the insistence of AI reviewer
> > and even broke the build in the process...)
> >
> > So naturally I support downgrading this message. With the commit message corrected,
> >
> > Acked-by: Marat Khalili <marat.khalili@huawei.com>
> 
> Looks like you added that message in:
>   55192fe168b9 ("bpf: introduce extensible load API")

It was moved from a different file, see above.
Maybe it applied to fewer code paths since they are now all merged.

Anyway it's not really important unless you think it's worth backporting.
Feel free to downgrade or remove this message.

RE: [PATCH 1/2] bpf: default log level should be NOTICE

[Marat Khalili]

> -----Original Message-----
> From: Stephen Hemminger <stephen@networkplumber.org>
> Sent: Wednesday 1 July 2026 23:35
> To: dev@dpdk.org
> Cc: Stephen Hemminger <stephen@networkplumber.org>; Konstantin Ananyev <konstantin.ananyev@huawei.com>;
> Marat Khalili <marat.khalili@huawei.com>
> Subject: [PATCH 1/2] bpf: default log level should be NOTICE
> 
> BPF is used by other services now and the default log
> level should match other libraries.
> 
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
>  lib/bpf/bpf.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/bpf/bpf.c b/lib/bpf/bpf.c
> index 740c080591..695454a2d4 100644
> --- a/lib/bpf/bpf.c
> +++ b/lib/bpf/bpf.c
> @@ -78,4 +78,4 @@ __rte_bpf_jit(struct rte_bpf *bpf)
>  	return rc;
>  }
> 
> -RTE_LOG_REGISTER_DEFAULT(rte_bpf_logtype, INFO);
> +RTE_LOG_REGISTER_DEFAULT(rte_bpf_logtype, NOTICE);
> --
> 2.53.0

Acked-by: Marat Khalili <marat.khalili@huawei.com>

(I wonder if this default should have a name like LIB_DEFAULT.)