On Sun, Apr 4, 2021 at 2:53 AM Hengqi Chen <chenhen...@outlook.com> wrote: > > Add missing ')' for KERNEL_VERSION macro. > > Signed-off-by: Hengqi Chen <chenhen...@outlook.com> > ---
The fix looks good, thank you. But your patch didn't make it into bpf/netdev patchworks instance ([0]) most probably due to too long CC list. Can you please re-send with just maintainers and bpf@ and netdev@ mailing lists in to/cc. Also for bpf and bpf-next tree, we ask to specify the tree with [PATCH bpf-next] prefix, so when re-submitting please adjust as well. Thanks. [0] https://patchwork.kernel.org/project/netdevbpf/list/ > tools/lib/bpf/bpf_helpers.h | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/tools/lib/bpf/bpf_helpers.h b/tools/lib/bpf/bpf_helpers.h > index cc2e51c64a54..b904128626c2 100644 > --- a/tools/lib/bpf/bpf_helpers.h > +++ b/tools/lib/bpf/bpf_helpers.h > @@ -51,7 +51,7 @@ > #endif > > #ifndef KERNEL_VERSION > -#define KERNEL_VERSION(a,b,c) (((a) << 16) + ((b) << 8) + ((c) > 255 ? 255 : > (c)) > +#define KERNEL_VERSION(a, b, c) (((a) << 16) + ((b) << 8) + ((c) > 255 ? 255 > : (c))) > #endif > > /* > -- > 2.25.1 >
