From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stephen Hemminger Subject: Re: [PATCH v3 1/2] gro: code cleanup Date: Tue, 2 Jan 2018 17:27:48 -0800 Message-ID: <20180102172748.311bed66@xeon-e3> References: <1513219779-100115-1-git-send-email-jiayu.hu@intel.com> <1513927544-97241-1-git-send-email-jiayu.hu@intel.com> <1513927544-97241-2-git-send-email-jiayu.hu@intel.com> <20180102112600.GA10276@bricha3-MOBL3.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: "Richardson, Bruce" , "Hu, Jiayu" , "dev@dpdk.org" , "Chen, Junjie J" , "Ananyev, Konstantin" , "Yigit, Ferruh" , "Yao, Lei A" To: "Tan, Jianfeng" Return-path: Received: from mail-pl0-f66.google.com (mail-pl0-f66.google.com [209.85.160.66]) by dpdk.org (Postfix) with ESMTP id 82DE11B014 for ; Wed, 3 Jan 2018 02:27:51 +0100 (CET) Received: by mail-pl0-f66.google.com with SMTP id 1so221222plv.0 for ; Tue, 02 Jan 2018 17:27:51 -0800 (PST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Wed, 3 Jan 2018 01:07:37 +0000 "Tan, Jianfeng" wrote: > > -----Original Message----- > > From: Richardson, Bruce > > Sent: Tuesday, January 2, 2018 7:26 PM > > To: Hu, Jiayu > > Cc: dev@dpdk.org; Tan, Jianfeng; Chen, Junjie J; Ananyev, Konstantin; > > stephen@networkplumber.org; Yigit, Ferruh; Yao, Lei A > > Subject: Re: [dpdk-dev] [PATCH v3 1/2] gro: code cleanup > > > > On Fri, Dec 22, 2017 at 03:25:43PM +0800, Jiayu Hu wrote: > > > - Remove needless check and variants > > > - For better understanding, update the programmer guide and rename > > > internal functions and variants > > > - For supporting tunneled gro, move common internal functions from > > > gro_tcp4.c to gro_tcp4.h > > > - Comply RFC 6864 to process the IPv4 ID field > > > > > > Signed-off-by: Jiayu Hu > > > --- > > > .../prog_guide/generic_receive_offload_lib.rst | 246 ++++++++------- > > > doc/guides/prog_guide/img/gro-key-algorithm.png | Bin 0 -> 28231 > > bytes > > > > Rather than binary PNG images, please use SVG files (note, real SVG, not > > an SVG file with a binary blob pasted into it). > > Based on my limited experience, there is no shortcut for this, but re-draw the picture with tools like visio. Inkscape is open source and produces svg files.