From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ed White Subject: Re: [PATCH v3 11/13] x86/altp2m: define and implement alternate p2m HVMOP types. Date: Mon, 06 Jul 2015 09:49:21 -0700 Message-ID: <559AB191.3020502@intel.com> References: <1435774177-6345-1-git-send-email-edmund.h.white@intel.com> <1435774177-6345-12-git-send-email-edmund.h.white@intel.com> <559A53D5.5030209@citrix.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <559A53D5.5030209@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: Andrew Cooper , xen-devel@lists.xen.org Cc: Ravi Sahita , Wei Liu , George Dunlap , Tim Deegan , Ian Jackson , Jan Beulich , tlengyel@novetta.com, Daniel De Graaf List-Id: xen-devel@lists.xenproject.org On 07/06/2015 03:09 AM, Andrew Cooper wrote: > On 01/07/15 19:09, Ed White wrote: >> Signed-off-by: Ed White > > I am still very much unconvinced by the argument against having a single > HVMOP_altp2m and a set of subops. do_domctl() and do_sysctl() are > examples of a subop style hypercall with different XSM settings for > different subops. > > Furthermore, factoring out a do_altp2m_op() handler would allow things > like the hvm_altp2m_supported() check to be made common. Factoring > further to having a named common header of a subop and a domid at the > head of every subop structure would allow all the domain rcu locking to > become common outside of the subop switch. > How do we get to a binding decision on whether making this change is a prerequisite for acceptance or not? Changing the HVMOP encoding means fairly extensive changes to the code in hvm.c, and the XSM patch, and the code Tamas has written. It also necessitates significant changes to all the code we use to test the intra-domain protection model. Feature freeze is Friday, and that's a lot to change, test, and get approved. Who owns the decision? Ed