User Details
- User Since
- Mar 22 2019, 4:46 AM (389 w, 3 d)
Thu, Sep 3
No longer relevant.
Wed, Sep 2
Tue, Sep 1
Apply pte_g in _pmap_alloc_l3() as well.
Mon, Aug 31
Tue, Aug 25
Fri, Aug 21
LGTM! Thanks for your work. I will make the necessary style tweaks before committing (I noted a couple).
This got committed. Closing.
Not very important.
@mav I found this review again. Seems the removal made it to the 2.3 release branch. Shall I proceed here?
Thu, Aug 20
Fri, Aug 14
Thu, Aug 13
Wed, Aug 12
Generally looks fine.
Prefer .Dv NUL and bump .Dd.
Likely unnecessary.
Closing some old reviews. I won't be pursuing this.
Closing some old reviews. I won't be pursuing this.
Tue, Aug 11
Hi, thanks for this submission.
I did not review in detail.
Sun, Aug 9
On arm64, the hwpmc interrupt (arm64_intr()) is dispatched from the PMU driver: pmu_intr() in sys/arm/arm/pmu.c.
Aug 5 2026
Seems fine.
spinlock_enter() is weird, and a somewhat confusing name. I believe it is the disabling of interrupts that really prevents hardclock() from interfering. It is somewhat misleading to say that pm->pm_runcount is "protected" by a spinlock.
Jul 28 2026
Jul 27 2026
Thanks. I was considering removing all this code, but it was premature. I am happy to keep it if it is useful.
Jul 21 2026
Jul 20 2026
Jul 17 2026
Jul 16 2026
Seems sensible as a starting point.
Per the replies in https://github.com/freebsd/freebsd-src/pull/2245, this ends up having no functional impact. Closing.
Jul 14 2026
So, this applies the EOI handling scheme preferred by many drivers, and endorsed by mmel@.