[PATCH 3/3] include: sbi: Lift the maximum harts to 1024
Anup Patel
anup at brainfault.org
Thu Oct 9 06:29:08 PDT 2025
On Thu, Oct 9, 2025 at 5:55 PM Wu Fei <atwufei at 163.com> wrote:
>
>
> On 9/30/25 19:36, Anup Patel wrote:
> > On Wed, Aug 6, 2025 at 6:39 AM <wu.fei9 at sanechips.com.cn> wrote:
> >> It is expected to have riscv server boards with many cores very soon.
> >>
> >> Signed-off-by: Wu Fei <wu.fei9 at sanechips.com.cn>
> >> ---
> >> include/sbi/sbi_hartmask.h | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/include/sbi/sbi_hartmask.h b/include/sbi/sbi_hartmask.h
> >> index 200ab6e..2f94f39 100644
> >> --- a/include/sbi/sbi_hartmask.h
> >> +++ b/include/sbi/sbi_hartmask.h
> >> @@ -20,7 +20,7 @@
> >> * also represents the maximum number of HART ids generic OpenSBI
> >> * can handle.
> >> */
> >> -#define SBI_HARTMASK_MAX_BITS 128
> >> +#define SBI_HARTMASK_MAX_BITS 1024
> > Instead of making this 8x, it is better to move to a more realistic
> > number like 256 or 512. NO ?
>
>
> Do you think we need to make this configurable? It sounds good to make
> it larger if it's configurable,
>
> 512 is not enough sooner or later on servers.
I would rather wait for a real platform with > 128 CPUs to show-up. Since
it is a one-line change and we can do it any day.
Regards,
Anup
>
>
> Thanks,
>
> Fei.
>
> >
> > Regards,
> > Anup
> >
> >> /** Representation of hartmask */
> >> struct sbi_hartmask {
> >> --
> >> 2.43.0
> >>
> >> --
> >> opensbi mailing list
> >> opensbi at lists.infradead.org
> >> http://lists.infradead.org/mailman/listinfo/opensbi
>
More information about the opensbi
mailing list