Josef 'Jeff' Sipek wrote:
> Currently, the annotation just forces the structures to be packed, and
> 4-byte aligned.
Semantic nitpick: in my definition of "annotation" this is more than
just an annotation.
An "__ondisk" annotation, to me, would allow something like sparse to
verify properly laid out on-disk structures, but would not affect the
actual runtime code - I think that would be quite useful. However, this
change actually impacts the bytecode; it is a functional change.
So I really do understand what you're trying to do, despite my
protestations. If there is some magical instruction to gcc which:
a) leaves all current "non-broken" ABIs and gcc implementations'
bytecode untouched (or at the very least, minimally/trivially modified), and
b) fixes all possible future ABIs so they will always have things
perfectly and properly aligned, again w/o undue molestation of the
resulting bytecode
then I could probably be convinced. :) But this seems like a tall
order, and would require much scrutiny.
I'm just very shy of a sweeping change like this, which has a material
impact on the most common architectures, and does not actually provide,
as far as I can see, any benefit to them - only risk.
And for now I'll shut up and let the sgi guys chime in eventually. :)
-Eric
> Signed-off-by: Josef 'Jeff' Sipek <jeffpc@xxxxxxxxxxxxxx>
>
> ---
> This is just an RFC, and the alignment needs to be verified against the
> offsets within the pages read from disk, and more xfsqa runs on various
> architectures are needed. (I don't want to be responsible for something like
> the bitops regression on ppc!)
>
> The .text segment shrinks on x86 and s390x, but grows in ia64 (3776 bytes ==
> 0.3%).
>
> text data bss dec hex filename
> 542054 3171 3084 548309 85dd5 xfs-x86-original.ko
> 542026 3171 3084 548281 85db9 xfs-x86-packed-aligned4.ko
> 1244057 70858 2480 1317395 141a13 xfs-ia64-original.ko
> 1247833 70858 2480 1321171 1428d3 xfs-ia64-packed-aligned4.ko
> 679901 19374 3112 702387 ab7b3 xfs-s390x-original.ko
> 679781 19374 3112 702267 ab73b xfs-s390x-packed-aligned4.ko
>
> The approximate number of instructions effectively stays the same on x86
> (goes up by 2), s390x gets smaller (by 12 instructions), but ia64 bloats by
> 708 instructions (0.34%).
>
> $ for x in *.ko; do objdump -d $x > `basename $x .ko`.dis ; done
> $ wc -l *.dis
> 141494 xfs-x86-original.dis
> 141496 xfs-x86-packed-aligned4.dis
> 208514 xfs-ia64-original.dis
> 209222 xfs-ia64-packed-aligned4.dis
> 121544 xfs-s390x-original.dis
> 121532 xfs-s390x-packed-aligned4.dis
>
> I could try to compile things on a sparc64, mips, and x86_64, but that's for
> another day - and depending on where this thread will lead.
>
> The patch keeps xfsqa happy on x86 - well, it dies in the 100-range, but I
> haven't had the time to check if that happens without this patch. Someone
> (not it!) should nurse xfsqa back to health :)
>
> Jeff.
|