Age | Commit message (Collapse) | Author |
|
when the request is unsupported.
So check mode sense(6) and mode sense(10) results for valid headers before
passing the results back.
Avoid overwriting any error that was returned.
ok jmatthew@
|
|
to replace various uses of '5' when calculating the amount of data in the
INQUIRY response. Matches up more naturally with SID_SCSI2_ALEN.
Also use to fix SCSIDEBUG display of INQUIRY responses to show correct count of
bytes received/available.
|
|
magic numbers. As the nearby comment says '< 2 is obsolete. >2 is reserved'. So
0x2 is just right.
|
|
struct scsi_rw_10.
ok gnezdo@ jmatthew@ (who also did sparc64 compile test)
|
|
allow shrinking the names arrays.
|
|
and let cdstart() use it if the requested i/o is unable to fit into a READ(10).
Certainly better than silently truncating the i/o into a READ(10)/WRITE(10).
|
|
|
|
|
|
are expected to fit into a 4-byte field.
|
|
have the caller assign it to xs->cmdlen. Pass in xs->cmd instead of xs. A
slightly cleaner API that provides the potential of detecting and reacting to a
failure to create the desired command.
|
|
xs fields to a more convenient location, shrinking upcoming diffs. Nuke some
Captain Obvious comments.
|
|
|
|
favour of simply using the device's claimed SCSI level of support. Except of
course for ATAPI/USB devices which often don't claim anything. Keep assuming
they are at least SCSI-2. Use consistant tests in
sdminphys/cdminphys/sdstart/cdstart.
|
|
and was always checked in concert with SDEV_ATAPI. Just rely on SDEV_ATAPI and
SDEV_UMASS in all but the one place sd(4) where SDEV_ONLYBIG is set
independently of SDEV_ATAPI/_UMASS.
ok jmatthew@
|
|
pointless timeout_set(sc_timoeut) and timeout_del(sc_timeout) calls and the
sc_timeout fields themselves.
|
|
|
|
|
|
SDEV_S_DYING instead. Makes it clear that the scsi_link and the [cd|sd|st]
device always agree on their state.
|
|
the bus. Use SLIST_FOREACH_SAFE() rather than 'while (!SLIST_EMPTY())'' as there
is a condition which would cause scsi_detach_link() to return without removing
the scsi_link from the SLIST.
|
|
contents thereof.
Compile tests by martjn@ (alpha), visa@ (sgi) jmatthew@ (sparc64) aoyama@
(luna88k)
|
|
XS_NO_CCB purge of 2017.
Nuke pointless ISSET()/CLR() checks and the #define's.
|
|
make the three variants more similar and easier to understand. Ensures
consistent error checks and eliminates pointless adapter_buswidth checks when
processing the list of scsi_links.
|
|
it.
Prepares for removal of #include <scsi/scsi_debug.h> from scsiconf.h as soon as
the alpha tester digs out from hurricane to compile last untested files.
|
|
nestled together. Rename scsibusprint() to scsibussubprint() since it is used
with scsibussubmatch().
|
|
|
|
leading whitespace. Rename some local functions.
|
|
scsibus_softc pointers around.
|
|
hex value after named flags. Make flag name arrays NULL terminated rathar than
fixed size.
|
|
|
|
driver successfully compiles on one or more of amd64, i386, hppa.
|
|
fields to struct scsibus_attach_args. Nuke the struct scsi_link *
(saa_sc_link) in scaibus_attach_args.
Explicitly initialize each field in scsibus_attach_args variables.
|
|
adapter_buswidth, adapter_softc) from struct scsi_link to struct
scsibus_attach_args.
Additional compile tests by jmatthew@ (sparc64) and aoyam@ (luna88k).
|
|
adapter_softc, adapter_buswidth) via link->bus rather than using
copies currently residing in the link.
|
|
initialize the scsi_link's on the bus. After sucking this information
out of the "prototype" link provided by the scsibus_attach_arg, no
need to keep a pointer to that prototype.
|
|
(ahc(4) and qlw(4)) can just compare the values of the "bus" member
directly.
A slightly different path to the same result that matthew@ traversed
in his work culminating in scsiconf.h r1.146.
|
|
points to the inquiry data contained in the struct scsi_link pointed
to by the other member, sa_sc_link.
|
|
2010.
Part of matthew@'s reverted scsiconf.h r1.146.
|
|
2015 in favour of ql*(4).
|
|
immediately before initializing the only field in the struct.
|
|
aassigned to adapter_buswidth to indicate the adapter is not a target
on the bus.
ok dlg@ as part of a larger diff.
|
|
'adapter_target' to a value greater than or equal to
'adapater_buswidth' to allow all possible targets to be probed.
Add SDEV_NO_ADAPTER_TARGET (0xffff), a value guaranteed to
be greater than or equal to the u_int16_t 'adapter_buswidth'.
ok dlg@ as part of a larger diff.
|
|
than scsi_mode_header[_big] pointers to the same data.
Continues the umtamgling and simplification of the mode sense code.
CID 1491299
CID 1491297
|
|
rather than after the only invocation.
No functional change.
|
|
always sizeof(struct scsi_mode_sense_buf).
No functional change.
|
|
xs->timeout.
Just set xs->timeout to 20000 and drop the parameter.
No functional change.
|
|
detecton of a 0 value.
CID 1488899
|
|
throat of physio().
Allows individual devices to eventually be modified to support larger
physio() (a.k.a. 'raw') i/o's if they prove capable of them.
No immediate functional change.
Tested for many weeks by and ok robert@.
|
|
the relationships between various scsi structs.
|
|
check dev_minphys for NULL before calling it.
|
|
'minphys()'. Just use & check for NULL instead, since 'minphys()' is
always called on the code path ([cd|sd|st]minphys) that calls
physio().
|