thr3ads.net - Xen devel - [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable) [Jul 2010]

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

eXeC001er

2010-Jul-11 08:16 UTC

[Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

Current source-code work if use following configuration:
config example:    disk=[''tap:vhd:/path/../disk.img,xvda,w'']

but not work if use:
config example:   
disk=[''tap:tapdisk:vhd:/path/../disk.img,xvda,w'']
Error during start DomU:    File ''vhd:/path/.../disk.img''
doesn''t exist.

this patch fix this bug.

Signed-off-by: eXeC001er <execooler@gmail.com>

Thanks.



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

Boris Derzhavets

2010-Jul-11 08:57 UTC

head link

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

After patching xen-unstable (changeset:   21769:a672af698bc3)
Rebuilt and reinstalled xen&tools.
Changed  "tap2:tapdisk:vhd" to "tap:vhd"

root@ServerLSX:~/NexentaStor-Community-3.0.2# xm create -c 
nexentastor-3.0.2-xen.cfg
Using config file "./nexentastor-3.0.2-xen.cfg".
root@ServerLSX:~/NexentaStor-Community-3.0.2# Error: Device 51952 not connected

root@ServerLSX:/usr/local/NexentaStor-Community-3.0.2# xm create -c 
nexentastor-3.0.2-xen.cfg
Using config file "./nexentastor-3.0.2-xen.cfg".
Error: Device /dev/xvdp (51952, tap2) is already connected.

Boris.

--- On Sun, 7/11/10, eXeC001er <execooler@gmail.com> wrote:

From: eXeC001er
 <execooler@gmail.com>
Subject: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk
configuration (xen-unstable)
To: "Xen-devel" <xen-devel@lists.xensource.com>
Date: Sunday, July 11, 2010, 4:16 AM

Current source-code work if use following configuration:config example:  
 disk=[''tap:vhd:/path/../disk.img,xvda,w'']


but not work if use:config example:  
 disk=[''tap:tapdisk:vhd:/path/../disk.img,xvda,w'']


Error during start DomU:    File ''vhd:/path/.../disk.img''
doesn''t exist.


this patch fix this bug.


Signed-off-by: eXeC001er <execooler@gmail.com>


Thanks.


-----Inline Attachment Follows-----

_______________________________________________
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

eXeC001er

2010-Jul-11 09:16 UTC

head link

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

2010/7/11 Boris Derzhavets <bderzhavets@yahoo.com>
> After patching xen-unstable (changeset:   21769:a672af698bc3)
> Rebuilt and reinstalled xen&tools.
> Changed  "tap2:tapdisk:vhd" to "tap:vhd"
>you can use:  "tap:vhd" or "tap:tapdisk:vhd"
>
> root@ServerLSX:~/NexentaStor-Community-3.0.2# xm create -c
> nexentastor-3.0.2-xen.cfg
> Using config file "./nexentastor-3.0.2-xen.cfg".
> root@ServerLSX:~/NexentaStor-Community-3.0.2# Error: Device 51952 not
> connected
>
> root@ServerLSX:/usr/local/NexentaStor-Community-3.0.2# xm create -c
> nexentastor-3.0.2-xen.cfg
> Using config file "./nexentastor-3.0.2-xen.cfg".
> Error: Device /dev/xvdp (51952, tap2) is already connected.
>
Yes, for fix these issues need to use other patch (I will send it during 30
min.)

>
> Boris.
>
> --- On *Sun, 7/11/10, eXeC001er <execooler@gmail.com>* wrote:
>
>
> From: eXeC001er <execooler@gmail.com>
> Subject: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk
> configuration (xen-unstable)
> To: "Xen-devel" <xen-devel@lists.xensource.com>
> Date: Sunday, July 11, 2010, 4:16 AM
>
>
> Current source-code work if use following configuration:
> config example:   
disk=[''tap:vhd:/path/../disk.img,xvda,w'']
>
> but not work if use:
> config example:   
disk=[''tap:tapdisk:vhd:/path/../disk.img,xvda,w'']
> Error during start DomU:    File ''vhd:/path/.../disk.img''
doesn''t exist.
>
> this patch fix this bug.
>
> Signed-off-by: eXeC001er <execooler@gmail.com>
>
> Thanks.
>
> -----Inline Attachment Follows-----
>
> _______________________________________________
> 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

Boris Derzhavets

2010-Jul-11 13:06 UTC

head link

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

Patching :-

diff -r c514d69c71d8 tools/python/xen/xend/XendBootloader.py
--- a/tools/python/xen/xend/XendBootloader.py    Fri Jun 04 11:39:06 2010 +0100
+++ b/tools/python/xen/xend/XendBootloader.py    Sun Jun 06 23:44:20 2010 +0400
@@ -38,10 +38,15 @@
         msg = "Bootloader isn''t executable"
         log.error(msg)
         raise VmError(msg)
-    if not os.access(disk, os.R_OK):
-        msg = "Disk isn''t accessible"
-        log.error(msg)
-        raise VmError(msg)
+    attempt = 0
+    while True:
+        if not os.access(disk, os.R_OK) and attempt > 3:
+            msg = "Disk isn''t accessible"
+            log.error(msg)
+            raise VmError(msg)
+        else:
+            break
+        attempt = attempt + 1

doesn''t help any longer under xen-unstable vs 4.0.1.

Boris.

--- On Sun, 7/11/10, eXeC001er <execooler@gmail.com> wrote:

From: eXeC001er <execooler@gmail.com>
Subject: Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk 
configuration (xen-unstable)
To: "Boris Derzhavets" <bderzhavets@yahoo.com>
Cc: "Xen-devel" <xen-devel@lists.xensource.com>
Date: Sunday, July 11, 2010, 5:16 AM



2010/7/11 Boris Derzhavets <bderzhavets@yahoo.com>

After patching xen-unstable (changeset:   21769:a672af698bc3)
Rebuilt and reinstalled xen&tools.
Changed  "tap2:tapdisk:vhd" to "tap:vhd"

you can use:  "tap:vhd" or "tap:tapdisk:vhd"
root@ServerLSX:~/NexentaStor-Community-3.0.2# xm create -c 
nexentastor-3.0.2-xen.cfg

Using config file "./nexentastor-3.0.2-xen.cfg".
root@ServerLSX:~/NexentaStor-Community-3.0.2# Error: Device 51952 not connected

root@ServerLSX:/usr/local/NexentaStor-Community-3.0.2# xm create -c 
nexentastor-3.0.2-xen.cfg

Using config file "./nexentastor-3.0.2-xen.cfg".
Error: Device /dev/xvdp (51952, tap2) is already connected.

Yes, for fix these issues need to use other patch (I will send it during 30
min.)
 

Boris.

--- On Sun, 7/11/10, eXeC001er <execooler@gmail.com> wrote:

From: eXeC001er
 <execooler@gmail.com>
Subject: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk
configuration (xen-unstable)
To: "Xen-devel" <xen-devel@lists.xensource.com>

Date: Sunday, July 11, 2010, 4:16 AM

Current source-code work if use following configuration:config example:  
 disk=[''tap:vhd:/path/../disk.img,xvda,w'']



but not work if use:config example:  
 disk=[''tap:tapdisk:vhd:/path/../disk.img,xvda,w'']



Error during start DomU:    File ''vhd:/path/.../disk.img''
doesn''t exist.



this patch fix this bug.



Signed-off-by: eXeC001er <execooler@gmail.com>



Thanks.



-----Inline Attachment Follows-----

_______________________________________________
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 - Jul 2010 - [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

[Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)

Re: [Xen-devel] [PATCH] blktap/blktap2: Fix for incorrect parse disk configuration (xen-unstable)