| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
member of 'struct scsi_xfer' has always been pointed at the 'struct scsi_generic
cmdstore' member of the same instance. So nuke 'cmdstore' and remove the '*'
from cmd. Take the address of cmd as required by the various casts.
No intentional functional change.
luna88k test by aoyama@, sparc64 test by jmatthew@
Identification of 2009's last *cmd use and ok jmatthew@
|
|
|
|
|
|
|
| |
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).
|
|
|
|
|
|
|
|
| |
In sparc64 autoconf access 'luns' via sb_luns and 'adapter_buswidth'
via sb_adapter_buswidth.
Removes last post-config uses of the copies of bus related
information in scsi_link.
|
|
|
|
| |
statement. All to make upcoming diff(s) smaller and easier to read.
|
|
|
|
| |
immediately before initializing the only field in the struct.
|
|
|
|
|
|
| |
While here remove some whitespace.
OK millert@
|
|
|
|
|
|
| |
indicate the adapter is not a target on the bus.
ok dlg@ as part of a larger diff.
|
|
|
|
|
|
| |
'minphys()'. Just use & check for NULL instead, since 'minphys()' is
always called on the code path ([cd|sd|st]minphys) that calls
physio().
|
|
|
|
|
|
|
|
|
|
|
| |
in drivers. Terse one liners, NULLs instead of 0's, explicitly specify
all members, etc.
Nuke #ifdef notyet blocks related to the scsi_adapter in aic.
No intentional functional change.
ok tedu@
|
|
|
|
| |
ok mpi@ bluhm@
|
|
|
|
|
|
| |
that will end up in config_detach() flags via scsi_detach_target().
ok jmatthew@ dlg@
|
|
|
|
|
|
| |
shrinks the code a bit on sparc64.
ok jmatthew@
|
|
|
|
| |
OK mpi@ deraadt@
|
|
|
|
|
|
| |
shaves ~256 bytes off of qla_put_data_seg on sparc64
ok kettenis@ jmatthew@
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
have any direct symbols used. Tested for indirect use by compiling
amd64/i386/sparc64 kernels.
ok tedu@ deraadt@
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
interrupt context to a taskq running in a thread. however, there
is a concern that if we do that then we allow accidental use of
sleeping APIs in this work, which will make it harder to move the
work back to interrupts in the future.
guenther and kettenis came up with the idea of marking a proc with
CANTSLEEP which the sleep paths can check and panic on.
this builds on that so you create taskqs that run with CANTSLEEP
set except when they need to sleep for more tasks to run.
the taskq_create api is changed to take a flags argument so users
can specify CANTSLEEP. MPSAFE is also passed via this flags field
now. this means archs that defined IPL_MPSAFE to 0 can now create
mpsafe taskqs too.
lots of discussion at s2k15
ok guenther@ miod@ mpi@ tedu@ pelikan@
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
when workqs were introduced, we provided a second argument so you
could pass a thing and some context to work on it in. there were
very few things that took advantage of the second argument, so when
i introduced pools i suggested removing it. since tasks were meant
to replace workqs, it was requested that we keep the second argument
to make porting from workqs to tasks easier.
now that workqs are gone, i had a look at the use of the second
argument again and found only one good use of it (vdsp(4) on sparc64
if you're interested) and a tiny handful of questionable uses. the
vast majority of tasks only used a single argument. i have since
modified all tasks that used two args to only use one, so now we
can remove the second argument.
so this is a mechanical change. all tasks only passed NULL as their
second argument, so we can just remove it.
ok krw@
|
| |
|
|
|
|
| |
ok mpi@ kspillner@
|
|
|
|
|
|
|
|
| |
CIRCLEQ_* is deprecated and not called in the tree. The other queue types
have *_END macros which were added for symmetry with CIRCLEQ_END. They are
defined as NULL. There's no reason to keep the other *_END macro calls.
ok millert@
|
|
|
|
| |
ok tedu
|
|
|
|
| |
after discussions with beck deraadt kettenis.
|
|
|
|
| |
it in one place.
|
|
|
|
|
|
|
|
|
| |
there to our port lists, then find the next available loop id and carry on
rather than restarting. This way, we're less likely to get stuck looping
when the firmware behaves inconsistently. We also don't need to treat domain
controller logins specially.
additional complications discovered by deraadt@
|
| |
|
|
|
|
| |
and set location on ports as best we can. Take the port mutex too.
|
|
|
|
| |
post-attach mailbox operations work reliably.
|
|
|
|
| |
Fix some parameters and wording too.
|
|
|
|
|
|
| |
matter if we don't see ourselves, and if we pick another starting point, the
first port we see might disappear half way through (I have seen this happen),
which makes it more complicated to identify when we're back at the start.
|
|
|
|
| |
add that, don't try loading firmware for them.
|
|
|
|
|
|
| |
firmware (2048 for 2k login firmware, 256 otherwise)
pointed out by deraadt@, ok dlg@
|
|
|
|
|
|
|
|
| |
interrupt. With this we can talk to Hitachi disk arrays, which sometimes
return so much sense data that a status continuation response is generated
in addition to the normal status response.
tested by deraadt@
|
|
|
|
|
| |
local devices it has logged in to. This will also be useful for hotplug
later on.
|
|
|
|
|
| |
before trying to boot it, so we can explain why it's not working rather
than printing cryptic errors.
|
|
|
|
| |
ok jmatthew
|
|
|
|
|
|
| |
attributes.
ok dlg@, jmatthew@
|
|
|
|
|
|
| |
hardware.
ok dlg@, jmatthew@
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
in a per generation structure. this avoids code on every isr to figure
out what version of the chip we are, which is silly since we know that
at attach time.
ok jmatthew@
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
bit is to post to the resp_out register once outside the loop rather
than inside every loop.
rework the code for finding the queue registers so its done once
on attach by pointing the sc at a map, rather than running big
conditionals in code on every register operation based on teh chip
version.
tested on 2200s by me and 2312s by jmatthew@
ok jmatthew@
|
|
|
|
|
| |
which means we don't need to use sgl continuation command buffers.
this keeps things much simpler on the iopool side of things.
|
|
|
|
| |
discovery stuff enabled for now.
|
|
|
|
| |
the fc4 types for fabric ports we find
|
|
|
|
|
|
|
| |
one copy when both isp(4) and qla(4) are enabled. This is a temporary
measure until qla(4) takes over completely.
looked at by miod@ and kettenis@
|