From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id AC216370 for ; Thu, 8 Dec 2016 18:50:38 +0100 (CET) Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga102.fm.intel.com with ESMTP; 08 Dec 2016 09:50:36 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,320,1477983600"; d="scan'208";a="15522434" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.237.220.29]) ([10.237.220.29]) by orsmga002.jf.intel.com with ESMTP; 08 Dec 2016 09:50:32 -0800 To: Shreyansh Jain , dev@dpdk.org References: <1480846288-2517-1-git-send-email-shreyansh.jain@nxp.com> <1480846288-2517-8-git-send-email-shreyansh.jain@nxp.com> Cc: thomas.monjalon@6wind.com, david.marchand@6wind.com, Adrien Mazarguil From: Ferruh Yigit Message-ID: <74db5496-6348-1f18-d4b8-457260cd6e97@intel.com> Date: Thu, 8 Dec 2016 17:50:30 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.5.1 MIME-Version: 1.0 In-Reply-To: <1480846288-2517-8-git-send-email-shreyansh.jain@nxp.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [PATCH 07/13] pci: replace probe and remove handlers with rte_driver 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: Thu, 08 Dec 2016 17:50:39 -0000 Hi Shreyansh, On 12/4/2016 10:11 AM, Shreyansh Jain wrote: > rte_pci_driver probe/remove callback are replaced with the rte_driver > based probe/remove. This patch changes all the PCI drivers which reference > rte_pci_driver->probe/remove. > > rte_pci_driver continues to have probe/remove callback which would be used > once eth_driver is removed in later patches. > > Only changes to PCI are done. VDEV changes require a different > model because of arguments being passed to vdev->probe/remove. > > Signed-off-by: Shreyansh Jain > --- > drivers/net/bnx2x/bnx2x_ethdev.c | 12 ++++++++---- > drivers/net/bnxt/bnxt_ethdev.c | 6 ++++-- > drivers/net/cxgbe/cxgbe_ethdev.c | 6 ++++-- > drivers/net/e1000/em_ethdev.c | 6 ++++-- > drivers/net/e1000/igb_ethdev.c | 12 ++++++++---- > drivers/net/ena/ena_ethdev.c | 6 ++++-- > drivers/net/enic/enic_ethdev.c | 6 ++++-- > drivers/net/fm10k/fm10k_ethdev.c | 6 ++++-- > drivers/net/i40e/i40e_ethdev.c | 6 ++++-- > drivers/net/i40e/i40e_ethdev_vf.c | 6 ++++-- > drivers/net/ixgbe/ixgbe_ethdev.c | 12 ++++++++---- > drivers/net/mlx4/mlx4.c | 6 ++++-- Generating compile error: .../net/mlx4/mlx4.c:5913:13: error: subobject initialization overrides initialization of other fields within its enclosing subobject [-Werror,-Winitializer-overrides] .driver = { ^ .../net/mlx4/mlx4.c:5909:13: note: previous initialization is here .driver = { ^ > drivers/net/mlx5/mlx5.c | 4 +++- .../net/mlx5/mlx5.c:736:13: error: subobject initialization overrides initialization of other fields within its enclosing subobject [-Werror,-Winitializer-overrides] .driver = { ^ .../net/mlx5/mlx5.c:732:13: note: previous initialization is here .driver = { ^ > drivers/net/nfp/nfp_net.c | 6 ++++-- > drivers/net/qede/qede_ethdev.c | 16 ++++++++++------ > drivers/net/szedata2/rte_eth_szedata2.c | 6 ++++-- > drivers/net/virtio/virtio_ethdev.c | 4 ++-- > drivers/net/vmxnet3/vmxnet3_ethdev.c | 6 ++++-- Following drivers seems missing: drivers/net/thunderx/nicvf_ethdev.c Also new added drivers/net/sfc/sfc_ethdev.c, but since sfc is not merged to main branch yet, perhaps fix may wait your patches to be applied first. > lib/librte_eal/common/eal_common_pci.c | 10 ++++++++-- > lib/librte_eal/common/include/rte_dev.h | 12 ++++++++++++ > lib/librte_ether/rte_ethdev.c | 16 ++++++++++++---- > lib/librte_ether/rte_ethdev.h | 6 +++--- > 22 files changed, 122 insertions(+), 54 deletions(-) > Also getting following with mlx5: In file included from .../x86_64-native-linuxapp-gcc/include/rte_mbuf.h:57:0, from .../x86_64-native-linuxapp-gcc/include/rte_ether.h:52, from .../drivers/net/mlx5/mlx5_trigger.c:38: /usr/include/infiniband/verbs.h: In function ‘verbs_get_device’: .../x86_64-native-linuxapp-gcc/include/rte_common.h:350:40: error: initialization discards ‘const’ qualifier from pointer target type [-Werror=discarded-qualifiers] typeof(((type *)0)->member) *_ptr = (ptr); \ <...>