From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-io0-f169.google.com (mail-io0-f169.google.com [209.85.223.169]) by dpdk.org (Postfix) with ESMTP id A52D45690 for ; Mon, 7 Sep 2015 22:52:54 +0200 (CEST) Received: by iofb144 with SMTP id b144so97572636iof.1 for ; Mon, 07 Sep 2015 13:52:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=JckWXk5mJkRyE6J446rSkKwDEUJmW23XcEYm/xXLy7s=; b=meL6kSgrPN5c9NmT3VE/mM+Oy+cJVxDr4x++Pz7vMUn/jVvTxCzw1kCsJJLM39ef2r n81b+edKGv1gczW6xaBrPMftkC5etSMbJ0NnGSnHDPe+Z48HBKtf4wXtbNYoEtoqgYol xMlCT6b3JfYQXyUD3lq0JlFbIZRC4GVg2/azc1t/pGkM0Ms/9cLA7HmeTmIeAtlV2k/G IE8UxkK0CmtVyJui3s2aEvsyzF3s+K1eHuy0YiE/kEaF+hM81EB2UY8HXIxSKBl1o8QA b/gJIhsWfRrXCwE3mVIIEspX0nbooIqq5Dlv18Ttcox+iBx0E8SSM1dsvGXJgIWzMRy6 FAAQ== MIME-Version: 1.0 X-Received: by 10.107.167.10 with SMTP id q10mr30985566ioe.66.1441659173984; Mon, 07 Sep 2015 13:52:53 -0700 (PDT) Received: by 10.79.109.22 with HTTP; Mon, 7 Sep 2015 13:52:53 -0700 (PDT) In-Reply-To: <1440807373-24770-1-git-send-email-marc.sune@bisdn.de> References: <1440804014-30815-1-git-send-email-marc.sune@bisdn.de> <1440807373-24770-1-git-send-email-marc.sune@bisdn.de> Date: Mon, 7 Sep 2015 22:52:53 +0200 Message-ID: From: Marc Sune To: Thomas Monjalon Content-Type: text/plain; charset=UTF-8 X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v4 0/2] ethdev: add port speed capability bitmap 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: Mon, 07 Sep 2015 20:52:55 -0000 2015-08-29 2:16 GMT+02:00 Marc Sune : > The current rte_eth_dev_info abstraction does not provide any mechanism to > get the supported speed(s) of an ethdev. > > For some drivers (e.g. ixgbe), an educated guess can be done based on the > driver's name (driver_name in rte_eth_dev_info), see: > > http://dpdk.org/ml/archives/dev/2013-August/000412.html > > However, i) doing string comparisons is annoying, and can silently > break existing applications if PMDs change their names ii) it does not > provide all the supported capabilities of the ethdev iii) for some drivers > it > is impossible determine correctly the (max) speed by the application > (e.g. in i40, distinguish between XL710 and X710). > > This small patch adds speed_capa bitmap in rte_eth_dev_info, which is > filled > by the PMDs according to the physical device capabilities. > > v2: rebase, converted speed_capa into 32 bits bitmap, fixed alignment > (checkpatch). > > v3: rebase to v2.1. unified ETH_LINK_SPEED and ETH_SPEED_CAP into > ETH_SPEED. > Converted field speed in struct rte_eth_conf to speeds, to allow a > bitmap > for defining the announced speeds, as suggested by M. Brorup. Fixed > spelling issues. > > v4: fixed errata in the documentation of field speeds of rte_eth_conf, and > commit 1/2 message. rebased to v2.1.0. v3 was incorrectly based on > ~2.1.0-rc1. > Thomas, Since mostly you were commenting for v1 and v2; any opinion on this one? Regards marc > > Marc Sune (2): > Added ETH_SPEED_ bitmap in rte_eth_dev_info > Filling speed capability bitmaps in the PMDs > > app/test-pmd/cmdline.c | 32 +++++++++++------------ > app/test/virtual_pmd.c | 2 +- > drivers/net/bonding/rte_eth_bond_8023ad.c | 14 +++++----- > drivers/net/cxgbe/base/t4_hw.c | 8 +++--- > drivers/net/e1000/em_ethdev.c | 20 +++++++++----- > drivers/net/e1000/igb_ethdev.c | 20 +++++++++----- > drivers/net/fm10k/fm10k_ethdev.c | 3 +++ > drivers/net/i40e/i40e_ethdev.c | 43 > +++++++++++++++++++------------ > drivers/net/i40e/i40e_ethdev_vf.c | 2 +- > drivers/net/ixgbe/ixgbe_ethdev.c | 32 +++++++++++++++-------- > drivers/net/mlx4/mlx4.c | 6 +++++ > drivers/net/vmxnet3/vmxnet3_ethdev.c | 2 +- > lib/librte_ether/rte_ethdev.h | 42 > +++++++++++++++++++++--------- > 13 files changed, 142 insertions(+), 84 deletions(-) > > -- > 2.1.4 > >