[dpdk-dev] [PATCH v2] doc: update release notes for 17.05

[Shreyansh Jain]

On Wednesday 10 May 2017 10:29 AM, Shreyansh Jain wrote:
> Hello John,
>
> Some super trivial comments inline:
>
> On Tuesday 09 May 2017 10:26 PM, John McNamara wrote:
>> Fix grammar, spelling and formatting of DPDK 17.05 release notes.
>>
>> Signed-off-by: John McNamara <john.mcnamara at intel.com>
>> ---
>>
>> V2: Added updated MLX4/5 text. This patch supercedes patch 24166.
>>
>>  doc/guides/rel_notes/release_17_05.rst | 197
>> +++++++++++++++------------------
>>  1 file changed, 91 insertions(+), 106 deletions(-)
>>
>> diff --git a/doc/guides/rel_notes/release_17_05.rst
>> b/doc/guides/rel_notes/release_17_05.rst
>> index 4b47ae1..f911711 100644
>> --- a/doc/guides/rel_notes/release_17_05.rst
>> +++ b/doc/guides/rel_notes/release_17_05.rst
>> @@ -41,16 +41,18 @@ New Features
>>       Also, make sure to start the actual text at the margin.
>>       =========================================================
>>
>> -* **Reorganized the mbuf structure.**
>> +* **Reorganized mbuf structure.**
>> +
>> +  The mbuf structure has been reorganized as follows:
>>
>>    * Align fields to facilitate the writing of ``data_off``,
>> ``refcnt``, and
>>      ``nb_segs`` in one operation.
>>    * Use 2 bytes for port and number of segments.
>> -  * Move the sequence number in the second cache line.
>> +  * Move the sequence number to the second cache line.
>>    * Add a timestamp field.
>>    * Set default value for ``refcnt``, ``next`` and ``nb_segs`` at
>> mbuf free.
>>
>> -* **Added mbuf raw free API**
>> +* **Added mbuf raw free API.**
>>
>>    Moved ``rte_mbuf_raw_free()`` and ``rte_pktmbuf_prefree_seg()``
>> functions to
>>    the public API.
>> @@ -58,14 +60,14 @@ New Features
>>  * **Added free Tx mbuf on demand API.**
>>
>>    Added a new function ``rte_eth_tx_done_cleanup()`` which allows an
>> application
>> -  to request the driver to release mbufs from their Tx ring that are
>> no longer
>> -  in use, independent of whether or not the ``tx_rs_thresh`` has been
>> crossed.
>> +  to request the driver to release mbufs from that are no longer in
>> use from a
>
> isn't the 'that' in '..mbufs from that are no...' above extra?
>

Actually, the 'from' is extra in the line above, not 'that'.