summaryrefslogtreecommitdiffstats
path: root/sys/dev/ciss/ciss.c
Commit message (Collapse)AuthorAgeFilesLines
* Correctly set the interrupt enable and disable bits. The previousscottl2008-08-021-4/+0
| | | | | | code interfered with Performant mode and legacy interrupts. Also remove a register read operation on the Simplq code that was effectively a time-wasting no-op.
* A number of significant enhancements to the ciss driver:scottl2008-07-111-93/+388
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. The FreeBSD driver was setting an interrupt coalesce delay of 1000us for reasons that I can only speculate on. This was hurting everything from lame sequential I/O "benchmarks" to legitimate filesystem metadata operations that relied on serialized barrier writes. One of my filesystem tests went from 35s to complete down to 6s. 2. Implemented the Performant transport method. Without the fix in (1), I saw almost no difference. With it, my filesystem tests showed another 5-10% improvement in speed. It was hard to measure CPU utilization in any meaningful way, so it's not clear if there was a benefit there, though there should have been since the interrupt handler was reduced from 2 or more PCI reads down to 1. 3. Implemented MSI-X. Without any docs on this, I was just taking a guess, and it appears to only work with the Performant method. This could be a programming or understanding mistake on my part. While this by itself made almost no difference to performance since the Performant method already eliminated most of the synchronous reads over the PCI bus, it did allow the CISS hardware to stop sharing its interrupt with the USB hardware, which in turn allowed the driver to become decoupled from the Giant-locked USB driver stack. This increased performance by almost 20%. The MSI-X setup was done with 4 vectors allocated, but only 1 vector used since the performant method was told to only use 1 of 4 queues. Fiddling with this might make it work with the simpleq method, not sure. I did not implement MSI since I have no MSI-specific hardware in my test lab. 4. Improved the locking in the driver, trimmed some data structures. This didn't improve test times in any measurable way, but it does look like it gave a minor improvement to CPU usage when many processes/threads were doing I/O in parallel. Again, this was hard to accurately test.
* Add support for the P212, P410, P410i, P411, and P812 HP Smart Arrayps2008-06-101-0/+5
| | | | | | controllers. Submitted by: Scott Benesh at HP
* Remove a check that didn't allow > 12 byte CDB's to be issued tops2008-05-161-6/+0
| | | | | | ciss. This should allow volumes > 2TB to work. Reported by: Emil Mikulic
* Add `hw.ciss.nop_message_heartbeat' tunable (default disabled) foriwasaki2008-02-101-1/+4
| | | | | | | | | | | NOP-message polling in ciss_periodic(). Note that setting the tunable to non-zero can be workaround only for `ADAPTER HEARTBEAT FAILED' problem, and may freeze the system w/o the problem. Reviewed by: scottl Reported by: Attila Nagy MFC after: 3 days
* Don't repeat error logging about NOP message sending ifiwasaki2008-01-281-1/+5
| | | | | | | | ciss_report_request() return an error (which is most likely data underrun). Noticed by: Mark Atkinson MFC after: 1 week
* Fix NOP message sending in ciss_periodic() which causes panic withiwasaki2007-11-051-3/+16
| | | | | | | | option INVARIANTS. Reviewed by: simokawa Tested by: noriyosi_kawano MFC after: 1 week
* Rename the kthread_xxx (e.g. kthread_create()) callsjulian2007-10-201-3/+3
| | | | | | | | | | | to kproc_xxx as they actually make whole processes. Thos makes way for us to add REAL kthread_create() and friends that actually make theads. it turns out that most of these calls actually end up being moved back to the thread version when it's added. but we need to make this cosmetic change first. I'd LOVE to do this rename in 7.0 so that we can eventually MFC the new kthread_xxx() calls.
* Add NOP-message polling to ciss_periodic().iwasaki2007-10-131-1/+80
| | | | | | | | Disable adapter by detecting adapter is dead. Tested by: Masaki YATSU(on RELENG_6) Reviewed by: scottl MFC after: 1 week
* Fix a mistake made in the MPSAFE commit that caused CAM to serialize requestsscottl2007-10-121-1/+2
| | | | to the controller.
* Prepare for future integration between CAM and newbus. xpt_bus_registerscottl2007-06-171-2/+2
| | | | | | | now takes a device_t to be the parent of the bus that is being created. Most SIMs have been updated with a reasonable argument, but a few exceptions just pass NULL for now. This argument isn't used yet and the newbus integration likely won't be ready until after 7.0-RELEASE.
* Satisfy witness during shutdownscottl2007-06-051-0/+2
|
* Eliminate M_TEMP.scottl2007-05-141-3/+3
|
* Streamline locking in ciss_free()scottl2007-05-021-26/+27
|
* MPSAFE ciss driverscottl2007-05-011-19/+44
|
* Remove Giant from CAM. Drivers (SIMs) now register a mutex that CAM willscottl2007-04-151-2/+2
| | | | | | | | | | | use to synchornize and protect all data objects that are used for that SIM. Drivers that are not yet MPSAFE register Giant and operate as usual. RIght now, no drivers are MPSAFE, though a few will be changed in the coming week as this work settles down. The driver API has changed, so all CAM drivers will need to be recompiled. The userland API has not changed, so tools like camcontrol do not need to be recompiled.
* o break newbus api: add a new argument of type driver_filter_t topiso2007-02-231-1/+1
| | | | | | | | | | | | | bus_setup_intr() o add an int return code to all fast handlers o retire INTR_FAST/IH_FAST For more info: http://docs.freebsd.org/cgi/getmsg.cgi?fetch=465712+0+current/freebsd-current Reviewed by: many Approved by: re@
* o ciss.ko depends on cam and pci.maxim2006-11-301-0/+2
| | | | | | PR: kern/105989 Submitted by: nork MFC after: 1 month
* 2nd and final commit that moves us to CAM_NEW_TRAN_CODEmjacob2006-11-021-9/+0
| | | | | | as the default. Reviewed by multitudes.
* The first of 3 major steps to move the CAM layer forward to usingmjacob2006-10-311-3/+21
| | | | | | | | | | | | | | | | | | | | | the CAM_NEW_TRAN_CODE that has been in the tree for some years now. This first step consists solely of adding to or correcting CAM_NEW_TRAN_CODE pieces in the kernel source tree such that a both a GENERIC (at least on i386) and a LINT build with CAM_NEW_TRAN_CODE as an option will compile correctly and run (at least with some the h/w I have). After a short settle time, the other pieces (making CAM_NEW_TRAN_CODE the default and updating libcam and camcontrol) will be brought in. This will be an incompatible change in that the size of structures related to XPT_PATH_INQ and XPT_{GET,SET}_TRAN_SETTINGS change in both size and content. However, basic system operation and basic system utilities work well enough with this change. Reviewed by: freebsd-scsi and specific stakeholders
* Spin until a request structure is available in the ioctl path.ps2006-05-301-2/+2
|
* Since DELAY() was moved, most <machine/clock.h> #includes have beenphk2006-05-161-1/+0
| | | | unnecessary.
* Free another memory leak when dealing with disk notification.ps2006-04-201-1/+4
|
* Free some previously leaked memory on module unload.ps2006-04-191-2/+9
|
* It seems ciss should ignore overrun and underrun on a SCSI INQUIRYps2005-12-161-1/+4
| | | | | | | command. This fixes some weird booting issues on newer versions of the firmware on the MSA20. Reported by: Philippe Pegon <Philippe dot Pegon at crc dot u-strasbg dot fr>
* Update PCI ids to add the E200, E200i, P400, and P400i storageps2005-11-111-3/+8
| | | | | | controllers. Remove the E400 since it is not a real product. Submitted by: HP
* There's no reason to check the valence. This allows ciss to workps2005-08-091-6/+0
| | | | on the P600.
* Remove bus_{mem,p}io.h and related code for a micro-optimization on i386nyan2005-05-291-1/+0
| | | | | | and amd64. The optimization is a trivial on recent machines. Reviewed by: -arch (imp, marcel, dfr)
* Support passthru ioctl commands from 32bit binaries.ps2005-05-181-1/+17
|
* Add support for the P600 and name the E400.ps2005-04-281-1/+2
|
* Provide a way to soft reset a proxy controller such as an MSA20 orps2005-04-191-0/+55
| | | | | MSA500. This is useful if you need to reset one of the storage arrays on reboot.
* handle ciss_lookup failuresam2005-03-291-0/+5
| | | | Noticed by: Coverity Prevent analysis tool
* Use BUS_PROBE_DEFAULT for pci probe return valueimp2005-03-051-1/+1
|
* - Remove dead code.scottl2005-02-211-3/+2
| | | | | | - Protect against negative values as array indexes. Submitted by: Coverity Prevent analysis tool
* Remove 6422, V100 and add the P600.ps2005-01-201-2/+1
| | | | Submitted by: John Cagle
* ciss's interrupt handler was missing the INTR_ENTROPY flag.peter2004-08-161-1/+2
|
* Name non-physical devices.ps2004-07-281-1/+1
|
* Reset the update flag when scanning for new devices.ps2004-06-221-0/+1
|
* Use the maximum physical bus when rescanning those targets. Thisps2004-06-221-1/+1
| | | | fixes a panic on cards which do not have any drives attached.
* on media exchange, update/rescan the drives. This allows a volumeps2004-06-211-0/+4
| | | | in a failed stated to come back on-line without a reboot.
* There is no need to call ciss_media_accept async anymore since allps2004-06-211-42/+16
| | | | notify events are handled in a kthread.
* Add SCSI passthrough support to CISS. This allows devices like tape drivesscottl2004-06-211-22/+260
| | | | | | | | | | | that are on a CISS bus to be exported up to CAM and made available as normal devices. This will typically add one or two buses to CAM, which will be numbered starting at 32 to allow room for CISS proxy buses. Also, the CISS firmware usually hides disk devices, but these can also be exposed as 'pass' devices if you set the hw.ciss.expose_hidden_physical tunable. Sponsored by: Tape Laboratories, Inc. MFC After: 3 days
* Do the dreaded s/dev_t/struct cdev */phk2004-06-161-3/+3
| | | | Bump __FreeBSD_version accordingly.
* If a device is not ONLINE, do not attempt to submit any commandsps2004-06-151-23/+4
| | | | | | | or I/O to the volume. This solves a panic when removing a disk in a RAID 0 volume. Reported by: le
* Add pci id's for the SmartArray 6422 and V100 controllers. Alsops2004-06-051-0/+17
| | | | | | add a whole bunch of pci id's for future controllers. Submitted by: John Cagle <first.last@hp.com>
* Turn SCSI pre-fetch ON. This is mainly for 64XX and 64X basedps2004-05-171-0/+4
| | | | | | | | controllers and allows the controller to prefetch 1-2k on certain PCI memory reads to the host. The spec says this should only be used for IA32 based systems. Informed of feature by: John Cagle <first.last@hp.com>
* make this compile/work with CISS_DEBUG defined.ps2004-04-261-8/+10
|
* There's no need to call ciss_report_request in the passthru ioctlps2004-04-191-3/+1
| | | | | | routine since the error will be reported back to the user buffer. This will quiet down the bootverbose case when using an ACU which does brute force discovery of the physical and logical devices.
* Report only new new events when initially attaching to the controller.ps2004-04-191-1/+4
|
* move the cleanup of the control device into ciss_free and add someps2004-04-181-2/+10
| | | | | ifdefs for the diffrent kthread_create API between -current and -stable
OpenPOWER on IntegriCloud