[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Xen-devel] Re: [Xen-changelog] [XENBUS] Avoid direct use of xen_start_info.



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?

On Mon, 2006-08-14 at 19:20 +0000, Xen patchbot-unstable wrote:
> # HG changeset patch
> # User kfraser@xxxxxxxxxxxxxxxxxxxxx
> # Node ID e061f14161b052b3287a1e41222cfa292a194fc2
> # Parent  9519445d9e9d7b8bd48fcd93d1bba485508ddbff
> [XENBUS] Avoid direct use of xen_start_info. It's unevailable when
> building as a separate driver for an HVM guest.
> Signed-off-by: Steven Smith <ssmith@xxxxxxxxxxxxx>
> Signed-off-by: Keir Fraser <keir@xxxxxxxxxxxxx>

...

> @@ -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.)

-- 
Hollis Blanchard
IBM Linux Technology Center


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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.