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

[Xen-devel] [PATCH] ioemu: Merge qcow2 fixes from qemu


  • To: xen-devel@xxxxxxxxxxxxxxxxxxx
  • From: Kevin Wolf <kwolf@xxxxxxx>
  • Date: Thu, 27 Mar 2008 13:15:41 +0100
  • Delivery-date: Thu, 27 Mar 2008 05:20:31 -0700
  • List-id: Xen developer discussion <xen-devel.lists.xensource.com>

The following two fixes by Jürgen Keil have been in qemu for a year now.
They are still missing in ioemu.

[Qemu-devel] PATCH: qcow2 image corruption
http://lists.gnu.org/archive/html/qemu-devel/2007-03/msg00773.html

[Qemu-devel] [PATCH] qcow2: release refcount table clusters of the old
table, after growing the refcount table
http://lists.gnu.org/archive/html/qemu-devel/2007-04/msg00043.html

Signed-off-by: Kevin Wolf <kwolf@xxxxxxx>
diff -r 81147306041a tools/ioemu/block-qcow2.c
--- a/tools/ioemu/block-qcow2.c Tue Mar 25 14:37:43 2008
+++ b/tools/ioemu/block-qcow2.c Thu Mar 27 13:59:33 2008
@@ -1886,6 +1886,8 @@
     int64_t table_offset;
     uint64_t data64;
     uint32_t data32;
+    int old_table_size;
+    int64_t old_table_offset;
 
     if (min_size <= s->refcount_table_size)
         return 0;
@@ -1931,10 +1933,14 @@
                     &data32, sizeof(data32)) != sizeof(data32))
         goto fail;
     qemu_free(s->refcount_table);
+    old_table_offset = s->refcount_table_offset;
+    old_table_size = s->refcount_table_size;
     s->refcount_table = new_table;
     s->refcount_table_size = new_table_size;
+    s->refcount_table_offset = table_offset;
 
     update_refcount(bs, table_offset, new_table_size2, 1);
+    free_clusters(bs, old_table_offset, old_table_size * sizeof(uint64_t));
     return 0;
  fail:
     free_clusters(bs, table_offset, new_table_size2);
_______________________________________________
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®.