lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:	Fri, 14 Nov 2008 08:14:36 -0800
From:	"Jeff Kirsher" <jeffrey.t.kirsher@...el.com>
To:	jeff@...zik.org, davem@...emloft.net
Cc:	netdev@...r.kernel.org,
	"Alexander Duyck" <alexander.h.duyck@...el.com>,
	"Jeff Kirsher" <jeffrey.t.kirsher@...el.com>,
	"Peter P Waskiewicz Jr" <peter.p.waskiewicz.jr@...el.com>
Subject: Re: [RFC][NET-NEXT PATCH 1/7] ixgbe: this patch adds support for DCB to the kernel and ixgbe driver

On Fri, Nov 7, 2008 at 4:51 PM, Jeff Kirsher
<jeffrey.t.kirsher@...el.com> wrote:
> From: Alexander Duyck <alexander.h.duyck@...el.com>
>
> This adds support for Data Center Bridging (DCB) features in the ixgbe driver and adds an rtnetlink interface for configuring DCB to the kernel.  The DCB feature support included are Priority Grouping (PG) - which allows bandwidth guarantees to be allocated to groups to traffic based on the 802.1q priority, and Priority Based Flow Control (PFC) - which introduces a new MAC control PAUSE frame which works at granularity of the 802.1p priority instead of the link (IEEE 802.3x).
>
> Signed-off-by: Alexander Duyck <alexander.h.duyck@...el.com>
> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@...el.com>
> Signed-off-by: Peter P Waskiewicz Jr <peter.p.waskiewicz.jr@...el.com>
> ---
>
>  drivers/net/ixgbe/Makefile          |    3
>  drivers/net/ixgbe/ixgbe.h           |   24 +
>  drivers/net/ixgbe/ixgbe_dcb.c       |  332 +++++++++++++++++
>  drivers/net/ixgbe/ixgbe_dcb.h       |  157 ++++++++
>  drivers/net/ixgbe/ixgbe_dcb_82598.c |  398 ++++++++++++++++++++
>  drivers/net/ixgbe/ixgbe_dcb_82598.h |   94 +++++
>  drivers/net/ixgbe/ixgbe_dcb_nl.c    |  352 ++++++++++++++++++
>  drivers/net/ixgbe/ixgbe_ethtool.c   |   30 +
>  drivers/net/ixgbe/ixgbe_main.c      |  185 ++++++++-
>  include/linux/dcbnl.h               |  230 +++++++++++
>  include/linux/netdevice.h           |    8
>  include/linux/rtnetlink.h           |    5
>  include/net/dcbnl.h                 |   44 ++
>  net/Kconfig                         |    1
>  net/Makefile                        |    3
>  net/dcb/Kconfig                     |   12 +
>  net/dcb/Makefile                    |    1
>  net/dcb/dcbnl.c                     |  704 +++++++++++++++++++++++++++++++++++
>  18 files changed, 2564 insertions(+), 19 deletions(-)

During testing of this patch, we found an issue where if Data Center
Bridging (DCB) is not enabled in the kernel, ixgbe does not compile
due to dependencies to DCB.  I have an updated patch which wraps all
the DCB code to be dependent on CONFIG_DCBNL, but before posting the
updated patch I wanted to get feedback on the following options:

1) Make all DCB code in the ixgbe driver dependent on CONFIG_DCBNL to
be present to build.
2) Have an IXGBE_CONFIG_DCB build option for ixgbe that becomes
available when CONFIG_DCBNL is present.

I would think that option 2 would be best, so that if users wanted
DCB, but not in ixgbe they could simple not enable DCB in ixgbe.


-- 
Cheers,
Jeff
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Powered by blists - more mailing lists