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 <[email protected]>
> > > 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
