]> www.infradead.org Git - users/jedix/linux-maple.git/commit
rtc: interface: Fix long-standing race when setting alarm
authorEsben Haabendal <esben@geanix.com>
Fri, 16 May 2025 07:23:35 +0000 (09:23 +0200)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Thu, 9 Oct 2025 21:34:51 +0000 (23:34 +0200)
commit795cda8338eab036013314dbc0b04aae728880ab
tree6d64bf980af4b9e8561b8b6846e2a7ebb5c46326
parent87064da2db7be537a7da20a25c18ba912c4db9e1
rtc: interface: Fix long-standing race when setting alarm

As described in the old comment dating back to
commit 6610e0893b8b ("RTC: Rework RTC code to use timerqueue for events")
from 2010, we have been living with a race window when setting alarm
with an expiry in the near future (i.e. next second).
With 1 second resolution, it can happen that the second ticks after the
check for the timer having expired, but before the alarm is actually set.
When this happen, no alarm IRQ is generated, at least not with some RTC
chips (isl12022 is an example of this).

With UIE RTC timer being implemented on top of alarm irq, being re-armed
every second, UIE will occasionally fail to work, as an alarm irq lost
due to this race will stop the re-arming loop.

For now, I have limited the additional expiry check to only be done for
alarms set to next seconds. I expect it should be good enough, although I
don't know if we can now for sure that systems with loads could end up
causing the same problems for alarms set 2 seconds or even longer in the
future.

I haven't been able to reproduce the problem with this check in place.

Cc: stable@vger.kernel.org
Signed-off-by: Esben Haabendal <esben@geanix.com>
Link: https://lore.kernel.org/r/20250516-rtc-uie-irq-fixes-v2-1-3de8e530a39e@geanix.com
Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/rtc/interface.c