From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ed White Subject: Re: [PATCH v3 12/13] x86/altp2m: Add altp2mhvm HVM domain parameter. Date: Mon, 06 Jul 2015 11:01:27 -0700 Message-ID: <559AC277.8050501@intel.com> References: <1435774177-6345-1-git-send-email-edmund.h.white@intel.com> <1435774177-6345-13-git-send-email-edmund.h.white@intel.com> <20150706174954.GJ1134@zion.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20150706174954.GJ1134@zion.uk.xensource.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: Wei Liu Cc: Ravi Sahita , George Dunlap , 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/06/2015 10:49 AM, Wei Liu wrote: >> diff --git a/tools/libxl/libxl_types.idl b/tools/libxl/libxl_types.idl >> index 23f27d4..66a89cf 100644 >> --- a/tools/libxl/libxl_types.idl >> +++ b/tools/libxl/libxl_types.idl >> @@ -437,6 +437,7 @@ libxl_domain_build_info = Struct("domain_build_info",[ >> ("mmio_hole_memkb", MemKB), >> ("timer_mode", libxl_timer_mode), >> ("nested_hvm", libxl_defbool), >> + ("altp2mhvm", libxl_defbool), > > It's redundant to have "hvm" in the name of this field. Calling it > "altp2m" would be fine IMHO. > When I originally started writing this code, I modelled the naming and some of the structure on nestedhvm, which is why so many things had hvm in the name. I've now removed a lot of those, but in this instance I wonder if doing so would cause confusion, since we have a Xen command-line parameter called altp2m. Ed