On Mon, Oct 18, 2021 at 9:07 AM John Fastabend <john.fastabend@xxxxxxxxx> wrote: > > Wan Jiabing wrote: > > Fix following checkincludes.pl warning: > > ./scripts/checkincludes.pl tools/testing/selftests/bpf/cgroup_helpers.c > > tools/testing/selftests/bpf/cgroup_helpers.c: unistd.h is included more > > than once. > > > > Signed-off-by: Wan Jiabing <wanjiabing@xxxxxxxx> > > --- > > Changelog: > > v2: > > - Fix the commit description. > > --- > > The Subject is a bit unusual. Typically it would be something like, > > "[PATCH bpf-next] selftests, remove duplicated include in cgroup_helpers" Also for selftests we use "selftests/bpf: " prefix (at least we generally try, for consistency). Fixed up, applied to bpf-next. Thanks. > > For the actual patch though LGTM. > > Acked-by: John Fastabend <john.fastabend@xxxxxxxxx>
