[Skiboot] [PATCH 2/2] core/init: Checksum romem after patching out traps

[Carol L Soto]

"Vasant Hegde" <hegdevasant at linux.vnet.ibm.com> wrote on 10/11/2019
05:16:46 AM:

> From: "Vasant Hegde" <hegdevasant at linux.vnet.ibm.com>
> To: "Oliver O'Halloran" <oohall at gmail.com>, skiboot at lists.ozlabs.org
> Cc: Carol L Soto/Poughkeepsie/IBM at IBMUS
> Date: 10/11/2019 05:16 AM
> Subject: Re: [Skiboot] [PATCH 2/2] core/init: Checksum romem after
> patching out traps
>
> On 10/11/19 9:57 AM, Oliver O'Halloran wrote:
> > Currently we checksum the read-only parts of skiboot's memory just
> > before loading and booting petitboot. Commit 9ddc1a6bfaef
> > ("core/util: trap based assertions") modifies the .text after this
> > point since it needs to disable the trap instructions that we use
> > to trigger an abort() before entering the kernel.
> >
> > We can fix this by moving the checksum to after the point where the
> > traps are patched out. We could do the patching sooner, but since
> > load_and_boot_kernel() is a fairly complex function it's perferable
> > to keep boot-time assertion infrastructure active until just before
> > we enter the kernel.
> >
> > Reported-by: Carol L Soto <clsoto at us.ibm.com>
> > Fixes: 9ddc1a6bfaef ("core/util: trap based assertions")
> > Signed-off-by: Oliver O'Halloran <oohall at gmail.com>
>
> Looks good to me and it fixed fast-reboot issue.
>
> Tested-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
>
>
> -Vasant

It fixed the issue. thanks
Tested-by:  Carol L Soto <clsoto at us.ibm.com>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ozlabs.org/pipermail/skiboot/attachments/20191011/14c0126f/attachment-0001.htm>