On Tue 10 Mar 2020 03:04:47 AM CET, John Snow wrote:
>> static int handle_copied(BlockDriverState *bs, uint64_t guest_offset,
>> - uint64_t *host_offset, uint64_t *bytes, QCowL2Meta **m)
>> + uint64_t *host_offset, uint64_t *bytes)
>> {
>> BDRVQcow2State *s = bs->opaque;
>> int l2_index;
>> @@ -1567,7 +1567,7 @@ again:
>> /*
>> * 2. Count contiguous COPIED clusters.
>> */
>> - ret = handle_copied(bs, start, &cluster_offset, &cur_bytes, m);
>> + ret = handle_copied(bs, start, &cluster_offset, &cur_bytes);
>> if (ret < 0) {
>> return ret;
>> } else if (ret) {
>>
>
> Seems OK to me -- it is definitely unused.
> (Is _that_ a problem? For qcow2 maintainers to know.)
It is unused now, but I'm using it in my subcluster allocation patches:
https://lists.gnu.org/archive/html/qemu-block/2019-12/msg00588.html
(I expect to send v4 soon, probably this week)
Berto
