[dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide

Kavanagh, Mark B mark.b.kavanagh at intel.com
Thu Oct 12 17:56:54 CEST 2017


>From: Mcnamara, John
>Sent: Thursday, October 12, 2017 4:29 PM
>To: Kavanagh, Mark B <mark.b.kavanagh at intel.com>; dev at dpdk.org
>Cc: Hu, Jiayu <jiayu.hu at intel.com>; Tan, Jianfeng <jianfeng.tan at intel.com>;
>Ananyev, Konstantin <konstantin.ananyev at intel.com>; Yigit, Ferruh
><ferruh.yigit at intel.com>; thomas at monjalon.net; Kavanagh, Mark B
><mark.b.kavanagh at intel.com>
>Subject: RE: [dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide
>
>
>
>> -----Original Message-----
>> From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Mark Kavanagh
>> Sent: Thursday, October 12, 2017 2:46 PM
>> To: dev at dpdk.org
>> Cc: Hu, Jiayu <jiayu.hu at intel.com>; Tan, Jianfeng
>> <jianfeng.tan at intel.com>; Ananyev, Konstantin
>> <konstantin.ananyev at intel.com>; Yigit, Ferruh <ferruh.yigit at intel.com>;
>> thomas at monjalon.net; Kavanagh, Mark B <mark.b.kavanagh at intel.com>
>> Subject: [dpdk-dev] [PATCH] doc: minor fixes for GSO prog_guide
>>
>> 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.
>>
>> Fixes: f6010c7 ("doc: add GSO programmer's guide")
>>
>> Signed-off-by: Mark Kavanagh <mark.b.kavanagh at intel.com>
>
>
>
>> +.. _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,

Hey John,

Thanks for the feedback - responses inline.

I'll post v2 now, and carry forward your Ack.

Cheers,
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.

I've moved the note back to its original position, but corrected the indentation, so this target can be removed.

>
>Also, the note isn't indented to the level of the bullet point. I don't know
>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.

As above (the syntax highlighting of my editor shows indented 'note' tags as incorrect, which is why indentation was incorrect in the first place!).

>
>With or without these changes:
>
>Acked-by: John McNamara <john.mcnamara at intel.com>
>
>



More information about the dev mailing list