Re: Re: [PATCH 1/1] Make user mmapped CQ arming flags field 32 bit size to remove 64 bit architecture dependency of siw.

[Doug Ledford <dledford@redhat.com>]

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

On Wed, 2019-08-07 at 17:49 +0000, Bernard Metzler wrote:
> 
> It hurts, but I did finally setup qemu with a ppc image to check,
> and so you are right!
> 
> ...
> 
> #ifdef __BIG_ENDIAN__
> 
> seem to be available in both kernel and user land...
> 
> But, general question: siw in its current shape isn't out
> for long, older versions from github are already broken with
> the abi. So, silently changing the abi at this stage of siw
> deployment is no option? It's a hassle to see an old mistake
> carried along forever with that #ifdef statement...no?

I was thinking about that myself.

This really hasn't been out long enough to completely tie our hands
here.  A point update to rdma-core will resolve any user side issues. 
Are they doing stable kernel patches for the last kernel?  If so, we can
fix it both places.  No distros have picked up the original ABI in this
short of a window and managed to get it into a shipped product, so we
can notify any that might have picked up the broken version and get that
updated too if we don't dilly dally and make the call quickly.

-- 
Doug Ledford <dledford@redhat.com>
    GPG KeyID: B826A3330E572FDD
    Fingerprint = AE6B 1BDA 122B 23B4 265B  1274 B826 A333 0E57 2FDD

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]