From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chuck Anderson Subject: Re: __HYPERVISOR_COMPAT_VIRT_START for HVM guest with PV drivers Date: Thu, 20 Aug 2009 13:48:41 -0700 Message-ID: <4A8DB6A9.2060202@oracle.com> References: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0848629599==" Return-path: In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: "xen-devel@lists.xensource.com" List-Id: xen-devel@lists.xenproject.org --===============0848629599== Content-Type: text/html; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Signed-off-by: Chuck Anderson <chuck.anderson@oracle.com>

I booted a 3.4.0 based hypervisor with changeset 20101 applied and verified that it resolved the problem.
Chuck

Keir Fraser wrote:
On 20/08/2009 17:27, "Keir Fraser" <keir.fraser@eu.citrix.com> wrote:

  
Do the guest virtual addresses passed by a hypervisor call on a 32-bit
HVM with PV drivers guest to a 64-bit hypervisor need to lie below
__HYPERVISOR_COMPAT_VIRT_START?  Both HVM and PVM domains have their
hv_compat_vstart set to __HYPERVISOR_COMPAT_VIRT_START.  The
hypervisor's compat layer is causing the balloon driver's
HYPERVISOR_memory_op(XENMEM_decrease_reservation, &reservation)
hypervisor call to error out in xen/common/compat/memory.c
compat_memory_op(), compat_handle_ok(), because the PVHVM guest's
extent_start lies above __HYPERVISOR_COMPAT_VIRT_START.  The following
patch appears to fix the problem but I want to make sure it is fixing
the real bug:
      
Yeah, you pretty much have it figured. I may just clean up for style (I
think an extra macro for this is a bit pointless), but the patch is
fundamentally good.
    

Take a look at xen-unstable.hg:20101

 -- Keir


  
--===============0848629599== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xensource.com http://lists.xensource.com/xen-devel --===============0848629599==--