RE: aacraid+dlt (or dat)

"Salyzyn, Mark" <[email protected]>
Newsgroups gmane.linux.drivers.aacraid.devel
Message-ID <[email protected]>
The driver Yaroslav posted is `severely old' and is just our channel release
driver for the 2120 with a printk added. It is effectively version
0.9.10dmb-29D, the `current' submitted to Alan is 1.1.2.

Sounds like regression to me ...

The one I provided recently as work-in-progress is a 1.1.2+ with sundry
debug prints turned on. It has one notable bugfix (a panic resulted from an
AIF transmission) over the one shipped to Alan that is pending. The
remaining code changes are an investigation for an enhancement dealing with
bringing a zero'd array on-line without the need for a reboot (or a echo
scsi add-single-device ? ? ? ? > /proc/scsi/scsi).

Sincerely -- Mark Salyzyn

-----Original Message-----
From: [email protected] [mailto:[email protected]]
Sent: Friday, April 25, 2003 5:54 PM
To: [email protected]
Cc: sean.upton-lttx/[email protected]; [email protected]
Subject: Re: aacraid+dlt (or dat)


On Fri, 2003-04-25 at 16:13, Funk Gabor wrote:
> I'd also volunteer to test, though it seems that for the time
> being, I'll be in need for a separate SCSI card for the DLT... 
> Anyway, no test - no final results, so I would gladly give a shot.
> From where is it possible to download that patch?

http://marc.theaimsgroup.com/?l=linux-scsi&m=104809886115519&w=2

was the post by Yaroslav Klyukin <[email protected]> to Alan Cox
which contained a copy of the driver (not a patch) which he said worked
in 2.4.20.  I'm not sure how he got it, I can't find it on Adaptec's
linux.adaptec.com web site.  I *believe* this is what Deanna sent to
Alan prior to her departure from Adaptec, and is what Alan claimed above
to have pending.

The diff between 2.4.x-BK-current and what is attached to the note above
looks like:

$ diffstat aacraid.patch
 Makefile   |    1
 aachba.c   |  200 +++++++++++++++++++++++++++++++++--------------------
 aacraid.h  |  158 ++++++++++++++++++++++++++++++++----------
 commctrl.c |  227
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 comminit.c |   37 ++++-----
 commsup.c  |   47 ++++++------
 compat.h   |  114 ++++++++++++++++++++++++++++++
 dpcsup.c   |   22 +++--
 linit.c    |  197 ++++++++++++++++++++++------------------------------
 9 files changed, 718 insertions(+), 285 deletions(-)


Ignoring all the whitespace changes made, you get:

 Makefile   |    1
 aachba.c   |  102 +++++++++++++++++++--------
 aacraid.h  |  150 +++++++++++++++++++++++++++++++---------
 commctrl.c |  227
+++++++++++++++++++++++++++++++++++++++++++++++++++++++++----
 comminit.c |   37 ++++-----
 commsup.c  |   43 ++++++-----
 compat.h   |  114 ++++++++++++++++++++++++++++++
 dpcsup.c   |   20 +++--
 linit.c    |  181 ++++++++++++++++++++----------------------------
 9 files changed, 654 insertions, 221 deletions


I'll attach both patches (one with whitespace changes in it, one that
ignores whitespace changes) if someone cares to decipher what changed
and summarize it for us.

One change I believe is in there fixes a bug with I/O requests that use
memory > 4GB.  I haven't run into that bug myself though, so I can't say
for sure...

Thanks,
Matt

-- 
Matt Domsch
Sr. Software Engineer, Lead Engineer, Architect
Dell Linux Solutions www.dell.com/linux
Linux on Dell mailing lists @ http://lists.us.dell.com

_______________________________________________
Linux-aacraid-devel mailing list
[email protected]
http://lists.us.dell.com/mailman/listinfo/linux-aacraid-devel
Please read the FAQ at http://lists.us.dell.com/faq or search the list archives at http://lists.us.dell.com/htdig/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.