> Date: Fri, 1 Oct 2021 20:10:35 +0200
> From: Martin Pieuchot <[email protected]>
>
> Diff below turns i386's pmap_collect() into a noop like it is on
> amd64/arm64/powerpc64... This is part of the UVM unlocking diff and
> might no longer be necessary now that pmap_extract() has been fixed.
I think it is no longer necessary.
> So I'd like to know if we want to align i386's behavior with other
> archs, which should help us debug MI issues or if I drop this diff.
>
> Since sthen@ tested it, I'd be in favor of putting it in. ok?
There are other architectures that do something similar and get even
less testing than i386. So I think it makes (some) sense to keep it.
> Index: arch/i386/i386/pmap.c
> ===================================================================
> RCS file: /cvs/src/sys/arch/i386/i386/pmap.c,v
> retrieving revision 1.217
> diff -u -p -r1.217 pmap.c
> --- arch/i386/i386/pmap.c 11 Sep 2021 18:08:32 -0000 1.217
> +++ arch/i386/i386/pmap.c 1 Oct 2021 17:04:20 -0000
> @@ -2259,13 +2259,6 @@ pmap_unwire_86(struct pmap *pmap, vaddr_
> void
> pmap_collect(struct pmap *pmap)
> {
> - /*
> - * free all of the pt pages by removing the physical mappings
> - * for its entire address space.
> - */
> -
> - pmap_do_remove(pmap, VM_MIN_ADDRESS, VM_MAX_ADDRESS,
> - PMAP_REMOVE_SKIPWIRED);
> }
>
> /*
>
>
