Page MenuHomeFreeBSD

ctl: Make max_luns and max_ports tunable variables instead of hardcoded defines.
ClosedPublic

Authored by manu on Oct 30 2017, 3:45 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 19 2024, 9:28 PM
Unknown Object (File)
Jan 13 2024, 10:30 PM
Unknown Object (File)
Jan 11 2024, 9:46 PM
Unknown Object (File)
Jan 8 2024, 11:14 AM
Unknown Object (File)
Dec 23 2023, 12:54 AM
Unknown Object (File)
Nov 29 2023, 7:55 AM
Unknown Object (File)
Nov 22 2023, 9:33 PM
Unknown Object (File)
Nov 17 2023, 1:39 PM
Subscribers

Details

Diff Detail

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

Event Timeline

bcr added a subscriber: bcr.

OK from manpages.

This revision is now accepted and ready to land.Oct 30 2017, 8:31 PM
sys/cam/ctl/ctl.c
422 ↗(On Diff #34459)

Could you add read-only (CTL_RDTUN) sysctl-s for these two, to make it possible to easily check the current values?

1896 ↗(On Diff #34459)

Could you reword it to indicate what the valid value would be?

sys/cam/ctl/ctl.c
422 ↗(On Diff #34459)

Yeah sure I'll do that.

1896 ↗(On Diff #34459)

Will do.

Address trasz@ comments.

This revision now requires review to proceed.Nov 6 2017, 10:25 AM
manu marked 4 inline comments as done.Nov 6 2017, 10:26 AM
This revision is now accepted and ready to land.Nov 6 2017, 6:01 PM

Fix compilation when CTL_LEGACY_STATS is defined.

This revision now requires review to proceed.Nov 7 2017, 10:18 AM
This revision was automatically updated to reflect the committed changes.
bjk added inline comments.
head/share/man/man4/ctl.4
202

mdoc style is to start a new line after a sentence-ending full stop. So, something like

Specifies the maximum number of ports supported, which must be a power of 2.
The default value is 256

(similarly for line 200.)

Please remember to update .Dd on commit. Thanks!

head/share/man/man4/ctl.4
195

s/The following/These/

200

"Specifies the" doesn't add anything and makes the reader search farther in the sentence for what this actually is.

Maximum number of LUNs supported.
Must be a power of 2.  Default 1024.
202

As above:

Maximum number of ports supported.
Must be a power of 2. Default 256.