[Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Zhang Haoyu
Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1 table between big-endian and host style. Signed-off-by: Zhang Haoyu zhan...@sangfor.com Reviewed-by: Max Reitz mre...@redhat.com --- v3 - v4: - convert local L1 table to host-style before copy it back to

Re: [Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Gonglei
On 2014/10/22 19:45, Zhang Haoyu wrote: Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1 table between big-endian and host style. Signed-off-by: Zhang Haoyu zhan...@sangfor.com Reviewed-by: Max Reitz mre...@redhat.com --- v3 - v4: - convert

Re: [Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Max Reitz
On 2014-10-22 at 13:59, Gonglei wrote: On 2014/10/22 19:45, Zhang Haoyu wrote: Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1 table between big-endian and host style. Signed-off-by: Zhang Haoyu zhan...@sangfor.com Reviewed-by: Max Reitz

Re: [Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Max Reitz
On 2014-10-22 at 14:01, Max Reitz wrote: On 2014-10-22 at 13:59, Gonglei wrote: On 2014/10/22 19:45, Zhang Haoyu wrote: Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1 table between big-endian and host style. Signed-off-by: Zhang Haoyu

Re: [Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Gonglei
On 2014/10/22 20:02, Max Reitz wrote: On 2014-10-22 at 14:01, Max Reitz wrote: On 2014-10-22 at 13:59, Gonglei wrote: On 2014/10/22 19:45, Zhang Haoyu wrote: Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1 table between big-endian and host style.

Re: [Qemu-devel] [PATCH v4] snapshot: use local variable to bdrv_pwrite_sync L1 table

2014-10-22 Thread Max Reitz
On 2014-10-22 at 14:21, Gonglei wrote: On 2014/10/22 20:02, Max Reitz wrote: On 2014-10-22 at 14:01, Max Reitz wrote: On 2014-10-22 at 13:59, Gonglei wrote: On 2014/10/22 19:45, Zhang Haoyu wrote: Use local variable to bdrv_pwrite_sync L1 table, needless to make conversion of cached L1