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

Re: [Xen-devel] [PATCH 2/2] x86/HVM: fix setting mem access to default



>>> On 03.04.14 at 02:13, <aravindp@xxxxxxxxx> wrote:
>> commit 3b0bcb89 ("x86/mm/p2m: Move p2m code in
>>HVMOP_[gs]et_mem_access into p2m.c") introduced an off-by-one mistake
>>forcing an input of HVMMEM_access_default to always fail. Since related, also
>>eliminate the inefficient setup of an on-stack array for each function
>>invocation.
> 
> Thanks for catching this. I do have a question. If the caller wants to set 
> the default access by specifying pfn == ~0ul, the call could be made with 
> access = HVMMEM_access_default. Should this be allowed?

I asked myself this question too, but wasn't really decided about the
usefulness of this, so left the behavior unchanged (i.e. allow this
pointless operation). I'll leave the decision whether to disable this to
the mm maintainers.

Jan


_______________________________________________
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®.