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

Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12512
Build 12787: arc lint + arc unit

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

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

1900

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

sys/cam/ctl/ctl.c
422

Yeah sure I'll do that.

1900

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 ↗(On Diff #34886)

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 ↗(On Diff #34886)

s/The following/These/

200 ↗(On Diff #34886)

"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 ↗(On Diff #34886)

As above:

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