Page MenuHomeFreeBSD
Differential D21546

Eliminate redundant vm code with a few new convenience functions.
ClosedPublic
Actions

Authored by jeff on Sep 6 2019, 12:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 20, 10:10 AM
Unknown Object (File)
Fri, Sep 11, 2:32 PM
Unknown Object (File)
Tue, Sep 8, 10:45 PM
Unknown Object (File)
Sat, Aug 29, 12:35 AM
Unknown Object (File)
Sat, Aug 29, 12:32 AM
Unknown Object (File)
Sat, Aug 29, 12:31 AM
Unknown Object (File)
Aug 13 2026, 1:21 PM
Unknown Object (File)
Aug 12 2026, 8:11 AM
View All Files
Subscribers
delphij
emaste
imp

Details

Reviewers
alc
markj
kib
dougm
Commits
rS352176: Replace redundant code with a few new vm_page_grab facilities:
Summary

This is a combination of the below two reviews.

https://reviews.freebsd.org/D21332
https://reviews.freebsd.org/D21331

I eliminated VM_ALLOC_VALID in favor of using VM_ALLOC_NOCREAT with vm_page_grab_valid(). It simplifies things a bit and I think is less ambiguous.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

jeff created this revision.Sep 6 2019, 12:35 PM
Herald added subscribers: emaste, delphij. · View Herald TranscriptSep 6 2019, 12:35 PM
Harbormaster completed remote builds in B26305: Diff 61723.Sep 6 2019, 12:35 PM
jeff retitled this revision from Eliminate redundant with a few new convenience functions. to Eliminate redundant vm code with a few new convenience functions..Sep 6 2019, 12:42 PM
jeff edited the summary of this revision. (Show Details)
jeff added reviewers: alc, markj, kib, dougm.
This revision was not accepted when it landed; it landed in state Needs Review.Sep 10 2019, 7:08 PM
Closed by commit rS352176: Replace redundant code with a few new vm_page_grab facilities: (authored by jeff). · Explain Why
This revision was automatically updated to reflect the committed changes.
jeff added a commit: rS352176: Replace redundant code with a few new vm_page_grab facilities:.
Herald added a subscriber: imp. · View Herald TranscriptSep 10 2019, 7:08 PM

Revision Contents
Changeset List

PathSize
head/
sys/
cddl/
contrib/
opensolaris/
uts/
common/
fs/
zfs/
61 lines
compat/
linuxkpi/
common/
src/
24 lines
dev/
drm2/
ttm/
7 lines
18 lines
xen/
gntdev/
2 lines
privcmd/
2 lines
kern/
25 lines
vm/
19 lines
12 lines
4 lines
177 lines

Diff 61905

View Options

head/sys/cddl/contrib/opensolaris/uts/common/fs/zfs/zfs_vnops.c

Loading...
View Options

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

Loading...
View Options

head/sys/dev/drm2/ttm/ttm_bo_vm.c

Loading...
View Options

head/sys/dev/drm2/ttm/ttm_tt.c

Loading...
View Options

head/sys/dev/xen/gntdev/gntdev.c

Loading...
View Options

head/sys/dev/xen/privcmd/privcmd.c

Loading...
View Options

head/sys/kern/uipc_shm.c

Loading...
View Options

head/sys/vm/vm_glue.c

Loading...
View Options

head/sys/vm/vm_object.c

Loading...
View Options

head/sys/vm/vm_page.h

Loading...
View Options

head/sys/vm/vm_page.c

Loading...