Differential D35790

vm_pager: Remove the default pager
ClosedPublic
Actions

Authored by markj on Jul 12 2022, 4:23 PM.

Tags

None

Referenced Files

	F157342651: D35790.id108062.diff
	Wed, May 20, 11:46 AM

	F157325442: D35790.id108247.diff
	Wed, May 20, 8:42 AM

	Unknown Object (File)
	Tue, May 19, 9:51 AM

	Unknown Object (File)
	Mon, May 18, 11:48 AM

	Unknown Object (File)
	Mon, May 18, 11:37 AM

	Unknown Object (File)
	Sun, May 17, 2:24 AM

	Unknown Object (File)
	Thu, May 14, 5:57 PM

	Unknown Object (File)
	Mon, Apr 27, 7:30 AM

Subscribers

Details

Reviewers

dougm
alc
kib
imp

Commits

rG1424f65bbcb3: vm_pager: Remove the default pager

Summary

No functional change intended.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 46409
Build 43298: arc lint + arc unit

Event Timeline

markj created this revision.Jul 12 2022, 4:23 PM

Herald added a subscriber: imp. · View Herald TranscriptJul 12 2022, 4:23 PM

markj requested review of this revision.Jul 12 2022, 4:23 PM

Harbormaster completed remote builds in B46387: Diff 108062.Jul 12 2022, 4:23 PM

markj added a parent revision: D35789: vm_object: Remove redundant OBJ_SWAP checks.Jul 12 2022, 4:23 PM

markj added a child revision: D35791: vm_pager: Remove references to KVME_TYPE_DEFAULT in the kernel.

It's time...

This revision is now accepted and ready to land.Jul 12 2022, 4:57 PM

kib accepted this revision.Jul 12 2022, 4:59 PM

kib added inline comments.

sys/vm/vm.h
93–94	I wonder if we want to keep the KBI there.

alc added inline comments.Jul 12 2022, 11:32 PM

sys/vm/vm.h
93–94	Maybe replace OBJT_DEFAULT by OBJT_RESERVED?

Add a placeholder object type to avoid renumbering the rest.
Redefine OBJT_DEFAULT as OBJT_SWAP. At least virtualbox creates OBJT_DEFAULT objects, but seems to require no other modification, so we can avoid breaking its build.

This revision now requires review to proceed.Jul 13 2022, 12:28 AM

Harbormaster completed remote builds in B46392: Diff 108078.Jul 13 2022, 12:28 AM

alc accepted this revision.Jul 13 2022, 6:14 AM

This revision is now accepted and ready to land.Jul 13 2022, 6:14 AM

kib accepted this revision.Jul 13 2022, 8:37 AM

Modify vm_pager_init() to handle holes in the object type space.

This revision now requires review to proceed.Jul 13 2022, 3:37 PM

Harbormaster completed remote builds in B46409: Diff 108107.Jul 13 2022, 3:37 PM

kib accepted this revision.Jul 13 2022, 3:50 PM

This revision is now accepted and ready to land.Jul 13 2022, 3:50 PM

alc accepted this revision.Jul 14 2022, 9:31 AM

Closed by commit rG1424f65bbcb3: vm_pager: Remove the default pager (authored by markj). · Explain WhyJul 17 2022, 11:26 AM

This revision was automatically updated to reflect the committed changes.

markj added a commit: rG1424f65bbcb3: vm_pager: Remove the default pager.

Revision Contents
Changeset List

Path

Size

sys/

conf/

1 line

vm/

default_pager.c

3 lines

5 lines

Diff 108107

sys/conf/files

Loading...

sys/vm/default_pager.c

Loading...

sys/vm/vm.h

Loading...

sys/vm/vm_pager.c

Loading...