diff for duplicates of <200804080421.45447.arnd@arndb.de> diff --git a/a/1.txt b/N1/1.txt index 8e0a89e..a1a7f1a 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,41 +1,32 @@ On Friday 04 April 2008, Jerone Young wrote: > > > +int __init ppc44x_idle_init(void) > > > +{ -> > > + void *func = modes[current_mode].entry; -> > > + struct device_node *node; +> > > +=A0=A0=A0void *func =3D modes[current_mode].entry; +> > > +=A0=A0=A0struct 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); -> > > + } +> > > +=A0=A0=A0node =3D of_find_node_by_path("/hypervisor"); +> > > +=A0=A0=A0if (node) { +> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0/* if we find /hypervisor node is i= +n device tree, +> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0 set idle mode to wait */ +> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0func =3D &ppc44x_idle; /* wait */ +> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0of_node_put(node); +> > > +=A0=A0=A0} > > > + -> > > + ppc_md.power_save = func; -> > > + return 0; +> > > +=A0=A0=A0ppc_md.power_save =3D func; +> > > +=A0=A0=A0return 0; > > > +} > > > + > > > +arch_initcall(ppc44x_idle_init); -> > -> > IIRC, this would over-ride the idle_param() below, is that the +> >=20 +> > IIRC, this would over-ride the idle_param() below, is that the =A0 > > intended behavior? -> +>=20 > Yes. At the moment if it detects a hypervisor in the kernel tree it > overrides what the command line says. -> +>=20 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 diff --git a/a/content_digest b/N1/content_digest index 18c5e27..647bf28 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -2,8 +2,8 @@ "ref\0835D8B13-CF90-4013-A172-8243BF93DE52@pobox.com\0" "ref\01207323761.6634.6.camel@thinkpadL\0" "From\0Arnd Bergmann <arnd@arndb.de>\0" - "Subject\0Re: [kvm-ppc-devel] [PATCH] [v2] Add idle wait support for 44x\0" - "Date\0Tue, 08 Apr 2008 02:21:44 +0000\0" + "Subject\0Re: [kvm-ppc-devel] [PATCH] [v2] Add idle wait support for 44x platforms\0" + "Date\0Tue, 8 Apr 2008 04:21:44 +0200\0" "To\0linuxppc-dev@ozlabs.org" " jyoung5@us.ibm.com\0" "Cc\0kvm-ppc-devel@lists.sourceforge.net" @@ -13,43 +13,34 @@ "On Friday 04 April 2008, Jerone Young wrote:\n" "> > > +int __init ppc44x_idle_init(void)\n" "> > > +{\n" - "> > > +\302\240\302\240\302\240void *func = modes[current_mode].entry;\n" - "> > > +\302\240\302\240\302\240struct device_node *node;\n" + "> > > +=A0=A0=A0void *func =3D modes[current_mode].entry;\n" + "> > > +=A0=A0=A0struct device_node *node;\n" "> > > +\n" - "> > > +\302\240\302\240\302\240node = of_find_node_by_path(\"/hypervisor\");\n" - "> > > +\302\240\302\240\302\240if (node) {\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240/* if we find /hypervisor node is in device tree,\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240 \302\240 set idle mode to wait */\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240func = &ppc44x_idle; /* wait */\n" - "> > > +\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240\302\240of_node_put(node);\n" - "> > > +\302\240\302\240\302\240}\n" + "> > > +=A0=A0=A0node =3D of_find_node_by_path(\"/hypervisor\");\n" + "> > > +=A0=A0=A0if (node) {\n" + "> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0/* if we find /hypervisor node is i=\n" + "n device tree,\n" + "> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0 =A0 set idle mode to wait */\n" + "> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0func =3D &ppc44x_idle; /* wait */\n" + "> > > +=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0=A0of_node_put(node);\n" + "> > > +=A0=A0=A0}\n" "> > > +\n" - "> > > +\302\240\302\240\302\240ppc_md.power_save = func;\n" - "> > > +\302\240\302\240\302\240return 0;\n" + "> > > +=A0=A0=A0ppc_md.power_save =3D func;\n" + "> > > +=A0=A0=A0return 0;\n" "> > > +}\n" "> > > +\n" "> > > +arch_initcall(ppc44x_idle_init);\n" - "> > \n" - "> > IIRC, this would over-ride the idle_param() below, is that the \302\240\n" + "> >=20\n" + "> > IIRC, this would over-ride the idle_param() below, is that the =A0\n" "> > intended behavior?\n" - "> \n" + ">=20\n" "> Yes. At the moment if it detects a hypervisor in the kernel tree it\n" "> overrides what the command line says.\n" - "> \n" + ">=20\n" "\n" "This is unusual behavior. Normally, we try to come up with reasonable\n" "defaults and give the user the chance to override it.\n" "\n" - "\tArnd <><\n" - "\n" - "-------------------------------------------------------------------------\n" - "This SF.net email is sponsored by the 2008 JavaOne(SM) Conference \n" - "Register now and save $200. Hurry, offer ends at 11:59 p.m., \n" - "Monday, April 7! Use priority code J8TLD2. \n" - "http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone\n" - "_______________________________________________\n" - "kvm-ppc-devel mailing list\n" - "kvm-ppc-devel@lists.sourceforge.net\n" - https://lists.sourceforge.net/lists/listinfo/kvm-ppc-devel + "\tArnd <><" -8b6bcbe1bc751a5a3801847bf5ed0fa52aaa40ff3ec3bbee54ac4c2c109b46cb +86949170530eb1cee2edbf010aa781aa97776690ecaf15f06288b8ddc71c1678
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.