DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: add kni changes to release note
@ 2018-11-05 18:29 Dan Gora
  2018-11-05 19:06 ` Ferruh Yigit
  2018-11-05 19:54 ` [dpdk-dev] [PATCH v2] " Dan Gora
  0 siblings, 2 replies; 6+ messages in thread
From: Dan Gora @ 2018-11-05 18:29 UTC (permalink / raw)
  To: dev; +Cc: Ferruh Yigit, Dan Gora, John McNamara, Marko Kovacevic

Add the new module parameter for the KNI kernel module, the new command
line flag for the KNI sample application, and the new API function
'rte_kni_update_link()' to the release note.

Signed-off-by: Dan Gora <dg@adax.com>

---
 doc/guides/rel_notes/release_18_11.rst | 13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/doc/guides/rel_notes/release_18_11.rst b/doc/guides/rel_notes/release_18_11.rst
index cfa92b8c0..43fe849db 100644
--- a/doc/guides/rel_notes/release_18_11.rst
+++ b/doc/guides/rel_notes/release_18_11.rst
@@ -301,6 +301,16 @@ New Features
   computation to the NIST Cryptographic Algorithm Validation Program (CAVP)
   test vectors.
 
+* **Updated KNI kernel module and sample application.**
+
+  Updated the KNI kernel module with a new kernel module parameter,
+  ``carrier=[on|off]`` to allow the user to control the default carrier
+  state of KNI kernel network interfaces.
+
+  Added a new command line flag ``-m`` to the KNI sample application to
+  monitor and automatically reflect the physical NIC carrier state to the
+  KNI kernel network interface with the new ``rte_kni_update_link()`` API.
+
 
 API Changes
 -----------
@@ -362,6 +372,9 @@ API Changes
   and seen as invalid because of its state ``RTE_ETH_DEV_UNUSED``.
   This new behaviour is enabled per driver for a migration period.
 
+* kni: Added the new API function ``rte_kni_update_link()`` to allow the user
+  to set the carrier state of the KNI kernel network interface.
+
 * A new device flag, RTE_ETH_DEV_NOLIVE_MAC_ADDR, changes the order of
   actions inside rte_eth_dev_start regarding MAC set. Some NICs do not
   support MAC changes once the port has started and with this new device
-- 
2.19.0

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

* Re: [dpdk-dev] [PATCH] doc: add kni changes to release note
  2018-11-05 18:29 [dpdk-dev] [PATCH] doc: add kni changes to release note Dan Gora
@ 2018-11-05 19:06 ` Ferruh Yigit
  2018-11-05 19:27   ` Dan Gora
  2018-11-05 19:54 ` [dpdk-dev] [PATCH v2] " Dan Gora
  1 sibling, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2018-11-05 19:06 UTC (permalink / raw)
  To: Dan Gora, dev; +Cc: John McNamara, Marko Kovacevic

On 11/5/2018 6:29 PM, Dan Gora wrote:
> Add the new module parameter for the KNI kernel module, the new command
> line flag for the KNI sample application, and the new API function
> 'rte_kni_update_link()' to the release note.
> 
> Signed-off-by: Dan Gora <dg@adax.com>
> 
> ---
>  doc/guides/rel_notes/release_18_11.rst | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
> 
> diff --git a/doc/guides/rel_notes/release_18_11.rst b/doc/guides/rel_notes/release_18_11.rst
> index cfa92b8c0..43fe849db 100644
> --- a/doc/guides/rel_notes/release_18_11.rst
> +++ b/doc/guides/rel_notes/release_18_11.rst
> @@ -301,6 +301,16 @@ New Features
>    computation to the NIST Cryptographic Algorithm Validation Program (CAVP)
>    test vectors.
>  
> +* **Updated KNI kernel module and sample application.**
> +
> +  Updated the KNI kernel module with a new kernel module parameter,
> +  ``carrier=[on|off]`` to allow the user to control the default carrier
> +  state of KNI kernel network interfaces.
> +
> +  Added a new command line flag ``-m`` to the KNI sample application to
> +  monitor and automatically reflect the physical NIC carrier state to the
> +  KNI kernel network interface with the new ``rte_kni_update_link()`` API.
> +
>  
>  API Changes
>  -----------
> @@ -362,6 +372,9 @@ API Changes
>    and seen as invalid because of its state ``RTE_ETH_DEV_UNUSED``.
>    This new behaviour is enabled per driver for a migration period.
>  
> +* kni: Added the new API function ``rte_kni_update_link()`` to allow the user
> +  to set the carrier state of the KNI kernel network interface.
> +

It is good to document new API, which can be on "new features" section above, we
mostly don't document new APIs in "api changes" section, but also it is
important to document behavior change too.
Now by default kni interfaces will have carrier off, and there won't be any
traffic and people may get stuck, this is to help them, that is why I mentioned
sysfs interface, since this patch replaces that one, can you please add those
information here?

>  * A new device flag, RTE_ETH_DEV_NOLIVE_MAC_ADDR, changes the order of
>    actions inside rte_eth_dev_start regarding MAC set. Some NICs do not
>    support MAC changes once the port has started and with this new device
> 

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

* Re: [dpdk-dev] [PATCH] doc: add kni changes to release note
  2018-11-05 19:06 ` Ferruh Yigit
