[SLOF] [PATCH slof v3 3/5] Revert "rtas: Improve error handling in instantiate-rtas"

Greg Kurz groug at kaod.org
Tue Oct 3 18:22:08 AEDT 2017


On Tue,  3 Oct 2017 16:15:21 +1100
Alexey Kardashevskiy <aik at ozlabs.ru> wrote:

> This reverts commit 1c17c13a57 "rtas: Improve error
> handling in instantiate-rtas" as it has improved a thing
> about to be reverted.
> 
> Signed-off-by: Alexey Kardashevskiy <aik at ozlabs.ru>
> ---

Well, this technically breaks bisectability since QEMU doesn't implement
KVMPPC_H_RTAS_UPDATE... maybe you should fold patch 3 and 4 actually.

>  board-qemu/slof/rtas.fs | 9 +++------
>  1 file changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/board-qemu/slof/rtas.fs b/board-qemu/slof/rtas.fs
> index 54d3929..0d2bf49 100644
> --- a/board-qemu/slof/rtas.fs
> +++ b/board-qemu/slof/rtas.fs
> @@ -173,13 +173,10 @@ rtas-node set-node
>  : instantiate-rtas ( adr -- entry )
>      dup rtas-base swap rtas-size move
>      dup rtas-entry rtas-base - +
> -    2dup hv-rtas-update dup 0 <> IF
> -	\ Ignore hcall not implemented error, print error otherwise
> -	dup -2 <> IF ." HV-RTAS-UPDATE error: " . cr ELSE drop THEN
> -    ELSE
> -	drop
> +    2dup hv-rtas-update 0 <> IF
> +    ." Failed to update RTAS " cr
>      THEN
> -    nip
> +    nip   
>  ;
>  
>  device-end



More information about the SLOF mailing list