From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ian Campbell Subject: Re: [OSSTEST Nested PATCH v9 5/9] Move the codes about memory size setting into prep() Date: Wed, 6 May 2015 11:57:07 +0100 Message-ID: <1430909827.2660.221.camel@citrix.com> References: <1430548098-5233-1-git-send-email-longtaox.pang@intel.com> <1430548098-5233-6-git-send-email-longtaox.pang@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1430548098-5233-6-git-send-email-longtaox.pang@intel.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: "longtao.pang" Cc: wei.liu2@citrix.com, robert.hu@intel.com, Ian.Jackson@eu.citrix.com, xen-devel@lists.xen.org List-Id: xen-devel@lists.xenproject.org On Sat, 2015-05-02 at 14:28 +0800, longtao.pang wrote: > From: "longtao.pang" > > Move the codes about memory size setting from outside into prep(), "code for setting memory size" > where these $ram_mb operations are only used; and in next patch, > for L2 guest installation, $ram_mb will be override by a runvar value, "overridden" > which needs $gho in scope. It is useful to say explicitly: This is pure code motion. No functional change. > Signed-off-by: longtao.pang With those minor changes: Acked-by: Ian Campbell