On Sat, 04/22 10:43, Ashijeet Acharya wrote: > Changes in v4: > - fix commit message in patch 1 (fam) > - drop size_to_clusters() function (fam) > - fix grammatical errors in function documentations (fam) > - factor out metadata loading coding in a separate patch (patch 4) (fam) > - rename vmdk_alloc_cluster_offset() to vmdk_alloc_clusters() (fam) > - break patch 4(in v3) into separate patches (patch 3 and 8) (fam) > - rename extent_size to extent_end (fam) > - use QEMU_ALIGN_UP instead of vmdk_align_offset. (fam) > - drop next and simply do m_data = m_data->next (fam)
I still don't like the m_data linked list but as this is a huge performance improvement, I think we can merge this once the few bugs are addressed, then clean up things on top. Thanks, Fam