Re: [kvm-ppc-devel] [PATCH] [v2] Add idle wait support for 44x

[Arnd Bergmann <arnd@arndb.de>]

On Friday 04 April 2008, Jerone Young wrote:
> > > +int __init ppc44x_idle_init(void)
> > > +{
> > > +   void *func = modes[current_mode].entry;
> > > +   struct device_node *node;
> > > +
> > > +   node = of_find_node_by_path("/hypervisor");
> > > +   if (node) {
> > > +           /* if we find /hypervisor node is in device tree,
> > > +              set idle mode to wait */
> > > +           func = &ppc44x_idle; /* wait */
> > > +           of_node_put(node);
> > > +   }
> > > +
> > > +   ppc_md.power_save = func;
> > > +   return 0;
> > > +}
> > > +
> > > +arch_initcall(ppc44x_idle_init);
> > 
> > IIRC, this would over-ride the idle_param() below, is that the  
> > intended behavior?
> 
> Yes. At the moment if it detects a hypervisor in the kernel tree it
> overrides what the command line says.
> 

This is unusual behavior. Normally, we try to come up with reasonable
defaults and give the user the chance to override it.

	Arnd <><

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Register now and save $200. Hurry, offer ends at 11:59 p.m., 
Monday, April 7! Use priority code J8TLD2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
kvm-ppc-devel mailing list
kvm-ppc-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-ppc-devel