From: Mark Lord <liml@rtr.ca>
To: Frans Pop <elendil@planet.nl>
Cc: Tejun Heo <htejun@gmail.com>,
linux-ide@vger.kernel.org, nico@cam.org, buytenh@wantstofly.org,
Byron Bradley <byron.bbradley@gmail.com>
Subject: Re: sata_mv on Orion: mv_err_intr when shutting down
Date: Sun, 15 Jun 2008 09:44:55 -0400 [thread overview]
Message-ID: <48551CD7.8020303@rtr.ca> (raw)
In-Reply-To: <200806151141.52326.elendil@planet.nl>
Frans Pop wrote:
> On Sunday 15 June 2008, Tejun Heo wrote:
>> The interesting thing here is that sata_mv can see the disk going off
>> the power, which is kind of cool. The controller and CPU are watching
>> itself as it goes off the life support and is drifting awaaaaaayyyy.
>>
>> The PSU has switched off and power to the drive was cut. I don't think
>> anything can go wrong at that point.
>
> What I still find most interesting is that with 2.6.25 the extra
> message(s) do not appear and the poweroff is "clean". And that it affects
> 2 different systems where one seems to live on just a bit longer than the
> other. So it does look as if something _has_ changed.
>
> I'll probably do a bisect to see if I can discover what/where/why...
..
Sure, go for it.
But I think that you will simply discover that,
the sata_mv driver now actually correctly detects/reports most errors,
which it didn't necessarily do in the past.
There were lots of EH bugfixes in sata_mv this time around.
Cheers
next prev parent reply other threads:[~2008-06-15 13:45 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-05-12 13:26 sata_mv on Orion: mv_err_intr when shutting down Martin Michlmayr
2008-05-12 15:39 ` Mark Lord
2008-05-12 15:46 ` Martin Michlmayr
2008-05-12 19:15 ` Martin Michlmayr
2008-06-12 18:18 ` Martin Michlmayr
2008-06-13 12:42 ` Mark Lord
2008-06-13 16:12 ` Frans Pop
2008-06-13 16:56 ` Mark Lord
2008-06-13 17:10 ` Frans Pop
2008-06-13 19:01 ` Nicolas Pitre
2008-06-13 19:20 ` Mark Lord
2008-06-13 19:31 ` Nicolas Pitre
2008-06-15 2:18 ` Tejun Heo
2008-06-15 9:41 ` Frans Pop
2008-06-15 10:54 ` Byron Bradley
2008-06-15 13:44 ` Mark Lord [this message]
2008-06-22 14:37 ` Martin Michlmayr
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=48551CD7.8020303@rtr.ca \
--to=liml@rtr.ca \
--cc=buytenh@wantstofly.org \
--cc=byron.bbradley@gmail.com \
--cc=elendil@planet.nl \
--cc=htejun@gmail.com \
--cc=linux-ide@vger.kernel.org \
--cc=nico@cam.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.