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