[Skiboot] [PATCH v2 1/2] debug_descriptor: Claim reserved field for host kernel log buffer

Benjamin Herrenschmidt benh at kernel.crashing.org
Mon May 28 23:17:35 AEST 2018


On Mon, 2018-05-28 at 13:25 +0930, Joel Stanley wrote:
> Signed-off-by: Joel Stanley <joel at jms.id.au>
> ---
> We could bump the version instead, but I don't think there's any call
> for that.
> 
> Another option would be to re-use some of the memcons fields that are
> currently only used by FSP.
> 
> A third option would be to use this field to link to yet another debug
> structure to allow for future flexibility.

Nah there's a version field already for that. You should bump it while
at it, to indicate that the field is valid.

Cheers,
Ben.

> ---
>  include/skiboot.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/skiboot.h b/include/skiboot.h
> index b4bdf37795dd..7f3740965412 100644
> --- a/include/skiboot.h
> +++ b/include/skiboot.h
> @@ -75,7 +75,7 @@ struct debug_descriptor {
>  					 * low 4 bits driver (e.g. uart). */
>  	u8	state_flags; /* various state flags - OPAL_BOOT_COMPLETE etc */
>  	u16	reserved2;
> -	u32	reserved[2];
> +	u64	log_buf_phys;		/* Pointer to kernel log buffer */
>  
>  	/* Memory console */
>  	u64	memcons_phys;


More information about the Skiboot mailing list