linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Justin T. Gibbs" <gibbs@scsiguy.com>
To: Stephan von Krawczynski <skraw@ithnet.com>,
	"Justin T. Gibbs" <gibbs@scsiguy.com>
Cc: linux-kernel@vger.kernel.org, willy@w.ods.org,
	marcelo@conectiva.com.br, green@namesys.com
Subject: Re: Undo aic7xxx changes (now rc7+aic20030603)
Date: Tue, 10 Jun 2003 09:38:31 -0600	[thread overview]
Message-ID: <23050000.1055259511@caspian.scsiguy.com> (raw)
In-Reply-To: <20030610122326.13fe8889.skraw@ithnet.com>

>> Stephan,
>>
>> Other than your most recent complaint that the driver doesn't function
>> correctly in an SMP kernel when you specify the nosmp option, you have
>> yet to provide any information that points to a problem in the aic7xxx
>> driver.
>
> Dear Justin,
>
> I am really not complaining about you not helping specifically _me_, I am
> complaining about your quite visible general opinion that this whole thing is
> really not serious, or maybe it is only that you are not making your efforts
> transparent to others, I don't know.

I never said that it wasn't serios, I just haven't seen any indication
that this problem is caused by my driver.  There is a big difference.
If your complaint is that I typically help people to solve their problems
*off-list*, then I'm sorry if that offends your sensibilities.
I personally don't think that I need to CC a million people while I'm
passing back various debugging information and asking for new output.  Its
just a lot of noise for the majority of people on the linux-kernel list.

>>  Without such information, I'm at a loss to help you.  One thing
>> that you forgot to mention in your "report" is that data corruption can
>> happen in many more places than just in the aic7xxx driver.
>
> <sarcasm>Did I mention the big magnet right beside the tape?</sarcasm>

I'm just sick of being blamed for anything that goes wrong on any system
that happens to have an aic7xxx controller in it.  99% or the time its
not my fault, but I suppose since I debug and resolve these issues off
list for people that contact me, the general assumption is that these
issues are the aic7xxx driver's fault.

>>  The data could be corrupted by a VM bug,
>
> VM is quite the same, tar'ing to /dev/tape or /var/bak/mybackfile.tar.

No, the VM activity is quite different.

>> a buffer layer bug, or a filesystem bug.
>
> /dev/tape with a filesystem? Have you read what we are talking about?

Where did you get the data to place on the tape?  /dev/zero?

>>  When testing our drivers against RHAS2.1 we found that the stock
>> kernel had data corruption issues very similar to what your are talking
>> about when run on very fast, hyperthreading, SMP machines.  The data
>> corruption occurred with any SCSI controller we tried, regardless of vendor.
>
> My question is: is it solved?

My understanding is that it was fixed in 2.4.18 level kernels, but since
I don't know the root cause of the corruption, it could have just been
made more difficult to reproduce.

>> If you continue to feel that the aic7xxx driver is at fault, I encourage you
>> to try to reproduce this failure with someone elses card.  I think you'll
>> find that the problem persists even with this change.
>
> This is not the first discussion about an instability in aic.

I'm not talking about *every case of aic7xxx driver instability*, I'm
talking about *this particular case* of driver instability.  Problems
that to the naive user look similar are typically not.

>> I will be more than happy to look into why the aic7xxx driver may not
>> operate correctly in an SMP kernel with the nosmp option.  Considering
>> that your complaint about this failure came into my email box just
>> yesterday, perhaps you can give me just a few days to look into this
>> before you decide to call me unresponsive.  Since I'm attending a
>> conference this whole week, I won't even be able to look at this
>> until I return on Monday of next week.
>
> Justin, this is nothing quite serious, I just mentioned it for a feedback to
> something _simple_.

It's the only thing you've mentioned that I have enough information to
look at.

>> I'm sorry that you are experiencing data corruption.  I take those
>> issues very seriously, but all of your panics and other reports point
>> to issues elsewhere in the kernel that should be resolved before you
>> conclude that the data corruption you are experiencing is somehow
>> the aic7xxx driver's fault.  I'll be more than happy to fess up to
>> and correct any defect that is found in the driver, but I cannot fix
>> bugs that I cannot reproduce and that have no usable debugging information
>> associated with them.
>
> What exactly is "elsewhere" if your data is bogus when tar'ing onto /dev/tape
> via aic and it is completely ok when tar'ing into a file via reiserfs/3ware ?
> There is not really much left between tar and the aic-driver and the tape.

I suggest you go browse the code that is exercised by such an activity
before you say that.

--
Jusitn


  reply	other threads:[~2003-06-10 15:30 UTC|newest]

Thread overview: 110+ 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
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 [this message]
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
     [not found] <20030507203025$6f60@gated-at.bofh.it>
     [not found] ` <20030509005011$6cee@gated-at.bofh.it>
     [not found]   ` <20030509101012$732a@gated-at.bofh.it>
     [not found]     ` <20030509122007$758f@gated-at.bofh.it>
     [not found]       ` <20030509131009$00f3@gated-at.bofh.it>
     [not found]         ` <20030611045008$03cf@gated-at.bofh.it>
     [not found]           ` <20030611203031$12de@gated-at.bofh.it>
     [not found]             ` <20030611211012$34cf@gated-at.bofh.it>
     [not found]               ` <20030613095017$1680@gated-at.bofh.it>
     [not found]                 ` <20030617210022$3e37@gated-at.bofh.it>
     [not found]                   ` <20030618111010$154f@gated-at.bofh.it>
2003-06-18 12:46                     ` Undo aic7xxx changes (now rc7+aic20030603) Pascal Schmidt
2003-06-18 12:49                       ` Stephan von Krawczynski

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=23050000.1055259511@caspian.scsiguy.com \
    --to=gibbs@scsiguy.com \
    --cc=green@namesys.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=marcelo@conectiva.com.br \
    --cc=skraw@ithnet.com \
    --cc=willy@w.ods.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 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).