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.mcnam...@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'.