qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] exec: Limit translation limiting in address_spa


From: Stefano Stabellini
Subject: Re: [Qemu-devel] [PATCH] exec: Limit translation limiting in address_space_translate to xen
Date: Thu, 27 Mar 2014 12:17:21 +0000
User-agent: Alpine 2.02 (DEB 1266 2009-07-14)

On Thu, 27 Mar 2014, Paolo Bonzini wrote:
> Il 27/03/2014 05:35, Alexey Kardashevskiy ha scritto:
> > The address_space_translate() function cuts the returned plen (page size)
> > to hardcoded TARGET_PAGE_SIZE. This function can be used on pages bigger
> > than that so this limiting should not be used on such pages.
> > 
> > Since originally the limiting was introduced for XEN, we can safely
> > limit this piece of code to XEN. So does the patch.
> > 
> > Suggested-by: Paolo Bonzini <address@hidden>
> > Signed-off-by: Alexey Kardashevskiy <address@hidden>
> > ---
> > 
> > I need this change for VFIO-on-sPAPR series as it is going to support
> > 16MB IOMMU pages. Thanks.
> > ---
> >  exec.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/exec.c b/exec.c
> > index 91513c6..cf12049 100644
> > --- a/exec.c
> > +++ b/exec.c
> > @@ -380,7 +380,7 @@ MemoryRegion *address_space_translate(AddressSpace *as,
> > hwaddr addr,
> >          as = iotlb.target_as;
> >      }
> > 
> > -    if (memory_access_is_direct(mr, is_write)) {
> > +    if (xen_enabled() && memory_access_is_direct(mr, is_write)) {
> >          hwaddr page = ((addr & TARGET_PAGE_MASK) + TARGET_PAGE_SIZE) -
> > addr;
> >          len = MIN(page, len);
> >      }
> > 
> 
> Stefano, can you include this in your Xen tree for 2.1?

Sure.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]