Query : Regarding overwriting of DTB loading region while clearing BSS section

[Kohli, Gaurav]

Adding Chandra , He has some made some patch related to this.

On 11/23/2017 11:51 PM, Mark Rutland wrote:

> On Thu, Nov 23, 2017 at 10:26:07PM +0530, Kohli, Gaurav wrote:
>> Hi Mark,
>>
>> Thanks for the reply,
>>
>> Actually, We solved our issue by removing unwanted code from kernel,
> I would very strongly recommend that you update your bootladoer/firmware
> to respect image_size, since any numbero f things may increase the size
> of the kernel in future.
>
> i.e. trimmign the kernel down is a workaround, but not a fix.
>
>> But my request can't we put some check itself in head.s during clear
>> of BSS region to check whether dtb is intact or not.  Because
>> ultimately it will fail in start kernel.
>> At that point(start_kernel),  we can not directly say what is the root cause
>> for this overriding.
> Perhaps we could add a check in setup_arch(). If the DTB pointer
> overlaps any part of the kerenl (BSS included), the bootloader is
> clearly broken.
>
> Even then, it's going to be a slient panic(), since we won't be able to
> parse anything out of the DTB...
>
> Thanks,
> Mark.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
-- 
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project.