Re: [Qemu-block] [PATCH v3 18/39] qcow2: Refactor get_cluster_table()

2018-02-01 Thread Max Reitz
On 2018-02-01 11:40, Alberto Garcia wrote: > On Wed 31 Jan 2018 09:11:48 PM CET, Max Reitz wrote: >> On 2018-01-26 15:59, Alberto Garcia wrote: >>> After the previous patch we're now always using l2_load() in >>> get_cluster_table() regardless of whether a new L2 table has to be >>> allocated or no

Re: [Qemu-block] [PATCH v3 18/39] qcow2: Refactor get_cluster_table()

2018-02-01 Thread Alberto Garcia
On Wed 31 Jan 2018 09:11:48 PM CET, Max Reitz wrote: > On 2018-01-26 15:59, Alberto Garcia wrote: >> After the previous patch we're now always using l2_load() in >> get_cluster_table() regardless of whether a new L2 table has to be >> allocated or not. >> >> This patch refactors that part of the c

Re: [Qemu-block] [PATCH v3 18/39] qcow2: Refactor get_cluster_table()

2018-01-31 Thread Max Reitz
On 2018-01-26 15:59, Alberto Garcia wrote: > After the previous patch we're now always using l2_load() in > get_cluster_table() regardless of whether a new L2 table has to be > allocated or not. > > This patch refactors that part of the code to use one single l2_load() > call. > > Signed-off-by:

Re: [Qemu-block] [PATCH v3 18/39] qcow2: Refactor get_cluster_table()

2018-01-26 Thread Eric Blake
On 01/26/2018 08:59 AM, Alberto Garcia wrote: > After the previous patch we're now always using l2_load() in > get_cluster_table() regardless of whether a new L2 table has to be > allocated or not. > > This patch refactors that part of the code to use one single l2_load() > call. > > Signed-off-b

[Qemu-block] [PATCH v3 18/39] qcow2: Refactor get_cluster_table()

2018-01-26 Thread Alberto Garcia
After the previous patch we're now always using l2_load() in get_cluster_table() regardless of whether a new L2 table has to be allocated or not. This patch refactors that part of the code to use one single l2_load() call. Signed-off-by: Alberto Garcia --- block/qcow2-cluster.c | 21 +++