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