Re: [PATCH net 3/3] net/mlx5: DR, Fix out-of-bounds write when sync to HW fails

[Yevgeny Kliteynik]

On 27-Oct-26 10:56, Paolo Abeni wrote:
> On 10/23/26 12:52, netdev-bot+sashiko@xxxxxxxxxx wrote:
> > - [Medium] After a single SYNC_STEERING failure the pool latches into a
> >    permanent error state while allocation stays open, so device SW ICM…
>
> This one looks legit.
>
> /P

Agree.

Actually, there should be two aspects to this fix.
First part is as you commented - stop serving new chunks once the pool
is in error state. Second part is allow for some transient sync failures.

I'll issue v2 that handles both: a failed sync no longer disables the pool
right away. hot_chunks_arr has spare room, the hot chunks stay queued, and the
following frees keep retrying the sync. Only if the sync keeps failing until
the spare room is used up does the pool go into an error state.

-- YK