[{"id":1761166,"web_url":"http://patchwork.ozlabs.org/comment/1761166/","msgid":"<20170831162334.GM6175@ovn.org>","list_archive_url":null,"date":"2017-08-31T16:23:34","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":67603,"url":"http://patchwork.ozlabs.org/api/people/67603/","name":"Ben Pfaff","email":"blp@ovn.org"},"content":"On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> IETF NSH draft will be approved by end of August, NSH header\n> format has been finalized and won't be change anymore, so we\n> need to follow this final spec to implement nsh.\n> \n> kernel data path also needs finalized uAPIs, they can't be\n> changed once they are merged.\n> \n> This patch adds new nsh key 'ttl', bits of flags and mdtype\n> fields are also changed to follow the final spec.\n> \n> A new action dec_nsh_ttl will be added in the following patch.\n> \n> Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n\nThis is a large change to just update the wire format.  Can you make it\nmore minimal?\n\nThanks,\n\nBen.","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>)","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 3xjnkD0hmRz9ryk\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 02:23:48 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id AD185910;\n\tThu, 31 Aug 2017 16:23:45 +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 A2C308EE\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 16:23:44 +0000 (UTC)","from relay3-d.mail.gandi.net (relay3-d.mail.gandi.net\n\t[217.70.183.195])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 28798E0\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 16:23:44 +0000 (UTC)","from ovn.org (unknown [208.91.2.3])\n\t(Authenticated sender: blp@ovn.org)\n\tby relay3-d.mail.gandi.net (Postfix) with ESMTPSA id 0A96DA80F9;\n\tThu, 31 Aug 2017 18:23:37 +0200 (CEST)"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-Originating-IP":"208.91.2.3","Date":"Thu, 31 Aug 2017 09:23:34 -0700","From":"Ben Pfaff <blp@ovn.org>","To":"Yi Yang <yi.y.yang@intel.com>","Message-ID":"<20170831162334.GM6175@ovn.org>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"dev@openvswitch.org, e@erig.me, jbenc@redhat.com","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1761170,"web_url":"http://patchwork.ozlabs.org/comment/1761170/","msgid":"<20170831162736.GN6175@ovn.org>","list_archive_url":null,"date":"2017-08-31T16:27:36","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":67603,"url":"http://patchwork.ozlabs.org/api/people/67603/","name":"Ben Pfaff","email":"blp@ovn.org"},"content":"On Thu, Aug 31, 2017 at 09:23:34AM -0700, Ben Pfaff wrote:\n> On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> > IETF NSH draft will be approved by end of August, NSH header\n> > format has been finalized and won't be change anymore, so we\n> > need to follow this final spec to implement nsh.\n> > \n> > kernel data path also needs finalized uAPIs, they can't be\n> > changed once they are merged.\n> > \n> > This patch adds new nsh key 'ttl', bits of flags and mdtype\n> > fields are also changed to follow the final spec.\n> > \n> > A new action dec_nsh_ttl will be added in the following patch.\n> > \n> > Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n> \n> This is a large change to just update the wire format.  Can you make it\n> more minimal?\n\nActually, let's skip it.  I think that we should release OVS 2.8 as is.\nWe delayed the release mostly to get NSH in, because Jan said that it\nwas important to them, but then it turns out that the NSH protocol is\nstill changing?  What a waste.  I will not make that mistake again.","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>)","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 3xjnpr20Cxz9sPm\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 02:27:48 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 21056B09;\n\tThu, 31 Aug 2017 16:27:46 +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 DF052901\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 16:27:44 +0000 (UTC)","from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net\n\t[217.70.183.194])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id B8C19E0\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 16:27:43 +0000 (UTC)","from ovn.org (unknown [208.91.2.3])\n\t(Authenticated sender: blp@ovn.org)\n\tby relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 293CFC5A53;\n\tThu, 31 Aug 2017 18:27:39 +0200 (CEST)"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-Originating-IP":"208.91.2.3","Date":"Thu, 31 Aug 2017 09:27:36 -0700","From":"Ben Pfaff <blp@ovn.org>","To":"Yi Yang <yi.y.yang@intel.com>","Message-ID":"<20170831162736.GN6175@ovn.org>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20170831162334.GM6175@ovn.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170831162334.GM6175@ovn.org>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"dev@openvswitch.org, e@erig.me, jbenc@redhat.com","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1761276,"web_url":"http://patchwork.ozlabs.org/comment/1761276/","msgid":"<CFF8EF42F1132E4CBE2BF0AB6C21C58D787F2D1F@ESESSMB107.ericsson.se>","list_archive_url":null,"date":"2017-08-31T19:07:29","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":68449,"url":"http://patchwork.ozlabs.org/api/people/68449/","name":"Jan Scheurich","email":"jan.scheurich@ericsson.com"},"content":"> -----Original Message-----\n> From: Ben Pfaff [mailto:blp@ovn.org]\n> Sent: Thursday, 31 August, 2017 18:28\n> >\n> > This is a large change to just update the wire format.  Can you make\n> > it more minimal?\n> \n> Actually, let's skip it.  I think that we should release OVS 2.8 as is.\n> We delayed the release mostly to get NSH in, because Jan said that it was\n> important to them, but then it turns out that the NSH protocol is still\n> changing?  What a waste.  I will not make that mistake again.\n\nHi Ben,\n\nIt was our mistake that we missed the late change of the NSH draft in June during the crunch time for OVS 2.8 and I apologize for that. \n\nI already suggested two weeks ago (https://mail.openvswitch.org/pipermail/ovs-dev/2017-August/337540.html) to release 2.8 as is and document the NSH support as experimental (with support for the pre-June draft version). So I support to follow this route. \n\nI believe we can create a much smaller correction which just fixes the wire format without adding any support for the new TTL field. That patch could be delivered on the 2.8 maintenance branch.\n\nFull compliance to the final RFC with support for the TTL field, the dec_nsh_ttl action and proper alignment with the net-next kernel datapath uAPI should then be targeted for OVS 2.9.\n\nRegards, Jan","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>)","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 3xjsMG4K0Rz9s8J\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 05:07:37 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 31633B1B;\n\tThu, 31 Aug 2017 19:07:34 +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 D899BAAB\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 19:07:32 +0000 (UTC)","from sessmg23.ericsson.net (sessmg23.ericsson.net [193.180.251.45])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 1AFCA79\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 19:07:31 +0000 (UTC)","from ESESSHC010.ericsson.se (Unknown_Domain [153.88.183.48])\n\tby sessmg23.ericsson.net (Symantec Mail Security) with SMTP id\n\t54.E8.22436.27E58A95; Thu, 31 Aug 2017 21:07:30 +0200 (CEST)","from ESESSMB107.ericsson.se ([169.254.7.26]) by\n\tESESSHC010.ericsson.se ([153.88.183.48]) with mapi id 14.03.0352.000; \n\tThu, 31 Aug 2017 21:07:29 +0200"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-AuditID":"c1b4fb2d-103ff700000057a4-06-59a85e7234a4","From":"Jan Scheurich <jan.scheurich@ericsson.com>","To":"Ben Pfaff <blp@ovn.org>, Yi Yang <yi.y.yang@intel.com>","Thread-Topic":"[PATCH v5 1/2] nsh: add new flow key 'ttl'","Thread-Index":"AQHTIPQi+dq82gHuKUuRIUePh5g18qKeh4wAgAABIACAAEEAcA==","Date":"Thu, 31 Aug 2017 19:07:29 +0000","Message-ID":"<CFF8EF42F1132E4CBE2BF0AB6C21C58D787F2D1F@ESESSMB107.ericsson.se>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20170831162334.GM6175@ovn.org> <20170831162736.GN6175@ovn.org>","In-Reply-To":"<20170831162736.GN6175@ovn.org>","Accept-Language":"en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-originating-ip":"[153.88.183.149]","MIME-Version":"1.0","X-Brightmail-Tracker":"H4sIAAAAAAAAA+NgFprDIsWRmVeSWpSXmKPExsUyM2K7gW5R3IpIg94/whavJjcwWhw9vYfZ\n\tYnHPCSaLFZtOM1v8/rqNyYHV48iig4wei/e8ZPJ4dvM/o8fzaz0sHu/3XWULYI3isklJzcks\n\tSy3St0vgynjSNZ+54DhXxenOJvYGxo0cXYycHBICJhK3OrvYuhi5OIQEjjBKHOrYyArhLGaU\n\taJ4ylb2LkYODTcBAYvZuB5AGEQEriXUftrCA1DALbGKUePjuACNIQljAXGL6iQtsEEUWEuff\n\tdbJC2E4Sm65dYAexWQRUJU7Mg6jhFfCVeHLyMNTm/YwSi9r/MIEkOAV0JJ48amIGsRkFxCS+\n\tn1oDFmcWEJe49WQ+E8TZAhJL9pxnhrBFJV4+/scKYStJrD28nQWiXk/ixtQpbBC2tsSyha+Z\n\tIRYLSpyc+YRlAqPoLCRjZyFpmYWkZRaSlgWMLKsYRYtTi4tz042M9VKLMpOLi/Pz9PJSSzYx\n\tAuPs4JbfujsYV792PMQowMGoxMPrHLQiUog1say4MvcQowQHs5II71IHoBBvSmJlVWpRfnxR\n\taU5q8SFGaQ4WJXFeh30XIoQE0hNLUrNTUwtSi2CyTBycUg2Ma5kKWzY+YQhccyhlbjqj8ewb\n\tudIuVdPbnaOXhbbqsir11BbsyPM+PmPJSlH2nxOnvL3urvxuy/mQIwFr/nxqXaFmy9Wzte6R\n\tj4epXvNbt5ORwLDrv5/zatL68g3VpTVaZ27Nne1qWatbfy70yc9LyTIzNuhk5HgeDNt38Y75\n\t7Uvrqg9uFuNVYinOSDTUYi4qTgQA1Fimsa8CAAA=","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"\"dev@openvswitch.org\" <dev@openvswitch.org>, \"e@erig.me\" <e@erig.me>,\n\t\"jbenc@redhat.com\" <jbenc@redhat.com>","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1761387,"web_url":"http://patchwork.ozlabs.org/comment/1761387/","msgid":"<20170831234359.GA11991@cran64.bj.intel.com>","list_archive_url":null,"date":"2017-08-31T23:44:00","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":68962,"url":"http://patchwork.ozlabs.org/api/people/68962/","name":"Yang, Yi","email":"yi.y.yang@intel.com"},"content":"On Thu, Aug 31, 2017 at 09:27:36AM -0700, Ben Pfaff wrote:\n> On Thu, Aug 31, 2017 at 09:23:34AM -0700, Ben Pfaff wrote:\n> > On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> > > IETF NSH draft will be approved by end of August, NSH header\n> > > format has been finalized and won't be change anymore, so we\n> > > need to follow this final spec to implement nsh.\n> > > \n> > > kernel data path also needs finalized uAPIs, they can't be\n> > > changed once they are merged.\n> > > \n> > > This patch adds new nsh key 'ttl', bits of flags and mdtype\n> > > fields are also changed to follow the final spec.\n> > > \n> > > A new action dec_nsh_ttl will be added in the following patch.\n> > > \n> > > Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n> > \n> > This is a large change to just update the wire format.  Can you make it\n> > more minimal?\n> \n> Actually, let's skip it.  I think that we should release OVS 2.8 as is.\n> We delayed the release mostly to get NSH in, because Jan said that it\n> was important to them, but then it turns out that the NSH protocol is\n> still changing?  What a waste.  I will not make that mistake again.\n\nBen, the confirmation I got from one of NSH draft authors is it won't be\nchanged except some text description, moreover it will be approved as\nIETF RFC very soon. So I think you can consider this again. Maybe OVS\nmaster can apply them if it is too late for OVS 2.8.","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>)","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 3xjzqr6CXjz9s8J\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri,  1 Sep 2017 09:59:20 +1000 (AEST)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 07565DFF;\n\tThu, 31 Aug 2017 23:58:49 +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 78843DFE\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 23:58:48 +0000 (UTC)","from mga04.intel.com (mga04.intel.com [192.55.52.120])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 0D697A3\n\tfor <dev@openvswitch.org>; Thu, 31 Aug 2017 23:58:47 +0000 (UTC)","from orsmga005.jf.intel.com ([10.7.209.41])\n\tby fmsmga104.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t31 Aug 2017 16:58:47 -0700","from unknown (HELO cran64.bj.intel.com) ([10.240.224.181])\n\tby orsmga005.jf.intel.com with ESMTP; 31 Aug 2017 16:58:46 -0700"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-ExtLoop1":"1","X-IronPort-AV":"E=Sophos;i=\"5.41,456,1498546800\"; d=\"scan'208\";a=\"144301764\"","Date":"Fri, 1 Sep 2017 07:44:00 +0800","From":"\"Yang, Yi\" <yi.y.yang@intel.com>","To":"Ben Pfaff <blp@ovn.org>","Message-ID":"<20170831234359.GA11991@cran64.bj.intel.com>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20170831162334.GM6175@ovn.org> <20170831162736.GN6175@ovn.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20170831162736.GN6175@ovn.org>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,\n\tRP_MATCHES_RCVD autolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"dev@openvswitch.org, e@erig.me, jbenc@redhat.com","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1797473,"web_url":"http://patchwork.ozlabs.org/comment/1797473/","msgid":"<20171101205315.GK27530@ovn.org>","list_archive_url":null,"date":"2017-11-01T20:53:15","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":67603,"url":"http://patchwork.ozlabs.org/api/people/67603/","name":"Ben Pfaff","email":"blp@ovn.org"},"content":"On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> IETF NSH draft will be approved by end of August, NSH header\n> format has been finalized and won't be change anymore, so we\n> need to follow this final spec to implement nsh.\n> \n> kernel data path also needs finalized uAPIs, they can't be\n> changed once they are merged.\n> \n> This patch adds new nsh key 'ttl', bits of flags and mdtype\n> fields are also changed to follow the final spec.\n> \n> A new action dec_nsh_ttl will be added in the following patch.\n> \n> Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n\nIt's time to take another look at NSH, by adding support for the latest\ndraft.  I posted a patch to kick this off:\n        https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340365.html\n\nThis is the sort of patch that could be backported to 2.8 if that is\ndesirable.\n\nAfter we get the protocol format fixed, I suggest rebasing and reposting\nthese patches so that we can get enhanced support into 2.9.\n\nThanks,\n\nBen.","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>)","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 3yS0mk4xqnz9t2V\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu,  2 Nov 2017 07:53:26 +1100 (AEDT)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 0809CD0B;\n\tWed,  1 Nov 2017 20:53:25 +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 94DA9D08\n\tfor <dev@openvswitch.org>; Wed,  1 Nov 2017 20:53:24 +0000 (UTC)","from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net\n\t[217.70.183.194])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 28D821A0\n\tfor <dev@openvswitch.org>; Wed,  1 Nov 2017 20:53:24 +0000 (UTC)","from ovn.org (unknown [208.91.3.26])\n\t(Authenticated sender: blp@ovn.org)\n\tby relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 970EAC5A46;\n\tWed,  1 Nov 2017 21:53:19 +0100 (CET)"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-Originating-IP":"208.91.3.26","Date":"Wed, 1 Nov 2017 13:53:15 -0700","From":"Ben Pfaff <blp@ovn.org>","To":"Yi Yang <yi.y.yang@intel.com>","Message-ID":"<20171101205315.GK27530@ovn.org>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"dev@openvswitch.org, e@erig.me, jbenc@redhat.com","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1799568,"web_url":"http://patchwork.ozlabs.org/comment/1799568/","msgid":"<20171106104244.GA52381@localhost.localdomain>","list_archive_url":null,"date":"2017-11-06T10:42:44","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":68962,"url":"http://patchwork.ozlabs.org/api/people/68962/","name":"Yang, Yi","email":"yi.y.yang@intel.com"},"content":"On Thu, Nov 02, 2017 at 04:53:15AM +0800, Ben Pfaff wrote:\n> On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> > IETF NSH draft will be approved by end of August, NSH header\n> > format has been finalized and won't be change anymore, so we\n> > need to follow this final spec to implement nsh.\n> > \n> > kernel data path also needs finalized uAPIs, they can't be\n> > changed once they are merged.\n> > \n> > This patch adds new nsh key 'ttl', bits of flags and mdtype\n> > fields are also changed to follow the final spec.\n> > \n> > A new action dec_nsh_ttl will be added in the following patch.\n> > \n> > Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n> \n> It's time to take another look at NSH, by adding support for the latest\n> draft.  I posted a patch to kick this off:\n>         https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340365.html\n> \n> This is the sort of patch that could be backported to 2.8 if that is\n> desirable.\n> \n> After we get the protocol format fixed, I suggest rebasing and reposting\n> these patches so that we can get enhanced support into 2.9.\n> \n> Thanks,\n\nBen, no problem, NSH kernel patch will be merged very soon, after that\nI'll rework this patch series, netlink message format for OVS_KEY_ATTR_NSH\nhas big change per kernel requirement, so userspace part will have some\nbig change for push_nsh and set. action names for data path also need\nto be changed.\n\nWe also can start backporting NSH kernel patches to OVS to support\ncompat mode, this requires to backport many kernel patches before NSH\nkernel patch, it will be great if OVS 2.9 can support both userspace and\nkernel data path.","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>)","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 3yVq5S69pxz9s82\n\tfor <incoming@patchwork.ozlabs.org>;\n\tMon,  6 Nov 2017 21:47:40 +1100 (AEDT)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 0AE53BAA;\n\tMon,  6 Nov 2017 10:47:37 +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 CFBE0BA9\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 10:47:35 +0000 (UTC)","from mga06.intel.com (mga06.intel.com [134.134.136.31])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 6C71ED4\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 10:47:35 +0000 (UTC)","from fmsmga006.fm.intel.com ([10.253.24.20])\n\tby orsmga104.jf.intel.com with ESMTP; 06 Nov 2017 02:47:34 -0800","from unknown (HELO localhost.localdomain) ([10.240.224.185])\n\tby fmsmga006.fm.intel.com with ESMTP; 06 Nov 2017 02:47:33 -0800"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-ExtLoop1":"1","X-IronPort-AV":"E=Sophos;i=\"5.44,352,1505804400\"; d=\"scan'208\";a=\"172494325\"","Date":"Mon, 6 Nov 2017 18:42:44 +0800","From":"\"Yang, Yi\" <yi.y.yang@intel.com>","To":"Ben Pfaff <blp@ovn.org>","Message-ID":"<20171106104244.GA52381@localhost.localdomain>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20171101205315.GK27530@ovn.org>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20171101205315.GK27530@ovn.org>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED,\n\tRP_MATCHES_RCVD autolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"\"dev@openvswitch.org\" <dev@openvswitch.org>, \"e@erig.me\" <e@erig.me>,\n\t\"jbenc@redhat.com\" <jbenc@redhat.com>","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1800044,"web_url":"http://patchwork.ozlabs.org/comment/1800044/","msgid":"<20171106221430.GJ27530@ovn.org>","list_archive_url":null,"date":"2017-11-06T22:14:30","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":67603,"url":"http://patchwork.ozlabs.org/api/people/67603/","name":"Ben Pfaff","email":"blp@ovn.org"},"content":"On Mon, Nov 06, 2017 at 06:42:44PM +0800, Yang, Yi wrote:\n> On Thu, Nov 02, 2017 at 04:53:15AM +0800, Ben Pfaff wrote:\n> > On Wed, Aug 30, 2017 at 02:21:01AM +0800, Yi Yang wrote:\n> > > IETF NSH draft will be approved by end of August, NSH header\n> > > format has been finalized and won't be change anymore, so we\n> > > need to follow this final spec to implement nsh.\n> > > \n> > > kernel data path also needs finalized uAPIs, they can't be\n> > > changed once they are merged.\n> > > \n> > > This patch adds new nsh key 'ttl', bits of flags and mdtype\n> > > fields are also changed to follow the final spec.\n> > > \n> > > A new action dec_nsh_ttl will be added in the following patch.\n> > > \n> > > Signed-off-by: Yi Yang <yi.y.yang@intel.com>\n> > \n> > It's time to take another look at NSH, by adding support for the latest\n> > draft.  I posted a patch to kick this off:\n> >         https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340365.html\n> > \n> > This is the sort of patch that could be backported to 2.8 if that is\n> > desirable.\n> > \n> > After we get the protocol format fixed, I suggest rebasing and reposting\n> > these patches so that we can get enhanced support into 2.9.\n> > \n> > Thanks,\n> \n> Ben, no problem, NSH kernel patch will be merged very soon, after that\n> I'll rework this patch series, netlink message format for OVS_KEY_ATTR_NSH\n> has big change per kernel requirement, so userspace part will have some\n> big change for push_nsh and set. action names for data path also need\n> to be changed.\n> \n> We also can start backporting NSH kernel patches to OVS to support\n> compat mode, this requires to backport many kernel patches before NSH\n> kernel patch, it will be great if OVS 2.9 can support both userspace and\n> kernel data path.\n\nSounds good.","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>)","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 3yW6LW68t0z9sBZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue,  7 Nov 2017 09:14:59 +1100 (AEDT)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id 3B7A9AD6;\n\tMon,  6 Nov 2017 22:14:43 +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 2078C94B\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 22:14:42 +0000 (UTC)","from relay2-d.mail.gandi.net (relay2-d.mail.gandi.net\n\t[217.70.183.194])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 988404FA\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 22:14:41 +0000 (UTC)","from ovn.org (unknown [IPv6:2001:470:108:1300:df7c:9be9:8c7:fd03])\n\t(Authenticated sender: blp@ovn.org)\n\tby relay2-d.mail.gandi.net (Postfix) with ESMTPSA id 32339C5A50;\n\tMon,  6 Nov 2017 23:14:35 +0100 (CET)"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","Date":"Mon, 6 Nov 2017 14:14:30 -0800","From":"Ben Pfaff <blp@ovn.org>","To":"\"Yang, Yi\" <yi.y.yang@intel.com>","Message-ID":"<20171106221430.GJ27530@ovn.org>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20171101205315.GK27530@ovn.org>\n\t<20171106104244.GA52381@localhost.localdomain>","MIME-Version":"1.0","Content-Disposition":"inline","In-Reply-To":"<20171106104244.GA52381@localhost.localdomain>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Status":"No, score=-0.7 required=5.0 tests=RCVD_IN_DNSWL_LOW\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"\"dev@openvswitch.org\" <dev@openvswitch.org>, \"e@erig.me\" <e@erig.me>,\n\t\"jbenc@redhat.com\" <jbenc@redhat.com>","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}},{"id":1800061,"web_url":"http://patchwork.ozlabs.org/comment/1800061/","msgid":"<CFF8EF42F1132E4CBE2BF0AB6C21C58D852C323B@ESESSMB107.ericsson.se>","list_archive_url":null,"date":"2017-11-06T22:26:38","subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","submitter":{"id":68449,"url":"http://patchwork.ozlabs.org/api/people/68449/","name":"Jan Scheurich","email":"jan.scheurich@ericsson.com"},"content":"Hi Ben,\n\n> > > It's time to take another look at NSH, by adding support for the latest\n> > > draft.  I posted a patch to kick this off:\n> > >         https://mail.openvswitch.org/pipermail/ovs-dev/2017-November/340365.html\n> > >\n> > > This is the sort of patch that could be backported to 2.8 if that is\n> > > desirable.\n> > >\n> > > After we get the protocol format fixed, I suggest rebasing and reposting\n> > > these patches so that we can get enhanced support into 2.9.\n> > >\n> > > Thanks,\n> >\n> > Ben, no problem, NSH kernel patch will be merged very soon, after that\n> > I'll rework this patch series, netlink message format for OVS_KEY_ATTR_NSH\n> > has big change per kernel requirement, so userspace part will have some\n> > big change for push_nsh and set. action names for data path also need\n> > to be changed.\n> >\n> > We also can start backporting NSH kernel patches to OVS to support\n> > compat mode, this requires to backport many kernel patches before NSH\n> > kernel patch, it will be great if OVS 2.9 can support both userspace and\n> > kernel data path.\n> \n> Sounds good.\n\nThanks for the initiative.\n\nI would still like to fix the wire format issue and 2-3 minor bugs on master and on the 2.8 branch before we start the refactoring on master when the NSH kernel datapath is finished. I think the nsh_ttl match field and the corresponding dec_nsh_ttl action can be left to master.\n\nI already had a small patch series prepared for master and branch-2.8 with the fixes. I will add your initialization of NSH TTL to 63 to it and send out shortly.\n\nBR, Jan","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>)","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 3yW6c44kPbz9sBZ\n\tfor <incoming@patchwork.ozlabs.org>;\n\tTue,  7 Nov 2017 09:26:44 +1100 (AEDT)","from mail.linux-foundation.org (localhost [127.0.0.1])\n\tby mail.linuxfoundation.org (Postfix) with ESMTP id ADB28B0B;\n\tMon,  6 Nov 2017 22:26:42 +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 EDA05AF3\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 22:26:41 +0000 (UTC)","from sesbmg23.ericsson.net (sesbmg23.ericsson.net [193.180.251.37])\n\tby smtp1.linuxfoundation.org (Postfix) with ESMTPS id 396D679\n\tfor <dev@openvswitch.org>; Mon,  6 Nov 2017 22:26:41 +0000 (UTC)","from ESESSHC003.ericsson.se (Unknown_Domain [153.88.183.27])\n\tby sesbmg23.ericsson.net (Symantec Mail Security) with SMTP id\n\tC8.E8.03220.F91E00A5; Mon,  6 Nov 2017 23:26:39 +0100 (CET)","from ESESSMB107.ericsson.se ([169.254.7.134]) by\n\tESESSHC003.ericsson.se ([153.88.183.27]) with mapi id 14.03.0352.000; \n\tMon, 6 Nov 2017 23:26:39 +0100"],"X-Greylist":"domain auto-whitelisted by SQLgrey-1.7.6","X-AuditID":"c1b4fb25-debff70000000c94-2c-5a00e19f8518","From":"Jan Scheurich <jan.scheurich@ericsson.com>","To":"Ben Pfaff <blp@ovn.org>, \"Yang, Yi\" <yi.y.yang@intel.com>","Thread-Topic":"[PATCH v5 1/2] nsh: add new flow key 'ttl'","Thread-Index":"AQHTIPQi+dq82gHuKUuRIUePh5g18qMAVDSAgAcxFACAAMFHAIAAEjzw","Date":"Mon, 6 Nov 2017 22:26:38 +0000","Message-ID":"<CFF8EF42F1132E4CBE2BF0AB6C21C58D852C323B@ESESSMB107.ericsson.se>","References":"<1504030862-14591-1-git-send-email-yi.y.yang@intel.com>\n\t<1504030862-14591-2-git-send-email-yi.y.yang@intel.com>\n\t<20171101205315.GK27530@ovn.org>\n\t<20171106104244.GA52381@localhost.localdomain>\n\t<20171106221430.GJ27530@ovn.org>","In-Reply-To":"<20171106221430.GJ27530@ovn.org>","Accept-Language":"en-US","Content-Language":"en-US","X-MS-Has-Attach":"","X-MS-TNEF-Correlator":"","x-originating-ip":"[153.88.183.154]","MIME-Version":"1.0","X-Brightmail-Tracker":"H4sIAAAAAAAAA+NgFprJIsWRmVeSWpSXmKPExsUyM2K7tO78hwxRBpsPKVm8mtzAaHH09B5m\n\ti8U9J5gsVmw6zWzx++s2JgdWjyOLDjJ6LN7zksnj2c3/jB7Pr/WweLzfd5UtgDWKyyYlNSez\n\tLLVI3y6BK+PFMZWChzwVX2+dY29gfM3ZxcjJISFgIjG95RNLFyMXh5DAYUaJw/f+skE4ixkl\n\tPrUvZuxi5OBgEzCQmL3bAaRBRMBW4siWXYwgNrNAoUTfuU4wW1jAXGL6iQtsEDUWEuffdbJC\n\t2G4StzuOg9WwCKhILDu8hwVkJK+Ar8THuZoQq74wSsw69JgZpIZTQFfi5pv/YHMYBcQkvp9a\n\twwSxS1zi1pP5TBBHC0gs2XOeGcIWlXj5+B8rhK0ksej2Z6h6HYkFuz+xQdjaEssWvgar5xUQ\n\tlDg58wnLBEbRWUjGzkLSMgtJyywkLQsYWVYxihanFiflphsZ66UWZSYXF+fn6eWllmxiBEbY\n\twS2/VXcwXn7jeIhRgINRiYf31yGGKCHWxLLiytxDjBIczEoivNvVgUK8KYmVValF+fFFpTmp\n\txYcYpTlYlMR5HfddiBASSE8sSc1OTS1ILYLJMnFwSjUwZhQfj5r0THjBpDCZlTLJ8/dM+c55\n\tM2XWq/43xi+6Lea7Nt4XkaqqVwg63Mb+ZuvWvL5ahZK6x7qfFOfUccvv9fqs86JqS8+Fh24L\n\tZnEX8vYqvV905NOVeoW5M10mZQeHzc+cVphV9CJKcNXzkH3rfoYoqadHLTNNF+9b33hgy4l7\n\tRwxX1F3/osRSnJFoqMVcVJwIAJMCcH+sAgAA","X-Spam-Status":"No, score=-2.3 required=5.0 tests=RCVD_IN_DNSWL_MED\n\tautolearn=disabled version=3.3.1","X-Spam-Checker-Version":"SpamAssassin 3.3.1 (2010-03-16) on\n\tsmtp1.linux-foundation.org","Cc":"\"dev@openvswitch.org\" <dev@openvswitch.org>, \"e@erig.me\" <e@erig.me>,\n\t\"jbenc@redhat.com\" <jbenc@redhat.com>","Subject":"Re: [ovs-dev] [PATCH v5 1/2] nsh: add new flow key 'ttl'","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=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"ovs-dev-bounces@openvswitch.org","Errors-To":"ovs-dev-bounces@openvswitch.org"}}]