summaryrefslogtreecommitdiffstats
path: root/sys/cam
Commit message (Collapse)AuthorAgeFilesLines
* Change the disk(9) API in order to make device removal more robust.phk2004-02-182-44/+50
| | | | | | | | | | | | | | | | | | | | | | | | Previously the "struct disk" were owned by the device driver and this gave us problems when the device disappared and the users of that device were not immediately disappearing. Now the struct disk is allocate with a new call, disk_alloc() and owned by geom_disk and just abandonned by the device driver when disk_create() is called. Unfortunately, this results in a ton of "s/\./->/" changes to device drivers. Since I'm doing the sweep anyway, a couple of other API improvements have been carried out at the same time: The Giant awareness flag has been flipped from DISKFLAG_NOGIANT to DISKFLAG_NEEDSGIANT A version number have been added to disk_create() so that we can detect, report and ignore binary drivers with old ABI in the future. Manual page update to follow shortly.
* Fix type in comment, 's/writtent/written/'johan2004-01-181-1/+1
| | | | | PR: 56954 Submitted by: Dan Langille <dan@langille.org>
* Move the ciss quirk to the right section, also update the commentps2003-12-081-8/+7
| | | | | stating that the controllers do not support SYNC CACHE since ciss only supports a small subset of the scsi spec.
* ciss doesn't like scsi SYNC CACHE. turn it offps2003-12-081-0/+8
|
* SS_FATAL|ENXIO rather than SS_RDEF for illegal track mode.obrien2003-12-011-1/+1
| | | | | | | | This reduces the 90+ lines boot output of spewage GEOM does for my Plextor SCSI burner. Submitted by: scottl Approved by: scottl
* - Implement selwakeuppri() which allows raising the priority of atanimura2003-11-091-1/+1
| | | | | | | | | | | | | thread being waken up. The thread waken up can run at a priority as high as after tsleep(). - Replace selwakeup()s with selwakeuppri()s and pass appropriate priorities. - Add cv_broadcastpri() which raises the priority of the broadcast threads. Used by selwakeuppri() if collision occurs. Not objected in: -arch, -current
* Sprinkle GIANT_REQUIRED asserts around the xpt layer to aid with locking thescottl2003-11-091-0/+58
| | | | SCSI drivers.
* Don't be so chatty when performing manual sense. This should make ATAPICAMscottl2003-11-081-2/+2
| | | | devices a lot more quiet.
* In camperiphdone(), make sure we check for fatal errors and bail outken2003-10-272-16/+17
| | | | | | | | | | | | | | | | | | | | | | instead of retrying them blindly. This should fix some of the problems people have been having with cdrom drives taking a long time to probe. This should also eliminate the need for the initial TUR in cdsize(). cam_periph.c: Don't keep retrying if the error we get back is a fatal error. This should help us detect the transition from "Logical unit not ready, cause not reportable" to "Medium not present" in the "TUR many" handler. (The TUR many handler gets triggered for Logical unit not ready, cause not reportable errors.) scsi_cd.c: Remove the initial test unit ready in cdsize(). Hopefully it isn't necessary after the above change. Submitted by: gibbs (mostly) Tested by: peter MFC After: 2 weeks
* Use bio_offset instead of bio_blknophk2003-10-181-2/+2
|
* There is no need to muck about with the B_PHYS flag here. We never evenphk2003-10-181-8/+0
| | | | get close to DEV_STRATEGY() which is the only place it is relevant.
* (dacleanup, dasysctlinit): Defend against calling sysctl_ctx_free onthomas2003-10-081-2/+5
| | | | | | | | an uninitialized sysctl_ctx, using flag DA_FLAG_SCTX_INIT. This prevents a panic encoutered with some umass units that probe correctly but fail to attach. Same problem, and same fix, as scsi_cd.c rev. 1.86. Reviewed by: njl, ken
* (cdcleanup): Defend against calling sysctl_ctx_free before the sysctl_ctxthomas2003-10-071-12/+15
| | | | | | | | | | | has been initialized. (cdsysctlinit): Set flag CD_FLAG_SCTX_INIT after sysctl_ctx has been initialized. This resolves a panic encountered when a cd drive is sucessfully probed but fails to attach. Reviewed by: ken
* Add disk_destroy() call to detach processing.phk2003-10-061-0/+15
| | | | Add short tempered TUR to cdsize() as a workaround.
* Be kind to 64bit architectures.simokawa2003-10-021-5/+5
|
* - Pick up a correct path_id for the bus on AC_PATH_REGISTERED.simokawa2003-09-301-6/+12
| | | | - Call cam_periph_invalidate() only if the periph is found.
* GEOMify.phk2003-09-301-78/+39
|
* Detach black hole device on AC_PATH_DEREGISTERED.simokawa2003-09-251-19/+20
|
* (scsi_request_sense): Set allocation length in REQUEST_SENSE CCB.thomas2003-09-211-0/+1
| | | | Reviewed by: ken
* (camperiphdone): When the cam_periph layer performs sense recovery,thomas2003-09-211-1/+1
| | | | | | | | completion of recovery is indicated by positioning the CAM_AUTOSNS_VALID bit in the status field of the CCB, not in the flags field. This fixes an endless loop of sense recovery actions. Reviewed by: ken
* I forgot whom I got this from- only set single initiator buffered modemjacob2003-09-131-2/+4
| | | | if we've recorded in our softc that we should set it.
* Disable the use of cloning use in floppy and CD drivers.phk2003-09-111-4/+4
| | | | | | | | | | This commit puts the relevant code snippets under #ifdef GONE_IN_5 (rather than #ifndef BURN_BRIDGES) thereby disabling the code now. The code wil be entirely removed before 5.2 unless we find reasons why this would be a bad idea. Approach suggested by: imp
* In case vmapbuf() fails, release all of the held resources.alc2003-09-101-5/+3
| | | | Submitted by: tegge
* Put the device cloning functions for disk-drivers under #ifndef BURN_BRIDGES.phk2003-09-051-0/+8
| | | | | | | | | | | For the floppy driver, use fdcontrol to manipulate density selection. For the CD drivers, the 'a' and 'c' suffix is without actual effect and any applications insisting on it can be satisfied with a symlink: ln -s /dev/cd0 /dev/cd0a Ongoing discussion may result in these pieces of code being removed before the 5-stable branch as opposed to after.
* Calling KNOTE with locks held may result in recursion when it calls backnjl2003-09-041-2/+13
| | | | | | | | into targreadfilt(). Unlock around calls to notify_user(). If an application is sending CCBs while the endpoint is shutting down, this may result in incomplete disable. A more complete solution will come with a "dying" flag. Submitted by: simokawa
* Upon receiving a CCB for a LUN that is not enabled, be sure to unlock thenjl2003-09-041-0/+1
| | | | | | softc on exit. Submitted by: simokawa
* Remove the quirk for the FujiFilm camera. Submitter indicates it is nownjl2003-09-041-8/+0
| | | | | | | | | | | working without the quirk. PR: Submitted by: guido Reviewed by: Approved by: Obtained from: MFC after: 30 days
* Unbreak buildworld. sys/taskqueue.h is a kernel-only include.ken2003-09-031-1/+1
| | | | Pointy Hat to: ken
* Move dynamic sysctl(8) variable creation for the cd(4) and da(4) driversken2003-09-032-43/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | out of cdregister() and daregister(), which are run from interrupt context. The sysctl code does blocking mallocs (M_WAITOK), which causes problems if malloc(9) actually needs to sleep. The eventual fix for this issue will involve moving the CAM probe process inside a kernel thread. For now, though, I have fixed the issue by moving dynamic sysctl variable creation for these two drivers to a task queue running in a kernel thread. The existing task queues (taskqueue_swi and taskqueue_swi_giant) run in software interrupt handlers, which wouldn't fix the problem at hand. So I have created a new task queue, taskqueue_thread, that runs inside a kernel thread. (It also runs outside of Giant -- clients must explicitly acquire and release Giant in their taskqueue functions.) scsi_cd.c: Remove sysctl variable creation code from cdregister(), and move it to a new function, cdsysctlinit(). Queue cdsysctlinit() to the taskqueue_thread taskqueue once we have fully registered the cd(4) driver instance. scsi_da.c: Remove sysctl variable creation code from daregister(), and move it to move it to a new function, dasysctlinit(). Queue dasysctlinit() to the taskqueue_thread taskqueue once we have fully registered the da(4) instance. taskqueue.h: Declare the new taskqueue_thread taskqueue, update some comments. subr_taskqueue.c: Create the new kernel thread taskqueue. This taskqueue runs outside of Giant, so any functions queued to it would need to explicitly acquire/release Giant if they need it. cd.4: Update the cd(4) man page to talk about the minimum command size sysctl/loader tunable. Also note that the changer variables are available as loader tunables as well. da.4: Update the da(4) man page to cover the retry_count, default_timeout and minimum_cmd_size sysctl variables/loader tunables. Remove references to /dev/r???, they aren't used any longer. cd.9: Update the cd(9) man page to describe the CD_Q_10_BYTE_ONLY quirk. taskqueue.9: Update the taskqueue(9) man page to describe the new thread task queue, and the taskqueue_swi_giant queue. MFC after: 3 days
* Remove quirk for Apacer Handydrive. Kevin Oberman <oberman@es.net> reportsnjl2003-08-251-8/+0
| | | | | | | that it works without the quirk. This and any other quirk changes will be MFCd after the release unless they fix a known problem. MFC after: 1 month
* Sort quirks into sections.njl2003-08-251-19/+20
|
* Add the DA_Q_NO_PREVENT quirk which keeps da(4) from sending PREVENT/ALLOWnjl2003-08-221-6/+15
| | | | | | | | | commands. Add a quirk for the Creative Nomad MuVo USB device that uses it as well as NO_SYNCHRONIZE_CACHE. PR: kern/53094 Submitted by: Richard Nyberg <rnyberg@it.su.se> MFC after: 3 days
* Quirk for Jungsoft NEXDISK USB flash key. Fails to mount withoutnjl2003-08-221-0/+8
| | | | | | | | NO_SYNCHRONIZE_CACHE. PR: kern/54737 Submitted by: David Thiel <lx@redundancy.redundancy.org> MFC after: 3 days
* Add quirks for the EXATELECOM i-Bead mp3 player.njl2003-08-221-0/+8
| | | | | | PR: kern/51675 Submitted by: Nicolas Jombart <ecu@ipv42.net> MFC after: 3 days
* Remove all USB quirks that only specify DA_Q_NO_6_BYTE. They are no longernjl2003-08-061-158/+0
| | | | | | | needed. Other USB quirks remain in hopes that user testing will reveal which ones are also no longer needed. Courtesy of: USENIX lunch break
* Remove unused cmd6workaround() calls.njl2003-08-061-15/+2
| | | | MFC after: 1 day
* Fix the new DA_OLD_QUIRKS option for normal and module compiles.njl2003-07-291-0/+1
| | | | Pointed out by: bde
* Deprecate USB and Firewire quirks. We should now never send 6 byte commandsnjl2003-07-291-18/+23
| | | | | | | to such devices. If a device fails due to this commit, add: options DA_OLD_QUIRKS to the kernel config and recompile. Then send the output of "camcontrol inquiry da0" to scsi@freebsd.org so the quirk can be re-enabled.
* Add a PATH_INQ flag, PIM_NO_6_BYTE, which indicates the SIM never wishes tonjl2003-07-283-1/+18
| | | | | | | | | | receive 6 byte commands. Add a check for this flag to da(4) and cd(4) so that they honor it. This is a quick workaround for many devices (especially USB) that require da(4) quirks to operate. The more complete approach is to finish the new transport code which will be aware of the SCSI version a transport implements. MFC after: 1 day
* Reference PR for IntelligentStick quirk entry.thomas2003-07-181-1/+1
| | | | PR: kern/53005
* Add quirk entry for IntelligentStick disc-on-key USB devices.thomas2003-07-171-0/+8
| | | | | | | Reported by Samuel Tardieu <sam@rfc1149.net>. Reviewed by: roberto MFC after: 1 week
* Merge common XPT_CALC_GEOMETRY functions into a single convenience function.njl2003-06-141-20/+1
| | | | | | | | | | | | | Devices below may experience a change in geometry. * Due to a bug, aic(4) never used extended geometry. Changes all drives >1G to now use extended translation. * sbp(4) drives exactly 1 GB in size now no longer use extended geometry. * umass(4) drives exactly 1 GB in size now no longer use extended geometry. For all other controllers in this commit, this should be a no-op. Looked over by: scottl
* Merge common XPT_CALC_GEOMETRY functions into a single convenience function.njl2003-06-142-0/+27
| | | | | | | | | | | | | | | | | | Devices below may experience a change in geometry. * Due to a bug, aic(4) never used extended geometry. Changes all drives >1G to now use extended translation. * sbp(4) drives exactly 1 GB in size now no longer use extended geometry. * umass(4) drives exactly 1 GB in size now no longer use extended geometry. For all other controllers in this commit, this should be a no-op. PR: Submitted by: Looked over by: scottl Approved by: Obtained from: MFC after:
* Use __FBSDID().obrien2003-06-1012-24/+40
|
* Use __FBSDID().obrien2003-06-105-11/+18
|
* Check in some months long pending minor debug output changes.mjacob2003-06-021-26/+47
| | | | | | Clarify that the implicit fallthrough was *not* intentional (thanks, Poul!) and reorganize the code so a correct fallthrough (with /* FALLTHROUGH */) occurs.
* Remove unused variables.phk2003-06-011-11/+1
| | | | | | | Remove break after return. Add XXX comment where intent is unclear. Found by: FlexeLint
* Add /* FALLTHROUGH */phk2003-06-011-0/+4
| | | | Found by: FlexeLint
* Add /* FALLTHROUGH */phk2003-05-313-9/+7
| | | | | | | Move /* FALLTHROUGH */ to correct location. Remove unused variable(s). Found by: FlexeLint
* Add /* FALLTHROUGH */phk2003-05-311-7/+4
| | | | | | | Remove unused variable(s). Order switch in canonical order. Found by: FlexeLint
OpenPOWER on IntegriCloud