linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Justin T. Gibbs" <gibbs@scsiguy.com>
To: Marcelo Tosatti <marcelo@conectiva.com.br>,
	lkml <linux-kernel@vger.kernel.org>
Subject: Re: Undo aic7xxx changes
Date: Thu, 08 May 2003 18:45:42 -0600	[thread overview]
Message-ID: <2804790000.1052441142@aslan.scsiguy.com> (raw)
In-Reply-To: <Pine.LNX.4.55L.0305071716050.17793@freak.distro.conectiva>

> Hi,
> 
> I've undone aic7xxx changes which were locking up some machines on
> initialization.

Hmm.  It would have been nice to have the oportunity to fix this correctly.
As it stands now, I have really no idea what people were testing or not
since by taking Alan's patch you have lost the complete change history
and the ability to step people through the changes.  I have preserved
this history in the bk send output that is available on my site if at
some point that is useful to you.

> The new driver is now named drivers/scsi/aic79xx and is under
> CONFIG_AIC79XX.

So we now have an extra copy of the assembler, the Config files, and
the aiclib files.  This is not a solution.  If you wanted to selectively
update the aic79xx driver, all you had to do was ask me for the requisite
change sets.  This is what a mainatiner is for.

> Justin, unfortunately I can't even THINK about updating aic7xxx to your
> new driver at the current release stage. I will do so in the 2.4.22.

Does this mean that you will actually take BK changes form me instead of
from just about anyone else that sends you aic7xxx driver updates?  I had
pretty much given up on this.

> The update also contains a PCI posting flush fix from Arjan.

Which is completely unnecessary and in fact will cause hangs and crashes
on many Dell servers.  The "fix" for the VIA systems that violate the
PCI spec is to either:

1) Update the driver correctly so that it's detection logic will
   automatically disable memory mapped I/O for these broken systems.

or 

2) Just disable the BIOS options that configure the system to violate
   the PCI prefetching rules.

Slowing down all systems, even the ones that are *not broken* by doing
extra, random, PCI read cycles is not a fix.

If you want some verification of the Dell issue (which I'm sure will
cause problems on other "fast" systems too), just ask Matt Domsh.

Again, if you have concerns about the aic7xxx or aic79xx drivers, my
mail box is always open.  Waiting to contact me until the last minute
where I can only sit on the sidelines and watch another train wreck is
not the best way to ensure that the drivers function correctly in 2.4.X.

What this basically boils down to is trust.  If you don't trust me,
tell me how I can build that trust.  Without it, I can only continue
to tell most people that contact me with bug reports, "It's already
fixed in the official driver.  You can pull the latest from ..."

--
Justin


  reply	other threads:[~2003-05-09  0:33 UTC|newest]

