mbox series

[bpf-next,0/3] samples/bpf: fix LLVM compilation warning with

Message ID 20221217153821.2285-1-danieltimlee@gmail.com (mailing list archive)
Headers show
Series samples/bpf: fix LLVM compilation warning with | expand

Message

Daniel T. Lee Dec. 17, 2022, 3:38 p.m. UTC
Currently, compiling samples/bpf with LLVM emits several warning. They
are only small details, but they do not appear when compiled with GCC.
Detailed compilation command and warning logs can be found from bpf CI.

Daniel T. Lee (3):
  samples/bpf: remove unused function with test_lru_dist
  samples/bpf: replace meaningless counter with tracex4
  samples/bpf: fix uninitialized warning with
    test_current_task_under_cgroup

 samples/bpf/test_current_task_under_cgroup_user.c | 6 ++++--
 samples/bpf/test_lru_dist.c                       | 5 -----
 samples/bpf/tracex4_user.c                        | 4 ++--
 3 files changed, 6 insertions(+), 9 deletions(-)

Comments

Yonghong Song Dec. 17, 2022, 5:48 p.m. UTC | #1
On 12/17/22 7:38 AM, Daniel T. Lee wrote:
> Currently, compiling samples/bpf with LLVM emits several warning. They
> are only small details, but they do not appear when compiled with GCC.
> Detailed compilation command and warning logs can be found from bpf CI.

Could you change the subject line to
   samples/bpf: fix LLVM compilation warning

> 
> Daniel T. Lee (3):
>    samples/bpf: remove unused function with test_lru_dist
>    samples/bpf: replace meaningless counter with tracex4
>    samples/bpf: fix uninitialized warning with
>      test_current_task_under_cgroup
> 
>   samples/bpf/test_current_task_under_cgroup_user.c | 6 ++++--
>   samples/bpf/test_lru_dist.c                       | 5 -----
>   samples/bpf/tracex4_user.c                        | 4 ++--
>   3 files changed, 6 insertions(+), 9 deletions(-)
>
Daniel T. Lee Dec. 18, 2022, 12:08 a.m. UTC | #2
On Sun, Dec 18, 2022 at 2:48 AM Yonghong Song <yhs@meta.com> wrote:
>
>
>
> On 12/17/22 7:38 AM, Daniel T. Lee wrote:
> > Currently, compiling samples/bpf with LLVM emits several warning. They
> > are only small details, but they do not appear when compiled with GCC.
> > Detailed compilation command and warning logs can be found from bpf CI.
>
> Could you change the subject line to
>    samples/bpf: fix LLVM compilation warning
>
> >
> > Daniel T. Lee (3):
> >    samples/bpf: remove unused function with test_lru_dist
> >    samples/bpf: replace meaningless counter with tracex4
> >    samples/bpf: fix uninitialized warning with
> >      test_current_task_under_cgroup
> >
> >   samples/bpf/test_current_task_under_cgroup_user.c | 6 ++++--
> >   samples/bpf/test_lru_dist.c                       | 5 -----
> >   samples/bpf/tracex4_user.c                        | 4 ++--
> >   3 files changed, 6 insertions(+), 9 deletions(-)
> >


Thanks for pointing this out.
I will send a v2 patch.