diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2015-04-20 09:08:49 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2015-04-20 09:08:49 -0700 |
commit | f4d03bd143c628b00f66cc2ec2c013767bdd1518 (patch) | |
tree | 0ff1571b048f1e47efbb1d1042562f3a92b4f24e /fs/cifs/smb2inode.c | |
parent | 09d51602cf84a1264946711dd4ea0dddbac599a1 (diff) |
smp: don't use 16-bit words for atomic accesses
Yes, it should work, but it's a bad idea. Not only did ARM64 not have
the 16-bit access code (there's a separate patch to add it), it's just
not a good atomic type. Some architectures fundamentally don't do
atomic accesses in them (alpha), and it's not like it saves any space
here anyway because of structure packing issues.
We normally should aim for flags to be "unsigned int" or "unsigned
long". And if space is at a premium, use a single byte (although that
causes problems on alpha again). There might be very special cases
where a 16-byte entity is really wanted, but this is not one of them.
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'fs/cifs/smb2inode.c')
0 files changed, 0 insertions, 0 deletions