Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-17 Thread Maxim Levitsky
On Thu, 2020-05-14 at 14:28 +0200, Max Reitz wrote: > On 10.05.20 15:40, Maxim Levitsky wrote: > > Some options are only useful for creation > > (or hard to be amended, like cluster size for qcow2), while some other > > options are only useful for amend, like upcoming keyslot management > > options

Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-17 Thread Maxim Levitsky
On Fri, 2020-05-15 at 12:24 -0500, Eric Blake wrote: > On 5/15/20 1:22 AM, Max Reitz wrote: > > > > > > > > > > +QCOW_COMMON_OPTIONS, > > > > > +{ /* end of list */ } > > > > > > ...the intended usage is to use the macro name followed by a comma, so > > > including a trailing com

Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-15 Thread Eric Blake
On 5/15/20 1:22 AM, Max Reitz wrote: +    QCOW_COMMON_OPTIONS, +    { /* end of list */ } ...the intended usage is to use the macro name followed by a comma, so including a trailing comma in the macro itself would lead to a syntax error. But why is that the indended usage? Is ther

Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-14 Thread Max Reitz
On 14.05.20 18:10, Eric Blake wrote: > On 5/14/20 7:28 AM, Max Reitz wrote: >> On 10.05.20 15:40, Maxim Levitsky wrote: >>> Some options are only useful for creation >>> (or hard to be amended, like cluster size for qcow2), while some other >>> options are only useful for amend, like upcoming keysl

Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-14 Thread Eric Blake
On 5/14/20 7:28 AM, Max Reitz wrote: On 10.05.20 15:40, Maxim Levitsky wrote: Some options are only useful for creation (or hard to be amended, like cluster size for qcow2), while some other options are only useful for amend, like upcoming keyslot management options for luks +#define QCOW_

Re: [PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-14 Thread Max Reitz
On 10.05.20 15:40, Maxim Levitsky wrote: > Some options are only useful for creation > (or hard to be amended, like cluster size for qcow2), while some other > options are only useful for amend, like upcoming keyslot management > options for luks > > Since currently only qcow2 supports amend, move

[PATCH v6 04/14] block/amend: separate amend and create options for qemu-img

2020-05-10 Thread Maxim Levitsky
Some options are only useful for creation (or hard to be amended, like cluster size for qcow2), while some other options are only useful for amend, like upcoming keyslot management options for luks Since currently only qcow2 supports amend, move all its options to a common macro and then include i