Message ID | 329CA03F-B2B1-4187-945D-59973EEDF368@cisco.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Yuanhan Liu |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [IPv6:::1]) by dpdk.org (Postfix) with ESMTP id 3C21A5A3E; Thu, 2 Feb 2017 13:05:12 +0100 (CET) Received: from rcdn-iport-7.cisco.com (rcdn-iport-7.cisco.com [173.37.86.78]) by dpdk.org (Postfix) with ESMTP id 062122C39 for <dev@dpdk.org>; Thu, 2 Feb 2017 13:05:09 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=942; q=dns/txt; s=iport; t=1486037110; x=1487246710; h=from:to:subject:date:message-id:references:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=mRV3NAxfKGVPdMDdXCoLswCon1ZAbbLlGeQprFX2lcA=; b=IcjRddhNJKJtxLZK5LV0A/fhbtwVlMje42Nn7E+dzBgJ6L0PiTkNLt2R /vQfMiLjFBeRy2oUYkrs5U2UkftanNpGkbcflg1I2Z0mnxN2xRENhGMl7 cp8AnOuoTuj66rI80vXtVARnATSAzz4kQr5TwRoqDz6khI31AontMXtqQ 0=; X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: A0CfBACSH5NY/5xdJa1dGQEBAQEBAQEBAQEBBwEBAQEBg1OBageDUJtzH5Mmgg+CDYYiAhqCMEAXAQIBAQEBAQEBYh0LhGoGIwQNVQIBHQUCJgICAjAVEAIEE4lxrRyBazqLLwEBAQEBAQEBAQEBAQEBAQEBAQEBAR2BC4VAggUIh0aCby6CMQWbXQGSBoFjjx0CSJI+ASABNoFLFUwBhjB1h3cBgQsBAQE X-IronPort-AV: E=Sophos;i="5.33,324,1477958400"; d="scan'208";a="201854350" Received: from rcdn-core-5.cisco.com ([173.37.93.156]) by rcdn-iport-7.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 02 Feb 2017 12:05:08 +0000 Received: from XCH-ALN-011.cisco.com (xch-aln-011.cisco.com [173.36.7.21]) by rcdn-core-5.cisco.com (8.14.5/8.14.5) with ESMTP id v12C58D8018831 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL) for <dev@dpdk.org>; Thu, 2 Feb 2017 12:05:08 GMT Received: from xch-aln-011.cisco.com (173.36.7.21) by XCH-ALN-011.cisco.com (173.36.7.21) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Thu, 2 Feb 2017 06:05:07 -0600 Received: from xch-aln-011.cisco.com ([173.36.7.21]) by XCH-ALN-011.cisco.com ([173.36.7.21]) with mapi id 15.00.1210.000; Thu, 2 Feb 2017 06:05:07 -0600 From: "Ido Barnea (ibarnea)" <ibarnea@cisco.com> To: "dev@dpdk.org" <dev@dpdk.org> Thread-Topic: [PATCH] Fill speed_capa for virtio Thread-Index: AQHSfTYZX54byr33jUq0qHryTKJNZqFWJN8A Date: Thu, 2 Feb 2017 12:05:07 +0000 Message-ID: <329CA03F-B2B1-4187-945D-59973EEDF368@cisco.com> References: <20170202092357.58508-1-ibarnea@cisco.com> In-Reply-To: <20170202092357.58508-1-ibarnea@cisco.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-messagesentrepresentingtype: 1 x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [64.103.125.48] Content-Type: text/plain; charset="utf-8" Content-ID: <AAF8C121FF4D4F4EB50A4F04E4E2BE14@emea.cisco.com> Content-Transfer-Encoding: base64 MIME-Version: 1.0 Subject: [dpdk-dev] [PATCH] Fill speed_capa for virtio X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <http://dpdk.org/ml/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://dpdk.org/ml/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <http://dpdk.org/ml/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Checks
Context | Check | Description |
---|---|---|
ci/Intel compilation | success | Compilation OK |
Commit Message
Ido Barnea (ibarnea)
Feb. 2, 2017, 12:05 p.m. UTC
From: Ido Barnea <ibarnea@cisco.com> Signed-off-by: Ido Barnea <ibarnea@cisco.com> --- drivers/net/virtio/virtio_ethdev.c | 1 + 1 file changed, 1 insertion(+)
Comments
2017-02-02 12:05, Ido Barnea: > From: Ido Barnea <ibarnea@cisco.com> > > Signed-off-by: Ido Barnea <ibarnea@cisco.com> > --- > drivers/net/virtio/virtio_ethdev.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c > index d1ff234..1d572b5 100644 > --- a/drivers/net/virtio/virtio_ethdev.c > +++ b/drivers/net/virtio/virtio_ethdev.c > @@ -1869,6 +1869,7 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) > (1ULL << VIRTIO_NET_F_HOST_TSO6); > if ((hw->guest_features & tso_mask) == tso_mask) > dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO; > + dev_info->speed_capa = ETH_LINK_SPEED_10G; Why 10G ? Yuanhan, any opinion?
Just wanted this to be consistent with below (From virtio_dev_link_update): link.link_speed = SPEED_10G; Need to put some value in speed_capa. Ido On 09/02/2017, 6:19 PM, "Thomas Monjalon" <thomas.monjalon@6wind.com> wrote: >2017-02-02 12:05, Ido Barnea: >> From: Ido Barnea <ibarnea@cisco.com> >> >> Signed-off-by: Ido Barnea <ibarnea@cisco.com> >> --- >> drivers/net/virtio/virtio_ethdev.c | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c >> index d1ff234..1d572b5 100644 >> --- a/drivers/net/virtio/virtio_ethdev.c >> +++ b/drivers/net/virtio/virtio_ethdev.c >> @@ -1869,6 +1869,7 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) >> (1ULL << VIRTIO_NET_F_HOST_TSO6); >> if ((hw->guest_features & tso_mask) == tso_mask) >> dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO; >> + dev_info->speed_capa = ETH_LINK_SPEED_10G; > >Why 10G ? >Yuanhan, any opinion?
2017-02-10 07:49, Ido Barnea: > On 09/02/2017, 6:19 PM, "Thomas Monjalon" <thomas.monjalon@6wind.com> wrote: > >2017-02-02 12:05, Ido Barnea: > >> From: Ido Barnea <ibarnea@cisco.com> > >> > >> Signed-off-by: Ido Barnea <ibarnea@cisco.com> > >> --- > >> drivers/net/virtio/virtio_ethdev.c | 1 + > >> 1 file changed, 1 insertion(+) > >> > >> diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c > >> index d1ff234..1d572b5 100644 > >> --- a/drivers/net/virtio/virtio_ethdev.c > >> +++ b/drivers/net/virtio/virtio_ethdev.c > >> @@ -1869,6 +1869,7 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) > >> (1ULL << VIRTIO_NET_F_HOST_TSO6); > >> if ((hw->guest_features & tso_mask) == tso_mask) > >> dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO; > >> + dev_info->speed_capa = ETH_LINK_SPEED_10G; > > > >Why 10G ? > >Yuanhan, any opinion? > > Just wanted this to be consistent with below (From virtio_dev_link_update): > link.link_speed = SPEED_10G; OK, that's the kind of justification which are good to have in the commit message.
diff --git a/drivers/net/virtio/virtio_ethdev.c b/drivers/net/virtio/virtio_ethdev.c index d1ff234..1d572b5 100644 --- a/drivers/net/virtio/virtio_ethdev.c +++ b/drivers/net/virtio/virtio_ethdev.c @@ -1869,6 +1869,7 @@ virtio_dev_info_get(struct rte_eth_dev *dev, struct rte_eth_dev_info *dev_info) (1ULL << VIRTIO_NET_F_HOST_TSO6); if ((hw->guest_features & tso_mask) == tso_mask) dev_info->tx_offload_capa |= DEV_TX_OFFLOAD_TCP_TSO; + dev_info->speed_capa = ETH_LINK_SPEED_10G; }/* -- 2.8.3