From: "Michael S. Tsirkin" <mst@mellanox.co.il>
To: Jeff Garzik <jeff@garzik.org>
Cc: linux-ide@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH][RFT] libata AHCI reset speed-up, improvements
Date: Wed, 4 Oct 2006 11:14:20 +0200 [thread overview]
Message-ID: <20061004091419.GA5683@mellanox.co.il> (raw)
In-Reply-To: <20060927025639.GA16969@havoc.gtf.org>
Quoting Jeff Garzik <jeff@garzik.org>:
> Subject: [PATCH][RFT] libata AHCI reset speed-up, improvements
>
>
> This patch, diff'd against 2.6.18-git5 (or later), attempts the
> following improvements:
>
> * Speed up reset, by polling rather than unconditionally sleeping
> for one seconds.
> * Save the Ports-Implemented register across controller resets.
> * Verify that AHCI mode was truly enabled.
>
> I'm quite interested in hearing people's test feedback on this patch.
> On my own ICH7 machine, it introduces a "softreset failed" error, but
> EH recovers nicely and nonetheless configures everything successfully.
> Probably just a slow ATAPI device, but I may have exposed a latent
> problem in the driver that was hidden until now by the ssleep(1).
Hi Jeff!
I tested this patch on my T60 (Intel ICH7)
against git 78b656b8bf933101b42409b4492734b23427bfc3
with the same result: during boot, I get
scsi0: ahci
ata1: port is slow to respond, pls be patient
ata1: port failed to respond (30 sec)
ata1: soft reset failed (device not ready)
ata1: soft reset failed retrying in 5 secs
etc
after a while everything recovers and seems to work OK
afterwards, but still this seems to add about a minute or so
to the boot time.
I'm not sure how to test whether I get the speedup that you
mention.
Hope this report helps.
--
MST
prev parent reply other threads:[~2006-10-04 9:13 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-09-27 2:56 Jeff Garzik
2006-10-04 9:14 ` Michael S. Tsirkin [this message]
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=20061004091419.GA5683@mellanox.co.il \
--to=mst@mellanox.co.il \
--cc=jeff@garzik.org \
--cc=linux-ide@vger.kernel.org \
--cc=linux-kernel@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
all inboxes | Powered by JetHome®