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

[Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

[-- Attachment #1: Type: text/plain, Size: 1254 bytes --]

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

-- 
Doug Ledford <dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>
              GPG KeyID: 0E572FDD



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 884 bytes --]