Re: [PATCH 1/2] xl: neuter vcpu-set --ignore-host.

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

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
> 
> -- 
> <<This happens because I choose it to happen!>> (Raistlin Majere)
> -----------------------------------------------------------------
> Dario Faggioli, Ph.D, http://about.me/dario.faggioli
> Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK)
>