An error occurred fetching the project authors.
  1. 21 Mar, 2007 2 commits
  2. 20 Mar, 2007 3 commits
    • Mark Haverkamp's avatar
      [SCSI] aacraid: Improved error handling · 03d44337
      Mark Haverkamp authored
      Received from Mark Salyzyn,
      
      This set of fixes improve error handling stability of the driver. A popular
      manifestation of the problems is an NULL pointer reference in the interrupt
      handler when referencing portions of the scsi command context, or in the
      scsi_done handling when an offlined device is referenced.
      
      The aacraid driver currently does not get notification of orphaned command
      completions due to devices going offline. The driver also fails to handle the
      commands that are finished by the error handler, and thus can complete again
      later at the hands of the adapter causing situations of completion of an
      invalid scsi command context. Test Unit Ready calls abort assuming that the
      abort was successful, but are not, and thus when the interrupt from the adapter
      occurs, they reference invalid command contexts. We add in a TIMED_OUT flag to
      inform the aacraid FIB context that the interrupt service should merely release
      the driver resources and not complete the command up. We take advantage of this
      with the abort handler as well for select abortable commands. And we detect and
      react if a command that can not be aborted is currently still outstanding to
      the controller when reissued by the retry mechanism.
      Signed-off-by: default avatarMark Haverkamp <markh@linux-foundation.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      03d44337
    • Mark Haverkamp's avatar
      [SCSI] aacraid: remove un-needed references to container id (cid) · 9e7c349c
      Mark Haverkamp authored
      Received from Mark Salyzyn,
      
      This little patch removes the ',cid)' container identification argument
      from some of the functions. The argument is used in some cases as merely
      a debug helper and thus not used, and in others, the value can be
      quickly acquired from the scsi command in their single solitary use in
      the procedure rather than wasting resources on passing the argument in
      from above.
      Signed-off-by: default avatarMark Haverkamp <markh@linux-foundation.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      9e7c349c
    • Mark Haverkamp's avatar
      [SCSI] aacraid: Fix blocking issue with container probing function (cast update) · fe76df42
      Mark Haverkamp authored
      Received from Mark Salyzyn,
      
      The aac_probe_container call blocks. This is an issue because it is called on
      occasion in the context of the queuecommand handler. Once in a blue moon this
      has resulted in a kernel panic sleeping during interrupt; or problems with some
      embedded system versions of the kernel that depend on queuecommand to not
      block. This ugly patch rewrites the aac_probe_container call into a new routine
      _aac_probe_container that is an asynchronous state machine to complete the
      series of operations. The legacy blocking aac_probe_container call used in
      other areas of the driver (during initialization scanning for all targets and
      in the separate hot-add/remove [aacraid] thread) merely issues
      _aac_probe_container and then simple spins calling schedule() waiting for
      completion.
      Signed-off-by: default avatarMark Haverkamp <markh@linux-foundation.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      fe76df42
  3. 14 Feb, 2007 1 commit
    • Tim Schmielau's avatar
      [PATCH] remove many unneeded #includes of sched.h · cd354f1a
      Tim Schmielau authored
      After Al Viro (finally) succeeded in removing the sched.h #include in module.h
      recently, it makes sense again to remove other superfluous sched.h includes.
      There are quite a lot of files which include it but don't actually need
      anything defined in there.  Presumably these includes were once needed for
      macros that used to live in sched.h, but moved to other header files in the
      course of cleaning it up.
      
      To ease the pain, this time I did not fiddle with any header files and only
      removed #includes from .c-files, which tend to cause less trouble.
      
      Compile tested against 2.6.20-rc2 and 2.6.20-rc2-mm2 (with offsets) on alpha,
      arm, i386, ia64, mips, powerpc, and x86_64 with allnoconfig, defconfig,
      allmodconfig, and allyesconfig as well as a few randconfigs on x86_64 and all
      configs in arch/arm/configs on arm.  I also checked that no new warnings were
      introduced by the patch (actually, some warnings are removed that were emitted
      by unnecessarily included header files).
      Signed-off-by: default avatarTim Schmielau <tim@physik3.uni-rostock.de>
      Acked-by: default avatarRussell King <rmk+kernel@arm.linux.org.uk>
      Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
      cd354f1a
  4. 27 Jan, 2007 2 commits
    • Mark Haverkamp's avatar
      [SCSI] aacraid: expanded expose physical device code (new) · e37ee4be
      Mark Haverkamp authored
      Received from Mark Salyzyn,
      
      Take the expose_physicals flag and allow the user to select default (physicals
      available via /dev/sg), exposed (physicals available via /dev/sd for
      experimental reasons) and hidden (physicals blocked from all access). This
      expands the functionality of the previous expose_physicals insmod parameter
      which was added to support some experimental configurations.
      
      Signed-off-by Mark Haverkamp <markh@linux-foundation.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      e37ee4be
    • Mark Haverkamp's avatar
      [SCSI] aacraid: rework packet support code · e8f32de5
      Mark Haverkamp authored
      Received from Mark Salyzyn,
      
      Replace all if/else packet formations with platform function calls. This is in
      recognition of the proliferation of read and write packet types, and in the
      need to migrate to up-and-coming packets for new products.
      
      Signed-off-by Mark Haverkamp <markh@linux-foundation.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      e8f32de5
  5. 13 Dec, 2006 1 commit
    • Robert P. J. Day's avatar
      [PATCH] getting rid of all casts of k[cmz]alloc() calls · 5cbded58
      Robert P. J. Day authored
      Run this:
      
      	#!/bin/sh
      	for f in $(grep -Erl "\([^\)]*\) *k[cmz]alloc" *) ; do
      	  echo "De-casting $f..."
      	  perl -pi -e "s/ ?= ?\([^\)]*\) *(k[cmz]alloc) *\(/ = \1\(/" $f
      	done
      
      And then go through and reinstate those cases where code is casting pointers
      to non-pointers.
      
      And then drop a few hunks which conflicted with outstanding work.
      
      Cc: Russell King <rmk@arm.linux.org.uk>, Ian Molton <spyro@f2s.com>
      Cc: Mikael Starvik <starvik@axis.com>
      Cc: Yoshinori Sato <ysato@users.sourceforge.jp>
      Cc: Roman Zippel <zippel@linux-m68k.org>
      Cc: Geert Uytterhoeven <geert@linux-m68k.org>
      Cc: Ralf Baechle <ralf@linux-mips.org>
      Cc: Paul Mackerras <paulus@samba.org>
      Cc: Kyle McMartin <kyle@mcmartin.ca>
      Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
      Cc: Martin Schwidefsky <schwidefsky@de.ibm.com>
      Cc: "David S. Miller" <davem@davemloft.net>
      Cc: Jeff Dike <jdike@addtoit.com>
      Cc: Greg KH <greg@kroah.com>
      Cc: Jens Axboe <jens.axboe@oracle.com>
      Cc: Paul Fulghum <paulkf@microgate.com>
      Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>
      Cc: Karsten Keil <kkeil@suse.de>
      Cc: Mauro Carvalho Chehab <mchehab@infradead.org>
      Cc: Jeff Garzik <jeff@garzik.org>
      Cc: James Bottomley <James.Bottomley@steeleye.com>
      Cc: Ian Kent <raven@themaw.net>
      Cc: Steven French <sfrench@us.ibm.com>
      Cc: David Woodhouse <dwmw2@infradead.org>
      Cc: Neil Brown <neilb@cse.unsw.edu.au>
      Cc: Jaroslav Kysela <perex@suse.cz>
      Cc: Takashi Iwai <tiwai@suse.de>
      Signed-off-by: default avatarAndrew Morton <akpm@osdl.org>
      Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
      5cbded58
  6. 24 Sep, 2006 2 commits
    • Mark Haverkamp's avatar
      [SCSI] aacraid: expose physical devices · 653ba58d
      Mark Haverkamp authored
      Received from Mark Salyzyn:
      
      I am placing this functionality into an insmod parameter. Normally the physical
      components are exported to sg, and are blocked from showing up in sd.
      
      Note that the pass-through I/O path via the driver through the Firmware to the
      physical disks is not an optimized path, the card is designed for Hardware
      RAID, elevator sorting and caching. This should not be used as a means for
      utilizing the aacraid based controllers as a generic scsi/SATA/SAS controller,
      performance should suck by a few percentage points, any RAID meta-data on the
      drives will confuse the controller about who owns the drives and there is a
      high risk of destroying content in both directions. Unreliable and for
      experimentation or strange controlled circumstances only.
      Signed-off-by: default avatarMark Haverkamp <markh@osdl.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      653ba58d
    • Mark Haverkamp's avatar
      [SCSI] aacraid: misc cleanup · 65101355
      Mark Haverkamp authored
      Received from Mark Salyzyn:
      
      Basically cleanup, nothing here will have an affect. Adjusting some
      error codes, removing superfluous definitions and code fragments.
      Signed-off-by: default avatarMark Haverkamp <markh@osdl.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      65101355
  7. 19 Aug, 2006 2 commits
  8. 23 Jun, 2006 1 commit
  9. 10 Jun, 2006 1 commit
  10. 06 Jun, 2006 1 commit
  11. 20 May, 2006 2 commits
    • Mark Haverkamp's avatar
      [SCSI] aacraid: adjustable timeouts · 404d9a90
      Mark Haverkamp authored
      Received From Mark Salyzyn
      
      Add the ability to adjust for unusual corner case failures. Both of
      these additional module parameters deal with embedded, non-intel or
      complicated system scenarios.
      
      Aif_timeout can be increased past the default 2 minute timeout to drop
      application registrations when a system has an unusually high event load
      resulting from continuing management requests, or simultaneous builds,
      or sluggish user space as a result of system load.
      
      Startup_timeout can be increased past the default 3 minute timeout to
      drop an adapter initialization for systems that have a very large number
      of targets, or slow to spin-up targets, or a complicated set of array
      configurations that extend the time for the firmware to declare that it
      is operational. This timeout would only have an affect on non-intel
      based systems, as the (more patient) BIOS would generally be where the
      startup delay would be dealt with.
      Signed-off-by: default avatarMark Haverkamp <markh@osdl.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      404d9a90
    • Mark Haverkamp's avatar
      [SCSI] aacraid: optimize sg alloc · 3c1e0cca
      Mark Haverkamp authored
      Received From Mark Salyzyn
      
      Slight space and speed efficiency improvement.
      Signed-off-by: default avatarMark Haverkamp <markh@osdl.org>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      3c1e0cca
  12. 13 Apr, 2006 4 commits
  13. 28 Mar, 2006 1 commit
  14. 04 Feb, 2006 2 commits
  15. 09 Nov, 2005 1 commit
  16. 29 Oct, 2005 1 commit
  17. 28 Oct, 2005 3 commits
  18. 26 Sep, 2005 4 commits
  19. 05 Sep, 2005 1 commit
  20. 28 Aug, 2005 1 commit
  21. 08 Aug, 2005 1 commit
    • James Bottomley's avatar
      [SCSI] aacraid: correct use of cmd->timeout field · 5262d085
      James Bottomley authored
      The cmd->timeout field has been obsolete for a while now.  While looking
      to remove it, I came across this use in the aacraid driver.  It looks
      like you want to initialise the firmware with the current timeout of the
      command (in seconds), so the value I think you should be using is
      cmd->timeout_per_command.
      
      Acked by: Mark Haverkamp <markh@osdl.org>
      Acked by: Mark Salyzyn <mark_salyzyn@adaptec.com>
      Signed-off-by: default avatarJames Bottomley <James.Bottomley@SteelEye.com>
      5262d085
  22. 05 Aug, 2005 2 commits
  23. 26 Jun, 2005 1 commit