[dm-devel] [PATCH 4/6] libmpathpersist: fix byte swapping for big endian systems
Martin Wilck
mwilck at suse.com
Fri Jun 22 23:27:04 UTC 2018
On Fri, 2018-06-22 at 16:19 -0700, Bart Van Assche wrote:
> On 06/22/18 16:16, Martin Wilck wrote:
> > - sa_key = 0;
> > - for (i = 0; i < 8; ++i){
> > - if (i > 0)
> > - sa_key <<= 8;
> > - sa_key |= paramp->sa_key[i];
> > - }
> > + sa_key = be64_to_cpu(*(uint64_t *)¶mp-
> > >sa_key[0]);
>
> Have you considered to use get_unaligned_be64() instead of the above
> construct with casts?
Yes, I did. I opted for this construct for patch brevity (and
admittedly, lazyness), because get_unaligned_be64() doesn't exist yet.
I can change that of course if you insist.
Thanks,
Martin
--
Dr. Martin Wilck <mwilck at suse.com>, Tel. +49 (0)911 74053 2107
SUSE Linux GmbH, GF: Felix Imendörffer, Jane Smithard, Graham Norton
HRB 21284 (AG Nürnberg)
More information about the dm-devel
mailing list