Re: [PATCH] usbmon: mark concurrent accesses to the monitored flag

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [Low] usbmon: unresolvable Fixes trailer SHA 1da177e4c3f4 in commit
  message
--

commit 68910eb830ad82fcbb29334f2601dba719edc500
Author: Zhu Jun <zhujun2@cmss.chinamobile.com>

> usbmon: mark concurrent accesses to the monitored flag
>
> USB submission and completion paths read bus->monitored without taking
> mon_lock or the monitor bus lock. Adding and removing readers changes the
> flag under those locks, so the plain accesses race, as reported by KCSAN.
>
> Use READ_ONCE() for the fast-path checks and WRITE_ONCE() for the updates.
> The monitor callbacks still serialize access to their reader lists using
> the existing bus locks.
>
> Fixes: 1da177e4c3f4 ("Linux-2.6.12-rc2")
> Signed-off-by: Zhu Jun <zhujun2@cmss.chinamobile.com>

[Severity: Low]
The commit SHA in the Fixes: trailer cannot be resolved in the repository
because the 12-character prefix 1da177e4c3f4 is ambiguous (the initial git
commit requires at least 13 characters, 1da177e4c3f41 ("Linux-2.6.12-rc2")).

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261111034138.12642-1-zhujun2@cmss.chinamobile.com?part=1