[Xen-devel] Re: [patch 1/1] hypervisor: Enable hypervisor detect pvhvm running on COMPAT mode

[Keir Fraser <keir.fraser@xxxxxxxxxxxxx>]

Thanks, I'll check in a clearer version of this.

 -- Keir

On 03/03/2009 06:46, "Joe Jin" <joe.jin@xxxxxxxxxx> wrote:

> Hi,
> 
> When 32bit pvhvm running on 64bit hypervisor, if guest OS create
> event-channel more than 32, Domain0 maybe could not communication
> with guest OS via new event-channel, also xenwatch thread state
> is TASK_UNINTERRUPTIBLE and could not got schedule, all inter-domain
> command almost no response.
> 
> How to reproduce:
>     Try to with "xm block-attach" attach more disk to the pvhvm
>     guest, the issue will appear.
> 
> This caused by hypervisor could not probe and decide if pvhvm guest
> running in COMPAT mode, and could set the right bits for guest os.
> 
> Please review, thanks.
> 
> Signed-off-by: Joe Jin <joe.jin@xxxxxxxxxx>
> ---
> 
>  domain.h |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff -r 98d48f7680db xen/include/asm-x86/domain.h
> --- a/xen/include/asm-x86/domain.h Wed Nov 19 19:13:22 2008 +0000
> +++ b/xen/include/asm-x86/domain.h Tue Mar 03 14:35:19 2009 +0800
> @@ -16,7 +16,7 @@
>  #define is_pv_32on64_domain(d) (0)
>  #endif
>  #define is_pv_32on64_vcpu(v)   (is_pv_32on64_domain((v)->domain))
> -#define IS_COMPAT(d)           (is_pv_32on64_domain(d))
> +#define IS_COMPAT(d)           (is_pv_32on64_domain(d) ||
> has_32bit_shinfo(d))
>  
>  struct trap_bounce {
>      uint32_t      error_code;
> 
> 



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel