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] [thread-next>] [day] [month] [year] [list]
Date:   Fri, 21 Oct 2022 16:37:57 -0700
From:   Jay Vosburgh <jay.vosburgh@...onical.com>
To:     Petr Machata <petrm@...dia.com>
cc:     "David S. Miller" <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Jakub Kicinski <kuba@...nel.org>,
        Paolo Abeni <pabeni@...hat.com>, netdev@...r.kernel.org,
        Veaceslav Falico <vfalico@...il.com>,
        Andy Gospodarek <andy@...yhouse.net>,
        Ido Schimmel <idosch@...dia.com>,
        Amit Cohen <amcohen@...dia.com>, Andrew Lunn <andrew@...n.ch>,
        Heiner Kallweit <hkallweit1@...il.com>,
        Russell King <linux@...linux.org.uk>,
        David Ahern <dsahern@...nel.org>, mlxsw@...dia.com
Subject: Re: [PATCH net-next 3/3] bonding: 3ad: Add support for 800G speed

Petr Machata <petrm@...dia.com> wrote:

>From: Amit Cohen <amcohen@...dia.com>
>
>Add support for 800Gbps speed to allow using 3ad mode with 800G devices.
>
>Signed-off-by: Amit Cohen <amcohen@...dia.com>
>Reviewed-by: Ido Schimmel <idosch@...dia.com>
>Signed-off-by: Petr Machata <petrm@...dia.com>

Acked-by: Jay Vosburgh <jay.vosburgh@...onical.com>


>---
> drivers/net/bonding/bond_3ad.c | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
>diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c
>index e58a1e0cadd2..455b555275f1 100644
>--- a/drivers/net/bonding/bond_3ad.c
>+++ b/drivers/net/bonding/bond_3ad.c
>@@ -75,6 +75,7 @@ enum ad_link_speed_type {
> 	AD_LINK_SPEED_100000MBPS,
> 	AD_LINK_SPEED_200000MBPS,
> 	AD_LINK_SPEED_400000MBPS,
>+	AD_LINK_SPEED_800000MBPS,
> };
> 
> /* compare MAC addresses */
>@@ -251,6 +252,7 @@ static inline int __check_agg_selection_timer(struct port *port)
>  *     %AD_LINK_SPEED_100000MBPS
>  *     %AD_LINK_SPEED_200000MBPS
>  *     %AD_LINK_SPEED_400000MBPS
>+ *     %AD_LINK_SPEED_800000MBPS
>  */
> static u16 __get_link_speed(struct port *port)
> {
>@@ -326,6 +328,10 @@ static u16 __get_link_speed(struct port *port)
> 			speed = AD_LINK_SPEED_400000MBPS;
> 			break;
> 
>+		case SPEED_800000:
>+			speed = AD_LINK_SPEED_800000MBPS;
>+			break;
>+
> 		default:
> 			/* unknown speed value from ethtool. shouldn't happen */
> 			if (slave->speed != SPEED_UNKNOWN)
>@@ -753,6 +759,9 @@ static u32 __get_agg_bandwidth(struct aggregator *aggregator)
> 		case AD_LINK_SPEED_400000MBPS:
> 			bandwidth = nports * 400000;
> 			break;
>+		case AD_LINK_SPEED_800000MBPS:
>+			bandwidth = nports * 800000;
>+			break;
> 		default:
> 			bandwidth = 0; /* to silence the compiler */
> 		}
>-- 
>2.35.3
>

Powered by blists - more mailing lists