[PATCH net-next 0/4] ixgbe: enable Relaxed Order for ARM64

[Ding Tianhong]

On 2017/4/1 16:39, Jeff Kirsher wrote:
> 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.

Thanks for your reminder, fix it in next version.:)

> 
>>
>> 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(-)