Removed tspe...@qti.qualcomm.com as the email address is not reachable. > > > > I'll come up with a new patch based on the discussion here. > > > > A few things noted, > > > > - we still want it to be 128B for generic build > > > > - we at least agreed on changing it to 64B for A72 > > > How will this be done? Will you add > > > config/arm/arm64_bluefield_linuxapp_gcc? > > I asked this question as there was a proposal containing 'a72' in the > > file name. IMO, the file name should contain 'bluefield', not on a72. > > Sorry for late reply. It's been buried for some reason. :-) No problem
> > Nope, I won't create such a file. That's for cross-compiler AFAIK. > I'm thinking about changing meson.build. Currently, one CL size is applied to > all kinds of cores. Consequently, for armv8a, both 'default' and 'a72' have to > have the same CL size. And one more thing raised from ARM was that > 'crypto' in -march can't be a default. Yes, crypto is optional. So, the distro builds (and any other builds for binary compatibility) should not expect crypto to be present by default. > > > > > - As Qualcomm Centriq CPU has 128B cache line with A72, they should > > > > create a profile in meson.build based on their impl_id. > > > Qualcomm is not A72 core. Can it not use RTE_CACHE_LINE_SIZE from > > > 'flags_generic' in config/arm/meson.build? > > We can add flags_qualcomm for their processor. > > Thanks, > Yongseok
