* [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).