[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v16 1/2] virtio-crypto: Add virtio crypto device
From: |
Halil Pasic |
Subject: |
Re: [Qemu-devel] [PATCH v16 1/2] virtio-crypto: Add virtio crypto device specification |
Date: |
Wed, 8 Feb 2017 15:27:36 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 |
On 02/07/2017 01:59 AM, Gonglei (Arei) wrote:
> Hi,
>
>> From: Michael S. Tsirkin [mailto:address@hidden
>> Sent: Tuesday, February 07, 2017 2:20 AM
>> Subject: Re: [Qemu-devel] [PATCH v16 1/2] virtio-crypto: Add virtio crypto
>> device specification
>>
>> On Mon, Feb 06, 2017 at 03:46:25PM +0000, Stefan Hajnoczi wrote:
>>> On Mon, Feb 06, 2017 at 01:48:09AM +0000, Gonglei (Arei) wrote:
>>>>>> +\item The device MUST set \field{max_size} to show the maximum size
>> of
>>>>> crypto request the device supports.
>>>>>
>>>>> In bytes?
>>>>>
>>>>>> +\item The device MUST set \field{max_cipher_key_len} to show the
>>>>> maximum length of cipher key if the device supports CIPHER service.
>>>>>
>>>>> In bits or bytes?
>>>>>
>>>>>> +\item The device MUST set \field{max_auth_key_len} to show the
>> maximum
>>>>> length of authenticated key if the device supports MAC service.
>>>>>
>>>>> In bits or bytes?
>>>>>
>>>>
>>>> All lengths in virtio crypto spec are bytes.
>>>
>>> Please move this before the first length field is defined:
>>>
>>> +\begin{note}
>>> +The basic unit of all data length the byte.
>>> +\end{note}
>>
>> And I'd rewrite as
>>
>> All data length fields specify the length in bytes.
>>
>
> Okay, will do it.
>
> BTW if no other objections or comments,
> I'd like to send v17 as the final version to ballot, can I?
>
> Thanks,
> -Gonglei
>
I think it's a bit early for a 'final version' and a 'ballot'.
We need at least one more iteration (IMHO).
Halil
>>>
>>> It should be at the beginning of the spec.
>>
>
>