From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id 5F5F91B5E0 for ; Mon, 16 Oct 2017 11:30:33 +0200 (CEST) Received: from orsmga003.jf.intel.com ([10.7.209.27]) by orsmga104.jf.intel.com with ESMTP; 16 Oct 2017 02:30:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,386,1503385200"; d="scan'208";a="1025596826" Received: from smonroyx-mobl.ger.corp.intel.com (HELO [10.252.25.241]) ([10.252.25.241]) by orsmga003.jf.intel.com with ESMTP; 16 Oct 2017 02:30:28 -0700 To: aviadye@dev.mellanox.co.il, dev@dpdk.org, pablo.de.lara.guarch@intel.com, aviadye@mellanox.com References: <1507987683-12315-1-git-send-email-aviadye@dev.mellanox.co.il> <1507987683-12315-5-git-send-email-aviadye@dev.mellanox.co.il> Cc: borisp@mellanox.com, akhil.goyal@nxp.com, hemant.agrawal@nxp.com, radu.nicolau@intel.com, declan.doherty@intel.com, liranl@mellanox.com, nelio.laranjeiro@6wind.com, thomas@monjalon.net From: Sergio Gonzalez Monroy Message-ID: Date: Mon, 16 Oct 2017 10:30:27 +0100 User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 MIME-Version: 1.0 In-Reply-To: <1507987683-12315-5-git-send-email-aviadye@dev.mellanox.co.il> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 05/11] examples/ipsec-secgw: Fixed transport X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Oct 2017 09:30:33 -0000 On 14/10/2017 14:27, aviadye@dev.mellanox.co.il wrote: > From: Aviad Yehezkel > > Seems like transport was broken for a long time Commit message needs to be improved. Just mentioned what is wrong or how do you fix it. Given that it is a fix, you should start the commit title with "fix ..." then also add the 'fixes' line with commit that added the bug. That way you can easily see since when it was introduced. Thanks, Sergio > Signed-off-by: Aviad Yehezkel > --- > examples/ipsec-secgw/esp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/examples/ipsec-secgw/esp.c b/examples/ipsec-secgw/esp.c > index 70bb81f..56ad7a0 100644 > --- a/examples/ipsec-secgw/esp.c > +++ b/examples/ipsec-secgw/esp.c > @@ -306,8 +306,8 @@ esp_outbound(struct rte_mbuf *m, struct ipsec_sa *sa, > sizeof(struct esp_hdr) + sa->iv_len); > memmove(new_ip, ip4, ip_hdr_len); > esp = (struct esp_hdr *)(new_ip + ip_hdr_len); > + ip4 = (struct ip *)new_ip; > if (likely(ip4->ip_v == IPVERSION)) { > - ip4 = (struct ip *)new_ip; > ip4->ip_p = IPPROTO_ESP; > ip4->ip_len = htons(rte_pktmbuf_data_len(m)); > } else {