On 7/9/26 5:25 PM, Sami Tolvanen wrote: > On Thu, Jul 9, 2026 at 1:45 AM Petr Pavlu <[email protected]> wrote: >> >> On 6/7/26 6:18 AM, Naveen Kumar Chaudhary wrote: >>> module_total_size() returns unsigned int but uses a signed int >>> accumulator. While the result is numerically correct, the type >>> mismatch is misleading. >>> >>> Change the accumulator to unsigned int to match the return type. >>> >>> Signed-off-by: Naveen Kumar Chaudhary <[email protected]> >> >> Queued on modules-next for v7.3-rc1. > > Note that the commit title still claims to fix a signed integer > overflow, which isn't accurate. Otherwise LGTM.
Thanks for noticing this. I tweaked the title to: module: procfs: use matching type for accumulator in module_total_size() -- Petr
