Hi Dave.

2006/11/28 15:23:53 -0500, Dave Anderson <[EMAIL PROTECTED]> wrote:
>> Wouldn't it be easier to create a additional per-segment "pls->zero_fill" 
>> field
>> that is based upon the (possibly) larger p->memsz value?  Then, if the 
>> request
>> does not fall into the (pls->phys_start <= startaddr < pls->phys_end) range,
>> a subsequent check can be made to see if it falls within the alternative
>> (pls->phys_end <= startaddr < pls->zero_fill) range?  If that's true,
>> then we can just return a block of zeroes.
>>
>>
>
>Hi Ken'ichi,
>
>Can you apply and verify the attached crash patch?
>
>I've add a new pls->zero_fill field, which will be initialized to
>(pls->phys_start + p->p_memsz) only if p->memsz != p->filesz;
>otherwise it will be set to zero.

Thank you for your patch! I found your patch much easier to understand
than my original one. And I verified that the crash utility worked
correctly both on i386 and x86_64 with you patch applied.

Please merge your patch into the crash utility.

Thanks
Ken'ichi Ohmichi
_______________________________________________
fastboot mailing list
[email protected]
https://lists.osdl.org/mailman/listinfo/fastboot

Reply via email to