netdev - Re: [PATCH net-next 9/9] net: txgbe: Implement SRIOV for AML devices

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20250519155833.GI365796@horms.kernel.org>
Date: Mon, 19 May 2025 16:58:33 +0100
From: Simon Horman <horms@...nel.org>
To: Jiawen Wu <jiawenwu@...stnetic.com>
Cc: netdev@...r.kernel.org, andrew+netdev@...n.ch, davem@...emloft.net,
	edumazet@...gle.com, kuba@...nel.org, richardcochran@...il.com,
	linux@...linux.org.uk, mengyuanlou@...-swift.com
Subject: Re: [PATCH net-next 9/9] net: txgbe: Implement SRIOV for AML devices

On Fri, May 16, 2025 at 05:32:20PM +0800, Jiawen Wu wrote:
> Support to bring VFs link up for AML 25G/10G devices.

Hi Jiawen,

I think this warrants a bit more explanation: what is required for
these devices; and perhaps how that differs from other devices.

> 
> Signed-off-by: Jiawen Wu <jiawenwu@...stnetic.com>
> ---
>  drivers/net/ethernet/wangxun/txgbe/txgbe_aml.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/net/ethernet/wangxun/txgbe/txgbe_aml.c b/drivers/net/ethernet/wangxun/txgbe/txgbe_aml.c
> index 6bcf67bef576..7dbcf41750c1 100644
> --- a/drivers/net/ethernet/wangxun/txgbe/txgbe_aml.c
> +++ b/drivers/net/ethernet/wangxun/txgbe/txgbe_aml.c
> @@ -10,6 +10,7 @@
>  #include "../libwx/wx_lib.h"
>  #include "../libwx/wx_ptp.h"
>  #include "../libwx/wx_hw.h"
> +#include "../libwx/wx_sriov.h"
>  #include "txgbe_type.h"
>  #include "txgbe_aml.h"
>  #include "txgbe_hw.h"
> @@ -315,6 +316,8 @@ static void txgbe_mac_link_up_aml(struct phylink_config *config,
>  	wx->last_rx_ptp_check = jiffies;
>  	if (test_bit(WX_STATE_PTP_RUNNING, wx->state))
>  		wx_ptp_reset_cyclecounter(wx);
> +	/* ping all the active vfs to let them know we are going up */
> +	wx_ping_all_vfs_with_link_status(wx, true);
>  }
>  
>  static void txgbe_mac_link_down_aml(struct phylink_config *config,
> @@ -329,6 +332,8 @@ static void txgbe_mac_link_down_aml(struct phylink_config *config,
>  	wx->speed = SPEED_UNKNOWN;
>  	if (test_bit(WX_STATE_PTP_RUNNING, wx->state))
>  		wx_ptp_reset_cyclecounter(wx);
> +	/* ping all the active vfs to let them know we are going down */
> +	wx_ping_all_vfs_with_link_status(wx, false);
>  }
>  
>  static void txgbe_mac_config_aml(struct phylink_config *config, unsigned int mode,
> -- 
> 2.48.1
>