rearrange attach so that the SDEV_VIRTUAL flag is set during scsi_probe,
rather than as a scan of all attached devices after scsibus is attached.

this will allow the cache enabling on virtual disks

rearrange attach so that the SDEV_VIRTUAL flag is set during scsi_probe,
rather than as a scan of all attached devices after scsibus is attached.

this will allow the cache enabling on virtual disks to run as part of the
disks attach routine.

show more ...

dont pass the dev_t from the scsi device drivers into the midlayer for
ioctl requests, and dont pass the proc pointers around for any ioctl
requests in scsi land at all. neither were used, so trim th

dont pass the dev_t from the scsi device drivers into the midlayer for
ioctl requests, and dont pass the proc pointers around for any ioctl
requests in scsi land at all. neither were used, so trim the fat.

ok krw@ marco@

show more ...

the virtual scsi disks that mpi(4) presents dont grok modifications to the
mode pages that control the caches. this adds code that talks to the mpi
chip directly on behalf of those disks so you can e

the virtual scsi disks that mpi(4) presents dont grok modifications to the
mode pages that control the caches. this adds code that talks to the mpi
chip directly on behalf of those disks so you can enable write caching on
them.

show more ...

Use a temporary variable for now to sidestep -Wbounded checking when
copying vendor[8]/product[16]/revision[4] out of struct scsi_inquiry_data
together with one memcopy.

ok krw

back out 1.143, it causes data corruption on the mpis in sun v20z boxes,
but i suspect it is common to all SPI mpi parts.

problem found and problem diff verified by landry.

ok krw@ landry@ jasper@

Fix gcc4 warning

use BUS_DMA_ZERO on alloc instead of bzeroing after.

ok dlg@, marco@

Fix cut 'n paste typo

i thought mpi gave each device all the openings on the bus, which was a
big motivation to implementing iopools. while looking at another issue i
noticed that openings were cut up for each disk.

this

i thought mpi gave each device all the openings on the bus, which was a
big motivation to implementing iopools. while looking at another issue i
noticed that openings were cut up for each disk.

this cranks openings to maxcmds.

show more ...

byteswap the number of blocks on physical disks for bioctl correctly.
makes output sane on sparc64 and other BE archs.

found by jason george

if there is no raid, do not allocate a 0-sized structure for sensors
and then start attaching it with 0 sensors attached
ok dlg

dont need to call scsi_done with splbio. the midlayer protects
itself now, its not the adapters responsibility anymore.

fix double free in an error path. the midlayer gets the ccb for scsi io
now, so it not our job to free it.

use SLISTs for managing the ccb free list rather than TAILQs.

modify mpi to provide an iopool as a way for the midlayer to manage access
to its free ccbs.

this allows the midlayer to schedule access to the bus in a roundrobin
fashion for all consumers on the b

modify mpi to provide an iopool as a way for the midlayer to manage access
to its free ccbs.

this allows the midlayer to schedule access to the bus in a roundrobin
fashion for all consumers on the bus, including io from devices and even
the internal mpi management commands used to poll the state of raid
devices. the result is fairer sharing between disks on the bus and more
reliable sensor updates.

ok krw@ beck@ marco@
tested by beck@

show more ...

dont allocate with M_TEMP and then free with M_DEVBUF. made even worse that
this was done for every sensor update, which really screwed up the memory
stats.

noticed by deraadt@

Change the scsi_cmd function member of scsi_adapter from int to
void. Use XS_NO_CCB error in the scsi command (xs) to report the
NO_CCB condition. Eliminates all SUCCESSFULLY_QUEUED and COMPLETE
conf

Change the scsi_cmd function member of scsi_adapter from int to
void. Use XS_NO_CCB error in the scsi command (xs) to report the
NO_CCB condition. Eliminates all SUCCESSFULLY_QUEUED and COMPLETE
confusion and untangles the midlayer from the adapter a bit more.

Eyes and some fixes by miod@

There may be some compile issues on little used (i.e. I don't have
any) drivers but the change is mechanical and thus easy to remedy.

ok dlg@

show more ...

rework the polling code to use the semantic krw@ proposed.

intercept the ccb_done handling so polled commands set a flag that mpi_poll
tests on. when ccb_done sets the variable, the poll loop breaks

rework the polling code to use the semantic krw@ proposed.

intercept the ccb_done handling so polled commands set a flag that mpi_poll
tests on. when ccb_done sets the variable, the poll loop breaks and
mpi_poll runs the original ccb_done handler for the ccb completion.

this is a lot simpler than the previous implementation and removes a
mutex.

ok beck@

show more ...

Zap all setting of ITSDONE in drivers that don't look at it. Nobody
else cares so it's just noise. Drivers that actually look at ITSDONE
are unchanged.

ok marco@ (for his files) dlg@ beck@

oops, get the order of args right for the header request in
mpi_bio_get_pg0_raid. the sensor updates dont poll at all now.

mpi_bio_get_pg0_raid is only called from a process context. let the cfg
requests sleep rather than poll for completion.

oops, forgot to set the ccb_cookie in mpi_wait. mpi_wait_done would fault
when it used the uninitialized cookie.

get rid of the last internal user of splbio. waiting for the completion of
a ccb can now be done with mpi_wait. this switches the cfg page handlers
over from their own tsleep stuff to mpi_wait.

rename ccb_xs to ccb_cookie, and switch it from a struct scsi_xfer * to a
void *. this will let me stash things other than scsi xfers in the ccb for
ccb_done handlers to use.

dont leak a ccb if we fail to get a reply in portenable