Re: [Qemu-devel] [PATCH v6] qcow2: Buffer L1 table in snapshot refcount update

2014-11-28 Thread Stefan Hajnoczi
On Tue, Nov 11, 2014 at 04:27:51PM +0100, Max Reitz wrote: > From: Zhang Haoyu > > Buffer the active L1 table in qcow2_update_snapshot_refcount() in order > to prevent in-place conversion of the L1 table buffer in the > BDRVQcowState to big endian and back, which would lead to data > corruption i

Re: [Qemu-devel] [PATCH v6] qcow2: Buffer L1 table in snapshot refcount update

2014-11-27 Thread Max Reitz
On 2014-11-20 at 15:32, Max Reitz wrote: On 2014-11-11 at 16:27, Max Reitz wrote: From: Zhang Haoyu Buffer the active L1 table in qcow2_update_snapshot_refcount() in order to prevent in-place conversion of the L1 table buffer in the BDRVQcowState to big endian and back, which would lead to dat

Re: [Qemu-devel] [PATCH v6] qcow2: Buffer L1 table in snapshot refcount update

2014-11-20 Thread Max Reitz
On 2014-11-11 at 16:27, Max Reitz wrote: From: Zhang Haoyu Buffer the active L1 table in qcow2_update_snapshot_refcount() in order to prevent in-place conversion of the L1 table buffer in the BDRVQcowState to big endian and back, which would lead to data corruption if that buffer was accessed c

[Qemu-devel] [PATCH v6] qcow2: Buffer L1 table in snapshot refcount update

2014-11-11 Thread Max Reitz
From: Zhang Haoyu Buffer the active L1 table in qcow2_update_snapshot_refcount() in order to prevent in-place conversion of the L1 table buffer in the BDRVQcowState to big endian and back, which would lead to data corruption if that buffer was accessed concurrently. This should not happen but bet