PR/51254 CVS commit: src/sys/uvm

[gnats-admin%netbsd.org@localhost]

The following reply was made to PR kern/51254; it has been noted by GNATS.

From: "Taylor R Campbell" <riastradh%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/51254 CVS commit: src/sys/uvm
Date: Wed, 14 Aug 2024 01:26:25 +0000

 Module Name:	src
 Committed By:	riastradh
 Date:		Wed Aug 14 01:26:25 UTC 2024
 
 Modified Files:
 	src/sys/uvm: uvm_map.c
 
 Log Message:
 uvm_map(9): Take vm map lock around uvm_unmap_remove.
 
 This was tripping one of the assertions I added.  While it is safe
 here not to hold the lock -- caller has exclusive access to the map
 at this point -- it is better if we can annotate the functions in
 question with executable notes about locking rules, and taking a
 single uncontended lock in the vm map destruction path is probably a
 tiny cost worth those executable notes.
 
 PR kern/51254: uvm assertion "!topdown || hint <= orig_hint" failed
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.420 -r1.421 src/sys/uvm/uvm_map.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.