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@xxxxxxxxxx>
GPG KeyID: 0E572FDD
Attachment:
signature.asc
Description: OpenPGP digital signature
