On 9/29/26 4:04 PM, [email protected] wrote:
Thank you for your contribution! Sashiko AI review found 3 potential issue(s) to consider.
Summary: Ignore, please merge.
Critical: 1 · High: 0 (1) · Medium: 1 (1) · Low: 1 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). - [Critical] ice_vsi_resize_stat_arrays() sizes the arrays from a snapshot taken before ice_vsi_decfg(): the PF pool,…
AI says that weird stuff, like CPU hotplug, must happen to trigger a problem, then, that the last patch fixes the problem. I will simply ignore (driver is not ready for such stuff in general).
- [Medium] ice_vsi_set_stat_arrays() publishes the new container with a plain store (`pf->vsi_stats[vsi->idx] = new_vsi_stat;`, ice_lib.c:592)…
In the body of the comment AI says that the ETF-vs-reset issue is pre-existing, reading more, the old code was doining the same level of changes prior to the series/patch, but due to "being new code" its not marked. Also, issue "seems plausible", but all works, perhaps due to other locks. I would like to leave it out of the series.
- [Low] This patch fixes a real heap out-of-bounds access but has no Fixes: tag, and the Fixes-tagged fix later in the series cannot be…
same comment as in patch 2, I'm against Fixes tags for "first half" of the fix, when for the same target commit and in the same series.
Pre-existing issues:
I simply want to merge code instead of endless growth of fixed area.
- [High] This problem wasn't introduced by this patch, but a fallback rebuild dereferences NULL queue maps. - [Medium] ice_vsi_free_unused_stat_arrays() calls kfree_rcu() on surplus ice_ring_stats entries while vsi->tx_rings[i]->ring_stats and…
> Sashiko AI review · https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20260925132636.123300-1-przemyslaw.kitszel%40intel.com
