Hi Markus, This seems to be ignored...
On 8/13/24 09:51, zhenwei pi wrote: > On 8/12/24 14:14, Philippe Mathieu-Daudé wrote: >> On 12/8/24 03:42, zhenwei pi wrote: >>> QCryptodevBackendServiceType was introduced by >>> bc304a6442e (cryptodev: Introduce server type in QAPI). However there >>> is a lack of member description. Thanks to Markus for pointing out >>> this. >>> >>> Signed-off-by: zhenwei pi >>> --- >>> qapi/cryptodev.json | 10 ++++++++++ >>> 1 file changed, 10 insertions(+) >>> >>> diff --git a/qapi/cryptodev.json b/qapi/cryptodev.json >>> index 68289f4984..9622c6d92b 100644 >>> --- a/qapi/cryptodev.json >>> +++ b/qapi/cryptodev.json >>> @@ -28,6 +28,16 @@ >>> # >>> # The supported service types of a crypto device. >>> # >>> +# @cipher: Symmetric Key Cipher service >>> +# >>> +# @hash: Hash service >>> +# >>> +# @mac: Message Authentication Codes service >>> +# >>> +# @aead: Authenticated Encryption with Associated Data service >>> +# >>> +# @akcipher: Asymmetric Key Cipher service >> >> Can we drop all trailing "service"s? >> > > I copied these descriptions from virtio-crypto spec. Removing them also > is fine to me. > >>> +# >>> # Since: 8.0 >>> ## >>> { 'enum': 'QCryptodevBackendServiceType', >> > > -- > zhenwei pi > -- zhenwei pi