Re: [Xen-devel] [PATCH 4/6] mini-os/x86-64 entry: remove unnecessary event blocking

[Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>]

Xu Zhang, le Fri 08 Mar 2013 15:30:17 -0600, a écrit :
> We don't need to block events here because:
>  - if we came from "hypervisor_callback", events are disabled at this point,
>    no need to block again;
>  - if we came from "error_entry", we shouldn't touch event mask, for
>    exception hanlding are meant to be interrupted by Xen events (virtual
>    irq).
> 
> Signed-off-by: Xu Zhang <xzhang@xxxxxxxxxx>

Acked-by: Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>

> ---
>  extras/mini-os/arch/x86/x86_64.S |    1 -
>  1 files changed, 0 insertions(+), 1 deletions(-)
> 
> diff --git a/extras/mini-os/arch/x86/x86_64.S 
> b/extras/mini-os/arch/x86/x86_64.S
> index 79e893f..5e0021b 100644
> --- a/extras/mini-os/arch/x86/x86_64.S
> +++ b/extras/mini-os/arch/x86/x86_64.S
> @@ -185,7 +185,6 @@ ENTRY(hypervisor_callback2)
>  
>  error_exit:
>       RESTORE_REST
> -     XEN_BLOCK_EVENTS(%rsi)
>  
>  retint_kernel:
>  retint_restore_args:
> -- 
> 1.7.7.6
> 

-- 
Samuel
 > Il [e2fsck] a bien démarré, mais il m'a rendu la main aussitot en me
 > disant "houlala, c'est pas beau à voir votre truc, je préfèrerai que
 > vous teniez vous même la tronçonneuse" (traduction libre)
 NC in Guide du linuxien pervers : "Bien configurer sa tronçonneuse."

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel