linux-kernel - Re: Linux 2.6.32.47

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <1320736973.16747.55.camel@dagon.hellion.org.uk>
Date:	Tue, 8 Nov 2011 07:22:52 +0000
From:	Ian Campbell <Ian.Campbell@...rix.com>
To:	Greg KH <gregkh@...e.de>
CC:	Konrad Rzeszutek Wilk <konrad.wilk@...cle.com>,
	Jeremy Fitzhardinge <Jeremy.Fitzhardinge@...rix.com>,
	Christoph Biedl <linux-kernel.bfrz@...chmal.in-ulm.de>,
	linux-kernel@...r.kernel.org, stable@...nel.org
Subject: Re: Linux 2.6.32.47

On Mon, 2011-11-07 at 16:30 -0800, Greg KH wrote:
> On Tue, Nov 08, 2011 at 12:43:49AM +0100, Christoph Biedl wrote:
> > Greg KH wrote...
> > 
> > > I'm announcing the release of the 2.6.32.47 kernel.
> > 
> > Unfortunately this does not build anymore on ARM (Debian armel):
> > 
> >   CHK     include/linux/version.h
> >   SYMLINK include/asm -> include/asm-arm
> > make[1]: `include/asm-arm/mach-types.h' is up to date.
> >   CHK     include/linux/utsrelease.h
> >   CALL    scripts/checksyscalls.sh
> >   CHK     include/linux/compile.h
> >   GEN     .version
> >   CHK     include/linux/compile.h
> >   UPD     include/linux/compile.h
> >   CC      init/version.o
> >   LD      init/built-in.o
> >   LD      .tmp_vmlinux1
> > drivers/built-in.o: In function `sysdev_resume':
> > ledtrig-default-on.c:(.text+0x2a97c): undefined reference to `irq_pm_syscore_resume'
> > make: *** [.tmp_vmlinux1] Error 1
> > 
> > 
> > The most likely culprit is
> > 
> >     commit 0f12a6ad9fa3a03f2bcee36c9cb704821e244c40
> >     Author: Ian Campbell <ian.campbell@...rix.com>
> >     Date:   Mon Oct 3 15:37:00 2011 +0100
> > 
> > 	genirq: Add IRQF_RESUME_EARLY and resume such IRQs earlier
> > 
> > as reverting that commit made the error go away.
> > 
> > The .config used is attached. FWIW, the target system is a DockStar.
> > 
> > Sorry for not having checked during the review period, I should have
> > done so.
> 
> Not good.
> 
> Ian, Konrad, any thoughts as to how to fix this?

Looks like I got the wrong CONFIG symbol for the function, or more
importantly the dummy version of it. Sorry about this.

Ian.

8<--------------------------------------------------------------

>From 33a64cf4a184b47493ef312b08468aca3361e112 Mon Sep 17 00:00:00 2001
From: Ian Campbell <ian.campbell@...rix.com>
Date: Tue, 8 Nov 2011 07:19:06 +0000
Subject: [PATCH] irq: fix build error due to missing irq_pm_syscore_resume

irq_pm_syscore_resume is only available iff CONFIG_PM_SLEEP (kernel/irq/pm.o is
only built if this is true). Move the definition (and the dummy definition)
under that umbrella.

Introduced by the backport of upstream 9bab0b7fbaceec47d32db51cd9e59c82fb071f5a
as 0f12a6ad9fa3a03f2bcee36c9cb704821e244c40.

Signed-off-by: Ian Campbell <ian.campbell@...rix.com>
Reported-by: Christoph Biedl <linux-kernel.bfrz@...chmal.in-ulm.de>
Reported-by: Antoine Martin <antoine@...afix.co.uk>
---
 include/linux/interrupt.h |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/include/linux/interrupt.h b/include/linux/interrupt.h
index c7e1aa5..a65ef60 100644
--- a/include/linux/interrupt.h
+++ b/include/linux/interrupt.h
@@ -199,16 +199,16 @@ extern void enable_irq(unsigned int irq);
 #ifdef CONFIG_GENERIC_HARDIRQS
 extern void suspend_device_irqs(void);
 extern void resume_device_irqs(void);
-extern void irq_pm_syscore_resume(void);
 #ifdef CONFIG_PM_SLEEP
 extern int check_wakeup_irqs(void);
+extern void irq_pm_syscore_resume(void);
 #else
 static inline int check_wakeup_irqs(void) { return 0; }
+static inline void irq_pm_syscore_resume(void) { };
 #endif
 #else
 static inline void suspend_device_irqs(void) { };
 static inline void resume_device_irqs(void) { };
-static inline void irq_pm_syscore_resume(void) { };
 static inline int check_wakeup_irqs(void) { return 0; }
 #endif
 
-- 
1.7.2.5



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