On 07/31/2012 10:51 AM, Dong Xu Wang wrote: > This is the implementation code for add-cow file format. Because image_file > might be very huge, then we can't read entire bitmap into memory, we must use > a cache. Since qcow-cache.c has implemted cache code, we can create our cache s/implemted/implemented/ > code based on it. Just wondering if Paolo's HBitmap code for drive-mirror might be a more efficient way to implement your caching. > > Signed-off-by: Dong Xu Wang > --- > block/Makefile.objs | 1 + > block/add-cow-cache.c | 206 +++++++++++++++++ > block/add-cow.c | 599 +++++++++++++++++++++++++++++++++++++++++++++++++ > block/add-cow.h | 101 +++++++++ > block_int.h | 2 + Rather than adding a new implementation for code duplication, can you refactor the existing implementation to be reusable, and update qcow-cache.c to call into the common refactored code? -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org