* [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header @ 2024-03-19 15:39 Daniel P. Berrangé 2024-03-19 16:06 ` Markus Armbruster 0 siblings, 1 reply; 5+ messages in thread From: Daniel P. Berrangé @ 2024-03-19 15:39 UTC (permalink / raw) To: qemu-devel; +Cc: Eric Blake, Daniel P. Berrangé, Markus Armbruster The 'detached-header' field in QCryptoBlockCreateOptionsLUKS was left over from earlier patch iterations. Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> --- qapi/crypto.json | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/qapi/crypto.json b/qapi/crypto.json index 931c88e688..ad9e3d0297 100644 --- a/qapi/crypto.json +++ b/qapi/crypto.json @@ -226,8 +226,6 @@ # @iter-time: number of milliseconds to spend in PBKDF passphrase # processing. Currently defaults to 2000. (since 2.8) # -# @detached-header: create a detached LUKS header. (since 9.0) -# # Since: 2.6 ## { 'struct': 'QCryptoBlockCreateOptionsLUKS', @@ -237,8 +235,7 @@ '*ivgen-alg': 'QCryptoIVGenAlgorithm', '*ivgen-hash-alg': 'QCryptoHashAlgorithm', '*hash-alg': 'QCryptoHashAlgorithm', - '*iter-time': 'int', - '*detached-header': 'bool'}} + '*iter-time': 'int' }} ## # @QCryptoBlockOpenOptions: -- 2.43.0 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header 2024-03-19 15:39 [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header Daniel P. Berrangé @ 2024-03-19 16:06 ` Markus Armbruster 2024-07-22 13:01 ` Daniel P. Berrangé 0 siblings, 1 reply; 5+ messages in thread From: Markus Armbruster @ 2024-03-19 16:06 UTC (permalink / raw) To: Daniel P. Berrangé; +Cc: qemu-devel, Eric Blake Daniel P. Berrangé <berrange@redhat.com> writes: > The 'detached-header' field in QCryptoBlockCreateOptionsLUKS > was left over from earlier patch iterations. > > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> > --- > qapi/crypto.json | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/qapi/crypto.json b/qapi/crypto.json > index 931c88e688..ad9e3d0297 100644 > --- a/qapi/crypto.json > +++ b/qapi/crypto.json > @@ -226,8 +226,6 @@ > # @iter-time: number of milliseconds to spend in PBKDF passphrase > # processing. Currently defaults to 2000. (since 2.8) > # > -# @detached-header: create a detached LUKS header. (since 9.0) > -# > # Since: 2.6 > ## > { 'struct': 'QCryptoBlockCreateOptionsLUKS', > @@ -237,8 +235,7 @@ > '*ivgen-alg': 'QCryptoIVGenAlgorithm', > '*ivgen-hash-alg': 'QCryptoHashAlgorithm', > '*hash-alg': 'QCryptoHashAlgorithm', > - '*iter-time': 'int', > - '*detached-header': 'bool'}} > + '*iter-time': 'int' }} > > ## > # @QCryptoBlockOpenOptions: Glad you caught this in time. Acked-by: Markus Armbruster <armbru@redhat.com> ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header 2024-03-19 16:06 ` Markus Armbruster @ 2024-07-22 13:01 ` Daniel P. Berrangé 2024-07-22 14:07 ` Markus Armbruster 0 siblings, 1 reply; 5+ messages in thread From: Daniel P. Berrangé @ 2024-07-22 13:01 UTC (permalink / raw) To: Markus Armbruster; +Cc: qemu-devel, Eric Blake On Tue, Mar 19, 2024 at 05:06:07PM +0100, Markus Armbruster wrote: > Daniel P. Berrangé <berrange@redhat.com> writes: > > > The 'detached-header' field in QCryptoBlockCreateOptionsLUKS > > was left over from earlier patch iterations. > > > > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> > > --- > > qapi/crypto.json | 5 +---- > > 1 file changed, 1 insertion(+), 4 deletions(-) > > > > diff --git a/qapi/crypto.json b/qapi/crypto.json > > index 931c88e688..ad9e3d0297 100644 > > --- a/qapi/crypto.json > > +++ b/qapi/crypto.json > > @@ -226,8 +226,6 @@ > > # @iter-time: number of milliseconds to spend in PBKDF passphrase > > # processing. Currently defaults to 2000. (since 2.8) > > # > > -# @detached-header: create a detached LUKS header. (since 9.0) > > -# > > # Since: 2.6 > > ## > > { 'struct': 'QCryptoBlockCreateOptionsLUKS', > > @@ -237,8 +235,7 @@ > > '*ivgen-alg': 'QCryptoIVGenAlgorithm', > > '*ivgen-hash-alg': 'QCryptoHashAlgorithm', > > '*hash-alg': 'QCryptoHashAlgorithm', > > - '*iter-time': 'int', > > - '*detached-header': 'bool'}} > > + '*iter-time': 'int' }} > > > > ## > > # @QCryptoBlockOpenOptions: > > Glad you caught this in time. > > Acked-by: Markus Armbruster <armbru@redhat.com> Urgh, it appears this patch got lost and was not in fact included in 9.0. What's your opinion on removing it from 9.1 with no deprecation period ? Functionally it did absolutely nothing even if used, so removing it at most would cause an incorrect field to now be rejected. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header 2024-07-22 13:01 ` Daniel P. Berrangé @ 2024-07-22 14:07 ` Markus Armbruster 2024-07-22 14:14 ` Daniel P. Berrangé 0 siblings, 1 reply; 5+ messages in thread From: Markus Armbruster @ 2024-07-22 14:07 UTC (permalink / raw) To: Daniel P. Berrangé; +Cc: qemu-devel, Eric Blake Daniel P. Berrangé <berrange@redhat.com> writes: > On Tue, Mar 19, 2024 at 05:06:07PM +0100, Markus Armbruster wrote: >> Daniel P. Berrangé <berrange@redhat.com> writes: >> >> > The 'detached-header' field in QCryptoBlockCreateOptionsLUKS >> > was left over from earlier patch iterations. >> > >> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> >> > --- >> > qapi/crypto.json | 5 +---- >> > 1 file changed, 1 insertion(+), 4 deletions(-) >> > >> > diff --git a/qapi/crypto.json b/qapi/crypto.json >> > index 931c88e688..ad9e3d0297 100644 >> > --- a/qapi/crypto.json >> > +++ b/qapi/crypto.json >> > @@ -226,8 +226,6 @@ >> > # @iter-time: number of milliseconds to spend in PBKDF passphrase >> > # processing. Currently defaults to 2000. (since 2.8) >> > # >> > -# @detached-header: create a detached LUKS header. (since 9.0) >> > -# >> > # Since: 2.6 >> > ## >> > { 'struct': 'QCryptoBlockCreateOptionsLUKS', >> > @@ -237,8 +235,7 @@ >> > '*ivgen-alg': 'QCryptoIVGenAlgorithm', >> > '*ivgen-hash-alg': 'QCryptoHashAlgorithm', >> > '*hash-alg': 'QCryptoHashAlgorithm', >> > - '*iter-time': 'int', >> > - '*detached-header': 'bool'}} >> > + '*iter-time': 'int' }} >> > >> > ## >> > # @QCryptoBlockOpenOptions: >> >> Glad you caught this in time. >> >> Acked-by: Markus Armbruster <armbru@redhat.com> > > Urgh, it appears this patch got lost and was not in fact included in > 9.0. What's your opinion on removing it from 9.1 with no deprecation > period ? Functionally it did absolutely nothing even if used, so > removing it at most would cause an incorrect field to now be rejected. Doc section "Deprecated features": In general features are intended to be supported indefinitely once introduced into QEMU. In the event that a feature needs to be removed, it will be listed in this section. The feature will remain functional for the release in which it was deprecated and one further release. After these two releases, the feature is liable to be removed. Deprecated features may also generate warnings on the console when QEMU starts up, or if activated via a monitor command, however, this is not a mandatory requirement. Note this talks about *features*. The software industry has a longstanding habit of declaring bugs features. By removing @detached-header without a deprecation period, we effectively do the opposite. The opposite of something bad must be good, right? ;-) Seriously, it's a judgement call. I'm happy to defer to your judgement here. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header 2024-07-22 14:07 ` Markus Armbruster @ 2024-07-22 14:14 ` Daniel P. Berrangé 0 siblings, 0 replies; 5+ messages in thread From: Daniel P. Berrangé @ 2024-07-22 14:14 UTC (permalink / raw) To: Markus Armbruster; +Cc: qemu-devel, Eric Blake On Mon, Jul 22, 2024 at 04:07:28PM +0200, Markus Armbruster wrote: > Daniel P. Berrangé <berrange@redhat.com> writes: > > > On Tue, Mar 19, 2024 at 05:06:07PM +0100, Markus Armbruster wrote: > >> Daniel P. Berrangé <berrange@redhat.com> writes: > >> > >> > The 'detached-header' field in QCryptoBlockCreateOptionsLUKS > >> > was left over from earlier patch iterations. > >> > > >> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com> > >> > --- > >> > qapi/crypto.json | 5 +---- > >> > 1 file changed, 1 insertion(+), 4 deletions(-) > >> > > >> > diff --git a/qapi/crypto.json b/qapi/crypto.json > >> > index 931c88e688..ad9e3d0297 100644 > >> > --- a/qapi/crypto.json > >> > +++ b/qapi/crypto.json > >> > @@ -226,8 +226,6 @@ > >> > # @iter-time: number of milliseconds to spend in PBKDF passphrase > >> > # processing. Currently defaults to 2000. (since 2.8) > >> > # > >> > -# @detached-header: create a detached LUKS header. (since 9.0) > >> > -# > >> > # Since: 2.6 > >> > ## > >> > { 'struct': 'QCryptoBlockCreateOptionsLUKS', > >> > @@ -237,8 +235,7 @@ > >> > '*ivgen-alg': 'QCryptoIVGenAlgorithm', > >> > '*ivgen-hash-alg': 'QCryptoHashAlgorithm', > >> > '*hash-alg': 'QCryptoHashAlgorithm', > >> > - '*iter-time': 'int', > >> > - '*detached-header': 'bool'}} > >> > + '*iter-time': 'int' }} > >> > > >> > ## > >> > # @QCryptoBlockOpenOptions: > >> > >> Glad you caught this in time. > >> > >> Acked-by: Markus Armbruster <armbru@redhat.com> > > > > Urgh, it appears this patch got lost and was not in fact included in > > 9.0. What's your opinion on removing it from 9.1 with no deprecation > > period ? Functionally it did absolutely nothing even if used, so > > removing it at most would cause an incorrect field to now be rejected. > > Doc section "Deprecated features": > > In general features are intended to be supported indefinitely once > introduced into QEMU. In the event that a feature needs to be removed, > it will be listed in this section. The feature will remain functional for the > release in which it was deprecated and one further release. After these two > releases, the feature is liable to be removed. Deprecated features may also > generate warnings on the console when QEMU starts up, or if activated via a > monitor command, however, this is not a mandatory requirement. > > Note this talks about *features*. > > The software industry has a longstanding habit of declaring bugs > features. By removing @detached-header without a deprecation period, we > effectively do the opposite. The opposite of something bad must be > good, right? ;-) > > Seriously, it's a judgement call. I'm happy to defer to your judgement > here. Ok, I'm going to include this in 9.1. The chances of something already using the new 'detached header' feature from 9.0 is small. Out of that small set, the chances of something setting the (useless) 'detached-header' QAPI field is even smaller. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :| ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-07-22 14:15 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-03-19 15:39 [PATCH for-9.0] qapi: drop unused QCryptoBlockCreateOptionsLUKS.detached-header Daniel P. Berrangé 2024-03-19 16:06 ` Markus Armbruster 2024-07-22 13:01 ` Daniel P. Berrangé 2024-07-22 14:07 ` Markus Armbruster 2024-07-22 14:14 ` Daniel P. Berrangé
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).