07/10/2020 09:44, David Marchand: > On Tue, Oct 6, 2020 at 10:23 PM Thomas Monjalon <tho...@monjalon.net> wrote: > > > > The sub-directories of config/ are maintained by > > different architecture maintainers. > > > > Some wildcards are used to describe the lib and drivers files > > which are specific to some architectures. > > I noticed "some" misses, see below. > > > > > The EAL Arm files have split responsibilities depending on 32/64 suffix, > > and the common files are shared between Armv7 and Armv8 sections. > > > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > > --- > > MAINTAINERS | 23 +++++++++++------------ > > 1 file changed, 11 insertions(+), 12 deletions(-) > > > > diff --git a/MAINTAINERS b/MAINTAINERS > > index 75a17d51c0..349fc9c38a 100644 > > --- a/MAINTAINERS > > +++ b/MAINTAINERS > > @@ -256,32 +256,31 @@ F: app/test/test_rand_perf.c > > ARM v7 > > M: Jan Viktorin <vikto...@rehivetech.com> > > M: Ruifeng Wang <ruifeng.w...@arm.com> > > +F: config/arm/ > > F: lib/librte_eal/arm/ > > +X: lib/librte_eal/arm/include/*_64.h > > > > ARM v8 > > M: Jerin Jacob <jer...@marvell.com> > > M: Ruifeng Wang <ruifeng.w...@arm.com> > > -F: lib/librte_eal/arm/include/*_64.h > > -F: lib/librte_net/net_crc_neon.h > > -F: lib/librte_acl/acl_run_neon.* > > -F: lib/librte_bpf/bpf_jit_arm64.c > > -F: lib/librte_lpm/rte_lpm_neon.h > > -F: lib/librte_hash/rte*_arm64.h > > -F: lib/librte_efd/rte*_arm64.h > > -F: lib/librte_table/rte*_arm64.h > > -F: drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c > > -F: drivers/net/i40e/i40e_rxtx_vec_neon.c > > -F: drivers/net/virtio/virtio_rxtx_simple_neon.c > > +F: config/arm/ > > +F: lib/librte_eal/arm/ > > +X: lib/librte_eal/arm/include/*_32.h > > +F: lib/*/*_neon.* > > +F: lib/*/*_arm64.* > > +F: drivers/*/*/*_neon.* > > app and examples directory are not caught in this: > > ./lib/librte_net/net_crc_neon.h > ./lib/librte_acl/acl_run_neon.h > ./lib/librte_acl/acl_run_neon.c > ./lib/librte_lpm/rte_lpm_neon.h > ./lib/librte_node/ip4_lookup_neon.h > ./examples/l3fwd/l3fwd_lpm_neon.h > ./examples/l3fwd/l3fwd_em_hlm_neon.h > ./examples/l3fwd/l3fwd_neon.h > ./app/test-pmd/macswap_neon.h > ./drivers/net/hns3/hns3_rxtx_vec_neon.h > ./drivers/net/bnxt/bnxt_rxtx_vec_neon.c > ./drivers/net/mlx5/mlx5_rxtx_vec_neon.h > ./drivers/net/virtio/virtio_rxtx_simple_neon.c > ./drivers/net/ixgbe/ixgbe_rxtx_vec_neon.c > ./drivers/net/i40e/i40e_rxtx_vec_neon.c > > Can we use a wildcard at dpdk root? > */*/*_neon.* > > The drivers/*/*/*_neon.* entry would still be needed though. > > > > > > IBM POWER (alpha) > > M: David Christensen <d...@linux.vnet.ibm.com> > > +F: config/ppc/ > > F: lib/librte_eal/ppc/ > > -F: drivers/net/i40e/i40e_rxtx_vec_altivec.c > > +F: drivers/*/*/*_altivec.* > > F: examples/l3fwd/*altivec.h > > Idem neon, lib files missing. > > ./lib/librte_eal/ppc/include/rte_altivec.h > ./lib/librte_acl/acl_run_altivec.c > ./lib/librte_acl/acl_run_altivec.h > ./lib/librte_lpm/rte_lpm_altivec.h > ./examples/l3fwd/l3fwd_altivec.h > ./examples/l3fwd/l3fwd_lpm_altivec.h > ./drivers/net/mlx5/mlx5_rxtx_vec_altivec.h > ./drivers/net/virtio/virtio_rxtx_simple_altivec.c > ./drivers/net/i40e/i40e_rxtx_vec_altivec.c > > > > > > Intel x86 > > M: Bruce Richardson <bruce.richard...@intel.com> > > M: Konstantin Ananyev <konstantin.anan...@intel.com> > > +F: config/x86/ > > F: lib/librte_eal/x86/ > > Nothing for avx/sse ?
I was not extending the scope too much. I think you're right, we can go further in apps and x86-specific files.
