DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: fix release notes for removed testpmd commands
@ 2019-10-09 13:47 Thomas Monjalon
  2019-10-09 14:57 ` David Marchand
  0 siblings, 1 reply; 3+ messages in thread
From: Thomas Monjalon @ 2019-10-09 13:47 UTC (permalink / raw)
  To: John McNamara, Marko Kovacevic; +Cc: dev, flavia.musatescu

The notes were not visible in the generated doc output because
of an indentation mistake.
While fixing the indentation, the formatting is improved.

Fixes: e5db17a1e54e ("app/testpmd: remove duplicated Rx offload commands")
Cc: flavia.musatescu@intel.com

Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
---
 doc/guides/rel_notes/release_19_11.rst | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/doc/guides/rel_notes/release_19_11.rst b/doc/guides/rel_notes/release_19_11.rst
index be91b04ad9..0062e012ed 100644
--- a/doc/guides/rel_notes/release_19_11.rst
+++ b/doc/guides/rel_notes/release_19_11.rst
@@ -91,14 +91,17 @@ Removed Items
    Also, make sure to start the actual text at the margin.
    =========================================================
 
-   * Removed duplicated set of commands for RX offloading configuration from app/testpmd:
-     “port config all crc-strip|scatter|rx-cksum|rx-timestamp|hw-vlan|hw-vlan-filter|
-     hw-vlan-strip|hw-vlan-extend on|off”.
-
-     The testpmd commands set that can be used instead in order to enable or disable Rx
-     offloading on all Rx queues of a port is:
-     "port config <port_id> rx_offload crc_strip|scatter|ipv4_cksum|udp_cksum|tcp_cksum|
-     timestamp|vlan_strip|vlan_filter|vlan_extend on|off"
+* Removed duplicated set of commands for Rx offload configuration from testpmd::
+
+    port config all crc-strip|scatter|rx-cksum|rx-timestamp|
+                    hw-vlan|hw-vlan-filter|hw-vlan-strip|hw-vlan-extend on|off
+
+  The testpmd commands set that can be used instead
+  in order to enable or disable Rx offloading on all Rx queues of a port is::
+
+    port config <port_id> rx_offload crc_strip|scatter|
+                                     ipv4_cksum|udp_cksum|tcp_cksum|timestamp|
+                                     vlan_strip|vlan_filter|vlan_extend on|off
 
 
 API Changes
-- 
2.23.0


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: fix release notes for removed testpmd commands
  2019-10-09 13:47 [dpdk-dev] [PATCH] doc: fix release notes for removed testpmd commands Thomas Monjalon
@ 2019-10-09 14:57 ` David Marchand
  2019-10-24 12:16   ` David Marchand
  0 siblings, 1 reply; 3+ messages in thread
From: David Marchand @ 2019-10-09 14:57 UTC (permalink / raw)
  To: Thomas Monjalon, John McNamara; +Cc: Marko Kovacevic, dev, Flavia

On Wed, Oct 9, 2019 at 3:47 PM Thomas Monjalon <thomas@monjalon.net> wrote:
>
> The notes were not visible in the generated doc output because
> of an indentation mistake.
> While fixing the indentation, the formatting is improved.
>
> Fixes: e5db17a1e54e ("app/testpmd: remove duplicated Rx offload commands")
> Cc: flavia.musatescu@intel.com
>
> Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> ---
>  doc/guides/rel_notes/release_19_11.rst | 19 +++++++++++--------
>  1 file changed, 11 insertions(+), 8 deletions(-)
>
> diff --git a/doc/guides/rel_notes/release_19_11.rst b/doc/guides/rel_notes/release_19_11.rst
> index be91b04ad9..0062e012ed 100644
> --- a/doc/guides/rel_notes/release_19_11.rst
> +++ b/doc/guides/rel_notes/release_19_11.rst
> @@ -91,14 +91,17 @@ Removed Items
>     Also, make sure to start the actual text at the margin.
>     =========================================================
>
> -   * Removed duplicated set of commands for RX offloading configuration from app/testpmd:
> -     “port config all crc-strip|scatter|rx-cksum|rx-timestamp|hw-vlan|hw-vlan-filter|
> -     hw-vlan-strip|hw-vlan-extend on|off”.
> -
> -     The testpmd commands set that can be used instead in order to enable or disable Rx
> -     offloading on all Rx queues of a port is:
> -     "port config <port_id> rx_offload crc_strip|scatter|ipv4_cksum|udp_cksum|tcp_cksum|
> -     timestamp|vlan_strip|vlan_filter|vlan_extend on|off"
> +* Removed duplicated set of commands for Rx offload configuration from testpmd::
> +
> +    port config all crc-strip|scatter|rx-cksum|rx-timestamp|
> +                    hw-vlan|hw-vlan-filter|hw-vlan-strip|hw-vlan-extend on|off
> +
> +  The testpmd commands set that can be used instead
> +  in order to enable or disable Rx offloading on all Rx queues of a port is::

A bit hard to digest: "can be used instead in order to".
Might be worth some proofreading, John?


> +
> +    port config <port_id> rx_offload crc_strip|scatter|
> +                                     ipv4_cksum|udp_cksum|tcp_cksum|timestamp|
> +                                     vlan_strip|vlan_filter|vlan_extend on|off
>
>
>  API Changes

But at least the generated doc now contains something :-)

Reviewed-by: David Marchand <david.marchand@redhat.com>



--
David Marchand


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [dpdk-dev] [PATCH] doc: fix release notes for removed testpmd commands
  2019-10-09 14:57 ` David Marchand
@ 2019-10-24 12:16   ` David Marchand
  0 siblings, 0 replies; 3+ messages in thread
From: David Marchand @ 2019-10-24 12:16 UTC (permalink / raw)
  To: Thomas Monjalon; +Cc: Marko Kovacevic, dev, Flavia, John McNamara

On Wed, Oct 9, 2019 at 4:57 PM David Marchand <david.marchand@redhat.com> wrote:
>
> On Wed, Oct 9, 2019 at 3:47 PM Thomas Monjalon <thomas@monjalon.net> wrote:
> >
> > The notes were not visible in the generated doc output because
> > of an indentation mistake.
> > While fixing the indentation, the formatting is improved.
> >
> > Fixes: e5db17a1e54e ("app/testpmd: remove duplicated Rx offload commands")

> >
> > Signed-off-by: Thomas Monjalon <thomas@monjalon.net>
> Reviewed-by: David Marchand <david.marchand@redhat.com>

Applied, thanks.


-- 
David Marchand


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-10-24 12:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-10-09 13:47 [dpdk-dev] [PATCH] doc: fix release notes for removed testpmd commands Thomas Monjalon
2019-10-09 14:57 ` David Marchand
2019-10-24 12:16   ` David Marchand

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).