Re: Drives missing at boot

[Mark Knecht]

On Tue, Jul 20, 2010 at 7:53 AM, Mark Knecht <markknecht@xxxxxxxxx> wrote:
> On Tue, Jul 20, 2010 at 7:14 AM, Tejun Heo <tj@xxxxxxxxxx> wrote:
>> Helo,
>>
>> On 07/20/2010 05:14 AM, Paul Check wrote:
>>> Hey Tejun: I guess this is the same patch that you sent me to fix my issue
>>> with missing drives. ÂGood news: I've been through about 10 reboots now
>>> and no problems. ÂBased on my prior experience, I'd say with the old
>>> setup, 10 clean boots in a row was probably less than a 1% event. ÂSo, it
>>> seems that this has fixed my problem.
>>
>> Yeap, it's the same one. ÂI'm forwarding the patch upstream now but,
>> Mark, please let me know the test result.
>>
>> Thanks.
>>
>> --
>> tejun
>>
>
> Tejun,
> Â I'm traveling but back tonight. I'll try it then.
>
> Thanks,
> Mark
>

OK, I was able to get into the machine remotely for a few minutes. I
think the patch applied correctly and the machine cold booted cleanly.
(My wife powered it up for me.)

I'll do more boots later when you confirm everything looks reasonable.
I don't see any print statement in the patch so I don't know what to
look for. I'm attaching a dmesg file for you to review.

Assuming I did this right then everything seems good so far.

Cheers,
Mark

c2stable linux # patch --verbose -p1 <~mark/Downloads/ata_piix-sidpr-lock.patch
Hmm...  Looks like a unified diff to me...
The text leading up to this was:
--------------------------
|diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
|index 7409f98..3971bc0 100644
|--- a/drivers/ata/ata_piix.c
|+++ b/drivers/ata/ata_piix.c
--------------------------
Patching file drivers/ata/ata_piix.c using Plan A...
Hunk #1 succeeded at 158.
Hunk #2 succeeded at 952.
Hunk #3 succeeded at 968.
Hunk #4 succeeded at 1573.
done
c2stable linux #

Attachment: dmesg.Patch-New.COLDTest1
Description: Binary data