[Skiboot] [PATCH 08/12] xive: make endian-clean

Nicholas Piggin npiggin at gmail.com
Wed Oct 2 14:29:50 AEST 2019


Oliver O'Halloran's on October 1, 2019 3:24 pm:
> On Sun, 2019-09-29 at 17:46 +1000, Nicholas Piggin wrote:
>> Convert xive opal calls, dt construction, and in-memory hardware tables
>> to use explicit endian conversions.
> 
> Cedric, ack/nak?
> 
> I'm a bit iffy about the __prefixes since the in-memory tables are
> homogenously BE. If we're going to be more concerned about endian
> safety going forward I'd prefer we use the __beXX annotations and some
> kind of automated checking rather than ad-hoc conventions.

Yeah good point about that. I actually renamed those only to get the
compiler to tell me about anything I had missed. I really intended to
rename them back in a final patch.

Cedric has some accessor scheme that sounds like it will make this a
lot tidier too.

Thanks,
Nick


More information about the Skiboot mailing list