kern/151564: [ciss] ciss(4) should increase CISS_MAX_LOGICAL to 107

[xenophon\+freebsd]

All:

I've created a patch (see attached) for the ciss driver that makes the
following changes:

*	creates a boot-time tunable (hw.ciss.max_logical) that allows
changing the maximum number of logical drives (defaults to
CISS_MAX_LOGICAL, which currently is 15 according to
src/sys/dev/ciss/cissvar.h)

*	changes the behavior of another boot-time tunable
(hw.ciss.expose_hidden_physical) such that a value of 2 exposes hidden
physical direct access devices as da(4) devices, where a value of 1
retains the original behavior of exposing them only as pass(4) devices

*	updates the ciss(4) manual page regarding the above
modifications

Would a committer be willing to review this patch for potential
inclusion into some future release of FreeBSD?  I hereby place this
patch into the public domain.

I tested this successfully using FreeBSD/x86 9.0-RELEASE-p5 on an HP
ProLiant DL380 G3 with an on-board Smart Array 5i controller.  For
example, I have 20 single-disk RAID-0 arrays.  Without the patch, the
ciss driver returns the error "adapter claims to report absurd number of
logical drives".  With the patch applied and hw.ciss.max_logical set to
32 in /boot/loader.conf, the ciss driver correctly detects and
configures all twenty logical drives.  Furthermore, with
hw.ciss.expose_hidden_physical to 2, the ciss driver also correctly
exposes the physical disks underlying each of the 20 arrays as da(4)
devices.

I plan to test this patch next week using FreeBSD/amd64 on a DL380 G4,
which has similar specs to my G3.

Best wishes,
Matthew

-- 
I FIGHT FOR THE USERS


-------------- next part --------------
A non-text attachment was scrubbed...
Name: ciss-tunable-maxlogical-exposehidden-da.patch
Type: application/octet-stream
Size: 6054 bytes
Desc: ciss-tunable-maxlogical-exposehidden-da.patch
URL: <http://lists.freebsd.org/pipermail/freebsd-scsi/attachments/20121220/b52ebf68/attachment.obj>