thr3ads.net - Xen devel - [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests [Dec 2009]

If this information is useful, please help other people find it:
Share via:

James (song wei)

2009-Dec-14 02:53 UTC

[Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

Signed-off-by: James Song Wei  <jsong@novell.com> 
diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
--- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11 09:17:09 2009
+0000
+++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14 10:41:48 2009
+0800
@@ -234,7 +234,7 @@
  
     def op_usb_del(self, op, req):
         self.acceptCommand(req)
-        return req.threadRequest(self.do_usb_add, op, req)
+        return req.threadRequest(self.do_usb_del, op, req)
 
     def do_usb_del(self, _, req):
         return self.xd.domain_usb_add(self.dom.getName(), req)
-- 
View this message in context:
http://old.nabble.com/-PATCH--fix-bug-in-c-s-20332-Add-commands-to-hotplug-usb-devices-to-hvm-guests-tp26772352p26772352.html
Sent from the Xen - Dev mailing list archive at Nabble.com.


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

Pasi Kärkkäinen

2009-Dec-14 08:00 UTC

head link

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

On Sun, Dec 13, 2009 at 06:53:46PM -0800, James (song wei)
wrote:> 
> Signed-off-by: James Song Wei  <jsong@novell.com> 
> diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
> --- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11 09:17:09 2009
> +0000
> +++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14 10:41:48 2009
> +0800
> @@ -234,7 +234,7 @@
>   
>      def op_usb_del(self, op, req):
>          self.acceptCommand(req)
> -        return req.threadRequest(self.do_usb_add, op, req)
> +        return req.threadRequest(self.do_usb_del, op, req)
>  
>      def do_usb_del(self, _, req):
>          return self.xd.domain_usb_add(self.dom.getName(), req)                                     ^^^

do_usb_del() is still using domain_usb_add() ..

-- Pasi


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

James (song wei)

2009-Dec-14 08:22 UTC

head link

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

Sorry, thanks Pasi!
-James

Signed-off-by: James Song Wei  <jsong@novell.com> 
diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
--- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11 09:17:09 2009
+0000
+++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14 16:18:27 2009
+0800
@@ -234,10 +234,10 @@
  
     def op_usb_del(self, op, req):
         self.acceptCommand(req)
-        return req.threadRequest(self.do_usb_add, op, req)
+        return req.threadRequest(self.do_usb_del, op, req)
 
     def do_usb_del(self, _, req):
-        return self.xd.domain_usb_add(self.dom.getName(), req)
+        return self.xd.domain_usb_del(self.dom.getName(), req)
 
     def render_POST(self, req):
         return self.perform(req)


Pasi Kärkkäinen wrote:> 
> On Sun, Dec 13, 2009 at 06:53:46PM -0800, James (song wei) wrote:
>> 
>> Signed-off-by: James Song Wei  <jsong@novell.com> 
>> diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
>> --- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11 09:17:09
2009
>> +0000
>> +++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14 10:41:48
2009
>> +0800
>> @@ -234,7 +234,7 @@
>>   
>>      def op_usb_del(self, op, req):
>>          self.acceptCommand(req)
>> -        return req.threadRequest(self.do_usb_add, op, req)
>> +        return req.threadRequest(self.do_usb_del, op, req)
>>  
>>      def do_usb_del(self, _, req):
>>          return self.xd.domain_usb_add(self.dom.getName(), req)
>                                      ^^^
> 
> do_usb_del() is still using domain_usb_add() ..
> 
> -- Pasi
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
> 
> 
-- 
View this message in context:
http://old.nabble.com/-PATCH--fix-bug-in-c-s-20332-Add-commands-to-hotplug-usb-devices-to-hvm-guests-tp26772352p26774299.html
Sent from the Xen - Dev mailing list archive at Nabble.com.


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

Pasi Kärkkäinen

2009-Dec-14 09:12 UTC

head link

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

On Mon, Dec 14, 2009 at 12:22:37AM -0800, James (song wei)
wrote:> 
> Sorry, thanks Pasi!
> -James
> 
Keir: This patch needs to go to xen-3.4-testing.hg aswell.

-- Pasi
> Signed-off-by: James Song Wei  <jsong@novell.com> 
> diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
> --- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11 09:17:09 2009
> +0000
> +++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14 16:18:27 2009
> +0800
> @@ -234,10 +234,10 @@
>   
>      def op_usb_del(self, op, req):
>          self.acceptCommand(req)
> -        return req.threadRequest(self.do_usb_add, op, req)
> +        return req.threadRequest(self.do_usb_del, op, req)
>  
>      def do_usb_del(self, _, req):
> -        return self.xd.domain_usb_add(self.dom.getName(), req)
> +        return self.xd.domain_usb_del(self.dom.getName(), req)
>  
>      def render_POST(self, req):
>          return self.perform(req)
> 
> 
> Pasi Kärkkäinen wrote:
> > 
> > On Sun, Dec 13, 2009 at 06:53:46PM -0800, James (song wei) wrote:
> >> 
> >> Signed-off-by: James Song Wei  <jsong@novell.com> 
> >> diff -r b928797213ac tools/python/xen/xend/server/SrvDomain.py
> >> --- a/tools/python/xen/xend/server/SrvDomain.py Fri Dec 11
09:17:09 2009
> >> +0000
> >> +++ b/tools/python/xen/xend/server/SrvDomain.py Mon Dec 14
10:41:48 2009
> >> +0800
> >> @@ -234,7 +234,7 @@
> >>   
> >>      def op_usb_del(self, op, req):
> >>          self.acceptCommand(req)
> >> -        return req.threadRequest(self.do_usb_add, op, req)
> >> +        return req.threadRequest(self.do_usb_del, op, req)
> >>  
> >>      def do_usb_del(self, _, req):
> >>          return self.xd.domain_usb_add(self.dom.getName(), req)
> >                                      ^^^
> > 
> > do_usb_del() is still using domain_usb_add() ..
> > 
> > -- Pasi
> > 
> > 
> > _______________________________________________
> > Xen-devel mailing list
> > Xen-devel@lists.xensource.com
> > http://lists.xensource.com/xen-devel
> > 
> > 
> 
> -- 
> View this message in context:
http://old.nabble.com/-PATCH--fix-bug-in-c-s-20332-Add-commands-to-hotplug-usb-devices-to-hvm-guests-tp26772352p26774299.html
> Sent from the Xen - Dev mailing list archive at Nabble.com.
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@lists.xensource.com
> http://lists.xensource.com/xen-devel
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xensource.com
http://lists.xensource.com/xen-devel

Xen devel - Dec 2009 - [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

[Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests

Re: [Xen-devel] [PATCH] fix bug in c/s 20332 Add commands to hotplug usb devices to hvm guests