Age | Commit message (Collapse) | Author |
|
low-hanging splbio/splx pairs that are no longer needed and see if
this reveals any hidden scsi flaws.
ok dlg@
|
|
gets rid of #include <sys/dkio.h> in sys/ioctl.h and adds #include
<sys/dkio.h> to the places that actually want and use the disk
ioctls.
this became an issue when krw@'s X build failed when he was testing
a change to dkio.h.
tested by krw@
help from and ok miod@
|
|
|
|
identifier, allowing the disk to be identified without relying on the
device name.
ok deraadt@ krw@ beck@ marco@ todd@
|
|
|
|
requested, just use a supported sample rate. makes paramter
negotiation much easier. problem noted and patch tested on
several sb(4) devices by J.C Roberts, thanks.
|
|
|
|
Driver written by Mikko Tolmunen <oskari@sefirosu.org>
|
|
for a long time (if ever?), so send it to the Attic.
"with prejudice" miod@; GENERIC reminder from kettenis@
|
|
void. Use XS_NO_CCB error in the scsi command (xs) to report the
NO_CCB condition. Eliminates all SUCCESSFULLY_QUEUED and COMPLETE
confusion and untangles the midlayer from the adapter a bit more.
Eyes and some fixes by miod@
There may be some compile issues on little used (i.e. I don't have
any) drivers but the change is mechanical and thus easy to remedy.
ok dlg@
|
|
is what the function pointer type uses.
ok grange
|
|
seagate/trm/aha1742 were the only drivers paranoid enough to check
ITSDONE on getting the xs to execute. And optimistic enough to think
simply restting the flag would be a good thing. Have them chill out
like everyone else, reducing ITSDONE noise some more.
|
|
in the drivers just before calling scsi_done().
ok dlg@ beck@
|
|
From Simon Perreault <simon dot perreault at viagenie dot ca> via tech@
|
|
ok md5@
|
|
TRY_AGAIN_LATER uses with equivalent NO_CCB. Eliminates confusion
between the two as was always intended. buf I/O's that can't be
started get pushed back onto the front of the queue and retried.
Others get sent back to originator as failures. No more epi-cycle
looping inside the SCSI midlayer hoping the problem goes away.
Various testers, no objection from miod@ as vs(4) was tested by
nick@.
|
|
Pointed out and ok by miod@.
|
|
can cause beeps on console. Reported by Greg Jones on misc@
|
|
since the scsi_done() wrapper will timeout_del() unconditionnaly and the
scsi midlayer no longer provides scsi_xfer with zeroed struct timeout.
ok dlg@
|
|
behaviour.
|
|
label. Use the existing label, since we now check and reuse some
values in it. And that's what all other drivers do. Bonus: we don't
need that label on the stack anymore.
ok deraadt@
|
|
return zero on success and nonzero on failure.
This commit only performs mechanical changes for the existing emulops to
always return zero.
|
|
|
|
into panic()'s.
ok deraadt@ jsing@
|
|
ok otto@, agreed by deraadt@
|
|
|
|
printf-like arguments,
into panic() and remove call to Debugger()
hint and ok deraadt@
|
|
|
|
errnos. Note that the error strings are being ignored, since we long ago
decided to not spam the console, and there is no other nice way to use the
errors (without changing the ioctls to pass it back)
The errno is now useful, since we can pass b_error from failing IO up, and
the drive can decide how to use that
ok miod
|
|
second vc is created. However, it was allocated using the geometry of
the second vc to allocate backing store for the first. Be sure to use
the proper values in case geometries differ.
|
|
to stop is not called because the ioctl function is a masterpiece)
|
|
|
|
ok krw@
|
|
"do it" marco@
|
|
ok jakemsr@
|
|
bus_space_unmap(), not bus_space_free(). Fortunately this file is only
used on i386 where bus_space_free() simply invokes bus_space_unmap().
|
|
|
|
provided by Parfait
ok oga
|
|
in an array. Problem found and ok deraadt@.
|
|
unsigned, like all the other callers of aps_do_io in aps do.
|
|
|
|
|
|
|
|
calculation of ticks. Saves 26 bytes.
ok oga `fine' deraadt
|
|
schsio_attach() then later reads the chip id to print which device we
found.
I noticed that isa_attach_args has a field called ia_aux which can be
used for driver specific stuff, so stuff the device id in there so we
don't need to get the value from the chip again during attach.
|
|
bus_space_.. API uses that.
|
|
|
|
For the possibility of sleeping, the first two flags are UVM_PLA_WAITOK
and UVM_PLA_NOWAIT. It is an error not to show intention, so assert that
one of the two is provided. Switch over every caller in the tree to
using the appropriate flag.
ok art@, ariane@
|
|
|
|
between instances, saving space in the kernel. feedback from many (some
incorporated, some left for future work).
ok deraadt, kettenis, "why not" miod.
|