On Sat, 2006-04-15 at 08:37 -0400, Jimi Xenidis wrote:
> On Apr 14, 2006, at 11:36 AM, Hollis Blanchard wrote:
> >>>> +/* opt_nohv: If true, hardware has no hypervisor (HV) mode or
> >>>> nohv boot
> >>>> + * parameter was specified.
> >>>> + */
> >>>> +int opt_nohv = 0;
> >
> > Don't need to explicitly initialize this.
>
> Hollis, this is global processor state. We don;t track processor
> capabilities blobally yet, untill we do we need this. I don't think
> that we should expect boot_of.c to init it.
>
> >
> >>>> +boolean_param("nohv", opt_nohv);
> >>>> +
> >>>> int opt_earlygdb = 0;
> >
> > (Or this; it's not your code but if you submit a patch I'd happily
> > apply.)
>
> No, I explicitly want this, I'd like to be able to force "nohv" mode
> on the command line for testing and profiling purposes.
I think you're missing my point: I'm talking about initializing opt_nohv
and opt_earlygdb. These are global variables, and so they will land in
the .bss section.
--
Hollis Blanchard
IBM Linux Technology Center
_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ppc-devel
|
Home