Index | Thread | Search

From:
Mark Kettenis <mark.kettenis@xs4all.nl>
Subject:
Re: Missing km_free(9)
To:
Martin Pieuchot <mpi@grenadille.net>
Cc:
tech@openbsd.org
Date:
Thu, 04 Sep 2025 22:39:02 +0200

Download raw body.

Thread
  • Martin Pieuchot:

    Missing km_free(9)

    • Mark Kettenis:

      Missing km_free(9)

> Date: Thu, 4 Sep 2025 17:53:35 +0200
> From: Martin Pieuchot <mpi@grenadille.net>
> 
> When KVA allocations have been converted to km_alloc(9) we missed this
> free path, ok?

ok kettenis@

> Index: uvm/uvm_glue.c
> ===================================================================
> RCS file: /cvs/src/sys/uvm/uvm_glue.c,v
> diff -u -p -r1.93 uvm_glue.c
> --- uvm/uvm_glue.c	7 Jul 2025 18:33:37 -0000	1.93
> +++ uvm/uvm_glue.c	4 Sep 2025 15:49:05 -0000
> @@ -255,7 +255,7 @@ uvm_vsunlock_device(struct proc *p, void
>  	uvm_km_pgremove_intrsafe(kva, kva + sz);
>  	pmap_kremove(kva, sz);
>  	pmap_update(pmap_kernel());
> -	uvm_km_free(kernel_map, kva, sz);
> +	km_free((void *)kva, sz, &kv_any, &kp_none);
>  }
>  
>  const struct kmem_va_mode kv_uarea = {
> 
> 
>