Differential D21093

tidy up swap_pager_putpages
ClosedPublic
Actions

Authored by dougm on Jul 28 2019, 4:38 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, May 22, 3:24 PM

	Unknown Object (File)
	May 17 2026, 4:54 PM

	Unknown Object (File)
	May 15 2026, 3:38 AM

	Unknown Object (File)
	May 6 2026, 10:48 PM

	Unknown Object (File)
	May 5 2026, 10:30 AM

	Unknown Object (File)
	May 1 2026, 10:32 AM

	Unknown Object (File)
	Apr 28 2026, 10:11 AM

	Unknown Object (File)
	Apr 28 2026, 4:25 AM

Subscribers

Details

Reviewers

alc
kib
markj

Commits

rS350392: In swap_pager_putpages, move the initialization of a free-blocks

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

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dougm created this revision.Jul 28 2019, 4:38 AM

alc added inline comments.Jul 28 2019, 3:59 PM

swap_pager.c
1366–1367	Isn't this assignment unnecessary?
1382	I think it's within the scope of this patch to fix the style issue here: Add spaces around the '+'.
1411–1412	Add spaces around the '+'.

alc added inline comments.Jul 28 2019, 4:04 PM

swap_pager.c
1355–1356	I think that this one line no longer applies.
1366	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.

alc added inline comments.Jul 28 2019, 4:47 PM

swap_pager.c
1335	Please fix the "whos".
1355–1356	Please capitalize and punctuate these sentences.
1387	Shouldn't this be a period instead of a comma?

Tidy up comments.

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

markj accepted this revision.Jul 28 2019, 6:34 PM

kib accepted this revision.Jul 28 2019, 7:05 PM

Closed by commit rS350392: In swap_pager_putpages, move the initialization of a free-blocks (authored by dougm). · Explain WhyJul 28 2019, 7:32 PM

This revision was automatically updated to reflect the committed changes.

dougm added a commit: rS350392: In swap_pager_putpages, move the initialization of a free-blocks.

Herald added a subscriber: imp. · View Herald TranscriptJul 28 2019, 7:32 PM

Revision Contents
Changeset List

Path

Size

44 lines

Diff 60211

swap_pager.c

Loading...