[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 01/13] qcrypto: add generic infrastructure for crypto options
From: |
Maxim Levitsky |
Subject: |
Re: [PATCH 01/13] qcrypto: add generic infrastructure for crypto options amendment |
Date: |
Wed, 29 Jan 2020 19:49:29 +0200 |
On Tue, 2020-01-28 at 16:59 +0000, Daniel P. Berrangé wrote:
> On Tue, Jan 14, 2020 at 09:33:38PM +0200, Maxim Levitsky wrote:
> > This will be used first to implement luks keyslot management.
> >
> > block_crypto_amend_opts_init will be used to convert
> > qemu-img cmdline to QCryptoBlockAmendOptions
> >
> > Signed-off-by: Maxim Levitsky <address@hidden>
> > ---
> > block/crypto.c | 17 +++++++++++++++++
> > block/crypto.h | 3 +++
> > crypto/block.c | 31 +++++++++++++++++++++++++++++++
> > crypto/blockpriv.h | 8 ++++++++
> > include/crypto/block.h | 22 ++++++++++++++++++++++
> > qapi/crypto.json | 16 ++++++++++++++++
> > 6 files changed, 97 insertions(+)
>
> Reviewed-by: Daniel P. Berrangé <address@hidden>
>
>
> > diff --git a/qapi/crypto.json b/qapi/crypto.json
> > index b2a4cff683..9faebd03d4 100644
> > --- a/qapi/crypto.json
> > +++ b/qapi/crypto.json
> > @@ -309,3 +309,19 @@
> > 'base': 'QCryptoBlockInfoBase',
> > 'discriminator': 'format',
> > 'data': { 'luks': 'QCryptoBlockInfoLUKS' } }
> > +
> > +
> > +
> > +##
> > +# @QCryptoBlockAmendOptions:
> > +#
> > +# The options that are available for all encryption formats
> > +# when initializing a new volume
>
> minor point, the comment needs updating
Done.
Thanks for the review!
Best regards,
Maxim Levitsky
>
> > +#
> > +# Since: 5.0
> > +##
> > +{ 'union': 'QCryptoBlockAmendOptions',
> > + 'base': 'QCryptoBlockOptionsBase',
> > + 'discriminator': 'format',
> > + 'data': {
> > + } }
>
> Regards,
> Daniel
- [PATCH 00/13] LUKS: encryption slot management using amend interface, Maxim Levitsky, 2020/01/14
- [PATCH 01/13] qcrypto: add generic infrastructure for crypto options amendment, Maxim Levitsky, 2020/01/14
- [PATCH 02/13] qcrypto-luks: implement encryption key management, Maxim Levitsky, 2020/01/14
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Markus Armbruster, 2020/01/21
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Maxim Levitsky, 2020/01/21
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Daniel P . Berrangé, 2020/01/28
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Daniel P . Berrangé, 2020/01/28
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Maxim Levitsky, 2020/01/29
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Kevin Wolf, 2020/01/30
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Daniel P . Berrangé, 2020/01/30
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Kevin Wolf, 2020/01/30
- Re: [PATCH 02/13] qcrypto-luks: implement encryption key management, Daniel P . Berrangé, 2020/01/30