From: "Justin T. Gibbs" <gibbs@scsiguy.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: axboe@suse.de (Jens Axboe),
scarfoglio@arpacoop.it (Carlo Scarfoglio),
linux-kernel@vger.kernel.org
Subject: Re: AIC7XXX 6.2.5 driver
Date: Thu, 21 Feb 2002 13:11:59 -0700 [thread overview]
Message-ID: <200202212011.g1LKBxI49280@aslan.scsiguy.com> (raw)
In-Reply-To: Your message of "Thu, 21 Feb 2002 19:30:19 GMT." <E16dyv5-0007xz-00@the-village.bc.nu>
>> included in certain 2.4.18-pre kernels which was the cause of the patch
>> not working for him. I can only assume you ran into the same problem.
>> My patch was relative to 2.4.17, not a more recent, yet unblessed, kernel.
>
>Relative to 2.4.18-rc would be good
Okay. I will do this sometime this weekend. I'm currently in the
middle of a business trip which makes testing a new patch difficult.
>> As to the CMD640 patch. Can you let me know why you believe it breaks
>> the CMD640? The current scheme leaks transactions on the bus and *will*
>
>The CMD640 can't do 32bit config space access stuff - thats why the code
>is there to hack around it. That code needs to know or check what pci
>config mode it should have used.
Right. If you look at the comments added to the code, the PCI APIs always
prefer direct config space access over using the PCI BIOS. The Linux
direct access methods only perform 32 bit cycles for 32bit API calls.
The ony time Linux will use the PCIBIOS (which may do everything as
32bit accesses) is if the direct access method fails. So, the machines
where direct access doesn't work can't be supported by the old, manual,
CMD640 code anyway. On machines that have working direct access, using
the PCI APIs is equivalent to the old method with the added bonus that
you don't blindly attempt to perform an access type that the chipset
does not support. This was the root cause of the "CMD640 driver performs
drive by shootings of other devices" issue.
>> Information about the SCSI mid-layer changes were posted to the SCSI list
>> and I believe CC'd to you. If you need that information again, I'd be
>> happy to resend it.
>
>The midlayer stuff Im more than happy with
Okay.
--
Justin
prev parent reply other threads:[~2002-02-21 20:12 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-02-20 22:36 Carlo Scarfoglio
2002-02-21 7:52 ` Jens Axboe
2002-02-21 14:01 ` Alan Cox
2002-02-21 13:58 ` Jens Axboe
2002-02-21 19:09 ` Justin T. Gibbs
2002-02-21 19:30 ` Alan Cox
2002-02-21 20:11 ` Justin T. Gibbs [this message]
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=200202212011.g1LKBxI49280@aslan.scsiguy.com \
--to=gibbs@scsiguy.com \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=axboe@suse.de \
--cc=linux-kernel@vger.kernel.org \
--cc=scarfoglio@arpacoop.it \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®