Message ID | 1291974667-30254-4-git-send-email-jeffrey.t.kirsher@intel.com |
---|---|
State | Changes Requested, archived |
Delegated to: | David Miller |
Headers | show |
On Fri, 2010-12-10 at 01:50 -0800, Jeff Kirsher wrote: > Update Intel Wired LAN igb documentation. > > Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> > --- > Documentation/networking/igb.txt | 22 +++++++++++++++++++--- > 1 files changed, 19 insertions(+), 3 deletions(-) > > diff --git a/Documentation/networking/igb.txt b/Documentation/networking/igb.txt > index ab2d718..393bdb7 100644 > --- a/Documentation/networking/igb.txt > +++ b/Documentation/networking/igb.txt > @@ -36,6 +36,7 @@ Default Value: 0 > This parameter adds support for SR-IOV. It causes the driver to spawn up to > max_vfs worth of virtual function. > > + > Additional Configurations > ========================= > > @@ -60,7 +61,8 @@ Additional Configurations > Ethtool > ------- > The driver utilizes the ethtool interface for driver configuration and > - diagnostics, as well as displaying statistical information. > + diagnostics, as well as displaying statistical information. The latest > + version of Ethtool can be found at: > > http://sourceforge.net/projects/gkernel. Please update this to: http://ftp.kernel.org/pub/software/network/ethtool/ > @@ -103,8 +105,8 @@ Additional Configurations > > NOTE: You need to have inet_lro enabled via either the CONFIG_INET_LRO or > CONFIG_INET_LRO_MODULE kernel config option. Additionally, if > - CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded > - before the igb driver. > + CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded before > + the igb driver. This should be removed as you don't use inet_lro any more. > You can verify that the driver is using LRO by looking at these counters in > Ethtool: > @@ -116,6 +118,20 @@ Additional Configurations > > NOTE: IPv6 and UDP are not supported by LRO. > > + MAC and VLAN anti-spoofing feature > + ---------------------------------- > + When a malicious driver attempts to send a spoofed packet, it is dropped by > + the hardware and not transmitted. An interrupt is sent to the PF driver > + notifying it of the spoof attempt. > + > + When a spoofed packet is detected the PF driver will send the following > + message to the system log (displayed by the "dmesg" command): > + > + Spoof event(s) detected on VF(n) > + > + Where n=the VF that attempted to do the spoofing. I can't see that message in the PF driver code; does this actually apply to the in-tree driver? Also I hope this is rate-limited. Ben.
From: Ben Hutchings <bhutchings@solarflare.com> Date: Fri, 10 Dec 2010 15:50:24 +0000 > On Fri, 2010-12-10 at 01:50 -0800, Jeff Kirsher wrote: >> Update Intel Wired LAN igb documentation. >> >> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >> --- >> Documentation/networking/igb.txt | 22 +++++++++++++++++++--- >> 1 files changed, 19 insertions(+), 3 deletions(-) >> >> diff --git a/Documentation/networking/igb.txt b/Documentation/networking/igb.txt >> index ab2d718..393bdb7 100644 >> --- a/Documentation/networking/igb.txt >> +++ b/Documentation/networking/igb.txt >> @@ -36,6 +36,7 @@ Default Value: 0 >> This parameter adds support for SR-IOV. It causes the driver to spawn up to >> max_vfs worth of virtual function. >> >> + >> Additional Configurations >> ========================= >> >> @@ -60,7 +61,8 @@ Additional Configurations >> Ethtool >> ------- >> The driver utilizes the ethtool interface for driver configuration and >> - diagnostics, as well as displaying statistical information. >> + diagnostics, as well as displaying statistical information. The latest >> + version of Ethtool can be found at: >> >> http://sourceforge.net/projects/gkernel. > > Please update this to: > http://ftp.kernel.org/pub/software/network/ethtool/ Indeed. Intel folks, please address all of Ben's feedback and respin this pull request, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Fri, Dec 10, 2010 at 07:50, Ben Hutchings <bhutchings@solarflare.com> wrote: > On Fri, 2010-12-10 at 01:50 -0800, Jeff Kirsher wrote: >> Update Intel Wired LAN igb documentation. >> >> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >> --- >> Documentation/networking/igb.txt | 22 +++++++++++++++++++--- >> 1 files changed, 19 insertions(+), 3 deletions(-) >> >> diff --git a/Documentation/networking/igb.txt b/Documentation/networking/igb.txt >> index ab2d718..393bdb7 100644 >> --- a/Documentation/networking/igb.txt >> +++ b/Documentation/networking/igb.txt >> @@ -36,6 +36,7 @@ Default Value: 0 >> This parameter adds support for SR-IOV. It causes the driver to spawn up to >> max_vfs worth of virtual function. >> >> + >> Additional Configurations >> ========================= >> >> @@ -60,7 +61,8 @@ Additional Configurations >> Ethtool >> ------- >> The driver utilizes the ethtool interface for driver configuration and >> - diagnostics, as well as displaying statistical information. >> + diagnostics, as well as displaying statistical information. The latest >> + version of Ethtool can be found at: >> >> http://sourceforge.net/projects/gkernel. > > Please update this to: > http://ftp.kernel.org/pub/software/network/ethtool/ > >> @@ -103,8 +105,8 @@ Additional Configurations >> >> NOTE: You need to have inet_lro enabled via either the CONFIG_INET_LRO or >> CONFIG_INET_LRO_MODULE kernel config option. Additionally, if >> - CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded >> - before the igb driver. >> + CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded before >> + the igb driver. > > This should be removed as you don't use inet_lro any more. > >> You can verify that the driver is using LRO by looking at these counters in >> Ethtool: >> @@ -116,6 +118,20 @@ Additional Configurations >> >> NOTE: IPv6 and UDP are not supported by LRO. >> >> + MAC and VLAN anti-spoofing feature >> + ---------------------------------- >> + When a malicious driver attempts to send a spoofed packet, it is dropped by >> + the hardware and not transmitted. An interrupt is sent to the PF driver >> + notifying it of the spoof attempt. >> + >> + When a spoofed packet is detected the PF driver will send the following >> + message to the system log (displayed by the "dmesg" command): >> + >> + Spoof event(s) detected on VF(n) >> + >> + Where n=the VF that attempted to do the spoofing. > > I can't see that message in the PF driver code; does this actually apply > to the in-tree driver? Also I hope this is rate-limited. > > Ben. > For once the documentation was ahead of the code... :) I will update the patch, and I have updated the anti-spoofing patch we have in testing to also update the documentation at the same time the code is updated.
On Fri, Dec 10, 2010 at 12:21, David Miller <davem@davemloft.net> wrote: > From: Ben Hutchings <bhutchings@solarflare.com> > Date: Fri, 10 Dec 2010 15:50:24 +0000 > >> On Fri, 2010-12-10 at 01:50 -0800, Jeff Kirsher wrote: >>> Update Intel Wired LAN igb documentation. >>> >>> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> >>> --- >>> Documentation/networking/igb.txt | 22 +++++++++++++++++++--- >>> 1 files changed, 19 insertions(+), 3 deletions(-) >>> >>> diff --git a/Documentation/networking/igb.txt b/Documentation/networking/igb.txt >>> index ab2d718..393bdb7 100644 >>> --- a/Documentation/networking/igb.txt >>> +++ b/Documentation/networking/igb.txt >>> @@ -36,6 +36,7 @@ Default Value: 0 >>> This parameter adds support for SR-IOV. It causes the driver to spawn up to >>> max_vfs worth of virtual function. >>> >>> + >>> Additional Configurations >>> ========================= >>> >>> @@ -60,7 +61,8 @@ Additional Configurations >>> Ethtool >>> ------- >>> The driver utilizes the ethtool interface for driver configuration and >>> - diagnostics, as well as displaying statistical information. >>> + diagnostics, as well as displaying statistical information. The latest >>> + version of Ethtool can be found at: >>> >>> http://sourceforge.net/projects/gkernel. >> >> Please update this to: >> http://ftp.kernel.org/pub/software/network/ethtool/ > > Indeed. > > Intel folks, please address all of Ben's feedback and respin this > pull request, thanks. > -- > Re-spinning the pull request now.
diff --git a/Documentation/networking/igb.txt b/Documentation/networking/igb.txt index ab2d718..393bdb7 100644 --- a/Documentation/networking/igb.txt +++ b/Documentation/networking/igb.txt @@ -36,6 +36,7 @@ Default Value: 0 This parameter adds support for SR-IOV. It causes the driver to spawn up to max_vfs worth of virtual function. + Additional Configurations ========================= @@ -60,7 +61,8 @@ Additional Configurations Ethtool ------- The driver utilizes the ethtool interface for driver configuration and - diagnostics, as well as displaying statistical information. + diagnostics, as well as displaying statistical information. The latest + version of Ethtool can be found at: http://sourceforge.net/projects/gkernel. @@ -103,8 +105,8 @@ Additional Configurations NOTE: You need to have inet_lro enabled via either the CONFIG_INET_LRO or CONFIG_INET_LRO_MODULE kernel config option. Additionally, if - CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded - before the igb driver. + CONFIG_INET_LRO_MODULE is used, the inet_lro module needs to be loaded before + the igb driver. You can verify that the driver is using LRO by looking at these counters in Ethtool: @@ -116,6 +118,20 @@ Additional Configurations NOTE: IPv6 and UDP are not supported by LRO. + MAC and VLAN anti-spoofing feature + ---------------------------------- + When a malicious driver attempts to send a spoofed packet, it is dropped by + the hardware and not transmitted. An interrupt is sent to the PF driver + notifying it of the spoof attempt. + + When a spoofed packet is detected the PF driver will send the following + message to the system log (displayed by the "dmesg" command): + + Spoof event(s) detected on VF(n) + + Where n=the VF that attempted to do the spoofing. + + Support =======
Update Intel Wired LAN igb documentation. Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> --- Documentation/networking/igb.txt | 22 +++++++++++++++++++--- 1 files changed, 19 insertions(+), 3 deletions(-)