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

Re: [Xen-devel] [PATCH v5 08/10] xen/arm: Add relinquish_p2m_mapping to remove reference on every mapped page



On Tue, 2013-12-17 at 14:08 +0000, Julien Grall wrote:
> On 12/17/2013 09:26 AM, Ian Campbell wrote:
> > On Mon, 2013-12-16 at 17:37 +0000, Julien Grall wrote:
> >> +        if ( op == RELINQUISH && count == LPAE_ENTRIES &&
> >> +             hypercall_preempt_check() )
> >> +        {
> > 
> > If hypercall_preempt_check returns false the first time it is called
> > then you won't ever try again. I think you want
> >     (count+1 % LPAE_ENTRIES) == 0
> > (the +1 avoids preempting without having done any work yet)
> > 
> > (credit to Jan for pointing this out in his review of Mukesh's x86
> > variant)
> 
> The code was copied from relinquish_shared_pages
> (arch/x86/mem_sharing.c), so if I'm not mistaken this code is also buggy.

Yes, Jan sent a fix for it this morning.

Ian.




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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.