This patch makes some needlessly global code static.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Changes to the NCR5380 driver between 2.6.9 and 2.6.10 replaced the
driver's home-grown delayed work implementation with a call to
schedule_delayed_work().  However, the delay argument was not passed
correctly, so the work was usually scheduled for _way_ too far in
the future.  This patch fixes this.

NCR5380_print_status() is called from NCR5380_abort() and from
NCR5380_bus_reset().  In at least the abort() case, the host lock
has already been acquired by scsi_error.c:scsi_try_to_abort_command().
NCR5380_print_status() calls NCR5380_proc_info() which also acquires
and releases the host lock.  This patch removes the call to
NCR5380_proc_info() from NCR5380_print_status.

Cosmetic: Remove duplicated lines of code from NCR5380_abort().
Signed-off-by: Kenn Humborg <kenn@linux.ie>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

To try and escape the madness of adding a zillion USB card readers
to the SCSI whitelist, I flipped the scan all lun's by default switch
in the fedora kernel recently to see just what breaks, in the hope
of moving from a whitelist solution to a blacklist.

Here's the first (hopefully of not too many) devices that broke.
(See https://bugzilla.redhat.com/beta/show_bug.cgi?id=149402 for more info)
Signed-off-by: Dave Jones <davej@redhat.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

From: Guennadi Liakhovetski <g.liakhovetski@gmx.de>

Removes the page_to_virt and maps sg lists dynamically.
This makes the driver work with highmem pages.
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Signed-off-by: Jamie Lenehan <lenehan@twibble.org>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

The patch at the end of this message modifies the st write semantics in
the following way: write a filemark before rewind, offline, or seek if the
previous operation was write.

This semantics is specified on the man pages of some Unices and some
software (e.g., cpio) seems to assume this. The change makes sure that the
last file on the tape is properly terminated with a filemark and reading the
file does not fail at the end.
Signed-off-by: Kai Makisara <kai.makisara@kolumbus.fi>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

The basic change is to allow target scans to be done
by generic device (which need only have a scsi host
as a parent).  Also changes most target functions to
take a struct scsi_target instead of the H/C/T numbers.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

This patch adds a NOREPORTLUN blacklist flag for a disk made by "WDC"
(Winchester?).  The drive's firmware crashes when it receives REPORT_LUNS,
even though it claims to be SCSI rev 04.
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

This is an updated and unmangled version of the patch sent in by Martin
Peschke.  Apparently some drives report Hardware Error sense for
problems which do improve after retrying, so the patch retries these
supposedly "unrecoverable" errors for such devices.

In addition to the IBM ESS drive it adds a blacklist entry for the drive
inside the MPIO HS200 Gigabox.
Signed-off-by: Martin Peschke <mpeschke@de.ibm.com>
Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

The patch at the end of this message converts the SCSI tape driver to support
also descriptor based sense data. Test for deferred sense data have been added
in a couple of places and the EOM tests have been unified. Some tests have been
simplified but the patch is not meant to change the current behavior. The
patch is against 2.6.11-rc4 and has been tested to some extent.

The patch also includes the msleep_interruptible change from from kernel
janitors.

Thanks to Doug Gilbert for doing a first version of this sense data
conversion.
Signed-off-by: Kai Makisara <kai.makisara@kolumbus.fi>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

It's lacking EH support and the hardware is supported by the qla1280
driver now.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

it oopses when trying to use it with my SmartRaid IV card, it's a mess
and the only hardware supported by it but not the eata driver are two
10-year old ISA boards.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

reading the /proc/scsi/qla1280/* files can easily corrupt kernel memory.

As the feature is deprecated, and the qla1280 implementation doesn't
return very usefull implementation but is so buggy that any serious
user would have noticed we just remove it.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

The static device_request_lock doesn't protect anything; remove it.
Signed-off-by: Matthew Wilcox <matthew@wil.cx>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

This patch fixes one of Christroph's fixme comments in the SCSI midlayer.

The selection of the serial number and pid for commands was done
by a global variable and not SMP safe. The race was not very
serious because it was only used for error handling, but it's
still better to fix it.

I audited all the drivers and none seemed to use them
for anything interesting, so I just made it a per host counter
protected by host lock.

(in fact they could be probably removed because they only see
to be used as a flag in exception handling and for debugging.
The patch would be unfortunately quite big because a lot
of driver printks use them)

This should be slight faster on SMP too because the cache line of
the counter won't bounce around the machine.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

This patch makes some needlessly global code static.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

From: 	James.Smart@Emulex.Com

Adds io statistics (requests, completions, error count) as generic
attributes for scsi devices.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

I think the hardware_version, firmware_version, rom_version and
driver_version don't belong to the FC transport class, there's
nothing specific to FC or even SCSI specific in them.
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

From: 	Moore, Eric Dean <Eric.Moore@lsil.com>

1) mptscsih.c: I have changed task management 
requests so they complete in same thread before returning
to os.  I removed the TMtimer code.  
 
(2) mptctl.c: I have deleted both TMtimer and timer code. Replaced
with wait_event_interruptible_timeout.  So commands are now
completed in same thread.  
 
(3) mptctl_do_fw_download - nasty polling of global parameters
that are set in contents of interrupt handler(mptctl_reply), now using
wait_event_interruptible_timeout. 
 
(4) I have cleaned up mptctl_reply.
Signed-off-by: Eric Moore <Eric.Moore@lsil.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Convert mptctl to compat_ioctl. I also changed it to unlocked_ioctl
while I was on it.
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Convert megaraid2 driver to new compat_ioctl entry points.
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Add compat_ioctl to SG driver
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Convert aacraid driver to the compat_ioctl entry points.

I don't have hardware, so this is only compile tested, but I just
did some transformations on the existing code.
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Add compat_ioctl to osst
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Add comment that SR doesn't support compat_ioctl (because it doesn't
pass down any driver ioctls right now)
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Call new compat_ioctl host vector from tape driver
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

Add compat_ioctl entry point to SD
Signed-off-by: Andi Kleen <ak@muc.de>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>

into ppc970.osdl.org:/home/torvalds/v2.6/linux

into ppc970.osdl.org:/home/torvalds/v2.6/linux

into ppc970.osdl.org:/home/torvalds/v2.6/linux

Not that low-level code really tends to care, but since loff_t is
a signed 64-bit entity and size_t is unsigned (and potentially
64-bit), mixing the two isn't very well-defined.. 

The actual user copy will do them too, but only for the
range that ends up being actually copied. That hides
bugs when the range has been clamped by file size or other
issues.

into ppc970.osdl.org:/home/torvalds/v2.6/linux

With Olaf Hering <olh@suse.de>.

Parameters were interchanged.
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

From: Prarit Bhargava <prarit@sgi.com>

I discovered an issue where a hwif_init() failure lead to /proc/ide files
being created for devices that failed probes.  This resulted in
oops/WARN_ON/BUG_ON executions through the kernel depending on what
actions were on going.

slightly changed by me (bart)
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

ide_hwgroup_t.polling field added.  0 in poll_timeout field
used to indicate inactive polling but because 0 is a valid
jiffy value, though slim, there's a chance that something
weird can happen.
Signed-off-by: Tejun Heo <tj@home-tj.org>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>

ide_drive_t.sleeping field added.  0 in sleep field used to
indicate inactive sleeping but because 0 is a valid jiffy
value, though slim, there's a chance that something can go
weird.  And while at it, explicit jiffy comparisons are
converted to use time_before() macros.
Signed-off-by: Tejun Heo <tj@home-tj.org>
Signed-off-by: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>