Re: [PATCH] fix 3.3-testing c/s 18434

[Jim Fehlig <jfehlig@novell.com>]

[-- Attachment #1: Type: text/plain, Size: 771 bytes --]

Keir Fraser wrote:
> On 8/10/08 18:47, "Jim Fehlig" <jfehlig@novell.com> wrote:
>
>   
>> c/s 18434 in xen-3.3-testing has a hunk that changes gatherDom() to
>> _gatherDom() in tools/python/xen/xend/image.py.  There is no such
>> method, thus simple "xm list" fails
>>
>> jfehlig4: # xm li
>> Error: _gatherDom
>> Usage: xm list [options] [Domain, ...]
>> ..
>>
>> Trivial fix attached.  BTW, I don't see this patch in xen-unstable.  Why
>> is this only in xen3.3-testing?
>>     
>
> It is in xen-unstable, as c/s 18564 (which is part of
> xen-3.3-testing:18434).

Missed that, sorry.

I've also found that this c/s causes /local/domain/0/backend/{vkbd, vfb,
console} paths to leak :(.  Patch attached.

Thanks,
Jim

    Signed-off-by: Jim Fehlig <jfehlig@novell.com>



[-- Attachment #2: xenstore-leak.patch --]
[-- Type: text/x-patch, Size: 572 bytes --]

diff -r 8cc28137805d tools/python/xen/xend/server/DevController.py
--- a/tools/python/xen/xend/server/DevController.py	Thu Oct 09 11:00:51 2008 +0100
+++ b/tools/python/xen/xend/server/DevController.py	Thu Oct 09 16:00:08 2008 -0600
@@ -257,7 +257,7 @@ class DevController:
 
         if force:
             frontpath = self.frontendPath(dev)
-            backpath = self.readVm(devid, "backend")
+            backpath = xstransact.Read(frontpath, "backend")
             if backpath:
                 xstransact.Remove(backpath)
             xstransact.Remove(frontpath)

[-- Attachment #3: Type: text/plain, Size: 138 bytes --]

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