From: David Fries <david@fries.net>
To: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
"David S. Miller" <davem@davemloft.net>,
Joao Ramos <joao.ramos@inov.pt>,
d.stussy@yahoo.com, linux-ide@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/2] two ide fixes for -rc4
Date: Tue, 6 Oct 2009 22:30:06 -0500 [thread overview]
Message-ID: <20091007033005.GE21612@spacedout.fries.net> (raw)
In-Reply-To: <200910070246.05283.bzolnier@gmail.com>
On Wed, Oct 07, 2009 at 02:46:05AM +0200, Bartlomiej Zolnierkiewicz wrote:
> On Wednesday 07 October 2009 00:35:24 Linus Torvalds wrote:
> >
> > David, can you test these on the sis5513 system that had problems with the
> > original pio0 mode setting?
> >
> > Thanks,
> > Linus
> >
> > On Wed, 7 Oct 2009, Bartlomiej Zolnierkiewicz wrote:
> > >
> > > patch #1 fixes old bug in sis5513 (uncovered by 2.6.30 device probing
> > > changes but it could be also triggered in earlier kernels on warm-plug
> > > operation)
>
> patch #1 contained a stupid mistake preventing it from working (thanks to
> David Fries for letting me know quickly), I guess that is what one gets for
> the late night hacking..
>
> The corrected version below, David please try it instead:
>
> From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
> Subject: [PATCH take 2] sis5513: fix PIO setup for ATAPI devices
2.6.31 plus this patch fixed the hang on IDE initialization with the
SIS5513.
That's with my testing mode, I'm rebuilding a kernel and will try some
benchmarks tomorrow. I've been testing by hibernating to disk,
booting the test kernel with a bad root and resume partition (so if it
worked it wouldn't modify the suspended image), then rebooting to
resume from disk with the good kernel. That let me more quickly
bisect and test kernels, I just haven't ran this kernel much past the
initialization is all.
--
David Fries <david@fries.net>
http://fries.net/~david/ (PGP encryption key available)
next prev parent reply other threads:[~2009-10-07 3:30 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-06 22:27 [PATCH 0/2] two ide fixes for -rc4 Bartlomiej Zolnierkiewicz
2009-10-06 22:35 ` Linus Torvalds
2009-10-07 0:46 ` Bartlomiej Zolnierkiewicz
2009-10-07 3:30 ` David Fries [this message]
2009-10-07 11:06 ` David Miller
2009-10-10 17:54 ` -
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=20091007033005.GE21612@spacedout.fries.net \
--to=david@fries.net \
--cc=bzolnier@gmail.com \
--cc=d.stussy@yahoo.com \
--cc=davem@davemloft.net \
--cc=joao.ramos@inov.pt \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=torvalds@linux-foundation.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).