> Subject: Re: [PATCH] powerpc: update ibm,client-architecture

Please give this a more appropriate name.  

> In order to boot with more than 64 cores on machines that support the
> ibm,client-architecture RTAS call a new field has been added to the
> structure.  This patch updates that field and adds a few others in the
> process.

Please detail what these are here.

> It would be good if this could go in as a bugfix.  
> 
> Signed-off-by: Joel Schopp <jsch...@austin.ibm.com>
> 
> Index: linux-2.6.git/arch/powerpc/kernel/prom_init.c
> ===================================================================
> --- linux-2.6.git.orig/arch/powerpc/kernel/prom_init.c
> +++ linux-2.6.git/arch/powerpc/kernel/prom_init.c
> @@ -654,6 +654,9 @@ static void __init early_cmdline_parse(v
>  #define OV5_CMO                      0x00
>  #endif
>  
> +/* Option Vector 6: IBM PAPR hints */
> +#define OV6_LINUX            0x02    /* Linux is our OS */
> +
>  /*
>   * The architecture vector has an array of PVR mask/value pairs,
>   * followed by # option vectors - 1, followed by the option vectors.
> @@ -665,7 +668,7 @@ static unsigned char ibm_architecture_ve
>       W(0xffffffff), W(0x0f000003),   /* all 2.06-compliant */
>       W(0xffffffff), W(0x0f000002),   /* all 2.05-compliant */
>       W(0xfffffffe), W(0x0f000001),   /* all 2.04-compliant and earlier */
> -     5 - 1,                          /* 5 option vectors */
> +     6 - 1,                          /* 6 option vectors */
>  
>       /* option vector 1: processor architectures supported */
>       3 - 2,                          /* length */
> @@ -697,12 +700,26 @@ static unsigned char ibm_architecture_ve
>       0,                              /* don't halt */
>  
>       /* option vector 5: PAPR/OF options */
> -     5 - 2,                          /* length */
> +     13 - 2,                         /* length */
>       0,                              /* don't ignore, don't halt */
>       OV5_LPAR | OV5_SPLPAR | OV5_LARGE_PAGES | OV5_DRCONF_MEMORY |
>       OV5_DONATE_DEDICATE_CPU | OV5_MSI,
>       0,
>       OV5_CMO,
> +     0,                              /* reserved */
> +     0,                              /* reserved */
> +     0,                              /* reserved */
> +     0,                              /* reserved */
> +     W(NR_CPUS/4),                   /* max cores supported */

4?

> +
> +     /* option vector 6: IBM PAPR hints */
> +     4 - 2,

Please put a comment next to these to say what it is.  

> +     0,                              /* reserved */
> +     0,                              /* 1 == no secondary pteg */
> +     OV6_LINUX,
> +
> +     /* option vector 7: IBM PAPR OS identification */
> +     /* a human readable ascii string will go here */

Can we do this now or remove the comment.  Maybe UTS_RELEASE or
something like that.

Mikey

>  };
>  
>  /* Old method - ELF header with PT_NOTE sections */
> 
> 
> _______________________________________________
> Linuxppc-dev mailing list
> Linuxppc-dev@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/linuxppc-dev
> 
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Reply via email to