Differential D20470

Use vm_page_unwire_noq() instead of vm_page_unwire(PQ_NONE).
ClosedPublic
Actions

Authored by markj on May 30 2019, 4:36 PM.

Tags

None

Referenced Files

	F110700300: D20470.diff
	Sat, Feb 22, 1:40 AM

	F110698477: D20470.id59490.diff
	Sat, Feb 22, 1:11 AM

	Unknown Object (File)
	Thu, Feb 20, 5:32 PM

	Unknown Object (File)
	Sat, Feb 8, 2:41 AM

	Unknown Object (File)
	Dec 30 2024, 5:23 AM

	Unknown Object (File)
	Dec 5 2024, 11:37 AM

	Unknown Object (File)
	Nov 30 2024, 12:34 AM

	Unknown Object (File)
	Nov 22 2024, 10:40 AM

View All 69 Files

Subscribers

Details

Reviewers

alc
kib
dougm

Commits

rS348785: Replace uses of vm_page_unwire(m, PQ_NONE) with vm_page_unwire_noq(m).

Summary

For managed pages, these two calls are not equivalent:
vm_page_unwire(PQ_NONE) will remove the page from its page queue, if
any, while vm_page_unwire_noq() leaves it alone. However, the existing
callers of vm_page_unwire(PQ_NONE) are applying it to fictitious or
unmanaged pages, so there should be no functional impact.

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 24602
Build 23385: arc lint + arc unit

Event Timeline

markj created this revision.May 30 2019, 4:36 PM

Herald added subscribers: emaste, andrew. · View Herald TranscriptMay 30 2019, 4:36 PM

Harbormaster completed remote builds in B24602: Diff 58077.May 30 2019, 4:37 PM

markj added reviewers: alc, kib, dougm.May 30 2019, 4:37 PM

kib accepted this revision.May 30 2019, 5:27 PM

kib added inline comments.

sys/vm/vm_glue.c
233	Just a note that the page is managed. It is because it is freed immediately after, we can use vm_page_unwire_noq().

This revision is now accepted and ready to land.May 30 2019, 5:27 PM

markj added inline comments.Jun 7 2019, 6:12 PM

sys/vm/vm_glue.c
233	This call was added in D19247, so it will not be committed with the rest of this diff.

Closed by commit rS348785: Replace uses of vm_page_unwire(m, PQ_NONE) with vm_page_unwire_noq(m). (authored by markj). · Explain WhyJul 7 2019, 4:44 AM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptJul 7 2019, 4:44 AM

Revision Contents
Changeset List

Path

Size

sys/

amd64/

amd64/

2 lines

arm/

arm/

2 lines

nvidia/

drm2/

2 lines

compat/

linuxkpi/

common/

src/

2 lines

dev/

drm2/

ttm/

ttm_page_alloc.c

2 lines

ti/

2 lines

xen/

gntdev/

2 lines

i386/

i386/

2 lines

mips/

mips/

2 lines

riscv/

riscv/

2 lines

vm/

8 lines

4 lines

2 lines

Diff 58077

sys/amd64/amd64/pmap.c

Loading...

sys/arm/arm/pmap-v6.c

Loading...

sys/arm/nvidia/drm2/tegra_bo.c

Loading...

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

Loading...

sys/dev/drm2/ttm/ttm_page_alloc.c

Loading...

sys/dev/ti/if_ti.c

Loading...

sys/dev/xen/gntdev/gntdev.c

Loading...

sys/i386/i386/pmap.c

Loading...

sys/mips/mips/pmap.c

Loading...

sys/riscv/riscv/pmap.c

Loading...

sys/vm/uma_core.c

Loading...

sys/vm/vm_glue.c

Loading...

sys/vm/vm_kern.c

Loading...