> > On 02/07/2017 01:59 AM, Gonglei (Arei) wrote: > > Hi, > > > >> From: Michael S. Tsirkin [mailto:m...@redhat.com] > >> 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 > Sure. Many comments should be addressed.
Thanks, -Gonglei