From mboxrd@z Thu Jan 1 00:00:00 1970 From: ZhouPeng Subject: Re: [PATCH] libxl: Support linux-stubdom in libxl Date: Mon, 6 Jun 2011 19:56:52 +0800 Message-ID: References: <1306933116.775.237.camel@zakaz.uk.xensource.com> <1307024747.775.275.camel@zakaz.uk.xensource.com> <1307025357.775.277.camel@zakaz.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <1307025357.775.277.camel@zakaz.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Ian Campbell Cc: Samuel Thibault , "Xen-devel@lists.xensource.com" , Jiageng Yu , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org I think class is better than mode? 'Mode' is somewhat misleading to new newbie? It seem be telling there is one device model instance, which has mutiple working mode. Just like kernel mode or user mode to describe one OS=91s current working m= ode. 1) device_model_class 2) device_model_deployment 3) device_model_instance 4) device_model_mode If I have one vote for all the above chances, I will select '...class' :) And I suggest one : device_modele_mechanism :) 2011/6/2 Ian Campbell : > On Thu, 2011-06-02 at 15:31 +0100, Stefano Stabellini wrote: >> On Thu, 2 Jun 2011, Ian Campbell wrote: >> > On Thu, 2011-06-02 at 14:40 +0100, Jiageng Yu wrote: >> > > diff -r 37c77bacb52a tools/libxl/libxl.idl >> > > --- a/tools/libxl/libxl.idl =A0 =A0Mon May 23 17:38:28 2011 +0100 >> > > +++ b/tools/libxl/libxl.idl =A0 =A0Wed Jun 01 03:24:57 2011 +0100 >> > > @@ -196,6 +196,7 @@ >> > > =A0 =A0 =A0("dom_name", =A0 =A0 =A0 =A0 string), >> > > =A0 =A0 =A0("device_model_version", libxl_device_model_version), >> > > =A0 =A0 =A0("device_model_stubdomain", bool), >> > > + =A0 ("device_model_linux_stubdomain", bool), >> > > =A0 =A0 =A0("device_model", =A0 =A0 string, False, "if you set this = you must set device_model_version too"), >> > > =A0 =A0 =A0("saved_state", =A0 =A0 =A0string), >> > > =A0 =A0 =A0("type", =A0 =A0 =A0 =A0 =A0 =A0 libxl_domain_type), >> > >> > I think what we actually want here is a single device_model_type >> > Enumeration, values are something like "process", "stub-linux", >> > "stub-minios", rather than multiple device_model_XXX_stubdom booleans. >> >> indeed >> >> >> > I'm not convinced device_model_type is a good name, hopefully someone >> > can suggest something better. (device_model_mode??) >> >> some suggestions: >> >> 1) device_model_class >> 2) device_model_deployment >> 3) device_model_instance >> >> I vote for 3) > > I don't think deployment or instance has the right meaning here. class > is better but still doesn't feel right. > > maybe ..._mode? > > > > Ian. > > > _______________________________________________ > Xen-devel mailing list > Xen-devel@lists.xensource.com > http://lists.xensource.com/xen-devel > --=20 Zhou Peng Operating System Technology Group Institute of Software, the Chinese Academy of Sciences (ISCAS)