From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by dpdk.org (Postfix) with ESMTP id 443952C7A for ; Tue, 10 Feb 2015 14:22:27 +0100 (CET) Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by orsmga103.jf.intel.com with ESMTP; 10 Feb 2015 05:17:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,550,1418112000"; d="scan'208";a="452576831" Received: from bricha3-mobl3.ger.corp.intel.com ([10.243.20.46]) by FMSMGA003.fm.intel.com with SMTP; 10 Feb 2015 05:07:47 -0800 Received: by (sSMTP sendmail emulation); Tue, 10 Feb 2015 13:22:21 +0025 Date: Tue, 10 Feb 2015 13:22:21 +0000 From: Bruce Richardson To: Marc Sune Message-ID: <20150210132220.GD18684@bricha3-MOBL3> References: <1415358037-424-1-git-send-email-marc.sune@bisdn.de> <54D9F2A1.2040102@bisdn.de> <20150210120230.GC18684@bricha3-MOBL3> <54D9F7E3.2060504@bisdn.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <54D9F7E3.2060504@bisdn.de> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] Adding RTE_KNI_PREEMPT configuration option X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 10 Feb 2015 13:22:27 -0000 On Tue, Feb 10, 2015 at 01:21:55PM +0100, Marc Sune wrote: > > On 10/02/15 13:02, Bruce Richardson wrote: > >On Tue, Feb 10, 2015 at 12:59:29PM +0100, Marc Sune wrote: > >>This patch of Nov 2014 hasn't been yet ACKed/NACKed. Could someone please > >>give some quick feedback? > >> > >>Thanks > >>marc > >> > >Idea is good, any chance it could be added as a run-time rather than > >compile-time option? > > It is also an option. I wasn't really thinking someone would want to change > this behaviour at runtime. If we think it is worth, I can have a closer look > on it. Any other opinions on this? > > If we would go for a runtime flag, we would either have to add a config > parameter to rte_kni_init() or add a specific call to turn on/off this knob, > depending on whether it is sufficient to change this behaviour at > bootstrapping time, or we want to also change it during 'operation'. In > either case we would require some communication, probably via ioctl(), from > user-space to kernel space. > > Thanks > Marc > Yes, I can't see it needing to be changed much at runtime, however, we may need to take account of those who are using precompiled DPDK libs. For those using prebuilt DPDK libs, they won't have any ability to modify compile-time values. However, that being said, I have no particular objection to taking this change in as-is for now. It only adds something, rather than taking it away. Regards, /Bruce