linux-kernel - Re: 2.6.22-rc4-mm1

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]

Date:	Wed, 6 Jun 2007 10:23:28 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Andy Whitcroft <apw@...dowen.org>
Cc:	Cedric Le Goater <clg@...ibm.com>, linux-kernel@...r.kernel.org
Subject: Re: 2.6.22-rc4-mm1

On Wed, 06 Jun 2007 14:48:52 +0100 Andy Whitcroft <apw@...dowen.org> wrote:

> Cedric Le Goater wrote:
> > Andrew,
> > 
> > here's a minor fix for x86_64,
> > 
> > C.
> > 
> > 
> > when CONFIG_PM=y and CONFIG_SOFTWARE_SUSPEND=n, 
> > 
> >   CC      arch/x86_64/kernel/e820.o
> > /home/legoater/linux/2.6.22-rc4-mm1/arch/x86_64/kernel/e820.c: In function `e820_mark_nosave_regions':
> > /home/legoater/linux/2.6.22-rc4-mm1/arch/x86_64/kernel/e820.c:248: warning: implicit declaration of function `register_nosave_region'
> > 
> > Signed-off-by: Cedric Le Goater <clg@...ibm.com>
> > ---
> >  include/linux/suspend.h |    8 ++++++++
> >  1 file changed, 8 insertions(+)
> > 
> > Index: 2.6.22-rc4-mm1/include/linux/suspend.h
> > ===================================================================
> > --- 2.6.22-rc4-mm1.orig/include/linux/suspend.h
> > +++ 2.6.22-rc4-mm1/include/linux/suspend.h
> > @@ -74,6 +74,14 @@
> >  extern void hibernation_set_ops(struct hibernation_ops *ops);
> >  extern int hibernate(void);
> >  #else /* CONFIG_SOFTWARE_SUSPEND */
> > +static inline void register_nosave_region(unsigned long b, unsigned long e)
> > +{
> > +}
> > +
> > +static inline void register_nosave_region_late(unsigned long b, unsigned long e)
> > +{
> > +}
> > +
> >  static inline int swsusp_page_is_forbidden(struct page *p) { return 0; }
> >  static inline void swsusp_set_page_free(struct page *p) {}
> >  static inline void swsusp_unset_page_free(struct page *p) {}
> > -
> 
> 
> Looks like this is the cause of a bunch of compile failures across our
> testing.  Will shove this through tko.
> 

Yeah.  I put a different fix into the hot-fixes directory, thanks.

The powerpc failure is nasty - probably a toolchain stupidity.  I reverted
slub-use-ilog2-instead-of-series-of-constant-comparisons.patch which
_should_ fix it, but really the problem is triggered by ilog2().

-
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/