Hi! > > > Why are we calling driver suspend routines in these ? This is _not_ a > > > good idea ! On various machines, the mecanisms for shutting down are > > > quite different from suspend/resume, and current drivers have too many > > > bugs to make that safe. I keep getting all sort of reports of machines > > > not shutting down anymore. > > > > For example, my Centrino laptop will restart instead of power down with > > -mm kernels. > > > > To "fix" this I can either: > > - unplug power. Shutdown works when on battery power. > > - attach an external USB hard disk => power down always works. > > - remove device_suspend(PMSG_SUSPEND) => power down always works.- > > Yes, this is just one of the gazillion setup that got broken by this > change. Drivers already have a shutdown() callback anyway, and if we > want to re-use the suspend one, then we need to define some sane > parameter, not "fake" a system suspend.
I'd like to get rid of shutdown callback. Having two copies of code (one in callback, one in suspend) is ugly. Pavel -- teflon -- maybe it is a trademark, but it should not be. - 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/