From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id 169995A83 for ; Tue, 3 Mar 2015 12:43:39 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga101.fm.intel.com with ESMTP; 03 Mar 2015 03:43:38 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.09,681,1418112000"; d="scan'208";a="693214213" Received: from bricha3-mobl3.ger.corp.intel.com ([10.243.20.27]) by orsmga002.jf.intel.com with SMTP; 03 Mar 2015 03:43:36 -0800 Received: by (sSMTP sendmail emulation); Tue, 03 Mar 2015 11:43:35 +0025 Date: Tue, 3 Mar 2015 11:43:35 +0000 From: Bruce Richardson To: Ouyang Changchun Message-ID: <20150303114334.GA4076@bricha3-MOBL3> References: <1424926669-14202-1-git-send-email-changchun.ouyang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1424926669-14202-1-git-send-email-changchun.ouyang@intel.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH] tools/dpdk_nic_bind: Fix can't bind virtio-pci issue 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, 03 Mar 2015 11:43:40 -0000 On Thu, Feb 26, 2015 at 12:57:49PM +0800, Ouyang Changchun wrote: First off, I think the title needs to be changed. How about something like: "dpdk_nic_bind: don't exit if an unused module is missing" > In virtio test, on guest > 1. Bind virtio port to igb_uio driver; > 2. Remove igb_uio module; > 3. Bind virtio port to virtio-pci driver, it fails and reports: > "Error - no supported modules are loaded" > > The tool should check the to-be-bound driver flag, if it is dpdk driver(igb_uio, vfio etc), > and the corresponding module is not loaded, then exit, otherwise, just report a warning, > and continue to bind the non-dpdk driver(like virtio-pci) to dev. This is a good description of the problem. Once you change the title, I think an initial sentence is needed here by way of intro - such as: "The dpdk_nic_bind script will not allow ports to be bound or unbound if none of the kernel modules supported by DPDK is loaded. This patch relaxes this restriction by checking if a DPDK module is actually requested. The example below illustrates this problem:" > > Signed-off-by: Changchun Ouyang > --- > tools/dpdk_nic_bind.py | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/tools/dpdk_nic_bind.py b/tools/dpdk_nic_bind.py > index 2483056..8523f82 100755 > --- a/tools/dpdk_nic_bind.py > +++ b/tools/dpdk_nic_bind.py > @@ -175,8 +175,11 @@ def check_modules(): > > # check if we have at least one loaded module > if True not in [mod["Found"] for mod in mods] and b_flag is not None: > - print "Error - no supported modules are loaded" > - sys.exit(1) > + if b_flag in dpdk_drivers: > + print "Error - no supported modules(DPDK driver) are loaded" > + sys.exit(1) > + else: > + print "Warning - no supported modules(DPDK driver) are loaded" > > # change DPDK driver list to only contain drivers that are loaded > dpdk_drivers = [mod["Name"] for mod in mods if mod["Found"]] > -- > 1.8.4.2 > This is a worthwhile change. However, I think it could be made better: * If we try to bind a device to ixgbe, and ixgbe is not loaded, we get no error from the script * If igb_uio is loaded and we try to bind a device to vfio which is not loaded again we get no error from the script. If would be nice if instead of this checking explicitly for DPDK modules, we just check if the module to be bound is present and give an error if not. /Bruce