d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't free the cursor boundary tag during vmem_destroy().

Browse Source 
The cursor boundary tag is statically allocated in the vmem instead of
from the vmem_bt_zone.  Explicitly remove it from the vmem's segment
list in vmem_destroy before freeing all the segments from the vmem.

Reviewed by:	markj
MFC after:	1 week
Sponsored by:	Chelsio Communications
Differential Revision:	https://reviews.freebsd.org/D21953
This commit is contained in:
John Baldwin 2019-10-09 21:20:39 +00:00
parent 2f9520d5fe
commit 97ecf6efa0
1 changed files with 2 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
sys/kern/subr_vmem.c
View File

@ -466,6 +466,7 @@ static void
bt_remseg(vmem_t *vm, bt_t *bt)
{
MPASS(bt->bt_type != BT_TYPE_CURSOR);
TAILQ_REMOVE(&vm->vm_seglist, bt, bt_seglist);
bt_free(vm, bt);
}
@ -843,6 +844,7 @@ vmem_destroy1(vmem_t *vm)
VMEM_LOCK(vm);
MPASS(vm->vm_nbusytag == 0);
TAILQ_REMOVE(&vm->vm_seglist, &vm->vm_cursor, bt_seglist);
while ((bt = TAILQ_FIRST(&vm->vm_seglist)) != NULL)
bt_remseg(vm, bt);