> This is great, because on PowerPC we're looking to drop struct
> start_info entirely. However, I notice there are still a fair number of
> 'xen_start_info' references in drivers/xen. Are there more of these
> patches coming Steven, or since HVM guests don't use those drivers
> you're not pursuing them?
I wasn't planning on doing so since, as you say, none of the others
are relevant to HVM guests.

> > @@ -1014,8 +1017,13 @@ static int __init xenbus_probe_init(void
> >             if (xsd_port_intf)
> >                     xsd_port_intf->read_proc = xsd_port_read;
> >  #endif
> > -   } else
> > +   } else {
> >             xenstored_ready = 1;
> > +           xen_store_evtchn = xen_start_info->store_evtchn;
> > +           xen_store_mfn = xen_start_info->store_mfn;
> > +   }
> > +
> > +   xen_store_interface = mfn_to_virt(xen_store_mfn);
> >  
> >     /* Initialize the interface to xenstore. */
> >     err = xs_init();
> Could we abstract this part more please? I think calling into an
> arch-provided xen_get_store_mfn() function would be good. (And of course
> the same for store_evtchn and the console_* equivalents.)
This'd probably reduce the number of #ifdef's in the PV-on-HVM patches
as well, so I'd be happy to see it.

Steven.

Attachment: signature.asc
Description: Digital signature

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

Reply via email to