[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RESEND 2/6] crypto: add AEAD algorithms framewor
From: |
Longpeng (Mike) |
Subject: |
Re: [Qemu-devel] [PATCH RESEND 2/6] crypto: add AEAD algorithms framework |
Date: |
Sat, 7 Jan 2017 15:33:38 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20120327 Thunderbird/11.0.1 |
Hi Daniel,
On 2017/1/6 19:28, Daniel P. Berrange wrote:
> On Thu, Jan 05, 2017 at 08:49:37AM +0800, Longpeng(Mike) wrote:
>> This patch introduce AEAD algorithms framework.
>
> So AEAD is essentially just encryption with extra metadata on
> input and output stages. As such I don't think this should be
> implemented via new objects in QEMU.
>
> Instead, we should add the extra APIs to the existing
> cipher.{c,h} file.
>
> You've only implemented AES support here, so the code duplication
> is not obvious, but if we were to extend AEAD to other algorithms
> we'd see alot of duplication with the way you've done things.
>
> So....
>
All right, I will rework this recently. :)
>> ......
>> - 'data': ['ecb', 'cbc', 'xts', 'ctr']}
>> + 'data': ['ccm', 'gcm', 'ecb', 'cbc', 'xts', 'ctr']}
>
> This is fine.
>
> Regards,
> Daniel
--
Regards,
Longpeng(Mike)
- [Qemu-devel] [PATCH RESEND 0/6] crypto:add AEAD algorithms support, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 4/6] crypto: implement gcrypt-backed AEAD algorithms, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 1/6] configure: add CONFIG_GCRYPT/NETTLE_AEAD item, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 5/6] crypto: implement other common funcs for AEAD algorithms, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 2/6] crypto: add AEAD algorithms framework, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 3/6] crypto: implement nettle-backed AEAD algorithms, Longpeng(Mike), 2017/01/04
- [Qemu-devel] [PATCH RESEND 6/6] crypto: add AEAD algorithms testcases, Longpeng(Mike), 2017/01/04