Differential D28810

Let the VM page allocator handle page zeroing
AbandonedPublic
Actions

Authored by markj on Feb 19 2021, 8:23 PM.

Tags

None

Referenced Files

	F171548771: D28810.id.diff
	Fri, Sep 11, 5:09 PM

	Unknown Object (File)
	Fri, Sep 11, 2:02 AM

	Unknown Object (File)
	Thu, Sep 10, 12:52 AM

	Unknown Object (File)
	Tue, Sep 8, 11:08 PM

	Unknown Object (File)
	Mon, Sep 7, 3:45 PM

	Unknown Object (File)
	Fri, Sep 4, 7:35 PM

	Unknown Object (File)
	Fri, Sep 4, 4:51 PM

	Unknown Object (File)
	Fri, Sep 4, 12:15 PM

Subscribers

Details

Reviewers

manu
kib
alc
jeff

Summary

An earlier diff modified vm_page_alloc() and friends to zero the page if
VM_ALLOC_ZERO is specified, rather than treating VM_ALLOC_ZERO as
advisory. This diff patches up most consumers to reflect that change.
A few consumers require a bit more care and are handled in separate
diffs.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 37207
Build 34096: arc lint + arc unit

Event Timeline

markj created this revision.Feb 19 2021, 8:23 PM

Herald added a reviewer: manu. · View Herald TranscriptFeb 19 2021, 8:23 PM

Herald added subscribers: linuxkpi, emaste, jhibbits and 2 others. · View Herald Transcript

markj requested review of this revision.Feb 19 2021, 8:23 PM

Harbormaster completed remote builds in B37207: Diff 84298.Feb 19 2021, 8:23 PM

markj added a parent revision: D28809: arm pmap: Let the page allocator handle page zeroing.Feb 19 2021, 8:23 PM

markj added reviewers: kib, alc, jeff.Feb 19 2021, 8:27 PM

kib accepted this revision.Feb 19 2021, 9:31 PM

This revision is now accepted and ready to land.Feb 19 2021, 9:31 PM

alc added inline comments.Feb 20 2021, 10:48 PM

sys/dev/xen/balloon/balloon.c
240–246	I would not delete this comment. Instead, move it up before the allocation.

Superseded by D31986.

sys/dev/xen/balloon/balloon.c
240–246	I handled this comment in the new patch series (where I again had deleted the comment).

Revision Contents
Changeset List

Path

Size

sys/

amd64/

amd64/

29 lines

6 lines

arm/

nvidia/

drm2/

2 lines

arm64/

arm64/

9 lines

6 lines

compat/

linuxkpi/

common/

src/

10 lines

dev/

xen/

balloon/

11 lines

gntdev/

7 lines

i386/

i386/

10 lines

kern/

2 lines

mips/

mips/

4 lines

6 lines

powerpc/

aim/

5 lines

6 lines

4 lines

booke/

5 lines

powerpc/

3 lines

riscv/

riscv/

15 lines

6 lines

vm/

8 lines

Diff 84298

sys/amd64/amd64/pmap.c

Loading...

sys/amd64/amd64/uma_machdep.c

Loading...

sys/arm/nvidia/drm2/tegra_bo.c

Loading...

sys/arm64/arm64/pmap.c

Loading...

sys/arm64/arm64/uma_machdep.c

Loading...

sys/compat/linuxkpi/common/src/linux_page.c

Loading...

sys/dev/xen/balloon/balloon.c

Loading...

sys/dev/xen/gntdev/gntdev.c

Loading...

sys/i386/i386/pmap.c

Loading...

sys/kern/uipc_shm.c

Loading...

sys/mips/mips/pmap.c

Loading...

sys/mips/mips/uma_machdep.c

Loading...

sys/powerpc/aim/mmu_oea64.c

Loading...

sys/powerpc/aim/mmu_radix.c

Loading...

sys/powerpc/aim/slb.c

Loading...

sys/powerpc/booke/pmap_64.c

Loading...

sys/powerpc/powerpc/uma_machdep.c

Loading...

sys/riscv/riscv/pmap.c

Loading...

sys/riscv/riscv/uma_machdep.c

Loading...

sys/vm/vm_kern.c

Loading...