Differential D35790

vm_pager: Remove the default pager
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F169391282: D35790.diff
	Tue, Sep 1, 9:53 AM

	F169388269: D35790.id.diff
	Tue, Sep 1, 9:42 AM

	F169314631: D35790.id108062.diff
	Tue, Sep 1, 4:36 AM

	F169314428: D35790.id108078.diff
	Tue, Sep 1, 4:35 AM

	F169295291: D35790.id108247.diff
	Tue, Sep 1, 3:45 AM

	F169294750: D35790.id108107.diff
	Tue, Sep 1, 3:44 AM

	Unknown Object (File)
	Mon, Aug 31, 4:12 AM

	Unknown Object (File)
	Sun, Aug 30, 11:16 PM

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 46387
Build 43276: 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
92	I wonder if we want to keep the KBI there.

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

sys/vm/vm.h
92	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

1 line

3 lines

Diff 108062

sys/conf/files

Loading...

sys/vm/default_pager.c

Loading...

sys/vm/vm.h

Loading...

sys/vm/vm_pager.c

Loading...