Re: [Xen-devel] [PATCH v8 1/5] x86/ioreq server: Release the p2m lock after mmio is handled.

[Yu Zhang <yu.c.zhang@xxxxxxxxxxxxxxx>]

On 3/21/2017 9:18 AM, Yu Zhang wrote:
> On 3/21/2017 1:03 AM, Jan Beulich wrote:
> > > > > On 18.03.17 at 11:53, <yu.c.zhang@xxxxxxxxxxxxxxx> wrote:
> > > --- a/xen/arch/x86/hvm/hvm.c
> > > +++ b/xen/arch/x86/hvm/hvm.c
> > > @@ -1870,18 +1870,14 @@ int hvm_hap_nested_page_fault(paddr_t gpa, 
> > > unsigned long gla,
> > >            (npfec.write_access &&
> > >             (p2m_is_discard_write(p2mt) || (p2mt == 
> > > p2m_ioreq_server))) )
> > >       {
> > > -        __put_gfn(p2m, gfn);
> > > -        if ( ap2m_active )
> > > -            __put_gfn(hostp2m, gfn);
> > > -
> > >           rc = 0;
> > >           if ( unlikely(is_pvh_domain(currd)) )
> > > -            goto out;
> > > +            goto out_put_gfn;
> > Did you forget to re-base onto staging before you've sent this?
> > is_pvh_domain() was gone before your submission already. When
> > re-basing, feel free to also drop the dead "rc = 0;" line.
> Oops...
> I did a rebase days ago, but did not send the patch directly. There 
> were some XenGT test performed based on these patch.
> Sorry, I should have a check.  Will resend the patch. :)
BTW, since there's another change - "drop the rc = 0", I'd like to send 
the new patchset with a new version nubmer. :-)
Yu
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> https://lists.xen.org/xen-devel

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