archive mirror
 help / color / mirror / Atom feed
From: Christoph Hellwig <>
To: Andrew Vasquez <>
Cc: Linux-Kernel <>,
	Linux-SCSI <>
Subject: Re: [ANNOUNCE] QLogic qla2xxx driver update available (v8.00.00b7).
Date: Tue, 9 Dec 2003 14:28:23 +0100	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

On Fri, Dec 05, 2003 at 05:15:14PM -0800, Andrew Vasquez wrote:
> On Friday, December 05, 2003 12:53 PM, Andrew Vasquez wrote:
> > All,
> > 
> > A new version of the 8.x series driver for Linux 2.6.x kernels has
> > been uploaded to SourceForge: 
> > 
> >
> >
> False start.  I've uploaded 8.00.00b8 to the SF site.  This driver
> instructs the mid-layer to perform its initial scan with
> scsi_scan_host().  During testing, I disable the scan.  Sorry for
> the confusion.

Second problem seems to be that the extras/ dir missed the executable

Here's some patches for you:

	use select for CONFIG_SCSI_QLA2XXX, shorten the name
	of the failover option, sanitize kconfig texts.

	make ioctl code optional.  this is a bit ugly because
	sometimes we use defines from exioct.h in non-ioctl
	related code.  Btw, is it just me or is the ioctl code
	never called actually?

	use a common and simpler variant of the down with timeout
	scheme.  We should probably try to get a proper down_timeout
	as this one still is quite suboptimal..

	use new-style module_param() everywhere (works also for
	the compiled-in case).  The paramter naming is quite
	inconsistant and you have global variables like Bind
	that might cause quite some trouble in the non-modular
	case.  You should probably switch to module_param_named
	to always use nice lower_case, descriptive names
	(without the ql2x prefix) and map them to variables with
	a driver-prefix.

	Move some debug defines from qla_def.h to qla_dbg.h

	Use kmalloc directly instead of qla2xxx_kmem_zalloc and
	associated wrappers.

	Misc changes that don't affect generated code.

  reply	other threads:[~2003-12-09 13:28 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-12-06  1:15 [ANNOUNCE] QLogic qla2xxx driver update available (v8.00.00b7) Andrew Vasquez
2003-12-09 13:28 ` Christoph Hellwig [this message]
2003-12-29 17:32 ` James Bottomley
  -- strict thread matches above, loose matches on Subject: below --
2004-01-08 19:29 Andrew Vasquez
2004-01-08 16:47 Andrew Vasquez
2004-01-08 16:54 ` Christoph Hellwig
2004-01-08 19:33   ` Mike Anderson
2004-01-08 19:34     ` Christoph Hellwig
2004-01-08 19:57       ` Mike Anderson
2004-01-08 20:06         ` Christoph Hellwig
2004-01-08 22:11           ` Mike Anderson
2003-12-05 20:52 Andrew Vasquez

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \ \ \ \ \ \

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).