[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <562A19DA02000078000ADEE0@prv-mh.provo.novell.com>
Date: Fri, 23 Oct 2015 03:28:26 -0600
From: "Jan Beulich" <JBeulich@...e.com>
To: "Joe Jin" <joe.jin@...cle.com>
Cc: "Ian Campbell" <Ian.Campbell@...rix.com>,
"Paul Durrant" <Paul.Durrant@...rix.com>,
"Wei Liu" <wei.liu2@...rix.com>,
"David S. Miller" <davem@...emloft.net>,
"xen-devel@...ts.xenproject.org" <xen-devel@...ts.xenproject.org>,
"Boris Ostrovsky" <boris.ostrovsky@...cle.com>,
"Konrad Rzeszutek Wilk" <konrad.wilk@...cle.com>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>
Subject: Re: [Xen-devel] [PATCH 0/2] limit xen vnic max queues number
to online cpu number
>>> On 23.10.15 at 11:05, <joe.jin@...cle.com> wrote:
> On 10/23/2015 04:47 PM, Paul Durrant wrote:
>>> -----Original Message-----
>>> From: netdev-owner@...r.kernel.org [mailto:netdev-
>>> owner@...r.kernel.org] On Behalf Of Joe Jin
>>> Sent: 23 October 2015 08:54
>>> To: Wei Liu; Ian Campbell; Boris Ostrovsky; Konrad Rzeszutek Wilk; David S.
>>> Miller
>>> Cc: netdev@...r.kernel.org; xen-devel@...ts.xenproject.org
>>> Subject: [PATCH 0/2] limit xen vnic max queues number to online cpu
>>> number
>>>
>>> Currently xen vnic allowed to create lots of queues by set module param
>>> max_queues(both netback and netfront), when queues number larger than
>>> cpu number, it does not help for performance but need more cpu time.
>>>
>>
>> But it's an override, so why would you want to limit it? The parameter
> should not be set in the common case.
>
> Always we can not stop people use it because we provided it :)
Well, it's always a question of whether preventing the admin to
shoot himself in the foot makes sense: When it leads to an unusable
system, it probably does. When it leads to a sub-optimal working
system, perhaps we should allow them their freedom?
Jan
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists