Re: FW: [PATCH for-next v1 0/2] Advertise atomic operations support in mlx5

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 12/23/2015 12:05 PM, eran ben elisha wrote:
>>>
>>> If the firmware supports host endianness, try to configure the
>>> hardware to work this way, and on success propagate this capability to
>>> the upper layers, otherwise advertise that atomic operations aren't supported.
>>
>> If you are not going to allow any support except for host endian, then why bother with all of the BE stuff, can't you just list it as supported or not?
> 
> The HW default state is BE reply, need to change it's mode internally.
> The user will never know about this, it do listed as supported/not.

Ok, I've pulled this in.

>>
>>> Thanks, Eran
>>>
>>> Changes from v0:
>>>       - Rewrite the commit message of the second patch in the series
>>>
>>>
>>> Eran Ben Elisha (2):
>>>   net/mlx5_core: Configure HW to support atomic request in host
>>>     endianness
>>
>> I can fix this up, but generally try to keep your subject of patches short enough to fit in the generally accepted length.
> 
> Sure, will keep this in mind for next time.
> 
>>
>>>   IB/mlx5: Advertise atomic capabilities in query device
>>>
>>>  drivers/infiniband/hw/mlx5/main.c              | 28 ++++++++++++-
>>>  drivers/net/ethernet/mellanox/mlx5/core/main.c | 57 +++++++++++++++++++++++++-
>>>  include/linux/mlx5/driver.h                    |  5 +++
>>>  include/linux/mlx5/mlx5_ifc.h                  | 22 ++++++----
>>>  4 files changed, 102 insertions(+), 10 deletions(-)
>>>
>>
>>
>> --
>> Doug Ledford <dledford@xxxxxxxxxx>
>>               GPG KeyID: 0E572FDD
>>
>>


-- 
Doug Ledford <dledford@xxxxxxxxxx>
              GPG KeyID: 0E572FDD


Attachment: signature.asc
Description: OpenPGP digital signature


[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux