Differential D17494

Add missing ZFS sysctls
ClosedPublic
Actions

Authored by allanjude on Oct 10 2018, 5:24 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Oct 23, 12:12 AM

	Unknown Object (File)
	Tue, Oct 21, 10:43 PM

	Unknown Object (File)
	Sep 12 2025, 11:19 AM

	Unknown Object (File)
	Aug 18 2025, 7:56 AM

	Unknown Object (File)
	Jul 15 2025, 10:27 AM

	Unknown Object (File)
	Jul 15 2025, 8:06 AM

	Unknown Object (File)
	Apr 17 2025, 6:03 PM

	Unknown Object (File)
	Jan 28 2025, 4:36 AM

View All 38 Files

Subscribers

Details

Reviewers

mav
mmacy
jpaetzel
avg

Group Reviewers

Commits

rS339298: Add missing sysctls for tuning vdev queue depths for new I/O types

Summary

During the merge of rS329502 (zpool remove) and rS337007 (zpool initialize),
freebsd specific code to connect the new tunables was not added.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

allanjude created this revision.Oct 10 2018, 5:24 AM

Herald added subscribers: delphij, imp. · View Herald TranscriptOct 10 2018, 5:24 AM

Harbormaster completed remote builds in B20107: Diff 48959.Oct 10 2018, 5:24 AM

LGTM. The change is trivial enough.

This revision is now accepted and ready to land.Oct 10 2018, 5:30 AM

allanjude marked an inline comment as done.Oct 10 2018, 5:34 AM

allanjude added inline comments.

sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c
260 ↗	(On Diff #48959)	@avg While you are here, do you know what this #undef is about? Was it meant to undef ZFS_VDEV_QUEUE_KNOB_MIN and ZFS_VDEV_QUEUE_KNOB_MAX and just got lost in time? Should it be removed?

avg added inline comments.Oct 10 2018, 6:37 AM

sys/cddl/contrib/opensolaris/uts/common/fs/zfs/vdev_queue.c
260 ↗	(On Diff #48959)	It seems that that's how I committed it in rS258632, but mostly likely it was indeed a result of incremental development where I initially had a single macro to define both sysctl-s.

mav accepted this revision.Oct 10 2018, 2:37 PM

Closed by commit rS339298: Add missing sysctls for tuning vdev queue depths for new I/O types (authored by allanjude). · Explain WhyOct 10 2018, 10:55 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

sys/

cddl/

contrib/

opensolaris/

uts/

common/

fs/

zfs/

4 lines

Diff 48995

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

Loading...