From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ed White Subject: Re: [PATCH v2 07/12] x86/altp2m: add control of suppress_ve. Date: Tue, 07 Jul 2015 09:24:38 -0700 Message-ID: <559BFD46.9030208@intel.com> References: <1434999372-3688-1-git-send-email-edmund.h.white@intel.com> <1434999372-3688-8-git-send-email-edmund.h.white@intel.com> <558ADCEE0200007800088FF7@mail.emea.novell.com> <558AEEA8.1000305@intel.com> <558BD42502000078000895BD@mail.emea.novell.com> <558C2E05.3040300@intel.com> <558D079F0200007800089F82@mail.emea.novell.com> <558D7D88.1050100@intel.com> <559ABC64.4080503@intel.com> <559AC926.7010707@eu.citrix.com> <559ACC64.5050705@intel.com> <559BA5A2.8080106@eu.citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <559BA5A2.8080106@eu.citrix.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: George Dunlap Cc: Ravi Sahita , Wei Liu , Ian Jackson , Tim Deegan , "xen-devel@lists.xen.org" , Jan Beulich , Andrew Cooper , tlengyel@novetta.com, Daniel De Graaf List-Id: xen-devel@lists.xenproject.org On 07/07/2015 03:10 AM, George Dunlap wrote: > 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). > I'm disappointed that you think that. I respect yours, Jan's, etc. role as maintainers, and your absolute right to reject anything you think is inappropriate. It's clear that Jan, and now apparently you, don't respect my abilities or desire to do good work. I won't make the changes you suggest because I don't think they represent good design, and I don't think they are the right way to solve the issue at hand. I'm going to hand the Intel end of further discussions off to Ravi Sahita. Ed