From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id D6C19152A for ; Mon, 9 Jan 2017 17:18:55 +0100 (CET) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by orsmga105.jf.intel.com with ESMTP; 09 Jan 2017 08:18:54 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,340,1477983600"; d="scan'208";a="1091791221" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.38]) ([10.237.220.38]) by fmsmga001.fm.intel.com with ESMTP; 09 Jan 2017 08:18:52 -0800 To: Shreyansh Jain , david.marchand@6wind.com References: <1482756644-13726-1-git-send-email-shreyansh.jain@nxp.com> <1482758645-23057-1-git-send-email-shreyansh.jain@nxp.com> <1482758645-23057-12-git-send-email-shreyansh.jain@nxp.com> <8484fc57-baa7-0f84-bc05-49d7f4ca79ad@intel.com> Cc: dev@dpdk.org, thomas.monjalon@6wind.com From: Ferruh Yigit Message-ID: <03401dd4-5d78-6ad2-e199-c076495e186d@intel.com> Date: Mon, 9 Jan 2017 16:18:52 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: <8484fc57-baa7-0f84-bc05-49d7f4ca79ad@intel.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH v5 11/12] drivers: update PMDs to use rte_driver probe and remove 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, 09 Jan 2017 16:18:56 -0000 On 1/9/2017 3:19 PM, Ferruh Yigit wrote: > On 12/26/2016 1:24 PM, Shreyansh Jain wrote: >> These callbacks now act as first layer of PCI interfaces from the Bus. >> Bus probe would enter the PMDs through the rte_driver->probe/remove >> callbacks, falling to rte_xxx_driver->probe/remove (Currently, all the >> drivers are rte_pci_driver). >> >> This patch also changes QAT which is the only crypto PMD based on PCI. >> All others would be changed in a separate patch focused on VDEV. >> >> Signed-off-by: Shreyansh Jain >> --- <...> >> >> diff --git a/drivers/crypto/qat/rte_qat_cryptodev.c b/drivers/crypto/qat/rte_qat_cryptodev.c >> index 1e7ee61..bc1a9c6 100644 >> --- a/drivers/crypto/qat/rte_qat_cryptodev.c >> +++ b/drivers/crypto/qat/rte_qat_cryptodev.c >> @@ -120,6 +120,10 @@ crypto_qat_dev_init(__attribute__((unused)) struct rte_cryptodev_driver *crypto_ >> >> static struct rte_cryptodev_driver rte_qat_pmd = { >> .pci_drv = { >> + .driver = { >> + .probe = rte_eal_pci_probe, >> + .remove = rte_eal_pci_remove, >> + }, > > Since this part is common for all PCI drivers, why not make this part of > RTE_PMD_REGISTER_PCI macro? > I have seen your comment [1] which looks like better idea. providing a rte_bus->probe(), and rte_eal_bus_probe() calls rte_bus->probe() for pci devices, rte_bus->probe = rte_eal_pci_probe .. [1] http://dpdk.org/ml/archives/dev/2017-January/054125.html Thanks, ferruh