On Sat, 2017-04-01 at 15:25 +0800, Ding Tianhong wrote: > The IXGBE_ALLOW_RELAXED_ORDER will enable Relaxed Ordering (RO) which > allows > transactions that do not have any order of completion requirements to > complete more efficiently compare to the Stricted Ordering (SO) for > ixbge > nic card. Some architecture will see high write-to-memory performance > when RO is > enabled on the data transactions just like the SPARC did. > > The aarch64 could both support Relaxed Ordering (RO) and Stricted > Ordering (SO), > so enable this config could get much more better performance, didn't > see any > adverse effects. > > The ARCH_WANT_RELAX_ORDER looks more general and would cause > misleading and > ambiguous, and till now only ixgbe could enable this "flag", so > rename this > config more specific. > > After discussion with the architecture maintainer, enable this config > in driver > looks more appropriate to compatible several architecture just like > SPARC and ARM64, > maybe we need more discussion about this, so let's begin by this > patch set. > > In the last patch 1a8b6d76(net:add one common config ...), Mao only > fix the > config name issue for 82599 pf, but the 82598 and 82599 vf still need > to be fixed, > so rename the config all in the drivers to instead of CONFIG_SPARC. Thank you for at least CC'ing me, but please remember to also CC intel- wired-lan at lists.osuosl.org mailing list. It make it easier for me to add your patches to my queue and makes it visible for the developers/validation people who will be reviewing and testing your changes. > > Ding Tianhong (4): > ? ixgbe: sparc: rename the ARCH_WANT_RELAX_ORDER to > ??? IXGBE_ALLOW_RELAXED_ORDER > ? ixgbe: ixgbevf: Clear the CONFIG_SPARC for ixgbevf and 82598 > ? ixgbe: move IXGBE_ALLOW_RELAXED_ORDER from architecture to driver > ? ixgbe: enable IXGBE_ALLOW_RELAXED_ORDER for ARM64 > > ?arch/Kconfig????????????????????????????????????? | 3 --- > ?arch/sparc/Kconfig??????????????????????????????? | 1 - > ?drivers/net/ethernet/intel/Kconfig??????????????? | 4 ++++ > ?drivers/net/ethernet/intel/ixgbe/ixgbe_82598.c??? | 4 ++-- > ?drivers/net/ethernet/intel/ixgbe/ixgbe_common.c?? | 2 +- > ?drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c | 2 +- > ?6 files changed, 8 insertions(+), 8 deletions(-) -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: This is a digitally signed message part URL: