Re: [PATCH RFC v2 0/2] libibverbs memory barrier fixes

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

 



On 5/25/2016 10:51 AM, Steve Wise wrote:
>> On 05/20/2016 03:43 PM, Steve Wise wrote:
>>> This is v2 of the ARM64 mb* support, plus an additional patch to fail
>>> compiles if there is no platform implementation of the memory barriers.
>>> I've included the 2nd patch because I think it is important to not
>>> assume any default barrier implementation.  Getting it wrong can result
>>> in data corruption.
>>>
>>> Changes since V1:
>>>
>>> Put the RFC tag back on because I want to retest this series.
>>>
>>> Implemented the ARM64 memory barrier macros from scratch using the AMD
>>> reference docs.
>>>
>>> Added 2nd patch to fail compiles if no mb* macros exist for the platform.
>>>
>>> Steve Wise (2):
>>>   libibverbs: add ARM64 memory barrier macros
>>>   Fail compiles if no platform specific memory barriers exist
>>>
>>>  include/infiniband/arch.h |   15 +++++++++------
>>>  1 files changed, 9 insertions(+), 6 deletions(-)
>>>
>>
>> Please remember to use --thread=shallow when sending patch emails using
>> git send-email (which is what I assume you did).
>>
>> Let me know how the testing comes out.  The patches look fine to me
>> (thanks for adding the second one, it's the right thing to do).
>>
> 
> Hey Doug, the series tests good with cxgb4 on ARM64.  Do I need to repost a
> final series, or can you merge v2? 
> 
> Hey Noa, have you all had a chance to test this series?
> 
> Thanks,
> 
> Steve.
> 

Repost a final please.

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