From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wr0-f194.google.com (mail-wr0-f194.google.com [209.85.128.194]) by dpdk.org (Postfix) with ESMTP id 22AD0591E for ; Mon, 27 Feb 2017 14:09:18 +0100 (CET) Received: by mail-wr0-f194.google.com with SMTP id l37so1471991wrc.3 for ; Mon, 27 Feb 2017 05:09:18 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:sender:in-reply-to:references:from:date:message-id :subject:to:cc; bh=oKJ9X3lXcZNDw2rzsWPh2O1/kvokhBifgFTlRUgHkxA=; b=kV+ZQd47Jxu1BVqDqmesiQvrjnBbNNkhzxhlgnW6AaGDElIH+oWs89dOjjZqym2v6j h+8DN8uke7ggqgwedmhHIeiqCzmaCSg/nM0hwGC+bScuz+D5fasLE4CjUJ0yHqz6GDfA kgKdrwir5OZgOBHDDy5CPzAMX/1cTB2F55aYFztgTjoF8ddO6sNlUrXnrPRTQIdT/iel g5oaojVV9DR39gMPwM6MQC9IEqxqForVjgIWo/61SrHlL47oqN9dtnY7x8Bw5K+qFnjJ WvdUeA6lHAEn/xH2hYlLJORlMvPVK92MwnLxcnCwu7Kibo7gXgnBj3q28yD+tF3g4aQf OGCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:sender:in-reply-to:references:from :date:message-id:subject:to:cc; bh=oKJ9X3lXcZNDw2rzsWPh2O1/kvokhBifgFTlRUgHkxA=; b=shVHLDIw8iwNyVG/9/m43tPIjvp8aW52ZqJg9HdCMzjvQAwygsdBXa7LhRiwMiniKL eWd/tZMg3GIRgQkEGTHODGdf2S1lRW4V1OZ9LXCbOHmlMViUrI6SZ29E9h5BZRKGN9rI kHefotP6/WxTrYnFn2JLGdu/e1pEn9c5tbwRiNvgqPm6XY0BfhO4arl/bdFW/ZJbqpc2 jfAOcc8y7B0/D0tnhCr67k/3cOzZjCNXWL+2611apcQXtvCo8Z2JyTobu1sRFTRAlCYU awj/Km2ohNpv78ExG1y8hujfX+DoGL11mRp4t3GdkGIXGZyx3K4yssBbCSEdYOXoMm8t jMEg== X-Gm-Message-State: AMke39nBCnsT7j8Ojc1RlEdrTbvmOaDj1tuE37pQ7ZHexb0DHNQ31bX2bV6m+a9ZN+i5rdrNvj54gc3zeprzWA== X-Received: by 10.223.132.166 with SMTP id 35mr13804320wrg.122.1488200957591; Mon, 27 Feb 2017 05:09:17 -0800 (PST) MIME-Version: 1.0 Sender: jblunck@gmail.com Received: by 10.28.211.20 with HTTP; Mon, 27 Feb 2017 05:09:16 -0800 (PST) In-Reply-To: <1488018496-995-1-git-send-email-jblunck@infradead.org> References: <1487600262-14382-1-git-send-email-jblunck@infradead.org> <1488018496-995-1-git-send-email-jblunck@infradead.org> From: Jan Blunck Date: Mon, 27 Feb 2017 14:09:16 +0100 X-Google-Sender-Auth: -tO1zXFpp5iWZks54ll9R88zq_4 Message-ID: To: dev Cc: David Marchand , Shreyansh Jain , Ferruh Yigit Content-Type: text/plain; charset=UTF-8 Subject: Re: [dpdk-dev] [PATCH v3 00/10] Rework vdev probing to use rte_bus infrastructure X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 27 Feb 2017 13:09:18 -0000 On Sat, Feb 25, 2017 at 11:28 AM, Jan Blunck wrote: > With the rte_bus infrastructure present in 17.02 it is possible to refactor > the virtual device probing into a bus. This series also introduces the > rte_vdev_device to better keep track of devices. > > This patchset depends on: > http://dpdk.org/dev/patchwork/patch/20416/ > http://dpdk.org/dev/patchwork/patch/20417/ > > Changes since version 2: > * implicit bus registration through rte_eal_vdrv_register() On a second thought I don't think that this is correct though since it opens up the possibility of racing an alternative "virtual" bus. I don't think that this is a good thing though. I'll fix this in v4. Thoughts? Thanks, Jan > * explicit delay probing of virtual bus in rte_bus_probe() > * addition of rte_vdev_device_args() helper > * make virtual driver probe and remove take rte_vdev_device > > Changes since version 1: > * addition of rte_vdev_device_name() helper > * removed rte_eal_dev_init() from *.map files > * use SOCKET_ID_ANY > > Jan Blunck (10): > eal: probe legacy PCI devices before other bus devices > eal: probe new virtual bus after other bus devices > eal: move virtual device probing into a bus > eal: remove unused rte_eal_dev_init() > eal: Refactor vdev driver probe/remove > eal: add struct rte_vdev_device > eal: add virtual device name helper function > eal: add virtual device arguments helper function > eal: make virtual bus use rte_vdev_device > eal: make virtual driver probe and remove take rte_vdev_device > > drivers/crypto/null/null_crypto_pmd.c | 18 +- > drivers/net/af_packet/rte_eth_af_packet.c | 11 +- > drivers/net/bonding/rte_eth_bond_pmd.c | 13 +- > drivers/net/mpipe/mpipe_tilegx.c | 10 +- > drivers/net/null/rte_eth_null.c | 13 +- > drivers/net/pcap/rte_eth_pcap.c | 12 +- > drivers/net/ring/rte_eth_ring.c | 9 +- > drivers/net/tap/rte_eth_tap.c | 10 +- > drivers/net/vhost/rte_eth_vhost.c | 10 +- > drivers/net/virtio/virtio_user_ethdev.c | 18 +- > drivers/net/xenvirt/rte_eth_xenvirt.c | 9 +- > lib/librte_eal/bsdapp/eal/eal.c | 9 +- > lib/librte_eal/bsdapp/eal/rte_eal_version.map | 1 - > lib/librte_eal/common/eal_common_bus.c | 16 +- > lib/librte_eal/common/eal_common_dev.c | 28 --- > lib/librte_eal/common/eal_common_vdev.c | 259 +++++++++++++++++++++--- > lib/librte_eal/common/include/rte_dev.h | 5 - > lib/librte_eal/common/include/rte_vdev.h | 26 ++- > lib/librte_eal/linuxapp/eal/eal.c | 9 +- > lib/librte_eal/linuxapp/eal/rte_eal_version.map | 1 - > 20 files changed, 355 insertions(+), 132 deletions(-) > > -- > 2.7.4 >