Thread overview: 109+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-05-07 20:22 Undo aic7xxx changes Marcelo Tosatti
2003-05-09  0:45 ` Justin T. Gibbs [this message]
2003-05-09 10:06   ` Stephan von Krawczynski
2003-05-09 12:06     ` Willy Tarreau
2003-05-09 13:02       ` Stephan von Krawczynski
2003-05-09 13:27         ` Willy Tarreau
2003-05-09 13:46           ` Stephan von Krawczynski
2003-05-09 14:56             ` Willy Tarreau
2003-05-09 15:08               ` Arjan van de Ven
2003-05-09 16:27                 ` Willy Tarreau
2003-05-09 15:18               ` Andreas Schwab
2003-05-09 15:19               ` William Lee Irwin III
2003-05-09 14:11           ` Stephan von Krawczynski
2003-05-09 14:57             ` Willy Tarreau
2003-05-12  9:02               ` Stephan von Krawczynski
2003-05-12 15:43                 ` Marc-Christian Petersen
2003-05-12 17:25                 ` Willy Tarreau
2003-05-23 10:38                 ` Stephan von Krawczynski
2003-05-23 12:58                   ` Justin T. Gibbs
2003-05-23 13:11                     ` Stephan von Krawczynski
2003-05-23 19:57                     ` Willy Tarreau
2003-05-24 10:52                       ` Stephan von Krawczynski
2003-05-24 11:16                         ` Willy Tarreau
2003-05-25 10:58                           ` Stephan von Krawczynski
2003-05-25 12:35                             ` Willy TARREAU
2003-05-25 12:47                             ` Marc-Christian Petersen
2003-05-25 13:50                               ` Stephan von Krawczynski
2003-05-25 14:01                                 ` Marc-Christian Petersen
2003-05-25 14:03                                 ` Geller Sandor
2003-05-26 15:00                               ` Stephan von Krawczynski
2003-05-26 16:44                                 ` Willy Tarreau
2003-05-30  8:09                                   ` Stephan von Krawczynski
2003-05-30  8:19                                     ` Marc-Christian Petersen
2003-05-30  8:21                                     ` Arjan van de Ven
2003-05-30  8:51                                       ` Stephan von Krawczynski
2003-05-30 13:34                                     ` Jeff Garzik
2003-05-30 13:59                                       ` Stephan von Krawczynski
2003-05-30 13:35                                     ` Jeff Garzik
2003-05-25 18:30                             ` Justin T. Gibbs
2003-06-05 15:05                           ` Undo aic7xxx changes (now rc7+aic20030603) Stephan von Krawczynski
2003-06-05 18:14                             ` Willy Tarreau
2003-06-06  8:17                               ` Oleg Drokin
2003-06-06  9:04                                 ` Stephan von Krawczynski
2003-06-06  9:17                                   ` Oleg Drokin
2003-06-06 15:24                                     ` short freezing while file re-creation Stephan von Krawczynski
2003-06-06 16:02                                       ` Oleg Drokin
2003-06-06 19:00                                         ` Chris Mason
2003-06-06 19:10                                           ` Oleg Drokin
2003-06-06 19:20                                             ` Chris Mason
2003-06-08 10:15                                     ` Undo aic7xxx changes (now rc7+aic20030603) Stephan von Krawczynski
2003-06-08 11:19                               ` Stephan von Krawczynski
2003-06-08 11:49                                 ` Stephan von Krawczynski
2003-06-08 16:07                                   ` Stephan von Krawczynski
2003-06-09 15:10                                   ` Stephan von Krawczynski
2003-06-09 15:32                                     ` Justin T. Gibbs
2003-06-10 10:23                                       ` Stephan von Krawczynski
2003-06-10 15:38                                         ` Justin T. Gibbs
2003-06-10 17:11                                           ` Stephan von Krawczynski
2003-06-10 18:07                                             ` Justin T. Gibbs
2003-06-11  0:51                                               ` Stephan von Krawczynski
2003-06-11  4:39                                                 ` Justin T. Gibbs
2003-06-11 20:23                                                   ` Stephan von Krawczynski
2003-06-11 21:01                                                     ` John Stoffel
2003-06-13  9:45                                                       ` Stephan von Krawczynski
2003-06-15 12:56                                                         ` Stephan von Krawczynski
2003-06-15 13:26                                                           ` John Stoffel
2003-06-17 20:47                                                         ` Marcelo Tosatti
2003-06-18 11:05                                                           ` Stephan von Krawczynski
2003-06-18 14:21                                                             ` John Stoffel
2003-06-18 14:54                                                               ` Stephan von Krawczynski
2003-06-20 19:59                                                             ` Marcelo Tosatti
2003-06-20 20:59                                                               ` Kevin P. Fleming
2003-06-20 21:13                                                                 ` Marcelo Tosatti
2003-06-20 22:03                                                                   ` Willy Tarreau
2003-06-20 23:48                                                                     ` Stephan von Krawczynski
2003-06-21 10:50                                                                       ` Willy TARREAU
2003-06-22 19:00                                                                         ` Stephan von Krawczynski
2003-06-23 11:30                                                                         ` Stephan von Krawczynski
2003-06-24 11:11                                                                           ` Stephan von Krawczynski
2003-06-24 17:43                                                                             ` Willy Tarreau
2003-06-24 21:26                                                                               ` Stephan von Krawczynski
2003-06-24 22:03                                                                                 ` Willy Tarreau
2003-06-24 23:43                                                                                   ` Stephan von Krawczynski
2003-06-25 19:16                                                                                     ` Willy Tarreau
2003-06-25 19:42                                                                                       ` Stephan von Krawczynski
2003-06-25 20:30                                                                                         ` John Stoffel
2003-06-26  9:36                                                                                           ` Stephan von Krawczynski
2003-06-26 11:34                                                                                           ` Stephan von Krawczynski
2003-06-30 10:10                                                                                             ` Stephan von Krawczynski
2003-06-30 11:39                                                                                               ` Marcelo Tosatti
2003-06-30 12:08                                                                                                 ` Stephan von Krawczynski
2003-06-25 23:04                                                                                       ` Bernd Eckenfels
2003-06-25  2:22                                                                                 ` Valdis.Kletnieks
2003-06-24 18:31                                                                     ` Bill Davidsen
2003-06-12 13:54                                                     ` Stephan von Krawczynski
2003-06-10  1:38                                     ` Zwane Mwaikambo
2003-06-10 10:30                                       ` Stephan von Krawczynski
2003-06-10 12:51                                         ` Zwane Mwaikambo
2003-06-10 13:38                                           ` Stephan von Krawczynski
2003-06-10 13:51                                             ` Zwane Mwaikambo
2003-06-10 15:55                                               ` Stephan von Krawczynski
2003-06-10 16:23                                                 ` Oleg Drokin
2003-06-10 17:44                                               ` Stephan von Krawczynski
2003-06-10 18:15                                                 ` Zwane Mwaikambo
2003-06-10 23:55                                                   ` Stephan von Krawczynski
2003-06-10 18:20                                                 ` Zwane Mwaikambo
2003-05-23 18:30                   ` Undo aic7xxx changes Marcelo Tosatti
2003-05-23 19:25                     ` Stephan von Krawczynski
2003-05-12 15:31 Klaus Dittrich

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=2804790000.1052441142@aslan.scsiguy.com \
    --to=gibbs@scsiguy.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo@conectiva.com.br \
    /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;
as well as URLs for NNTP newsgroup(s).