Re: 2.6.16-rc4 libata + AHCI patched for suspend fails on ICH6

[Randy.Dunlap]

On Sun, 19 Feb 2006, Jens Axboe wrote:

> On Sun, Feb 19 2006, Ariel Garcia wrote:
> > Hi Jens,
> >
> > regarding your suspend support patch for libata:
> >
> > > author  Jens Axboe <axboe@xxxxxxx>
> > >    Fri, 6 Jan 2006 08:28:07 +0000 (09:28 +0100)
> > > commit  9b847548663ef1039dd49f0eb4463d001e596bc3
> >
> > >  [PATCH] Suspend support for libata
> > >  This patch adds suspend patch to libata, and ata_piix in particular.
> > > For most low level drivers, they should just need to add the 4 hooks to
> > > work. As I can only test ata_piix, I didn't enable it for more though.
> >
> > i tested the trivial "4-hooks" patch on kernel 2.6.16-rc4, on my laptop
> > (i915, ICH6 chipset, sata hd - a Fujitsu-Siemens 7020)
> > but it doesn't work as it should:
> >    after resume the drive fails to respond to the commands so it
> >    ends up remounted read-only.
> >
> > I am attaching:
> >    - the trivial patch i used
> >    - the output of lsmod (lsmod-clean.txt)
> >    - the output of lspci -vv  before (lspci-clean.txt)
> >         and after resuming (lspci-resume.txt)
> >    - the output of dmesg (glueing the full boot + resuming messages)
> >
> > All this was done running in single mode. I also tried suspending after
> > removing all unnecessary modules (usb, snd,ide,...), same result.
> >
> > BTW, running a    'diff lspci-clean.txt lspci-resume.txt'
> > i also noticed that after resume some pci devices get a different
> > "BusMaster" polarity, but the SATA controller doesn't.
> >
> > I would be glad to test patches/debug other things, feel free to ask.
>
> The first thing to try is to add the acpi addon from Randy and see if
> that helps at all. Looking at the log, the first command we issue after
> resume times out which smells a lot like an unlock command missing
> (which is typically in the GTF list from acpi).

Ariel-
These patches (for 2.6.16-rc3) are at
http://www.xenotime.net/linux/SATA/2.6.16-rc3/libata-rollup-2616-rc3.patch

in case you didn't find them yet.

> So try this patch on 2.6.16-rc3 (or -rc4, if it applies, haven't
> checked) and make sure to keep the ahci patch you have that adds the 4
> needed hooks as well.

-- 
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/