DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] doc: update ixgbe doc
@ 2017-05-03  5:54 Wenzhuo Lu
  2017-05-09 16:38 ` Mcnamara, John
  2017-05-10  1:39 ` [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf Wenzhuo Lu
  0 siblings, 2 replies; 6+ messages in thread
From: Wenzhuo Lu @ 2017-05-03  5:54 UTC (permalink / raw)
  To: dev; +Cc: john.mcnamara, Wenzhuo Lu

Add more explanation about how to disable MDD on
kernel PF.

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---
 doc/guides/nics/ixgbe.rst | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index 130765b..6773fe5 100644
--- a/doc/guides/nics/ixgbe.rst
+++ b/doc/guides/nics/ixgbe.rst
@@ -186,9 +186,12 @@ the advanced context descriptor should be set and set it. And DPDK has to ask
 the info about the header length from the upper layer, because parsing the
 packet itself is not acceptable. So, it's too expensive to support MDD.
 When using kernel PF + DPDK VF on x550, please make sure using the kernel
-driver that disables MDD or can disable MDD. (Some kernel driver can use
-this CLI 'insmod ixgbe.ko MDD=0,0' to disable MDD. Some kernel driver disables
-it by default.)
+PF driver that disables MDD or can disable MDD.
+Some kernel driver already disables MDD by default.
+Some kernel driver can use this CLI "insmod ixgbe.ko MDD=0,0" to disable MDD.
+Every "0" in the CLI means a port. Users need to add more "0"s if the machine
+has more ports. For example, if there're 6 ixgbe ports, the CLI should be
+changed to "insmod ixgbe.ko MDD=0,0,0,0,0,0".
 
 
 Statistics
-- 
1.9.3

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

* Re: [dpdk-dev] [PATCH] doc: update ixgbe doc
  2017-05-03  5:54 [dpdk-dev] [PATCH] doc: update ixgbe doc Wenzhuo Lu
@ 2017-05-09 16:38 ` Mcnamara, John
  2017-05-10  1:03   ` Lu, Wenzhuo
  2017-05-10  1:39 ` [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf Wenzhuo Lu
  1 sibling, 1 reply; 6+ messages in thread
From: Mcnamara, John @ 2017-05-09 16:38 UTC (permalink / raw)
  To: Lu, Wenzhuo, dev



> -----Original Message-----
> From: Lu, Wenzhuo
> Sent: Wednesday, May 3, 2017 6:55 AM
> To: dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>
> Subject: [PATCH] doc: update ixgbe doc
> 
> Add more explanation about how to disable MDD on kernel PF.
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
> ---
>  doc/guides/nics/ixgbe.rst | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
> 
> diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst index
> 130765b..6773fe5 100644
> --- a/doc/guides/nics/ixgbe.rst
> +++ b/doc/guides/nics/ixgbe.rst
> @@ -186,9 +186,12 @@ the advanced context descriptor should be set and set
> it. And DPDK has to ask  the info about the header length from the upper
> layer, because parsing the  packet itself is not acceptable. So, it's too
> expensive to support MDD.
>  When using kernel PF + DPDK VF on x550, please make sure using the kernel
> -driver that disables MDD or can disable MDD. (Some kernel driver can use
> -this CLI 'insmod ixgbe.ko MDD=0,0' to disable MDD. Some kernel driver
> disables -it by default.)
> +PF driver that disables MDD or can disable MDD.
> +Some kernel driver already disables MDD by default.
> +Some kernel driver can use this CLI "insmod ixgbe.ko MDD=0,0" to disable
> MDD.
> +Every "0" in the CLI means a port. Users need to add more "0"s if the
> +machine has more ports. For example, if there're 6 ixgbe ports, the CLI
> +should be changed to "insmod ixgbe.ko MDD=0,0,0,0,0,0".

Something like this would be better:

...
When using kernel PF + DPDK VF on x550, please make sure to use a kernel
PF driver that disables MDD or can disable MDD.

Some kernel drivers already disable MDD by default while some kernels can use
the command ``insmod ixgbe.ko MDD=0,0`` to disable MDD. Each "0" in the
command refers to a port. For example, if there are 6 ixgbe ports, the command
should be changed to ``insmod ixgbe.ko MDD=0,0,0,0,0,0``.

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

* Re: [dpdk-dev] [PATCH] doc: update ixgbe doc
  2017-05-09 16:38 ` Mcnamara, John
@ 2017-05-10  1:03   ` Lu, Wenzhuo
  0 siblings, 0 replies; 6+ messages in thread
From: Lu, Wenzhuo @ 2017-05-10  1:03 UTC (permalink / raw)
  To: Mcnamara, John, dev

Hi John,

> -----Original Message-----
> From: Mcnamara, John
> Sent: Wednesday, May 10, 2017 12:39 AM
> To: Lu, Wenzhuo; dev@dpdk.org
> Subject: RE: [PATCH] doc: update ixgbe doc
> 
> 
> 
> > -----Original Message-----
> > From: Lu, Wenzhuo
> > Sent: Wednesday, May 3, 2017 6:55 AM
> > To: dev@dpdk.org
> > Cc: Mcnamara, John <john.mcnamara@intel.com>; Lu, Wenzhuo
> > <wenzhuo.lu@intel.com>
> > Subject: [PATCH] doc: update ixgbe doc
> >
> > Add more explanation about how to disable MDD on kernel PF.
> >
> > Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
> > ---
> >  doc/guides/nics/ixgbe.rst | 9 ++++++---
> >  1 file changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
> > index
> > 130765b..6773fe5 100644
> > --- a/doc/guides/nics/ixgbe.rst
> > +++ b/doc/guides/nics/ixgbe.rst
> > @@ -186,9 +186,12 @@ the advanced context descriptor should be set
> and
> > set it. And DPDK has to ask  the info about the header length from the
> > upper layer, because parsing the  packet itself is not acceptable. So,
> > it's too expensive to support MDD.
> >  When using kernel PF + DPDK VF on x550, please make sure using the
> > kernel -driver that disables MDD or can disable MDD. (Some kernel
> > driver can use -this CLI 'insmod ixgbe.ko MDD=0,0' to disable MDD.
> > Some kernel driver disables -it by default.)
> > +PF driver that disables MDD or can disable MDD.
> > +Some kernel driver already disables MDD by default.
> > +Some kernel driver can use this CLI "insmod ixgbe.ko MDD=0,0" to
> > +disable
> > MDD.
> > +Every "0" in the CLI means a port. Users need to add more "0"s if the
> > +machine has more ports. For example, if there're 6 ixgbe ports, the
> > +CLI should be changed to "insmod ixgbe.ko MDD=0,0,0,0,0,0".
> 
> Something like this would be better:
> 
> ...
> When using kernel PF + DPDK VF on x550, please make sure to use a kernel
> PF driver that disables MDD or can disable MDD.
> 
> Some kernel drivers already disable MDD by default while some kernels can
> use the command ``insmod ixgbe.ko MDD=0,0`` to disable MDD. Each "0" in
> the command refers to a port. For example, if there are 6 ixgbe ports, the
> command should be changed to ``insmod ixgbe.ko MDD=0,0,0,0,0,0``.
Thanks for your comments. I'll reword the patch.

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

* [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf
  2017-05-03  5:54 [dpdk-dev] [PATCH] doc: update ixgbe doc Wenzhuo Lu
  2017-05-09 16:38 ` Mcnamara, John
@ 2017-05-10  1:39 ` Wenzhuo Lu
  2017-05-10  9:36   ` Mcnamara, John
  1 sibling, 1 reply; 6+ messages in thread
From: Wenzhuo Lu @ 2017-05-10  1:39 UTC (permalink / raw)
  To: dev; +Cc: john.mcnamara, Wenzhuo Lu

Add a better explanation about how to disable MDD (Malicious Driver
Detection) in a kernel PF.

Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
---

V2: fix wording.

 doc/guides/nics/ixgbe.rst | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/doc/guides/nics/ixgbe.rst b/doc/guides/nics/ixgbe.rst
index 130765b..696ff69 100644
--- a/doc/guides/nics/ixgbe.rst
+++ b/doc/guides/nics/ixgbe.rst
@@ -185,10 +185,13 @@ There's significant performance impact to support MDD. DPDK should check if
 the advanced context descriptor should be set and set it. And DPDK has to ask
 the info about the header length from the upper layer, because parsing the
 packet itself is not acceptable. So, it's too expensive to support MDD.
-When using kernel PF + DPDK VF on x550, please make sure using the kernel
-driver that disables MDD or can disable MDD. (Some kernel driver can use
-this CLI 'insmod ixgbe.ko MDD=0,0' to disable MDD. Some kernel driver disables
-it by default.)
+When using kernel PF + DPDK VF on x550, please make sure to use a kernel
+PF driver that disables MDD or can disable MDD.
+
+Some kernel drivers already disable MDD by default while some kernels can use
+the command ``insmod ixgbe.ko MDD=0,0`` to disable MDD. Each "0" in the
+command refers to a port. For example, if there are 6 ixgbe ports, the command
+should be changed to ``insmod ixgbe.ko MDD=0,0,0,0,0,0``.
 
 
 Statistics
-- 
1.9.3

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

* Re: [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf
  2017-05-10  1:39 ` [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf Wenzhuo Lu
@ 2017-05-10  9:36   ` Mcnamara, John
  2017-05-10 20:15     ` Thomas Monjalon
  0 siblings, 1 reply; 6+ messages in thread
From: Mcnamara, John @ 2017-05-10  9:36 UTC (permalink / raw)
  To: Lu, Wenzhuo, dev



> -----Original Message-----
> From: Lu, Wenzhuo
> Sent: Wednesday, May 10, 2017 2:40 AM
> To: dev@dpdk.org
> Cc: Mcnamara, John <john.mcnamara@intel.com>; Lu, Wenzhuo
> <wenzhuo.lu@intel.com>
> Subject: [PATCH v2] doc: update ixgbe doc about mdd in kernel pf
> 
> Add a better explanation about how to disable MDD (Malicious Driver
> Detection) in a kernel PF.
> 
> Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>


Acked-by: John McNamara <john.mcnamara@intel.com>

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

* Re: [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf
  2017-05-10  9:36   ` Mcnamara, John
@ 2017-05-10 20:15     ` Thomas Monjalon
  0 siblings, 0 replies; 6+ messages in thread
From: Thomas Monjalon @ 2017-05-10 20:15 UTC (permalink / raw)
  To: Lu, Wenzhuo; +Cc: dev, Mcnamara, John

10/05/2017 11:36, Mcnamara, John:
> From: Lu, Wenzhuo
> > 
> > Add a better explanation about how to disable MDD (Malicious Driver
> > Detection) in a kernel PF.
> > 
> > Signed-off-by: Wenzhuo Lu <wenzhuo.lu@intel.com>
> 
> Acked-by: John McNamara <john.mcnamara@intel.com>

Applied, thanks

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

end of thread, other threads:[~2017-05-10 20:15 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-03  5:54 [dpdk-dev] [PATCH] doc: update ixgbe doc Wenzhuo Lu
2017-05-09 16:38 ` Mcnamara, John
2017-05-10  1:03   ` Lu, Wenzhuo
2017-05-10  1:39 ` [dpdk-dev] [PATCH v2] doc: update ixgbe doc about mdd in kernel pf Wenzhuo Lu
2017-05-10  9:36   ` Mcnamara, John
2017-05-10 20:15     ` 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).