Page MenuHomeFreeBSD

CISS: Support >48 JBOD drives, fix SES enumeration, no panic on unplug, sysctl tunables
AbandonedPublic

Authored by zarychtam_plan-b.pwste.edu.pl on Aug 11 2024, 8:56 AM.
Tags
None
Referenced Files
F110229912: D46270.diff
Sat, Feb 15, 10:55 AM
Unknown Object (File)
Fri, Jan 24, 8:04 AM
Unknown Object (File)
Jan 10 2025, 6:13 PM
Unknown Object (File)
Dec 31 2024, 2:32 AM
Unknown Object (File)
Dec 18 2024, 2:44 PM
Unknown Object (File)
Nov 22 2024, 4:47 PM
Unknown Object (File)
Nov 18 2024, 4:08 PM
Unknown Object (File)
Nov 15 2024, 9:43 AM

Details

Reviewers
None
Summary

Not to hijack, but to boost the thread I am uploading this patch rebased on the most recent CURRENT. Perhaps a wider audience will find it also useful as I found this patch by Peter Eriksson. Thank you for sharing your work Peter !

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

zarychtam_plan-b.pwste.edu.pl updated this revision to Diff 141985.

Not to hijack, but to boost the thread I am uploading this patch rebased on the most recent CURRENT. Perhaps a wider audience will find it also useful as I found to. The original patch by Peter Eriksson submitted to review D25155

Thank you for sharing your work Peter !

So... This wasn't committed before because it intermixed the verbosity changes and the other changes.
The maintainer of CISS at the time demured on that aspect of it, but had already started to disengage from the community so wasn't good about communicating that.
Please separate the two sets out. It will be needed to upstream anyway. Since you've started a new review, I'd suggest removing the verbosity ones from this one and creating a new one. (or if you are more comfortable, moving to a github pull request where we can look at both at the same time, and you can also have two commits).
Also, there's no enough context here. Please re-upload with -U99999 if you decide to continue here and not as a pull request.

The other reason I didn't commit this before was that I had no CISS hardware to test with, and my line on getting that dried up. I also don't think I have any disks to test with, but I may be mistaken about that detail.

It would also be useful to know when receive diagnostic commands are sent/received and why we're justified adding them to that list. But given the history of this, we might not get that detail.

zarychtam_plan-b.pwste.edu.pl retitled this revision from CISS: Support >48 JBOD drives, fix SES enumeration, no panic on unplug, sysctl tunables &more verbosity to CISS: Support >48 JBOD drives, fix SES enumeration, no panic on unplug, sysctl tunables.

Verbosity and RECEIVE_DIAGNOSTIC were removed, the rest remains.

The original review D25155 (https://reviews.freebsd.org/D25155) has been committed in rGf03e1a42e92e: ciss: Minor formatting nit. (authored by pen_lysator.liu.se, committed by imp https://reviews.freebsd.org/rGf03e1a42e92eff76dcf474655b600db37b04ae2b)

If there's anything else that needs to be done, or if I botched something in the merge, please let me know.