Page MenuHomeFreeBSD

tidy up swap_pager_putpages
ClosedPublic

Authored by dougm on Jul 28 2019, 4:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 27 2023, 4:15 PM
Unknown Object (File)
Dec 20 2023, 3:32 AM
Unknown Object (File)
Nov 14 2023, 3:13 AM
Unknown Object (File)
Nov 12 2023, 6:18 AM
Unknown Object (File)
Nov 5 2023, 10:03 AM
Unknown Object (File)
Nov 5 2023, 9:06 AM
Unknown Object (File)
Oct 11 2023, 5:20 AM
Unknown Object (File)
Oct 8 2023, 11:57 AM
Subscribers

Details

Summary

(1) convert the panic() at the function's start into a KASSERT(), (2) convert sync into a bool-type variable async, (3) make the variable definitions style(9) compliant, and (4) move the calls to swp_pager_init_freerange() and swp_pager_freeswapspace() out from under the object lock.

Diff Detail

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

Event Timeline

swap_pager.c
1365 ↗(On Diff #60208)

Isn't this assignment unnecessary?

1384 ↗(On Diff #60208)

I think it's within the scope of this patch to fix the style issue here: Add spaces around the '+'.

1414 ↗(On Diff #60208)

Add spaces around the '+'.

swap_pager.c
1355 ↗(On Diff #60208)

I think that this one line no longer applies.

1364 ↗(On Diff #60208)

I would delete this blank line because the above block comment describes the below initialization to 'async'.

Apply reviewer suggestions. Make another style(9) fix.

swap_pager.c
1335 ↗(On Diff #60211)

Please fix the "whos".

1355–1356 ↗(On Diff #60211)

Please capitalize and punctuate these sentences.

1387 ↗(On Diff #60211)

Shouldn't this be a period instead of a comma?

alc added reviewers: kib, markj.

I think that this is a good change and ready to commit. Give Kostik and Mark a chance to comment.

This revision is now accepted and ready to land.Jul 28 2019, 5:07 PM