Page MenuHomeFreeBSD
Differential D12663

Move swapout code into vm_swapout.c.
ClosedPublic
Actions

Authored by kib on Oct 13 2017, 7:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Sep 13, 4:13 PM
Unknown Object (File)
Sun, Sep 13, 12:57 AM
Unknown Object (File)
Sat, Sep 12, 2:56 AM
Unknown Object (File)
Fri, Sep 11, 9:07 AM
Unknown Object (File)
Thu, Sep 10, 12:16 PM
Unknown Object (File)
Thu, Sep 10, 8:41 AM
Unknown Object (File)
Sun, Sep 6, 10:13 AM
Unknown Object (File)
Sun, Sep 6, 2:33 AM
View All Files
Subscribers
imp

Details

Reviewers
alc
markj
Commits
rS324795: Move swapout code into vm/vm_swapout.c.
Summary

There is no NO_SWAPPING #ifdef left in the code.

Test Plan

Compile-tested only ATM. Both !NO_SWAPPING and NO_SWAPPING configs.

Diff Detail

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

Event Timeline

kib created this revision.Oct 13 2017, 7:05 PM
Herald added a subscriber: imp. · View Herald TranscriptOct 13 2017, 7:05 PM
kib edited the summary of this revision. (Show Details)Oct 13 2017, 7:09 PM
markj accepted this revision.Oct 13 2017, 7:31 PM

Looks ok to me. I like this approach better than, e.g., ffs_softdep.c, where the stubs are kept in the same file.

sys/vm/vm_swapout_dummy.c
71 ↗(On Diff #33965)

Extra newline.

This revision is now accepted and ready to land.Oct 13 2017, 7:31 PM
kib marked an inline comment as done.Oct 13 2017, 8:06 PM
kib updated this revision to Diff 33969.Oct 13 2017, 8:13 PM

Remove empty line.
Do not ever return from NO_SWAPPING swapper().

This revision now requires review to proceed.Oct 13 2017, 8:13 PM
alc accepted this revision.Oct 19 2017, 5:14 AM
This revision is now accepted and ready to land.Oct 19 2017, 5:14 AM
Closed by commit rS324795: Move swapout code into vm/vm_swapout.c. (authored by kib). · Explain WhyOct 20 2017, 9:10 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
conf/
4 lines
sys/
4 lines
vm/
1 line
501 lines
9 lines
395 lines
975 lines
122 lines

Diff 34171

View Options

head/sys/conf/files

Loading...
View Options

head/sys/sys/_kstack_cache.h

Loading...
View Options

head/sys/vm/vm_extern.h

Loading...
View Options

head/sys/vm/vm_glue.c

Loading...
View Options

head/sys/vm/vm_pageout.h

Loading...
View Options

head/sys/vm/vm_pageout.c

Loading...
View Options

head/sys/vm/vm_swapout.c

Loading...
View Options

head/sys/vm/vm_swapout_dummy.c

Loading...