Re: [Xen-devel] Re: pvusb drivers for pvops 2.6.32.x kernel

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2011-01-03 at 11:41 +0000, Pasi KÃrkkÃinen wrote:
> Hello,
> 
> Jeremy: See the included patch. If it's OK it'd be nice
> to get it into xen/stable-2.6.32.x branch.
> 
> Thanks Nathanael!

Yes thanks!

I think the correct path for this functionality is to first get it
accepted into the upstream kernel by working with the USB subsystem
maintainer+list, fixing the issue arising from their review etc.

The stuff necessary to get the frontend upstream has been upstream for
ages. For the backend basic dom0 boot support is in 2.6.37 and the
generic scaffolding for backends is currently in linux-next (via
Konrad's tree) and is intended to be in the next merge window.

I think the backend just looks like a regular USB driver to the host
system so it probably belongs in drivers/usb/<something>/xen-usbback/
and not drivers/xen. (I'm not sure what the <something> should be,
perhaps "misc").

IIRC when the PV USB drivers were originally submitted there was no
toolstack support included. Has this since been implemented? Seems to be
missing from libxl at least. Is someone looking into this?

Ian.

> 
> -- Pasi
> 
> ----- Forwarded message from Nathanael Rensen <nathanael@xxxxxxxxxxxxxxxx> 
> -----
> 
> From: Nathanael Rensen <nathanael@xxxxxxxxxxxxxxxx>
> To: Pasi KÃrkkÃinen <pasik@xxxxxx>
> Cc: n_iwamatsu@xxxxxxxxxxxxxx
> Date: Mon, 3 Jan 2011 19:32:23 +0800
> Subject: Re: pvusb for pvops
> 
> On 1 January 2011 23:19, Nathanael Rensen <nathanael@xxxxxxxxxxxxxxxx> wrote:
> > On 1 January 2011 22:43, Pasi KÃrkkÃinen <pasik@xxxxxx> wrote:
> >> On Sat, Jan 01, 2011 at 03:10:58AM +0800, Nathanael Rensen wrote:
> >>> Hi Noboru,
> >>>
> >>> In case it is helpful ...
> >>>
> >>> I've attached a diff that adds the pvusb drivers to the debian squeeze
> >>> 2.6.32 pvops kernel.
> >>> So far I've only tested with a USB drive on xen 4.0.1 between debian
> >>> squeeze pvops dom0 and
> >>> debian squeeze pvops domU.
> >>>
> >>> The diff is based on the SLES SP1 diffs posted here:
> >>>   http://code.google.com/p/gentoo-xen-kernel/downloads/list
> >>> It almost applies cleanly to xen/stable-2.6.32.x except for the patch to
> >>> include/xen/interface/io/ring.h which is not required for 
> >>> xen/stable-2.6.32.x.
> >>> I haven't tested the xen/stable-2.6.32.x kernel though.
> >>>
> >>
> >> Great, thanks!
> >> Is it OK to forward this email to xen-devel mailinglist?
> >>
> >> Btw did you check from novell's kernel git tree if there
> >> has been fixes to the pvusb code after that patch on
> >> code.google.com was posted?
> >>
> >> -- Pasi
> >
> > For sure, you're welcome to forward it, but maybe hold off a bit because
> > I've refactored the pvusb frontend code to be consistent with the way the
> > other frontend drivers are within the pvops tree. I'll also take a look at
> > Novell's tree to check for updates.
> >
> > I'll do some testing on a kernel built from the pvops stable-2.6.32.x tree
> > and send an updated diff against that tree in the next couple of days.
> >
> > Nathanael
> >
> 
> I've attached an updated diff against the pvops xen/stable-2.6.32.x tree.
> I've tested by attaching an af9015 USB TV tuner to a pvops domu from
> a pvops dom0 and recording a few hours of TV.
> 
> I also tried to attach various USB devices to a WinXP domu with GPLPV
> drivers, but the domu kept crashing. That needs more investigation, but
> I suspect a problem with the GPLPV drivers. I'm keen for that to be fixed,
> but I don't have time immediately to dig into it.
> 
> I took a look at the patches at
> http://gitorious.org/opensuse/kernel-source/trees/SLE11-SP1/patches.xen
> and as far as I can tell, everything that relates to pvusb is included in the
> attached patch.
> 
> Nathanael
> 
> 
> 
> ----- End forwarded message -----



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