From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH 2/3] usertools: add hv_uio_setup script Date: Thu, 5 Apr 2018 15:39:25 -0700 Message-ID: <20180405153925.7b571d2a@xeon-e3> References: <20180405191305.26813-1-sthemmin@microsoft.com> <1607753.vleKbttEpa@xps> <20180405140329.3595edb8@xeon-e3> <21769641.JDxLthIqDH@xps> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: Stephen Hemminger , dev@dpdk.org To: Thomas Monjalon Return-path: Received: from mail-pl0-f67.google.com (mail-pl0-f67.google.com [209.85.160.67]) by dpdk.org (Postfix) with ESMTP id AAFB81CDCB for ; Fri, 6 Apr 2018 00:39:28 +0200 (CEST) Received: by mail-pl0-f67.google.com with SMTP id bj1-v6so18911766plb.8 for ; Thu, 05 Apr 2018 15:39:28 -0700 (PDT) In-Reply-To: <21769641.JDxLthIqDH@xps> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Thu, 05 Apr 2018 23:13:54 +0200 Thomas Monjalon wrote: > 05/04/2018 23:03, Stephen Hemminger: > > On Thu, 05 Apr 2018 22:43:39 +0200 > > Thomas Monjalon wrote: > > > > > 05/04/2018 21:13, Stephen Hemminger: > > > > Small script to rebind netvsc kernel device to Hyper-V > > > > networking PMD. It could be integrated in dpdk-bind, but dpdk-bind > > > > is focused on PCI, and that would get messy. > > > > > > > > Eventually, this functionality will be built into netvsc driver > > > > (see vdev_netvsc as an example). > > > > > > I believe we should avoid creating such script. > > > The direction to go, for hotplug, is to remove dpdk-devbind.py, > > > and implement kernel binding in PMDs (with EAL helpers). > > > > > > In order to make this change happen, we should not > > > add this hv_uio_setup.sh script. > > > > Yes, this is a temporary script like dpdk-bind, want to get rid of it > > and do everything inside driver. That is the next step. > > If this is temporary, it is a step in the wrong direction which > could confuse users. > > If it becomes definitive (design discussion in progress), then it should > be merged in dpdk-devbind.py. Right now PCI does it the same way. It doesn't have a good cold plug interface. The whole PCI probe logic expects that the device will have a vfio/uio already bound. When PCI is fixed, then VMBUS can be changed to the same thing.