From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 7CCCF1B3C2 for ; Thu, 12 Oct 2017 17:28:51 +0200 (CEST) Received: from fmsmga004.fm.intel.com ([10.253.24.48]) by fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Oct 2017 08:28:42 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.43,366,1503385200"; d="scan'208";a="322509329" Received: from irsmsx110.ger.corp.intel.com ([163.33.3.25]) by fmsmga004.fm.intel.com with ESMTP; 12 Oct 2017 08:28:42 -0700 Received: from irsmsx103.ger.corp.intel.com ([169.254.3.49]) by irsmsx110.ger.corp.intel.com ([163.33.3.25]) with mapi id 14.03.0319.002; Thu, 12 Oct 2017 16:28:41 +0100 From: "Mcnamara, John" To: "Kavanagh, Mark B" , "dev@dpdk.org" CC: "Hu, Jiayu" , "Tan, Jianfeng" , "Ananyev, Konstantin" , "Yigit, Ferruh" , "thomas@monjalon.net" , "Kavanagh, Mark B" Thread-Topic: [dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide Thread-Index: AQHTQ2CC/9vjurcCp0OAkUgEFEqhrqLgVUbw Date: Thu, 12 Oct 2017 15:28:40 +0000 Message-ID: References: <1507815955-23275-1-git-send-email-mark.b.kavanagh@intel.com> In-Reply-To: <1507815955-23275-1-git-send-email-mark.b.kavanagh@intel.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_IC x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZjMwN2Q5ODQtNGQ5ZC00ZmM5LTgxYjEtNGZjMjNmOGU0ZTg4IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6IkpRMUNQREp5ZWg4U1p5MEhKMGdFekdyV002a3Z4NVllUWdYVUxcL1JNS1lVPSJ9 dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action x-originating-ip: [163.33.239.181] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Subject: Re: [dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide 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: Thu, 12 Oct 2017 15:28:52 -0000 > -----Original Message----- > From: dev [mailto:dev-bounces@dpdk.org] On Behalf Of Mark Kavanagh > Sent: Thursday, October 12, 2017 2:46 PM > To: dev@dpdk.org > Cc: Hu, Jiayu ; Tan, Jianfeng > ; Ananyev, Konstantin > ; Yigit, Ferruh ; > thomas@monjalon.net; Kavanagh, Mark B > Subject: [dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide >=20 > Correct two minor issues in the GSO programmer's guide: > - a note is rendered incorrectly in the middle of an unordered list; > this results in the remainder of the list appearing inside the note. > Move note to end of the list to resolve same. > - two minor visual artifacts are present in the 'three-part-output- > segment' > diagram. Remove same. >=20 > Fixes: f6010c7 ("doc: add GSO programmer's guide") >=20 > Signed-off-by: Mark Kavanagh > +.. _note: > +.. note:: > + > + An application may use the same pool for both direct and indirect > + buffers. However, since indirect mbufs simply store a pointer, the Hi Mark, The _note target should be more specific since the these targets are used across all the documentation. So something like _gso_note would be better. Or alternatively leave out the target and just say something in the text like "See the note below.", without the link, since the note is quite close to the referring text. Also, the note isn't indented to the level of the bullet point. I don't kno= w if that is intentional or not but I would say there should be an additional 3 spaces indentation from ..note:: to the end of the note. With or without these changes: Acked-by: John McNamara