Re: [Xen-ia64-devel] [Patch] Fix for re-enabling PV-on-HVM on IPF

[Doi.Tsunehisa@xxxxxxxxxxxxxx]

You (yamahata) said:
>>> Probably it should looks like the following.
>>> if (page->count_info != 1) { /* no one but us is using this page */
>>>     put_page(page);
>>>     goto out;
>>> }
>>> __set_bit(&page->count_info, _PGC_allocated);
> 
>>   Does `page->count_info' have to be operated with atomic ?
> 
> Atomic operation is safe bet.
> XENMAPSPACE hypercall isn't performance critical.
> If you want to be paranoia, use "if (test_and_set_bit())" to check whether
> some else set the bit.

  I'll modify it with this policy.

Thanks,
- Tsunehisa Doi

_______________________________________________
Xen-ia64-devel mailing list
Xen-ia64-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ia64-devel