2.6.25-rc1 ahci regression
in [Kernel]
|
Prev: [PATCH 06/12] oom: remove warning for in mm-less task __oom_kill_process()
Next: [PATCH] acer-wmi/hp-wmi: use platform_device_unregister instead of platform_device_del in module_exit
From: Anton Vorontsov on 3 Jun 2010 09:20 On Thu, Jun 03, 2010 at 08:35:31PM +0800, Luming Yu wrote: > On Thu, Jun 3, 2010 at 5:39 PM, Anton Vorontsov <cbouatmailru(a)gmail.com> wrote: > > On Thu, Jun 03, 2010 at 05:18:40PM +0800, Luming Yu wrote: > > [...] > >> Linux version 2.6.32-25.el6.x86_64 > > > > Hm. Can you really boot the kernels that has > > 365cfa1ed5 applied and reverted? > > > > That is, do > > > > $ git reset --hard 365cfa1ed5 > > $ make > > $ ...boot... > Linux version 2.6.34-rc7 (root(a)localhost) (gcc version 4.4.4 20100503 [...] > > And then > > > > $ git reset --hard 365cfa1ed5^ > > $ make > > $ ...boot... > > > Linux version 2.6.34-rc7 (root(a)localhost) (gcc version 4.4.4 20100503 > (Red Hat 4.4.4-2) (GCC) ) #24 SMP Thu Jun 3 08:07:58 EDT 2010 Thanks. But for some reason you don't include debug messages from the not working kernel. :-/ I presume you use different methods to capture the log, you should pass 'debug' kernel command line option to actually enable these messages. Please try to capture the debugging messages. Meanwhile, here is a guess: initrd isn't happy about the new libahci.ko dependency for ahci.ko, and so it doesn't load the module? Why? I don't know. Let me check it... -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Luming Yu on 3 Jun 2010 09:40 > Meanwhile, here is a guess: initrd isn't happy about > the new libahci.ko dependency for ahci.ko, and so it > doesn't load the module? Why? I don't know. Let > me check it... > ah, your no function changed code creates a new module libahci and it's not in initrd? so I have this problem, a good guess I think I will check it -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Anton Vorontsov on 3 Jun 2010 09:40 On Thu, Jun 03, 2010 at 05:14:14PM +0400, Anton Vorontsov wrote: > On Thu, Jun 03, 2010 at 08:35:31PM +0800, Luming Yu wrote: > > On Thu, Jun 3, 2010 at 5:39 PM, Anton Vorontsov <cbouatmailru(a)gmail.com> wrote: > > > On Thu, Jun 03, 2010 at 05:18:40PM +0800, Luming Yu wrote: > > > [...] > > >> Linux version 2.6.32-25.el6.x86_64 > > > > > > Hm. Can you really boot the kernels that has > > > 365cfa1ed5 applied and reverted? > > > > > > That is, do > > > > > > $ git reset --hard 365cfa1ed5 > > > $ make > > > $ ...boot... > > Linux version 2.6.34-rc7 (root(a)localhost) (gcc version 4.4.4 20100503 > [...] > > > And then > > > > > > $ git reset --hard 365cfa1ed5^ > > > $ make > > > $ ...boot... > > > > > Linux version 2.6.34-rc7 (root(a)localhost) (gcc version 4.4.4 20100503 > > (Red Hat 4.4.4-2) (GCC) ) #24 SMP Thu Jun 3 08:07:58 EDT 2010 > > Thanks. But for some reason you don't include debug > messages from the not working kernel. :-/ I presume > you use different methods to capture the log, you > should pass 'debug' kernel command line option to > actually enable these messages. Please try to > capture the debugging messages. > > Meanwhile, here is a guess: initrd isn't happy about > the new libahci.ko dependency for ahci.ko, and so it > doesn't load the module? Why? I don't know. Let > me check it... Oh. It seems I found it. It must be the module order loading thing. Can you try the patch down below? And thanks for the patience! diff --git a/drivers/ata/Makefile b/drivers/ata/Makefile index 20c5251..092d9e0 100644 --- a/drivers/ata/Makefile +++ b/drivers/ata/Makefile @@ -1,7 +1,7 @@ obj-$(CONFIG_ATA) += libata.o -obj-$(CONFIG_SATA_AHCI) += ahci.o libahci.o +obj-$(CONFIG_SATA_AHCI) += libahci.o ahci.o obj-$(CONFIG_SATA_SVW) += sata_svw.o obj-$(CONFIG_ATA_PIIX) += ata_piix.o obj-$(CONFIG_SATA_PROMISE) += sata_promise.o -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Anton Vorontsov on 3 Jun 2010 10:00 On Thu, Jun 03, 2010 at 05:23:19PM +0400, Anton Vorontsov wrote: [...] > Oh. It seems I found it. It must be the module order loading > thing. > > Can you try the patch down below? OTOH... libahci is a link-time dependency. And the order doesn't matter here. So you should just have the libahci module in the initrd (actually, mkinitrd or whatever you use should have included this automatically). So, normally, this patch should be unnecessary. > diff --git a/drivers/ata/Makefile b/drivers/ata/Makefile > index 20c5251..092d9e0 100644 > --- a/drivers/ata/Makefile > +++ b/drivers/ata/Makefile > @@ -1,7 +1,7 @@ > > obj-$(CONFIG_ATA) += libata.o > > -obj-$(CONFIG_SATA_AHCI) += ahci.o libahci.o > +obj-$(CONFIG_SATA_AHCI) += libahci.o ahci.o > obj-$(CONFIG_SATA_SVW) += sata_svw.o > obj-$(CONFIG_ATA_PIIX) += ata_piix.o > obj-$(CONFIG_SATA_PROMISE) += sata_promise.o -- Anton Vorontsov email: cbouatmailru(a)gmail.com irc://irc.freenode.net/bd2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Luming Yu on 3 Jun 2010 10:20
On Thu, Jun 3, 2010 at 9:50 PM, Anton Vorontsov <cbouatmailru(a)gmail.com> wrote: > On Thu, Jun 03, 2010 at 05:23:19PM +0400, Anton Vorontsov wrote: > [...] >> Oh. It seems I found it. It must be the module order loading >> thing. >> >> Can you try the patch down below? > > OTOH... libahci is a link-time dependency. And the order doesn't > matter here. So you should just have the libahci module in > the initrd (actually, mkinitrd or whatever you use should > have included this automatically). > Neither ahci nor libachi is automatically included in initrd.. Manually added ahci and libachi into initrd seems to make the problem go away.. Any idea why that could happen? -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |