Message ID | 20190318095613.28167-2-ndabilpuram@marvell.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Ferruh Yigit |
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 [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7ACF92C30; Mon, 18 Mar 2019 10:56:54 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by dpdk.org (Postfix) with ESMTP id 66865152A for <dev@dpdk.org>; Mon, 18 Mar 2019 10:56:53 +0100 (CET) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.27/8.16.0.27) with SMTP id x2I9tI2E002323; Mon, 18 Mar 2019 02:56:52 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : content-transfer-encoding : mime-version; s=pfpt0818; bh=J6LXGVKrtqmI90btEYOtqBiRZyF3PlnR0o2olRp60V0=; b=eEhWjLi4GUkjeE4Ti6YPfDm8e+Cepx8jxfMfAWNr7DfyGX58c7R63S4YhjCAyyD6/iWi bkc3c9IyntXQtzmq7Wr+MAb0YXOfMeqTFRfMQcz69mDI3uJxfWBZSUvJS47Pw/b+v0OL SLVXzhckzXqRnByiVexIx4C50UV7K0mokXiHLaByuEcRqSUOXriTvUpLrHFv0ca83TFy LCCO10yjYzBoM11TXpTDSYFdie/erYTNjcn563bKhrq0K5eadhvRlSYaTSXfDBqrkJMW Im5pH3ZLkphw9Co7+DSTEGiwGUSbEb9ZBiwGDibAFb25rYEzretT5FknCc0Hv4Rc5odq gA== Received: from sc-exch03.marvell.com ([199.233.58.183]) by mx0a-0016f401.pphosted.com with ESMTP id 2ra80cg58n-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Mon, 18 Mar 2019 02:56:52 -0700 Received: from SC-EXCH02.marvell.com (10.93.176.82) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server (TLS) id 15.0.1367.3; Mon, 18 Mar 2019 02:56:41 -0700 Received: from NAM01-SN1-obe.outbound.protection.outlook.com (104.47.32.51) by SC-EXCH02.marvell.com (10.93.176.82) with Microsoft SMTP Server (TLS) id 15.0.1367.3 via Frontend Transport; Mon, 18 Mar 2019 02:56:41 -0700 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.onmicrosoft.com; s=selector1-marvell-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=J6LXGVKrtqmI90btEYOtqBiRZyF3PlnR0o2olRp60V0=; b=oVE/zdiPRNwDvHluwzZLzcjlJVssB9dn/B3aTn9B17TTRcKp/t/PKPBr2hk6mshQwk61RvP0fPbtp5U4slGapgumywQI5bhOxrE4XmWuNx6LgELUxaD1EU+YMZAOFrjnBVDXOF7+nI0uXPEisNusy1eCniAx+TILNZpx+M+HJ+c= Received: from DM6PR18MB2427.namprd18.prod.outlook.com (20.179.104.21) by DM6PR18MB2507.namprd18.prod.outlook.com (20.179.105.79) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1709.13; Mon, 18 Mar 2019 09:56:40 +0000 Received: from DM6PR18MB2427.namprd18.prod.outlook.com ([fe80::59e4:7c16:2667:7140]) by DM6PR18MB2427.namprd18.prod.outlook.com ([fe80::59e4:7c16:2667:7140%2]) with mapi id 15.20.1709.015; Mon, 18 Mar 2019 09:56:40 +0000 From: Nithin Kumar Dabilpuram <ndabilpuram@marvell.com> To: Wenzhuo Lu <wenzhuo.lu@intel.com>, Jingjing Wu <jingjing.wu@intel.com>, Bernard Iremonger <bernard.iremonger@intel.com> CC: "dev@dpdk.org" <dev@dpdk.org>, Ferruh Yigit <ferruh.yigit@intel.com>, Nithin Kumar Dabilpuram <ndabilpuram@marvell.com>, "shahafs@mellanox.com" <shahafs@mellanox.com> Thread-Topic: [PATCH v2 2/2] app/testpmd: fix tx qinq insert enable Thread-Index: AQHU3XDi/62Y0ELiGEm2Ccsm/XSu/w== Date: Mon, 18 Mar 2019 09:56:40 +0000 Message-ID: <20190318095613.28167-2-ndabilpuram@marvell.com> References: <20190219064840.12046-1-ndabilpuram@marvell.com> <20190318095613.28167-1-ndabilpuram@marvell.com> In-Reply-To: <20190318095613.28167-1-ndabilpuram@marvell.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-clientproxiedby: PN1PR01CA0092.INDPRD01.PROD.OUTLOOK.COM (2603:1096:c00:1::32) To DM6PR18MB2427.namprd18.prod.outlook.com (2603:10b6:5:180::21) x-mailer: git-send-email 2.8.4 x-ms-exchange-messagesentrepresentingtype: 1 x-originating-ip: [115.113.156.2] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 1261988c-3a8f-459b-b442-08d6ab880472 x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(7168020)(4627221)(201703031133081)(201702281549075)(8990200)(5600127)(711020)(4605104)(2017052603328)(7153060)(7193020); SRVR:DM6PR18MB2507; x-ms-traffictypediagnostic: DM6PR18MB2507: x-microsoft-antispam-prvs: <DM6PR18MB2507584A9A54F65D4C5DD597AF470@DM6PR18MB2507.namprd18.prod.outlook.com> x-forefront-prvs: 098076C36C x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(366004)(136003)(346002)(376002)(396003)(39850400004)(189003)(199004)(76176011)(6116002)(2616005)(26005)(11346002)(97736004)(6506007)(386003)(446003)(3846002)(6436002)(102836004)(486006)(52116002)(55236004)(256004)(14444005)(6486002)(476003)(6512007)(71190400001)(71200400001)(66066001)(186003)(53936002)(99286004)(25786009)(54906003)(8936002)(1076003)(4326008)(5660300002)(8676002)(81166006)(81156014)(2906002)(36756003)(50226002)(14454004)(305945005)(86362001)(478600001)(105586002)(7736002)(110136005)(68736007)(4744005)(106356001)(316002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM6PR18MB2507; H:DM6PR18MB2427.namprd18.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; received-spf: None (protection.outlook.com: marvell.com does not designate permitted sender hosts) x-ms-exchange-senderadcheck: 1 x-microsoft-antispam-message-info: p+opomKiQ5dERdqkljn6KIszemJiVB/ZwmSQ23UIWSniA5zIeCIIJ/wfpLW0/GjiXt5p0N/7n2W0kHxZEUWU+eED8wPmgtt4hH+0us1TexbxkXxI0HKf+nJ6ZCVS1e/QNF9z424erdKnbw0y2CncOdTI/6tOvwm8j5GDnyOWmlJ8o4DFsFz4dusa9igPjW2EZWDjJayK1Kz4zm0Pvyzd9taaod982MCYwRR+GpSSs1sMyK9SrSwGypkg6stnHucabKBzfGHxzUBWyn2R2cQ0M0eVUx5PmQrTcQie7DZJahtueEDOEU4JE4gx627hD5lN45m355g+38vDnRyxba9o4jrNYG80vp9sukbUQXLGI7qhi3Rk4hYYBYXxSdRPNT/tKj+/5Vk7+s+l4upM2JVXHyMPqgxbsME96jArdRDXn9w= Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-Network-Message-Id: 1261988c-3a8f-459b-b442-08d6ab880472 X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Mar 2019 09:56:40.3055 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR18MB2507 X-OriginatorOrg: marvell.com X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:, , definitions=2019-03-18_07:, , signatures=0 Subject: [dpdk-dev] [PATCH v2 2/2] app/testpmd: fix tx qinq insert enable X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org Sender: "dev" <dev-bounces@dpdk.org> |
Series |
[v2,1/2] app/testpmd: fix tx vlan and qinq dependency
|
|
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | warning | coding style issues |
ci/Intel-compilation | success | Compilation OK |
Commit Message
Nithin Dabilpuram
March 18, 2019, 9:56 a.m. UTC
Enable DEV_TX_OFFLOAD_VLAN_INSERT also along with
DEV_TX_OFFLOAD_VLAN_QINQ in tx_qinq_set() as it takes
both vlan id's as arguments.
Fixes: 597f9fafe13b ("app/testpmd: convert to new Tx offloads API")
Cc: shahafs@mellanox.com
Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com>
---
v2:
* Split change into two seperate patches as suggested.
app/test-pmd/config.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
Comments
Hi Nithin <snip> > Subject: [PATCH v2 2/2] app/testpmd: fix tx qinq insert enable > ./dpdk/devtools/check-git-log.sh -1 Wrong headline lowercase: app/testpmd: fix tx qinq insert enable The commit line does not seem correct as it is actually fixing the tx_qinq_set() function. Otherwise Acked-by: Bernard Iremonger <bernard.iremonger@intel.com> > Enable DEV_TX_OFFLOAD_VLAN_INSERT also along with > DEV_TX_OFFLOAD_VLAN_QINQ in tx_qinq_set() as it takes both vlan id's as > arguments. > > Fixes: 597f9fafe13b ("app/testpmd: convert to new Tx offloads API") > Cc: shahafs@mellanox.com > > Signed-off-by: Nithin Dabilpuram <ndabilpuram@marvell.com> > --- > v2: > * Split change into two seperate patches as suggested. > > app/test-pmd/config.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index > f800503..0243f07 100644 > --- a/app/test-pmd/config.c > +++ b/app/test-pmd/config.c > @@ -2994,7 +2994,8 @@ tx_qinq_set(portid_t port_id, uint16_t vlan_id, > uint16_t vlan_id_outer) > } > > tx_vlan_reset(port_id); > - ports[port_id].dev_conf.txmode.offloads |= > DEV_TX_OFFLOAD_QINQ_INSERT; > + ports[port_id].dev_conf.txmode.offloads |= > (DEV_TX_OFFLOAD_VLAN_INSERT | > + > DEV_TX_OFFLOAD_QINQ_INSERT); > ports[port_id].tx_vlan_id = vlan_id; > ports[port_id].tx_vlan_id_outer = vlan_id_outer; } > -- > 2.8.4
diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c index f800503..0243f07 100644 --- a/app/test-pmd/config.c +++ b/app/test-pmd/config.c @@ -2994,7 +2994,8 @@ tx_qinq_set(portid_t port_id, uint16_t vlan_id, uint16_t vlan_id_outer) } tx_vlan_reset(port_id); - ports[port_id].dev_conf.txmode.offloads |= DEV_TX_OFFLOAD_QINQ_INSERT; + ports[port_id].dev_conf.txmode.offloads |= (DEV_TX_OFFLOAD_VLAN_INSERT | + DEV_TX_OFFLOAD_QINQ_INSERT); ports[port_id].tx_vlan_id = vlan_id; ports[port_id].tx_vlan_id_outer = vlan_id_outer; }