Re: [XenPPC] [PATCH] add machine info to /proc/cpuinfo & change platform to "Xen"

[Jerone Young <jyoung5@xxxxxxxxxx>]

Ouch forgot to remove CHRP. Will send a new patch.

On Mon, 2007-03-12 at 18:04 -0500, Jerone Young wrote:
> The output of /proc/cpuinfo from this patch looks like the following:
> 
> [root@leaf7 ~]# cat /proc/cpuinfo 
> processor       : 0
> cpu             : PPC970, altivec supported
> clock           : 1600.000000MHz
> revision        : 2.2 (pvr 0039 0202)
> 
> processor       : 1
> cpu             : PPC970, altivec supported
> clock           : 1600.000000MHz
> revision        : 2.2 (pvr 0039 0202)
> 
> timebase        : 199838713
> platform        : Xen
> machine         : CHRP IBM,8842-21X
> 
> 
> Looks nice doesn't it :-) notice you see the machine type... ready for
> the tree.
> 
> 
> 
> On Thu, 2007-03-01 at 17:09 -0600, Hollis Blanchard wrote:
> > On Thu, 2007-03-01 at 15:27 -0600, Jerone Young wrote:
> > > This patch adds machine information to /proc/cpuinfo as well as
> > > changes the platform for "Xen-Maple" to Xen.
> > 
> > Jimi, this is the patch I was talking about.
> > 
> > Jerone, can you please paste cpuinfo from a system with this patch
> > applied? Also, see below.
> > 
> > > diff -r 3e512eaf2a6f arch/powerpc/platforms/xen/setup.c
> > > --- a/arch/powerpc/platforms/xen/setup.c        Thu Mar 01 10:27:39
> > > 2007 -0600
> > > +++ b/arch/powerpc/platforms/xen/setup.c        Thu Mar 01 14:27:23
> > > 2007 -0600
> > > @@ -39,6 +39,18 @@ extern struct machdep_calls mach_maple_m
> > >  extern struct machdep_calls mach_maple_md;
> > >  extern void maple_pci_init(void);
> > > 
> > > +static void xen_show_cpuinfo(struct seq_file *m)
> > > +{
> > > +       struct device_node *root;
> > > +       const char *model = "";
> > > +
> > > +       root = of_find_node_by_path("/");
> > > +       if (root)
> > > +               model = get_property(root, "model", NULL);
> > > +       seq_printf(m, "machine\t\t: CHRP %s\n", model);
> > > +       of_node_put(root);
> > > +}
> > 
> > Why CHRP? I don't think that should be there (think about PA Semi),
> > except *maybe* if the system actually is CHRP.
> > 
> > >  int direct_remap_pfn_range(struct vm_area_struct *vma,
> > >                 unsigned long address, 
> > >                 unsigned long mfn,
> > > @@ -272,9 +284,10 @@ void xen_machine_crash_shutdown(struct p
> > >  #endif
> > > 
> > >  define_machine(xen) {
> > > -       .name                   = "Xen-Maple",
> > > +       .name                   = "Xen",
> > >         .probe                  = xen_probe,
> > >         .setup_arch             = xen_setup_arch,
> > > +       .show_cpuinfo           = xen_show_cpuinfo,
> > >         .init_early             = xen_init_early,
> > >  #ifdef CONFIG_SHARE_MPIC
> > >         .init_IRQ               = xen_maple_init_IRQ, 
> > 
> 
> 
> _______________________________________________
> Xen-ppc-devel mailing list
> Xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
> http://lists.xensource.com/xen-ppc-devel


_______________________________________________
Xen-ppc-devel mailing list
Xen-ppc-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-ppc-devel