src-test2/sys/dev/ips/ips_commands.c, branch release/7.2.0_cvsFreeBSD source treehttps://cgit-dev.freebsd.org/src-test2/atom?h=release%2F7.2.0_cvs2009-05-01T02:51:58ZCopy releng/7.2 to release/7.2.0 for FreeBSD 7.2-RELEASE.2009-05-01T02:51:58ZKen Smithkensmith@FreeBSD.org2009-05-01T02:51:58Zurn:sha1:b205d83df7763ca1f099bce5ac12a05862a626c4
Approved by: re (implicit)
This commit was manufactured to restore the state of the 7.2-RELEASE image.
Separate the hardware definitions into ipsreg.h so they can be used by2005-11-29T09:39:41ZScott Longscottl@FreeBSD.org2005-11-29T09:39:41Zurn:sha1:f94dfeb4810245d485d85e8a494a3d339d7f1d95
future userland tools.
Fix build.2005-09-27T09:11:44ZGleb Smirnoffglebius@FreeBSD.org2005-09-27T09:11:44Zurn:sha1:5cd88eb04abc52308f5a4435a807fbc50b5ad55cReport status in hex, not decimal.2005-09-27T01:59:32ZScott Longscottl@FreeBSD.org2005-09-27T01:59:32Zurn:sha1:722e0ae489d52e8b4c00f28e2205e77a019ef0f0Overhaul error handling in the IPS driver. Don't use a magic value for2005-09-25T17:12:41ZScott Longscottl@FreeBSD.org2005-09-25T17:12:41Zurn:sha1:2eea70515c189e5c7c5457efd3e875ed395d736b
driver-induced errors, instead be better about propagating error status
upwards. Add more error definitions, courtesy of the linux driver. Fix
a command leak in the ioctl handler. Re-arrange some of the command handlers
to localize error handling.
MFC After: 3 days
Lock the IPS driver and bring it out from under Giant. Also do some2005-01-28T05:02:13ZScott Longscottl@FreeBSD.org2005-01-28T05:02:13Zurn:sha1:03a908f2ce4394b8fd8ec8e4734938c002989d0f
significant clean up and optimizations:
- don't call bioq_disksort() on every command, the hardware will do that for
us.
- remove all of the complicated bio deferral code. bio's that can't be
serviced immediately can just wait on the bioq.
- Only reserve one command object for doing control commands to the card.
This simplifies a lot of code and significantly reduces the size of the
command struct.
- Allocate commands out of a slab instead of embedding them into the softc.
- Call the command action method directly instead of having ips_get_free_cmd()
call it indirectly.
MFC After: 1 week
Change the return value of sema_timedwait() so it returns 0 on2004-06-14T18:19:05ZJohn Polstrajdp@FreeBSD.org2004-06-14T18:19:05Zurn:sha1:4717d22a7c97938d0e99d91f8dd1d55eeaae0fcd
success and a proper errno value on failure. This makes it
consistent with cv_timedwait(), and paves the way for the
introduction of functions such as sema_timedwait_sig() which can
fail in multiple ways.
Bump __FreeBSD_version and add a note to UPDATING.
Approved by: scottl (ips driver), arch
Use a unique malloc type rather than M_DEVBUF.2004-05-30T04:01:29ZScott Longscottl@FreeBSD.org2004-05-30T04:01:29Zurn:sha1:dd83a01e81b7f4729a40700faf8ae29b254878b4Switch from using mutexes to using semaphores to protect against early2004-02-28T19:14:41ZScott Longscottl@FreeBSD.org2004-02-28T19:14:41Zurn:sha1:b234a120c867ff3291e66666644af78f7a894095
completion of synchronous commands. Also switch to a per-array bioq as it
appears to improve performance.
Submitted by: mbr, imp.ch (bioq change)
The reset_type should not be 0x80, it should be set to zero.2004-01-01T10:22:10ZMartin Blappmbr@FreeBSD.org2004-01-01T10:22:10Zurn:sha1:aa8689ef95a6b48e9f309ccbedea714d2752b274
0x80 can cause the command to be rejected as invalid. This bug
exists also in the Linux IPS ffdc code.
Submitted by: David Jeffery
MFC after: 1 week