On 9/22/26 4:58 PM, [email protected] wrote: > Thank you for your contribution! Sashiko AI review found more severe > issue(s) in other patches of this series. This mail is only an FYI, in case > you want to address the nit picks below when you respin. > > Low priority / nit picks for your consideration: > > - [Low] Inaccurate newly added comment in __ovs_ct_lookup() > (net/openvswitch/conntrack.c:820-826).
Same thing as for the previous patch. Comments do not need to spell out everything that happens in the code. The intention of the comment and the code seems clear enough to me. So, I'd keep it as-is, unless someone thinks it needs more clarification. > > Pre-existing issues: > - [Low] Concern (INVALIDATED by later patch in the same series): the > identical helper/expectation use-after-free was claimed to remain… Preexisting and fixed later in the set. Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
