From: Martin Dalecki <dalecki@evision-ventures.com>
To: Jens Axboe <axboe@suse.de>
Cc: "Adam J. Richter" <adam@yggdrasil.com>, linux-kernel@vger.kernel.org
Subject: Re: Big patch: linux-2.5.2-pre11/drivers/scsi compilation fixes
Date: Fri, 11 Jan 2002 14:45:56 +0100 [thread overview]
Message-ID: <3C3EEC94.7020001@evision-ventures.com> (raw)
In-Reply-To: <20020111051456.A12788@baldur.yggdrasil.com> <20020111141850.B19814@suse.de>
Jens Axboe wrote:
>On Fri, Jan 11 2002, Adam J. Richter wrote:
>
>> Today I plan to post patches to make everything in
>>linux-2.5.2-pre11/drivers/scsi compile, at least everything under
>>x86, compiled as modules. They compile, and, the only undefined
>>symbol in when I boot is scsi_mark_host_reset in BusLogic.c.
>>However, the 2.5.2-pre11 patches are completely untested.
>>
>
>Please hang on with this for a week or so, there will be other changes
>to SCSI drivers required. You'll just end up doing the work twice.
>
For the record. We have a file called: sun3_NCR5380.c inside dirvers/scis.
There appers to be code there, which is using some of the oboslted error
handling facilities.
But under a compilation switch, there is code for the new error
management as well
already there, with some notes about the midlayer having problems.
If there is nobody supporting this driver, pleas consider switching
those macros... the
oboslete stuff,won't compile anyway.
However I doubt, that sun3 is supported anyway at all... so if not, then
please
consider this driver seriously for total removal ftom the kernel soruce
tree.
Here is the actual cided code:
#if 1 /* XXX Should now be done by midlevel code, but it's broken XXX */
/* XXX see below XXX */
/* MSch: old-style reset: actually abort all command processing here */
/* After the reset, there are no more connected or disconnected commands
* and no busy units; to avoid problems with re-inserting the commands
* into the issue_queue (via scsi_done())
And later down:
#else /* 1 */
/* MSch: new-style reset handling: let the mid-level do what it can */
/* ++guenther: MID-LEVEL IS STILL BROKEN.
* Mid-level is supposed to requeue all commands that were active on the
* various low-l
Both are quite at the end of the file...
next prev parent reply other threads:[~2002-01-11 13:46 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-01-11 13:14 Big patch: linux-2.5.2-pre11/drivers/scsi compilation fixes Adam J. Richter
2002-01-11 13:18 ` Jens Axboe
2002-01-11 13:45 ` Martin Dalecki [this message]
2002-01-11 14:20 ` Alan Cox
2002-01-11 14:42 ` Alan Cox
-- strict thread matches above, loose matches on Subject: below --
2002-01-11 16:59 Adam J. Richter
2002-01-11 17:27 ` Alan Cox
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:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3C3EEC94.7020001@evision-ventures.com \
--to=dalecki@evision-ventures.com \
--cc=adam@yggdrasil.com \
--cc=axboe@suse.de \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* 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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.