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

Re: [Xen-devel] [PATCH v2 07/12] x86/altp2m: add control of suppress_ve.



On 07/06/2015 07:43 PM, Ed White wrote:
>> Introducing yet another layer -- particularly in a hooked interface like
>> this -- just seems clunky.  It's not the worst thing in the world; if I
>> thought this would be the difference between making it or not, I might
>> just say fix it later.  But I don't think it will; and these little
>> things add up.
>>
> 
> I don't want to change set/get everywhere, and Tim already made it clear
> that coupling suppress_ve with p2m_type_t is not acceptable.
> 
> How can I provide an implementation that does not do either of the above
> but does allow access to suppress_ve in a way that is acceptable?
> 
> Tell me and I will do it.

The only reason I can think that you don't want to change get/set is
that you think it's too much work.

So here you go, I modified your patch; it took me 10 minutes, which is
less than what it would have taken me to continue arguing with you.
I've compile-tested it, but not done anything else (including porting
subsequent patches onto it).

 -George

Attachment: 0001-x86-altp2m-add-control-of-suppress_ve.patch
Description: Text Data

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