Subject: Re: [PATCH net] net/mlx5: Avoid MMIO when the error is detected

From: Saeed Mahameed <[email-masked]>
Date: Mon, 21 Aug 2023 13:51:56 -0700

On 07 Aug 14:02, Ganesh Goudar wrote:
>When the drivers are notfied about the pci error, All
>the IO to the card must be stopped, Else the recovery would
>fail, Avoid memory-mapped IO until the device recovers
>from pci error.
>
>Signed-off-by: Ganesh Goudar <[email-masked]>
>---
> drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c | 3 +++
> 1 file changed, 3 insertions(+)
>
>diff --git a/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c b/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c
>index 932fbc843c69..010dee4eec14 100644
>--- a/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c
>+++ b/drivers/net/ethernet/mellanox/mlx5/core/lib/clock.c
>@@ -221,10 +221,13 @@ static void mlx5_timestamp_overflow(struct work_struct *work)
> 	clock = container_of(timer, struct mlx5_clock, timer);
> 	mdev = container_of(clock, struct mlx5_core_dev, clock);
>
>+	if (mdev->state == MLX5_DEVICE_STATE_INTERNAL_ERROR)
>+		goto out;
> 	write_seqlock_irqsave(&clock->lock, flags);
> 	timecounter_read(&timer->tc);
> 	mlx5_update_clock_info_page(mdev);
> 	write_sequnlock_irqrestore(&clock->lock, flags);
>+out:
> 	schedule_delayed_work(&timer->overflow_work, timer->overflow_period);
> }
>

Another version of this fix was already submitted and accepted:

https://patchwork.kernel.org/project/netdevbpf/patch/[email-masked]/

>-- 
>2.40.1