On Mon, Jun 07, 2004 at 09:41:49AM -0700, Dave Hansen wrote: > On Mon, 2004-06-07 at 07:08, Ashok Raj wrote:plugging. > > + if (systemcfg->platform == PLATFORM_PSERIES_LPAR) > > + cpu->no_control=1; > > That condition is also inverted. Should read: > > if (systemcfg->platform != PLATFORM_PSERIES_LPAR) > cpu->no_control = 1; Good catch...I knew i mentioned untested :-) fixed patch attached... Cheers, ashok