Evaluate the real size of the sblk_zone.
ClosedPublic
Actions

Authored by kib on Oct 13 2017, 3:28 PM.

Tags

None

Referenced Files

	F168115004: D12660.id33958.diff
	Wed, Aug 26, 11:12 AM

	F168110471: D12660.id33957.diff
	Wed, Aug 26, 10:45 AM

	F168106857: D12660.id.diff
	Wed, Aug 26, 10:23 AM

	F168104575: D12660.diff
	Wed, Aug 26, 10:07 AM

	F168067367: D12660.diff
	Wed, Aug 26, 5:36 AM

	Unknown Object (File)
	Fri, Aug 14, 11:01 AM

	Unknown Object (File)
	Sun, Aug 9, 5:33 PM

	Unknown Object (File)
	Sun, Aug 9, 3:22 PM

View All Files

Subscribers

imp

Details

Reviewers

alc
markj

Commits

rS324600: Evaluate the real size of the sblk_zone.

Summary

This allows to not waste some small amount of memory, if available.

Submitted by: ota@j.email.ne.jp
PR: 221356

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, 3:28 PM

Herald added a subscriber: imp. · View Herald TranscriptOct 13 2017, 3:28 PM

I think that the change is fine, but it needs a comment. Otherwise, this is going to look awfully strange to someone who isn't familiar with the inner works of UMA.

Makes sense to me.

This revision is now accepted and ready to land.Oct 13 2017, 3:37 PM

Add comment.

This revision now requires review to proceed.Oct 13 2017, 3:50 PM

alc accepted this revision.Oct 13 2017, 4:05 PM

alc added inline comments.

sys/vm/swap_pager.c
546–548 ↗	(On Diff #33958)	"Often uma_zone_reserve_kva() cannot reserve exactly the requested size. Account for the difference when calculating swap_maxpages.