> On 11 Feb 2019, at 04:06, Alexei Starovoitov <alexei.starovoi...@gmail.com>
> wrote:
>
> On Fri, Feb 08, 2019 at 05:41:21PM +0000, Jiong Wang wrote:
>> At the moment, all kernel bpf objects are listed under BPF_OBJ_FILES.
>> Listing them manually sometimes causing patch conflict when people are
>> adding new testcases simultaneously.
>>
>> It is better to centre all the related source files under a subdir
>> "kern_progs", then auto-generate the object file list.
>>
>> Suggested-by: Alexei Starovoitov <a...@kernel.org>
>> Reviewed-by: Jakub Kicinski <jakub.kicin...@netronome.com>
>> Signed-off-by: Jiong Wang <jiong.w...@netronome.com>
>> ---
>> tools/testing/selftests/bpf/Makefile | 26
>> +++++-----------------
>> .../selftests/bpf/{ => kern_progs}/bpf_flow.c | 0
>> .../selftests/bpf/{ => kern_progs}/connect4_prog.c | 0
>> .../selftests/bpf/{ => kern_progs}/connect6_prog.c | 0
>> .../selftests/bpf/{ => kern_progs}/dev_cgroup.c | 0
>
> Thanks a lot for the patch.
> A tiny bit of bikeshedding...
> 'kern_progs' feels a bit too long and awkward to type.
> May be just 'progs’ ?
Ack, will do the change in v2.
Regards,
Jiong
