On Wed, 06 Jun 2007 14:48:52 +0100 Andy Whitcroft <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> > > --- > > 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 [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/