Re: [PATCH net V2 10/11] net/mlx5e: Update and set Xon/Xoff upon port speed set

On 15/09/2025 10:38, Tariq Toukan wrote:

On 11/09/2025 17:36, Jakub Kicinski wrote:

quoted

On Thu, 11 Sep 2025 17:25:22 +0300 Mark Bloch wrote:

quoted

On 11/09/2025 16:47, Jakub Kicinski wrote:

quoted

On Wed, 10 Sep 2025 17:00:11 -0700 Jakub Kicinski wrote:

quoted

Hi, this is breaking dual host CX7 w/ 28.45.1300 (but I think most
older FW versions, too). Looks like the host is not receiving any
mcast (ping within a subnet doesn't work because the host receives
no ndisc), and most traffic slows down to a trickle.
Lost of rx_prio0_buf_discard increments.

Please TAL ASAP, this change went to LTS last week.

Any news on this? I heard that it also breaks DCB/QoS configuration
on 6.12.45 LTS.

We are looking into this, once we have anything I'll update.
Just to make sure, reverting this is one commit solves the
issue you are seeing?

It did for me, but Daniel (who is working on the PSP series)
mentioned that he had reverted all three to get net-next working:

   net/mlx5e: Set local Xoff after FW update
   net/mlx5e: Update and set Xon/Xoff upon port speed set
   net/mlx5e: Update and set Xon/Xoff upon MTU set

Hi Jakub,

Thanks for reporting.
We're investigating and will update soon.

Regards,
Tariq

Hi,

We prefer reverting the single patch [1] for now. We'll submit a fixed 
version later.

Regarding the other two patches [2], initial testing showed no issues.
Can you/Daniel share more info? What issues you see, and the repro steps.

Thanks,
Tariq

[1]
net/mlx5e: Update and set Xon/Xoff upon port speed set

[2]
net/mlx5e: Set local Xoff after FW update
net/mlx5e: Update and set Xon/Xoff upon MTU set