powerpc64: dec_rearm: don't disable interrupts around atomic operations
authorcheloha <cheloha@openbsd.org>
Sat, 4 Feb 2023 23:20:54 +0000 (23:20 +0000)
committercheloha <cheloha@openbsd.org>
Sat, 4 Feb 2023 23:20:54 +0000 (23:20 +0000)
We don't need to disable interrupts around this singular mtdec() call.
The mtdec operation is atomic.

Link: https://marc.info/?l=openbsd-tech&m=167494864124073&w=2
ok gkoehler@

sys/arch/powerpc64/powerpc64/clock.c

index dee92d7..e9e111d 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: clock.c,v 1.9 2023/02/04 19:19:36 cheloha Exp $       */
+/*     $OpenBSD: clock.c,v 1.10 2023/02/04 23:20:54 cheloha Exp $      */
 
 /*
  * Copyright (c) 2020 Mark Kettenis <kettenis@openbsd.org>
@@ -58,7 +58,6 @@ void  cpu_startclock(void);
 void
 dec_rearm(void *unused, uint64_t nsecs)
 {
-       u_long s;
        uint32_t cycles;
 
        if (nsecs > dec_nsec_max)
@@ -66,9 +65,7 @@ dec_rearm(void *unused, uint64_t nsecs)
        cycles = (nsecs * dec_nsec_cycle_ratio) >> 32;
        if (cycles > UINT32_MAX >> 1)
                cycles = UINT32_MAX >> 1;
-       s = intr_disable();
        mtdec(cycles);
-       intr_restore(s);
 }
 
 void