[PATCH v2] generic: mips: support harts to boot from mips_warm_boot
Chao-ying Fu
icebergfu at gmail.com
Wed Jul 23 13:37:26 PDT 2025
On Tue, Jul 22, 2025 at 5:18 PM Samuel Holland
<samuel.holland at sifive.com> wrote:
>
> Hi Chao-ying,
>
> On 2025-07-22 6:04 PM, Chao-ying Fu wrote:
> > We program reset base for harts (other than hart 0) to boot at
> > mips_warm_boot that jumps to _start_warm. This helps to skip some code
> > sequence to speed up.
> >
> > Signed-off-by: Chao-ying Fu <cfu at mips.com>
> > ---
> > platform/generic/mips/p8700.c | 17 +++++++++++++++++
> > 1 file changed, 17 insertions(+)
> >
> > diff --git a/platform/generic/mips/p8700.c b/platform/generic/mips/p8700.c
> > index 888a45c..8c93ebf 100644
> > --- a/platform/generic/mips/p8700.c
> > +++ b/platform/generic/mips/p8700.c
> > @@ -140,6 +140,20 @@ static void power_up_other_cluster(u32 hartid)
> > }
> > #endif
> >
> > +static void __attribute__((naked,no_instrument_function,aligned(4096)))
>
> Please follow Jess's suggestion and use an assembly file like we already do for
> platform/generic/andes/sleep.S. This will be more reliable than forever fighting
> the compiler to keep it from inserting any instructions on its own. (Are you
> _sure_ that this list of attributes covers every toolchain and configuration?)
>
Hi Sam,
Ok. I will change to use an assembly file and send out v3 soon,
similar to sleep.S.
Thanks a lot!
Regards,
Chao-ying
More information about the opensbi
mailing list