linux-scsi.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjanv@redhat.com>
To: Luben Tuikov <luben_tuikov@adaptec.com>
Cc: Andi Kleen <ak@suse.de>, linux-scsi@vger.kernel.org
Subject: Re: [PATCH] Add proper module ID tables to Adaptec aic7[9x]xxdrivers
Date: Mon, 21 Jun 2004 18:12:27 +0200	[thread overview]
Message-ID: <20040621161227.GD30236@devserv.devel.redhat.com> (raw)
In-Reply-To: <40D70837.8070303@adaptec.com>

[-- Attachment #1: Type: text/plain, Size: 1310 bytes --]

On Mon, Jun 21, 2004 at 12:09:27PM -0400, Luben Tuikov wrote:
> Arjan van de Ven wrote:
> >On Mon, 2004-06-21 at 17:30, Luben Tuikov wrote:
> >
> >>Thank you Andi for the patches. aic7xxx drivers are due for update
> >>to use the new timeout infrastructure. I'll incorporate them in,
> >>and do some testing.
> >
> >
> >cool.
> >Would you please consider the update to be done in small incremental
> >changes/steps ? Eg each separate change/bugfix a separate patch ?
> >Would make life a lot easier for me (distro kernel guy) and for James to
> >be able to diagnose regressions a lot better by binary searching
> >changesets etc etc.
> 
> Yep, no problem.
> 
> Current kernel version of, say, aic79xxx is 1.3.11, and I have
> 2.0.12, they are 226 patches away. Each patch is a submission

I was actually hoping you would have started from the kernel.org driver not
the adaptec weird fork justin has been maintaining ;(


> into perforce.  I can extract each and every one of those 226
> and can post them somewhere on a website (hosting anyone? ;-) ),
> so that anyone can review them.

it's not just about review, it's also about merging it in parts so that you
don't need to binary chop through 226 patches if something fails, but only
through like 30 or so.
And then a week later the next 30 go in etc etc


[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

  reply	other threads:[~2004-06-21 16:12 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-21 14:14 [PATCH] Add proper module ID tables to Adaptec aic7[9x]xx drivers Andi Kleen
2004-06-21 13:44 ` Sergey Vlasov
2004-06-21 16:24   ` Andi Kleen
2004-06-21 16:10     ` Sergey Vlasov
2004-06-21 15:30 ` Luben Tuikov
2004-06-21 15:37   ` Arjan van de Ven
2004-06-21 16:09     ` [PATCH] Add proper module ID tables to Adaptec aic7[9x]xxdrivers Luben Tuikov
2004-06-21 16:12       ` Arjan van de Ven [this message]
2004-06-21 18:03   ` [PATCH] Add proper module ID tables to Adaptec aic7[9x]xx drivers Andi Kleen
2004-06-21 16:10     ` Luben Tuikov

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=20040621161227.GD30236@devserv.devel.redhat.com \
    --to=arjanv@redhat.com \
    --cc=ak@suse.de \
    --cc=linux-scsi@vger.kernel.org \
    --cc=luben_tuikov@adaptec.com \
    /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).