From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 7FAC38D39 for ; Thu, 19 May 2016 09:58:26 +0200 (CEST) Received: from int-mx14.intmail.prod.int.phx2.redhat.com (int-mx14.intmail.prod.int.phx2.redhat.com [10.5.11.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id BC93B7AE91; Thu, 19 May 2016 07:58:25 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org (vpn1-5-221.ams2.redhat.com [10.36.5.221]) by int-mx14.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id u4J7wNk3018657; Thu, 19 May 2016 03:58:24 -0400 To: Neil Horman , dev@dpdk.org References: <1463431287-4551-1-git-send-email-nhorman@tuxdriver.com> <1463605687-649-1-git-send-email-nhorman@tuxdriver.com> <1463605687-649-3-git-send-email-nhorman@tuxdriver.com> Cc: Bruce Richardson , Thomas Monjalon , Stephen Hemminger From: Panu Matilainen Message-ID: Date: Thu, 19 May 2016 10:58:23 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.0 MIME-Version: 1.0 In-Reply-To: <1463605687-649-3-git-send-email-nhorman@tuxdriver.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.27 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.25]); Thu, 19 May 2016 07:58:25 +0000 (UTC) Subject: Re: [dpdk-dev] [PATCHv2 2/4] drivers: Update driver registration macro usage 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: Thu, 19 May 2016 07:58:26 -0000 On 05/19/2016 12:08 AM, Neil Horman wrote: > Modify the PMD_REGISTER_DRIVER macro, bifurcating it into two > (PMD_REGISTER_DRIVER_PDEV and PMD_REGISTER_DRIVER_VDEV. Both of these do the > same thing the origional macro did, but both add the definition of a string > variable that informs interested parties of the name of the pmd, and the former > also defines an second string that holds the symbol name of the pci table that > is registered by this pmd. > > pmdinfo uses this information to extract hardware support from an object file > and create a json string to make hardware support info discoverable later. > > Signed-off-by: Neil Horman > CC: Bruce Richardson > CC: Thomas Monjalon > CC: Stephen Hemminger > CC: Panu Matilainen > --- > drivers/Makefile | 2 ++ > drivers/crypto/aesni_gcm/aesni_gcm_pmd.c | 4 +++- > drivers/crypto/aesni_mb/rte_aesni_mb_pmd.c | 4 +++- > drivers/crypto/null/null_crypto_pmd.c | 4 +++- > drivers/crypto/qat/rte_qat_cryptodev.c | 4 +++- > drivers/crypto/snow3g/rte_snow3g_pmd.c | 4 +++- > drivers/net/af_packet/rte_eth_af_packet.c | 4 +++- > drivers/net/bnx2x/bnx2x_ethdev.c | 6 ++++-- > drivers/net/bonding/rte_eth_bond_pmd.c | 7 ++++++- > drivers/net/cxgbe/cxgbe_ethdev.c | 4 +++- > drivers/net/e1000/em_ethdev.c | 3 ++- > drivers/net/e1000/igb_ethdev.c | 6 ++++-- > drivers/net/ena/ena_ethdev.c | 3 ++- > drivers/net/enic/enic_ethdev.c | 3 ++- > drivers/net/fm10k/fm10k_ethdev.c | 3 ++- > drivers/net/i40e/i40e_ethdev.c | 3 ++- > drivers/net/i40e/i40e_ethdev_vf.c | 3 ++- > drivers/net/ixgbe/ixgbe_ethdev.c | 6 ++++-- > drivers/net/mlx4/mlx4.c | 3 ++- > drivers/net/mlx5/mlx5.c | 3 ++- > drivers/net/mpipe/mpipe_tilegx.c | 4 ++-- > drivers/net/nfp/nfp_net.c | 3 ++- > drivers/net/null/rte_eth_null.c | 3 ++- > drivers/net/pcap/rte_eth_pcap.c | 4 +++- > drivers/net/ring/rte_eth_ring.c | 3 ++- > drivers/net/szedata2/rte_eth_szedata2.c | 3 ++- > drivers/net/vhost/rte_eth_vhost.c | 3 ++- > drivers/net/virtio/virtio_ethdev.c | 3 ++- > drivers/net/vmxnet3/vmxnet3_ethdev.c | 3 ++- > drivers/net/xenvirt/rte_eth_xenvirt.c | 2 +- > lib/librte_eal/common/include/rte_dev.h | 20 ++++++++++++++++---- > 31 files changed, 93 insertions(+), 37 deletions(-) > drivers/net/qede is missing and causes a build failure with a fresh config. It seems to be missing in v1 but I managed to test it, guess it must've been an old .config generated before QEDE got merged. - Panu -