On Thu, Dec 19, 2019 at 4:19 AM Andrew Cooper <andrew.coop...@citrix.com> wrote:
>
> On 18/12/2019 19:40, Tamas K Lengyel wrote:
> > diff --git a/xen/arch/x86/hvm/hvm.c b/xen/arch/x86/hvm/hvm.c
> > index 5a3a962fbb..1e888b403b 100644
> > --- a/xen/arch/x86/hvm/hvm.c
> > +++ b/xen/arch/x86/hvm/hvm.c
> > @@ -1902,12 +1902,11 @@ int hvm_hap_nested_page_fault(paddr_t gpa, unsigned
> > long gla,
> > if ( npfec.write_access && (p2mt == p2m_ram_shared) )
> > {
> > ASSERT(p2m_is_hostp2m(p2m));
> > - sharing_enomem =
> > - (mem_sharing_unshare_page(currd, gfn, 0) < 0);
> > + sharing_enomem = mem_sharing_unshare_page(currd, gfn, 0);
>
> This is a logical change. Is it intended to be in a later patch?
While it may look like one it's actually not. The variable
sharing_enomem is declared as an int and the function only has two
possible return values, 0 and -ENOMEM.
Tamas
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
