[{"id":1764417,"web_url":"http://patchwork.ozlabs.org/comment/1764417/","msgid":"<9D5B4962-7D34-4820-A8D5-2D8F52315EA2@vmware.com>","list_archive_url":null,"date":"2017-09-06T22:44:47","subject":"Re: [ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","submitter":{"id":68212,"url":"http://patchwork.ozlabs.org/api/people/68212/","name":"Darrell Ball","email":"dball@vmware.com"},"content":"On 9/6/17, 5:12 AM, \"ovs-dev-bounces@openvswitch.org on behalf of Zoltán Balogh\" <ovs-dev-bounces@openvswitch.org on behalf of zoltan.balogh@ericsson.com> wrote:\r\n\r\n    DPDK uses dp-packet pool for storing received packets. The pool is\r\n    reused by rxq_recv funcions of the DPDK netdevs. The datapath is\r\n    capable to modify the packet_type property of packets. For instance\r\n    when encapsulated L3 packets are received on a ptap gre port.\r\n    In this case the packet_type property of struct dp_packet can be\r\n    modified and later the same dp_packet with the modified packet_type\r\n    can be reused in the rxq_rec function, so it can contain corrupted\r\n    data.\r\n    \r\n    The dp_packet_batch_init_cutlen() in the rxq_recv functions iterates\r\n    over dp_packets and sets their cutlen. So I modified this function\r\n    to set packet_type to Ethernet for the dp_packets as well. I also\r\n    renamed this function because of the added functionality.\r\n    \r\n    The dp_packet_batch_init_cutlen() iterates over batch->count dp_packet.\r\n    Therefore setting of batch->count = nb_rx needs to be done before the\r\n    former function is invoked. This is an additional fix.\r\n    \r\n    Signed-off-by: Zoltán Balogh <zoltan.balogh@ericsson.com>\r\n    Signed-off-by: László Sürü <laszlo.suru@ericsson.com>\r\n    Co-authored-by: László Sürü <laszlo.suru@ericsson.com>\r\n    CC: Jan Scheurich <jan.scheurich@ericsson.com>\r\n    CC: Sugesh Chandran <sugesh.chandran@intel.com>\r\n    CC: Darrell Ball <dlu998@gmail.com>\r\n    ---\r\n     lib/dp-packet.h   | 4 +++-\r\n     lib/netdev-dpdk.c | 5 +++--\r\n     2 files changed, 6 insertions(+), 3 deletions(-)\r\n    \r\n    diff --git a/lib/dp-packet.h b/lib/dp-packet.h\r\n    index 046f3ab..0046d0a 100644\r\n    --- a/lib/dp-packet.h\r\n    +++ b/lib/dp-packet.h\r\n    @@ -805,12 +805,14 @@ dp_packet_delete_batch(struct dp_packet_batch *batch, bool may_steal)\r\n     }\r\n     \r\n     static inline void\r\n    -dp_packet_batch_init_cutlen(struct dp_packet_batch *batch)\r\n    +dp_packet_batch_init_transient_fields(struct dp_packet_batch *batch)\r\n\r\n[Darrell]\r\nCan we just call this function dp_packet_batch_init_packet_fields() now that it has wider scope than just cutlen,\r\nas the name makes it clear that we are initing the individual packet fields of a batch, rather than the batch context fields ?\r\n‘transient’ is maybe implied ?\r\n\r\n\r\n     {\r\n         struct dp_packet *packet;\r\n     \r\n         DP_PACKET_BATCH_FOR_EACH (packet, batch) {\r\n             dp_packet_reset_cutlen(packet);\r\n    +        /* Set packet_type to Ethernet. */\r\n    +        packet->packet_type = PACKET_TYPE_BE(OFPHTN_ONF, 0x0000);\r\n\r\n\r\n[Darrell] The original dp_packet init code for PTAP has\r\n b->packet_type = htonl(PT_ETH);\r\nThis form seems preferred and intuitive (comment would even be unnecessary); can we use this form ?\r\n\r\nAlso, the original dp_packet init code for PTAP was in dp_packet_init__ and then moved to dp_packet_init_dpdk\r\nI think we can just remove it from there, as it is redundant with the new code.\r\n\r\n\r\n         }\r\n     }\r\n     \r\n    diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c\r\n    index f58e9be..2422741 100644\r\n    --- a/lib/netdev-dpdk.c\r\n    +++ b/lib/netdev-dpdk.c\r\n    @@ -1644,8 +1644,9 @@ netdev_dpdk_vhost_rxq_recv(struct netdev_rxq *rxq,\r\n                                              nb_rx, dropped);\r\n         rte_spinlock_unlock(&dev->stats_lock);\r\n     \r\n    -    dp_packet_batch_init_cutlen(batch);\r\n         batch->count = (int) nb_rx;\r\n\r\n[Darrell] Although nothing to do with this patch, can we remove the cast ‘(int)’ above ?\r\n\r\n\r\n    +    dp_packet_batch_init_transient_fields(batch);\r\n\r\n[Darrell]\r\nGood catch; this should work better than initializing 0 of the entries (\r\n\r\n    +\r\n         return 0;\r\n     }\r\n     \r\n    @@ -1684,8 +1685,8 @@ netdev_dpdk_rxq_recv(struct netdev_rxq *rxq, struct dp_packet_batch *batch)\r\n             rte_spinlock_unlock(&dev->stats_lock);\r\n         }\r\n     \r\n    -    dp_packet_batch_init_cutlen(batch);\r\n         batch->count = nb_rx;\r\n    +    dp_packet_batch_init_transient_fields(batch);\r\n\r\n[Darrell]\r\nSame comment as above.\r\n     \r\n         return 0;\r\n     }\r\n    -- \r\n    1.9.1\r\n    \r\n    _______________________________________________\r\n    dev mailing list\r\n    dev@openvswitch.org\r\n    https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwIFAw&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=zcjvUT1lz85oUJWStkTlRzIdXuQnJ4HL-n6oiK99iuM&s=KmEO2izD43oNVigXXu6TlPCLNruTmVtKMGdxVb6ANYs&e=","headers":{"Return-Path":"<ovs-dev-bounces@openvswitch.org>","X-Original-To":["incoming@patchwork.ozlabs.org","dev@openvswitch.org"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","ovs-dev@mail.linuxfoundation.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=openvswitch.org\n\t(client-ip=140.211.169.12; helo=mail.linuxfoundation.org;\n\tenvelope-from=ovs-dev-bounces@openvswitch.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=onevmw.onmicrosoft.com\n\theader.i=@onevmw.onmicrosoft.com header.b=\"MVP7aF2b\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=dball@vmware.com; "],"Received":["from mail.linuxfoundation.org (mail.linuxfoundation.org\n\t[140.211.169.12])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xndvC51ZGz9sCZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu,  7 Sep 2017 08:44:55 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 2E759B77;\n\tWed,  6 Sep 2017 22:44:52 +0000 (UTC)","from smtp1.linuxfoundation.org (smtp1.linux-foundation.org\n\t[172.17.192.35])\n\tby mail.linuxfoundation.org (Postfix) with ESMTPS id B6FD5AE7\n\tfor <dev@openvswitch.org>; Wed,  6 Sep 2017 22:44:50 +0000 (UTC)","from NAM01-BY2-obe.outbound.protection.outlook.com\n\t(mail-by2nam01on0055.outbound.protection.outlook.com [104.47.34.55])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 93D87124\n\tfor <dev@openvswitch.org>; Wed,  6 Sep 2017 22:44:49 +0000 (UTC)","from BY2PR05MB614.namprd05.prod.outlook.com (10.141.218.148) by\n\tBY2PR05MB1894.namprd05.prod.outlook.com (10.163.32.24) with Microsoft\n\tSMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.35.3; Wed, 6 Sep 2017 22:44:47 +0000","from BY2PR05MB614.namprd05.prod.outlook.com ([10.141.218.148]) by\n\tBY2PR05MB614.namprd05.prod.outlook.com ([10.141.218.148]) with\n\tmapi id 15.20.0035.010; Wed, 6 Sep 2017 22:44:47 +0000"],"X-Greylist":"whitelisted by SQLgrey-1.7.6","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=onevmw.onmicrosoft.com; s=selector1-vmware-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=+v8dtpZ7YmACM5Io+8f7/YxELiPoj2kjkZso1pSsSXc=;\n\tb=MVP7aF2bWhnIanQqqO/1AbFjsiFcrlyFnKHNETF9W9IIshe9Rdigdd+WDQgdU+lQzQK/pI9wBmd4Irc6s+heYf3Za+Ui8I9zJMmLH43EBgEPQFXbIStkulXiaXPhPyN8SJGMBiBW0S3HxjUWUDQCsCYp4iKytF0e80MDHme1W/8=","From":"Darrell Ball <dball@vmware.com>","To":"=?utf-8?q?Zolt=C3=A1n_Balogh?= <zoltan.balogh@ericsson.com>,\n\t\"'dev@openvswitch.org'\" <dev@openvswitch.org>","Thread-Topic":"[ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","Thread-Index":"AQHTJ2G9k5XLIhOlkUitAiRLku5ikg==","Date":"Wed, 6 Sep 2017 22:44:47 +0000","Message-ID":"<9D5B4962-7D34-4820-A8D5-2D8F52315EA2@vmware.com>","References":"<AM2PR07MB1042D848A7965231EA9EB2878A970@AM2PR07MB1042.eurprd07.prod.outlook.com>","In-Reply-To":"<AM2PR07MB1042D848A7965231EA9EB2878A970@AM2PR07MB1042.eurprd07.prod.outlook.com>","Accept-Language":"en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","user-agent":"Microsoft-MacOutlook/f.23.0.170610","x-originating-ip":"[96.90.255.131]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; BY2PR05MB1894;\n\t20:IhpDyFxUK66LQuKwGsAbXBWhREAkIqQe8RWmb9jr4JFxuxUcGPHz7tO8B9xBCP9IjQ/D9VFZJUmp+gDTdhDfgc2aeUgmftKJjKJ6oJQ5Oim/L5Rz3rGTzO/2uX5LemCS0Sa+v9SaVV19yHbFx1FmphDWVpKrWI7mh0RXZU8NQWM=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"b976334e-d83e-41b8-3dd7-08d4f578e05c","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(300000502095)(300135100095)(22001)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:BY2PR05MB1894; ","x-ms-traffictypediagnostic":"BY2PR05MB1894:","x-exchange-antispam-report-test":"UriScan:(37575265505322)(10436049006162)(216315784871565)(228905959029699);","x-microsoft-antispam-prvs":"<BY2PR05MB18947DD5CF6B3D1D6EFB4F23C8970@BY2PR05MB1894.namprd05.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(100000703101)(100105400095)(3002001)(10201501046)(93006095)(93001095)(6041248)(20161123560025)(20161123555025)(20161123558100)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123564025)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:BY2PR05MB1894; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:BY2PR05MB1894; ","x-forefront-prvs":"0422860ED4","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(24454002)(189002)(199003)(377454003)(99286003)(575784001)(6246003)(478600001)(2950100002)(105586002)(6116002)(36756003)(102836003)(86362001)(3846002)(14454004)(106356001)(68736007)(6506006)(8676002)(54356999)(50986999)(305945005)(25786009)(229853002)(77096006)(966005)(2906002)(6486002)(8936002)(82746002)(6436002)(7736002)(81166006)(76176999)(81156014)(6512007)(5660300001)(53546010)(3280700002)(83506001)(101416001)(3660700001)(4001350100001)(6306002)(97736004)(33656002)(2900100001)(83716003)(53936002)(66066001)(189998001)(491001);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:BY2PR05MB1894;\n\tH:BY2PR05MB614.namprd05.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: vmware.com does not designate\n\tpermitted sender hosts)","authentication-results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=openvswitch.org\n\t(client-ip=140.211.169.12; helo=mail.linuxfoundation.org;\n\tenvelope-from=ovs-dev-bounces@openvswitch.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=onevmw.onmicrosoft.com\n\theader.i=@onevmw.onmicrosoft.com header.b=\"MVP7aF2b\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=dball@vmware.com; "],"spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","Content-ID":"<6F6B9CCA0E15154AB1B5DCCADA3F804A@namprd05.prod.outlook.com>","MIME-Version":"1.0","X-OriginatorOrg":"vmware.com","X-MS-Exchange-CrossTenant-originalarrivaltime":"06 Sep 2017 22:44:47.3073\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"b39138ca-3cee-4b4a-a4d6-cd83d9dd62f0","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"BY2PR05MB1894","X-Spam-Status":"No, score=0.0 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,\n\tRCVD_IN_DNSWL_NONE autolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Subject":"Re: [ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","X-BeenThere":"ovs-dev@openvswitch.org","X-Mailman-Version":"2.1.12","Precedence":"list","List-Id":"<ovs-dev.openvswitch.org>","List-Unsubscribe":"<https://mail.openvswitch.org/mailman/options/ovs-dev>,\n\t<mailto:ovs-dev-request@openvswitch.org?subject=unsubscribe>","List-Archive":"<http://mail.openvswitch.org/pipermail/ovs-dev/>","List-Post":"<mailto:ovs-dev@openvswitch.org>","List-Help":"<mailto:ovs-dev-request@openvswitch.org?subject=help>","List-Subscribe":"<https://mail.openvswitch.org/mailman/listinfo/ovs-dev>,\n\t<mailto:ovs-dev-request@openvswitch.org?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1765160,"web_url":"http://patchwork.ozlabs.org/comment/1765160/","msgid":"<AM2PR07MB104236FA513D2B6E57F53C768A950@AM2PR07MB1042.eurprd07.prod.outlook.com>","list_archive_url":null,"date":"2017-09-08T08:45:23","subject":"Re: [ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","submitter":{"id":67904,"url":"http://patchwork.ozlabs.org/api/people/67904/","name":"Zoltan Balogh","email":"zoltan.balogh@ericsson.com"},"content":"Hi Darrell,\r\n\r\nThank you for your comments! I created a second version and sent it to the dev list:\r\nhttps://patchwork.ozlabs.org/patch/811387/\r\n\r\nBest regards,\r\nZoltan\r\n\r\n> -----Original Message-----\r\n> From: Darrell Ball [mailto:dball@vmware.com]\r\n> Sent: Thursday, September 07, 2017 12:45 AM\r\n> To: Zoltán Balogh <zoltan.balogh@ericsson.com>; 'dev@openvswitch.org' <dev@openvswitch.org>\r\n> Subject: Re: [ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused dp_packets\r\n> \r\n> \r\n> \r\n> On 9/6/17, 5:12 AM, \"ovs-dev-bounces@openvswitch.org on behalf of Zoltán Balogh\" <ovs-dev-bounces@openvswitch.org on\r\n> behalf of zoltan.balogh@ericsson.com> wrote:\r\n> \r\n>     DPDK uses dp-packet pool for storing received packets. The pool is\r\n>     reused by rxq_recv funcions of the DPDK netdevs. The datapath is\r\n>     capable to modify the packet_type property of packets. For instance\r\n>     when encapsulated L3 packets are received on a ptap gre port.\r\n>     In this case the packet_type property of struct dp_packet can be\r\n>     modified and later the same dp_packet with the modified packet_type\r\n>     can be reused in the rxq_rec function, so it can contain corrupted\r\n>     data.\r\n> \r\n>     The dp_packet_batch_init_cutlen() in the rxq_recv functions iterates\r\n>     over dp_packets and sets their cutlen. So I modified this function\r\n>     to set packet_type to Ethernet for the dp_packets as well. I also\r\n>     renamed this function because of the added functionality.\r\n> \r\n>     The dp_packet_batch_init_cutlen() iterates over batch->count dp_packet.\r\n>     Therefore setting of batch->count = nb_rx needs to be done before the\r\n>     former function is invoked. This is an additional fix.\r\n> \r\n>     Signed-off-by: Zoltán Balogh <zoltan.balogh@ericsson.com>\r\n>     Signed-off-by: László Sürü <laszlo.suru@ericsson.com>\r\n>     Co-authored-by: László Sürü <laszlo.suru@ericsson.com>\r\n>     CC: Jan Scheurich <jan.scheurich@ericsson.com>\r\n>     CC: Sugesh Chandran <sugesh.chandran@intel.com>\r\n>     CC: Darrell Ball <dlu998@gmail.com>\r\n>     ---\r\n>      lib/dp-packet.h   | 4 +++-\r\n>      lib/netdev-dpdk.c | 5 +++--\r\n>      2 files changed, 6 insertions(+), 3 deletions(-)\r\n> \r\n>     diff --git a/lib/dp-packet.h b/lib/dp-packet.h\r\n>     index 046f3ab..0046d0a 100644\r\n>     --- a/lib/dp-packet.h\r\n>     +++ b/lib/dp-packet.h\r\n>     @@ -805,12 +805,14 @@ dp_packet_delete_batch(struct dp_packet_batch *batch, bool may_steal)\r\n>      }\r\n> \r\n>      static inline void\r\n>     -dp_packet_batch_init_cutlen(struct dp_packet_batch *batch)\r\n>     +dp_packet_batch_init_transient_fields(struct dp_packet_batch *batch)\r\n> \r\n> [Darrell]\r\n> Can we just call this function dp_packet_batch_init_packet_fields() now that it has wider scope than just cutlen,\r\n> as the name makes it clear that we are initing the individual packet fields of a batch, rather than the batch\r\n> context fields ?\r\n> ‘transient’ is maybe implied ?\r\n> \r\n> \r\n>      {\r\n>          struct dp_packet *packet;\r\n> \r\n>          DP_PACKET_BATCH_FOR_EACH (packet, batch) {\r\n>              dp_packet_reset_cutlen(packet);\r\n>     +        /* Set packet_type to Ethernet. */\r\n>     +        packet->packet_type = PACKET_TYPE_BE(OFPHTN_ONF, 0x0000);\r\n> \r\n> \r\n> [Darrell] The original dp_packet init code for PTAP has\r\n>  b->packet_type = htonl(PT_ETH);\r\n> This form seems preferred and intuitive (comment would even be unnecessary); can we use this form ?\r\n> \r\n> Also, the original dp_packet init code for PTAP was in dp_packet_init__ and then moved to dp_packet_init_dpdk\r\n> I think we can just remove it from there, as it is redundant with the new code.\r\n> \r\n> \r\n>          }\r\n>      }\r\n> \r\n>     diff --git a/lib/netdev-dpdk.c b/lib/netdev-dpdk.c\r\n>     index f58e9be..2422741 100644\r\n>     --- a/lib/netdev-dpdk.c\r\n>     +++ b/lib/netdev-dpdk.c\r\n>     @@ -1644,8 +1644,9 @@ netdev_dpdk_vhost_rxq_recv(struct netdev_rxq *rxq,\r\n>                                               nb_rx, dropped);\r\n>          rte_spinlock_unlock(&dev->stats_lock);\r\n> \r\n>     -    dp_packet_batch_init_cutlen(batch);\r\n>          batch->count = (int) nb_rx;\r\n> \r\n> [Darrell] Although nothing to do with this patch, can we remove the cast ‘(int)’ above ?\r\n> \r\n> \r\n>     +    dp_packet_batch_init_transient_fields(batch);\r\n> \r\n> [Darrell]\r\n> Good catch; this should work better than initializing 0 of the entries (\r\n> \r\n>     +\r\n>          return 0;\r\n>      }\r\n> \r\n>     @@ -1684,8 +1685,8 @@ netdev_dpdk_rxq_recv(struct netdev_rxq *rxq, struct dp_packet_batch *batch)\r\n>              rte_spinlock_unlock(&dev->stats_lock);\r\n>          }\r\n> \r\n>     -    dp_packet_batch_init_cutlen(batch);\r\n>          batch->count = nb_rx;\r\n>     +    dp_packet_batch_init_transient_fields(batch);\r\n> \r\n> [Darrell]\r\n> Same comment as above.\r\n> \r\n>          return 0;\r\n>      }\r\n>     --\r\n>     1.9.1\r\n> \r\n>     _______________________________________________\r\n>     dev mailing list\r\n>     dev@openvswitch.org\r\n>     https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-\r\n> 2Ddev&d=DwIFAw&c=uilaK90D4TOVoH58JNXRgQ&r=BVhFA09CGX7JQ5Ih-uZnsw&m=zcjvUT1lz85oUJWStkTlRzIdXuQnJ4HL-\r\n> n6oiK99iuM&s=KmEO2izD43oNVigXXu6TlPCLNruTmVtKMGdxVb6ANYs&e=\r\n>","headers":{"Return-Path":"<ovs-dev-bounces@openvswitch.org>","X-Original-To":["incoming@patchwork.ozlabs.org","dev@openvswitch.org"],"Delivered-To":["patchwork-incoming@bilbo.ozlabs.org","ovs-dev@mail.linuxfoundation.org"],"Authentication-Results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=openvswitch.org\n\t(client-ip=140.211.169.12; helo=mail.linuxfoundation.org;\n\tenvelope-from=ovs-dev-bounces@openvswitch.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ericsson.onmicrosoft.com\n\theader.i=@ericsson.onmicrosoft.com header.b=\"i3vr0o6M\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=zoltan.balogh@ericsson.com; "],"Received":["from mail.linuxfoundation.org (mail.linuxfoundation.org\n\t[140.211.169.12])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xpW9n4PJLz9s3w\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  8 Sep 2017 18:45:33 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 3963F9B9;\n\tFri,  8 Sep 2017 08:45:31 +0000 (UTC)","from smtp1.linuxfoundation.org (smtp1.linux-foundation.org\n\t[172.17.192.35])\n\tby mail.linuxfoundation.org (Postfix) with ESMTPS id A680D40F\n\tfor <dev@openvswitch.org>; Fri,  8 Sep 2017 08:45:29 +0000 (UTC)","from sessmg22.ericsson.net (sessmg22.ericsson.net [193.180.251.58])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 149BAD3\n\tfor <dev@openvswitch.org>; Fri,  8 Sep 2017 08:45:27 +0000 (UTC)","from ESESSHC008.ericsson.se (Unknown_Domain [153.88.183.42])\n\tby sessmg22.ericsson.net (Symantec Mail Security) with SMTP id\n\t04.62.20899.6A852B95; Fri,  8 Sep 2017 10:45:26 +0200 (CEST)","from EUR01-VE1-obe.outbound.protection.outlook.com (153.88.183.145)\n\tby oa.msg.ericsson.com (153.88.183.42) with Microsoft SMTP Server\n\t(TLS) id 14.3.352.0; Fri, 8 Sep 2017 10:45:25 +0200","from AM2PR07MB1042.eurprd07.prod.outlook.com (10.162.37.27) by\n\tAM2PR07MB0722.eurprd07.prod.outlook.com (10.160.56.16) with Microsoft\n\tSMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id\n\t15.20.35.3; Fri, 8 Sep 2017 08:45:23 +0000","from AM2PR07MB1042.eurprd07.prod.outlook.com\n\t([fe80::7e:2b4e:3774:4ef3]) by\n\tAM2PR07MB1042.eurprd07.prod.outlook.com\n\t([fe80::7e:2b4e:3774:4ef3%13]) with mapi id 15.20.0035.016;\n\tFri, 8 Sep 2017 08:45:23 +0000"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-AuditID":"c1b4fb3a-617ff700000051a3-fe-59b258a64a4f","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=ericsson.onmicrosoft.com; s=selector1-ericsson-com;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version;\n\tbh=eOFSON9K3zDuJabdMjkpk1d5VMpS4yazpw/SCU5dZ7E=;\n\tb=i3vr0o6Mxeh1eSQ7roe8FLiZfwK1Wtl6RIggUXb6ypR4krdNuHxQHHpRMNjokqxk9xlzfFQLU8oSTsaGDyUsJXJk6F2dQaHpSXSTBwTgRMlHQDGrrwQX6Lsqf0fU9Yo7VSGkYe4Pbc9PB/5lw/6UA0dXf63HcniFjKWaZgMOBuQ=","From":"=?utf-8?q?Zolt=C3=A1n_Balogh?= <zoltan.balogh@ericsson.com>","To":"Darrell Ball <dball@vmware.com>, \"'dev@openvswitch.org'\"\n\t<dev@openvswitch.org>","Thread-Topic":"[ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","Thread-Index":"AdMnCD3cQyrtf468Qh6oge0Asz5RMwAWX+6AAEcvq1A=","Date":"Fri, 8 Sep 2017 08:45:23 +0000","Message-ID":"<AM2PR07MB104236FA513D2B6E57F53C768A950@AM2PR07MB1042.eurprd07.prod.outlook.com>","References":"<AM2PR07MB1042D848A7965231EA9EB2878A970@AM2PR07MB1042.eurprd07.prod.outlook.com>\n\t<9D5B4962-7D34-4820-A8D5-2D8F52315EA2@vmware.com>","In-Reply-To":"<9D5B4962-7D34-4820-A8D5-2D8F52315EA2@vmware.com>","Accept-Language":"en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-originating-ip":"[91.82.100.59]","x-ms-publictraffictype":"Email","x-microsoft-exchange-diagnostics":"1; AM2PR07MB0722;\n\t6:qyE3/GcPUS4LwDFRsLNKXHo6JtOd1XPc2iC8th5ohpydlguAi/Vwl7jMg4zXPs/uO2qADHk8mFxpBliYsTXaB9sgJ/B9U/wVlE6l6kOnXzY/4Cw7c2YOqhwdovewqLH8UBxbQA9TpJlDlcEmqBPlmB5Yk3gMSvdEtwE6OFz6xgk+C3nKHSPA759SXAen7uma0QPVoFBQjqEBbE9PwkgNfodlYOqNqZ2SO71bJTxJQpjUNW54z5Yp0u85eW2HYmSI27WLcM0EaSp2DcHkRz6ryxlkCnYOAOJL2hPTPL+azLfP+jkfKiEGlO8eBbZj9XxiDufJVnmVaeg5B/RriGgtTg==;\n\t5:lZHqkuVtAbiC8MGeWUM4LszLZtc7lRj2QzaVmKUM+x7GqDh95qo4MK0mb+KhEcGWMRIZCIuxmS7abopgWz0vTSzKXANe6mr+v/GqFt8unFozUSdo9rmoXg2Ha6duSN3gAAlziJyFW9KI9Oypo+m8Lg==;\n\t24:u8CJyiJbdMP95L/0ZIIe57dIxmuYXjuVCKWaQwokrActD1UI85eouSY9xM6uSYAn+NxkwIX0PJDFO2G+8qJuUkwanUyWTrwo8/FFuM5hPIY=;\n\t7:aqkbqLHML9TOstt4KJzoZwfGHP0I3XhZfC++qUqYmVSsz/XnGMpoRXs2yVJKTOYihW+I7VwFWhhWDH7LIC2MnhJHEGMm4sawcjkgijyhXx87N7gfHCgTrl0bwXOPZzpdPwsUENwQ7i7gs8bZolWrU1qDmIIYm4aQU0Dzb2bRzHIwnABRAz4NadfAlQ6EAw2NMFhE1eb8AY/XuyupsZZ2pLYXP5MIm/aqSG7jF5Em4zM=","x-ms-exchange-antispam-srfa-diagnostics":"SSOS;","x-ms-office365-filtering-correlation-id":"2fd0a2cc-580f-4128-583f-08d4f695f1ea","x-microsoft-antispam":"UriScan:; BCL:0; PCL:0;\n\tRULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254152)(300000503095)(300135400095)(2017052603199)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095);\n\tSRVR:AM2PR07MB0722; ","x-ms-traffictypediagnostic":"AM2PR07MB0722:","authentication-results":["ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=openvswitch.org\n\t(client-ip=140.211.169.12; helo=mail.linuxfoundation.org;\n\tenvelope-from=ovs-dev-bounces@openvswitch.org;\n\treceiver=<UNKNOWN>)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=ericsson.onmicrosoft.com\n\theader.i=@ericsson.onmicrosoft.com header.b=\"i3vr0o6M\"; \n\tdkim-atps=neutral","spf=none (sender IP is )\n\tsmtp.mailfrom=zoltan.balogh@ericsson.com; "],"x-exchange-antispam-report-test":"UriScan:(37575265505322)(61668805478150)(10436049006162)(216315784871565)(228905959029699);","x-microsoft-antispam-prvs":"<AM2PR07MB0722920947CEF722F079C9748A950@AM2PR07MB0722.eurprd07.prod.outlook.com>","x-exchange-antispam-report-cfa-test":"BCL:0; PCL:0;\n\tRULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(5005006)(8121501046)(93006095)(93001095)(3002001)(10201501046)(100000703101)(100105400095)(6041248)(20161123564025)(20161123555025)(20161123560025)(20161123562025)(201703131423075)(201702281528075)(201703061421075)(201703061406153)(20161123558100)(6072148)(201708071742011)(100000704101)(100105200095)(100000705101)(100105500095);\n\tSRVR:AM2PR07MB0722; BCL:0; PCL:0;\n\tRULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095);\n\tSRVR:AM2PR07MB0722; ","x-forefront-prvs":"04244E0DC5","x-forefront-antispam-report":"SFV:NSPM;\n\tSFS:(10009020)(6009001)(39860400002)(377454003)(189002)(24454002)(199003)(13464003)(6436002)(8676002)(7696004)(3846002)(102836003)(6116002)(99286003)(305945005)(86362001)(55016002)(575784001)(6306002)(9686003)(7736002)(54356999)(53936002)(76176999)(6246003)(101416001)(81156014)(81166006)(74316002)(8936002)(50986999)(2900100001)(5660300001)(229853002)(2950100002)(66066001)(105586002)(85202003)(106356001)(6506006)(33656002)(3280700002)(3660700001)(53546010)(25786009)(85182001)(5250100002)(478600001)(14454004)(68736007)(189998001)(966005)(97736004)(2906002)(345774005)(491001);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:AM2PR07MB0722;\n\tH:AM2PR07MB1042.eurprd07.prod.outlook.com; FPR:; SPF:None;\n\tPTR:InfoNoRecords; MX:1; A:1; LANG:en; ","received-spf":"None (protection.outlook.com: ericsson.com does not designate\n\tpermitted sender hosts)","spamdiagnosticoutput":"1:99","spamdiagnosticmetadata":"NSPM","MIME-Version":"1.0","X-MS-Exchange-CrossTenant-originalarrivaltime":"08 Sep 2017 08:45:23.3747\n\t(UTC)","X-MS-Exchange-CrossTenant-fromentityheader":"Hosted","X-MS-Exchange-CrossTenant-id":"92e84ceb-fbfd-47ab-be52-080c6b87953f","X-MS-Exchange-Transport-CrossTenantHeadersStamped":"AM2PR07MB0722","X-OriginatorOrg":"ericsson.com","X-Brightmail-Tracker":"H4sIAAAAAAAAA+NgFmpjleLIzCtJLcpLzFFi42KZGbFdS3dZxKZIg95fghbPXyxktjh6eg+z\n\tA5PHs5v/GT3ezX/LFsAUxWWTkpqTWZZapG+XwJVxe8Z7poIv5hUHp61naWDcY9bFyMEhIWAi\n\tsXFJbRcjF4eQwBFGif9zTrBDOMcZJW68vgvmsAj0Mks8WfubGSIziUni5obvUGX3GCWuXN/P\n\t0sXIycEm4CzR9/g0G4gtIhAicXnrHbC4sECoRE/bLyaIeJjEmw+TWCBsK4mOnqXMIDaLgIrE\n\tg33n2UFsXoEYif875rBCLJjAKHH7xAqwZk4BO4kT2z6ANTMKiEl8P7UGLM4sIC5x68l8MFtC\n\tQEBiyZ7zzBC2qMTLx/9YIWwFiU0L3rND2LISl+Z3M4IskBBoY5eY9/c6VEJPYuvEt4wQtq/E\n\t9BcfoIoeMUlc7V3AApHQkti/diMjxBWxEtNbu9kg4jUS6+dtgbKzJbZMuA3VfJpV4vfxiyyQ\n\t8JaReHkjASL+nlVi3cZTLBMYdWYh+WIWUBmzgKbE+l36EGFFiSndD9lngUNGUOLkzCcsCxhZ\n\tVjGKFqcWF+emGxnppRZlJhcX5+fp5aWWbGIEpo6DW35b7WA8+NzxEKMAB6MSD+8Pt02RQqyJ\n\tZcWVuYcYJTiYlUR4L/oBhXhTEiurUovy44tKc1KLDzFKc7AoifM67LsQISSQnliSmp2aWpBa\n\tBJNl4uCUamCce/3lV8Ef65hfKv2W+tGyVNK4p21ja96tknttlz97GVrn7/yc8vmd1uKq4Cqn\n\tlwKnxb7f2ynUkKn3ysz1rez61QcYzdJluyylCwxEBR/s6K98paV954HjtmU3TJgNtgXkWyo1\n\tVOkwLp1pWt0jmLuxobInzVSOoZrP6sci6XlCJhrnGV7d3K3EUpyRaKjFXFScCACEJ3V5GQMA\n\tAA==","X-Spam-Status":"No, score=-2.3 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,\n\tRCVD_IN_DNSWL_MED autolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Subject":"Re: [ovs-dev] [PATCH] netdev-dpdk: reset packet_type for reused\n\tdp_packets","X-BeenThere":"ovs-dev@openvswitch.org","X-Mailman-Version":"2.1.12","Precedence":"list","List-Id":"<ovs-dev.openvswitch.org>","List-Unsubscribe":"<https://mail.openvswitch.org/mailman/options/ovs-dev>,\n\t<mailto:ovs-dev-request@openvswitch.org?subject=unsubscribe>","List-Archive":"<http://mail.openvswitch.org/pipermail/ovs-dev/>","List-Post":"<mailto:ovs-dev@openvswitch.org>","List-Help":"<mailto:ovs-dev-request@openvswitch.org?subject=help>","List-Subscribe":"<https://mail.openvswitch.org/mailman/listinfo/ovs-dev>,\n\t<mailto:ovs-dev-request@openvswitch.org?subject=subscribe>","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}}]