@ 2018-11-05 19:27   ` Dan Gora
  0 siblings, 0 replies; 6+ messages in thread
From: Dan Gora @ 2018-11-05 19:27 UTC (permalink / raw)
  To: Ferruh Yigit; +Cc: dev, John McNamara, Marko Kovacevic

On Mon, Nov 5, 2018 at 5:06 PM Ferruh Yigit <ferruh.yigit@intel.com> wrote:
> > +* kni: Added the new API function ``rte_kni_update_link()`` to allow the user
> > +  to set the carrier state of the KNI kernel network interface.
> > +
>
> It is good to document new API, which can be on "new features" section above, we
> mostly don't document new APIs in "api changes" section, but also it is
> important to document behavior change too.

That doesn't really make any sense.. APIs which are removed are
documented here, why wouldn't API additions be documented there as
well?

The new API function is an API change, it's not really a new DPDK
"feature" like a new subsystem or something..

> Now by default kni interfaces will have carrier off, and there won't be any
> traffic and people may get stuck, this is to help them, that is why I mentioned
> sysfs interface, since this patch replaces that one, can you please add those
> information here?

The sysfs trick was never really an "interface", it is/was completely
external to DPDK, but I'll add a blurb about it if you think that
it'll help..

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

* [dpdk-dev] [PATCH v2] doc: add kni changes to release note
  2018-11-05 18:29 [dpdk-dev] [PATCH] doc: add kni changes to release note Dan Gora
  2018-11-05 19:06 ` Ferruh Yigit
@ 2018-11-05 19:54 ` Dan Gora
  2018-11-14  0:34   ` Ferruh Yigit
  1 sibling, 1 reply; 6+ messages in thread
From: Dan Gora @ 2018-11-05 19:54 UTC (permalink / raw)
  To: dev; +Cc: Ferruh Yigit, Dan Gora, John McNamara, Marko Kovacevic

Add the new module parameter for the KNI kernel module, the new command
line flag for the KNI sample application, and the new API function
'rte_kni_update_link()' to the release note.

Signed-off-by: Dan Gora <dg@adax.com>
---
 doc/guides/rel_notes/release_18_11.rst | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

diff --git a/doc/guides/rel_notes/release_18_11.rst b/doc/guides/rel_notes/release_18_11.rst
index cfa92b8c0..9486faa1f 100644
--- a/doc/guides/rel_notes/release_18_11.rst
+++ b/doc/guides/rel_notes/release_18_11.rst
@@ -301,6 +301,25 @@ New Features
   computation to the NIST Cryptographic Algorithm Validation Program (CAVP)
   test vectors.
 
+* **Updated KNI kernel module, rte_kni library, and KNI sample application.**
+
+  Updated the KNI kernel module with a new kernel module parameter,
+  ``carrier=[on|off]`` to allow the user to control the default carrier
+  state of KNI kernel network interfaces.  The default carrier state
+  is now set to ``off``, so the interfaces cannot be used until the
+  carrier state is set to ``on`` via ``rte_kni_update_link`` or
+  by writing ``1`` to ``/sys/devices/virtual/net/<iface>/carrier``.
+  In previous versions the default carrier state was left undefined.
+  See :doc:`../prog_guide/kernel_nic_interface` for more information.
+
+  Added the new API function ``rte_kni_update_link`` to allow the user
+  to set the carrier state of the KNI kernel network interface.
+
+  Added a new command line flag ``-m`` to the KNI sample application to
+  monitor and automatically reflect the physical NIC carrier state to the
+  KNI kernel network interface with the new ``rte_kni_update_link`` API.
+  See :doc:`../sample_app_ug/kernel_nic_interface` for more information.
+
 
 API Changes
 -----------
-- 
2.19.0

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

* Re: [dpdk-dev] [PATCH v2] doc: add kni changes to release note
  2018-11-05 19:54 ` [dpdk-dev] [PATCH v2] " Dan Gora
@ 2018-11-14  0:34   ` Ferruh Yigit
  2018-11-18 23:17     ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Ferruh Yigit @ 2018-11-14  0:34 UTC (permalink / raw)
  To: Dan Gora, dev; +Cc: John McNamara, Marko Kovacevic

On 11/5/2018 7:54 PM, Dan Gora wrote:
> Add the new module parameter for the KNI kernel module, the new command
> line flag for the KNI sample application, and the new API function
> 'rte_kni_update_link()' to the release note.
> 
> Signed-off-by: Dan Gora <dg@adax.com>

Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: add kni changes to release note
  2018-11-14  0:34   ` Ferruh Yigit
@ 2018-11-18 23:17     ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2018-11-18 23:17 UTC (permalink / raw)
  To: Dan Gora; +Cc: dev, Ferruh Yigit, John McNamara, Marko Kovacevic

14/11/2018 01:34, Ferruh Yigit:
> On 11/5/2018 7:54 PM, Dan Gora wrote:
> > Add the new module parameter for the KNI kernel module, the new command
> > line flag for the KNI sample application, and the new API function
> > 'rte_kni_update_link()' to the release note.
> > 
> > Signed-off-by: Dan Gora <dg@adax.com>
> 
> Acked-by: Ferruh Yigit <ferruh.yigit@intel.com>

Applied, thanks

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

end of thread, other threads:[~2018-11-18 23:17 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-11-05 18:29 [dpdk-dev] [PATCH] doc: add kni changes to release note Dan Gora
2018-11-05 19:06 ` Ferruh Yigit
2018-11-05 19:27   ` Dan Gora
2018-11-05 19:54 ` [dpdk-dev] [PATCH v2] " Dan Gora
2018-11-14  0:34   ` Ferruh Yigit
2018-11-18 23:17     ` Thomas Monjalon

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