linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeremy Higdon <jeremy@sgi.com>
To: "Dailey, Nate" <Nate.Dailey@stratus.com>
Cc: linux-ide@vger.kernel.org
Subject: Re: sata_vsc.c cache line size question
Date: Wed, 7 Feb 2007 00:38:50 -0800	[thread overview]
Message-ID: <20070207083850.GA138013@sgi.com> (raw)
In-Reply-To: <92952AEF1F064042B6EF2522E0EEF43703EE318A@EXNA.corp.stratus.com>

On Thu, Jan 18, 2007 at 09:42:44AM -0500, Dailey, Nate wrote:
> Okay, trying again as an attachment...
> 
> Nate

Hi Nate, sorry about the delay.  This patch works fine.  Please see
http://linux.yyz.us/patch-format.html and send a new patch, if you would.
Then presumably jgarzik will take it, since it's a good fix.
Or if you want, I could generate the new patch.

Thanks,

jeremy



> 
> -----Original Message-----
> From: Jeremy Higdon [mailto:jeremy@sgi.com] 
> Sent: Thursday, January 18, 2007 2:12 AM
> To: Dailey, Nate
> Cc: linux-ide@vger.kernel.org
> Subject: Re: sata_vsc.c cache line size question
> 
> On Mon, Jan 15, 2007 at 11:26:59AM -0500, Dailey, Nate wrote:
> > Here's a patch that does what you suggest.
> > 
> > Because the default cache line size on my system is 0x10, I tested the
> > patch by checking against this value rather than 0... it worked as
> > expected.
> > 
> > This patch is against 2.6.19.2 that I just downloaded from kernel.org.
> I
> > actually tested on RHEL4 update 4, a 2.6.9 kernel, but I'll try
> building
> > the 2.6.19.2 on my system to make sure it works in that version as
> well.
> > 
> > Nate
> 
> Hi Nate,
> 
> The patch looks fine, except that your mailer wrapped the lines.
> Try sending it as a text attachment instead.
> 
> Thanks
> 
> jeremy

  reply	other threads:[~2007-02-07  8:38 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-01-18 14:42 sata_vsc.c cache line size question Dailey, Nate
2007-02-07  8:38 ` Jeremy Higdon [this message]
  -- strict thread matches above, loose matches on Subject: below --
2007-01-15 16:26 Dailey, Nate
2007-01-18  7:11 ` Jeremy Higdon
2007-01-12 19:45 Dailey, Nate
2007-01-14  8:03 ` Jeremy Higdon
2007-01-14 14:47   ` Alan
2007-01-14 18:21     ` Jeff Garzik

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=20070207083850.GA138013@sgi.com \
    --to=jeremy@sgi.com \
    --cc=Nate.Dailey@stratus.com \
    --cc=linux-ide@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 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).