Re: [PATCH net v4 5/5] ice: fix stats array overflow when VF requests more queues
From: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Date: 2026-09-25 07:13:13
On 9/24/26 2:23 PM, netdev-bot+sashiko@kernel.org wrote:
Thank you for your contribution! Sashiko AI review found 2 potential issue(s) to consider. Critical: 0 · High: 1 · Medium: 1 (2) · Low: 0 If you believe none of the issues are legitimate, please reply to each issue of Medium severity or higher and explain why the AI is wrong. If any of them are legitimate and you plan to respin, please let patchwork know by including "pw-bot: cr" as a separate line at the end of your reply (one such reply per series is enough). - [High] Incomplete fix: the identical stats-array out-of-bounds write remains reachable through the PF devlink `driver_reinit` reload path.
Confirmed. The issue is pre-existing though (reproduces w/o the series). This series was targeted at fixing issue for VF, and it does. I will send another commit to fix PF case - normally I would argue to leave it for followup, but will check if a simple fix is enough (compare to this commit - a oneliner that changed into 5 patches and few months...)
- [Medium] The patch introduces a new abort point in the middle of an already-started hardware VF reset.
this warrants a respin, I will have a look at pre-existing issues, but will rather try to avoid touching more than needed pw-bot: cr