[PATCH V4 1/2] ACPI / EC: Fix broken 64bit big-endian users of 'global_lock'

Viresh Kumar viresh.kumar at linaro.org
Fri Sep 25 13:25:49 PDT 2015


On 25 September 2015 at 13:33, Rafael J. Wysocki <rjw at rjwysocki.net> wrote:
> You're going to change that into bool in the next patch, right?

Yeah.

> So what if bool is a byte and the field is not word-aligned

Its between two 'unsigned long' variables today, and the struct isn't packed.
So, it will be aligned, isn't it?

> and changing
> that byte requires a read-modify-write.  How do we ensure that things remain
> consistent in that case?

I didn't understood why a read-modify-write is special here? That's
what will happen
to most of the non-word-sized fields anyway?

Probably I didn't understood what you meant..

--
viresh



More information about the linux-arm-kernel mailing list