From mboxrd@z Thu Jan 1 00:00:00 1970 From: Konrad Rzeszutek Wilk Subject: Re: [PATCH 1/2] xl: neuter vcpu-set --ignore-host. Date: Thu, 26 Sep 2013 08:45:05 -0400 Message-ID: <20130926124505.GD5792@konrad-lan.dumpdata.com> References: <1380141617-8981-1-git-send-email-konrad.wilk@oracle.com> <1380141617-8981-2-git-send-email-konrad.wilk@oracle.com> <1380180208.30061.4.camel@Abyss> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Content-Disposition: inline In-Reply-To: <1380180208.30061.4.camel@Abyss> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Dario Faggioli Cc: George.Dunlap@eu.citrix.com, xen-devel@lists.xensource.com, Ian.Jackson@eu.citrix.com, Ian.Campbell@citrix.com List-Id: xen-devel@lists.xenproject.org On Thu, Sep 26, 2013 at 09:23:28AM +0200, Dario Faggioli wrote: > On mer, 2013-09-25 at 16:40 -0400, Konrad Rzeszutek Wilk wrote: > > This patch keeps said option but neuters the check so that we > > can overcommit. In other words - by default the user is > > allowed to set as many vCPUs as they would like. > > > What about use the parameter to silence the warning? I mean, by default > we allow more vCPUs than vCPUs and print the warning. With '-i' we allow > that too (of course) and _do_not_ print the warning. Good point. Let me redo it that way. > > It's definitely not a big deal, it's just one way of not having a > completely useless and neglected param around... It materialized in my > mind while reading the description of the change, and I felt like I was > sharing it. :-) > > Regards, > Dario > > -- > <> (Raistlin Majere) > ----------------------------------------------------------------- > Dario Faggioli, Ph.D, http://about.me/dario.faggioli > Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) >