Source-Changes-D archive
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]
Re: CVS commit: src/sys/uvm
[resent without typo in To]
> Module Name: src
> Committed By: kbowling
> Date: Sat Jun 27 09:39:57 UTC 2026
>
> Modified Files:
> src/sys/uvm: uvm_km.c
>
> Log Message:
> uvm_km_pgremove_intrsafe: drain TLB shootdown before freeing pages
>
> Call pmap_update(pmap_kernel()) between pmap_kremove and the page
> free so the shootdown completes first.
>
> Reviewed by skrll@
Uh oh! This sounds bad, and if applicable, something that should be
pulled up to release branches, 9, 10, and 11. Is there a PR for this?
I suggest filing one and (for future changes) citing it in the commit
message like
PR kern/654321: uvm_km(9): missing pmap_update on free
so we can use it to track the pullup state (open -> once fixed in
HEAD, needs-pullups -> once pullups are submitted, pending-pullups ->
once pulled up to all applicable branches, closed).
Home |
Main Index |
Thread Index |
Old Index