Don Zickus <[EMAIL PROTECTED]> writes:

>> Looking at my build it appears bytes_out is being placed in the .bss.
>> A little odd since it is zero initialized but no big deal.
>> Could you confirm that bytes_out is being placed in the .bss section 
>> by inspecting arch/x86_64/boot/compresssed/misc.o and
>> arch/x86_64/boot_compressed/vmlinux.   "readelf -a $file" and then
>> looking up the section number and looking at the section table to see
>> which section it is was my technique.
>> 
>> If bytes_out is in the .bss for you then I suspect something is not
>> correctly zeroing the .bss.  Or else the .bss is being stomped.
>> 
>> I'm not certain how rep stosb can be done wrong but some bad pointer
>> math could have done it.
>> 
>> Eric
>
> It seems Vivek came up with a solution that works.  He sent it to me this
> morning.  We tested a bunch of machines and things seem to work now.  It
> looks like it mimics the i386 behaviour now.

Yes, this looks right.  It looks like I forgot to make this change when
the logic from i386 was adopted to x86_64, ages ago.

This is exactly the place in the code I would have expected a bug
from the symptoms you were seeing.

Thanks all I will include this in my version of the patches.

Eric
_______________________________________________
fastboot mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/fastboot

Reply via email to