> > +#ifdef CONFIG_XMON_DEFAULT > > +int __initdata xmon_on = 1; > > +#else > > +int __initdata xmon_on = 0; > > +#endif > > This shouldn't be __initdata, should it? It's used at runtime below. Thanks for spotting! New patch on the way. Anton