[PATCH 4/5] rust: hrtimer: use READ_ONCE instead of read_volatile

Gary Guo gary at garyguo.net
Tue Jan 6 07:23:00 PST 2026


On Tue, 06 Jan 2026 13:37:34 +0100
Andreas Hindborg <a.hindborg at kernel.org> wrote:

> "FUJITA Tomonori" <fujita.tomonori at gmail.com> writes:
> >
> > Sorry, of course this should be:
> >
> > +__rust_helper ktime_t rust_helper_hrtimer_get_expires(const struct hrtimer *timer)
> > +{
> > +	return hrtimer_get_expires(timer);
> > +}
> >  
> 
> This is a potentially racy read. As far as I recall, we determined that
> using read_once is the proper way to handle the situation.
> 
> I do not think it makes a difference that the read is done by C code.

If that's the case I think the C code should be fixed by inserting the
READ_ONCE?

Best,
Gary



More information about the linux-arm-kernel mailing list