[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device spec
From: |
Michael S. Tsirkin |
Subject: |
Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification |
Date: |
Wed, 3 Aug 2016 18:46:37 +0300 |
On Wed, Aug 03, 2016 at 03:17:06PM +0000, Zeng, Xin wrote:
>
>
> > -----Original Message-----
> > From: Michael S. Tsirkin [mailto:address@hidden
> > Sent: Wednesday, August 3, 2016 10:30 PM
> > To: Gonglei <address@hidden>
> > Cc: address@hidden; address@hidden;
> > address@hidden; address@hidden;
> > address@hidden; address@hidden;
> > address@hidden; address@hidden;
> > address@hidden; address@hidden; Zeng, Xin
> > <address@hidden>; Keating, Brian A <address@hidden>; Ma,
> > Liang J <address@hidden>; Griffin, John <address@hidden>;
> > address@hidden; address@hidden
> > Subject: Re: [PATCH v6 0/2] virtio-crypto: virtio crypto device
> > specification
> >
> > On Mon, Aug 01, 2016 at 05:20:19PM +0800, Gonglei wrote:
> > > This is the specification (version 6) about a new virtio crypto device.
> > > After a big reconstruction, the spec (symmetric algos) is near to
> > > stabilize.
> > > This version fix some problems of formating and return value, etc.
> > >
> > > If you have any comments, please let me know, thanks :)
> >
> > You might want to open a jira tracker in oasis jira to add this.
> >
>
> Could we hold on for a few days? I am still preparing patchset
> based on this one.
Of course. Please see
process for voting and issue tracking for virtio spec
basically ideally the final proposal includes the # of the
issue it addesses, but as long as it's just material for
discussion, there's no rush to track it.
> >
> > > CC: Michael S. Tsirkin <address@hidden>
> > > CC: Cornelia Huck <address@hidden>
> > > CC: Stefan Hajnoczi <address@hidden>
> > > CC: Lingli Deng <address@hidden>
> > > CC: Jani Kokkonen <address@hidden>
> > > CC: Ola Liljedahl <address@hidden>
> > > CC: Varun Sethi <address@hidden>
> > > CC: Zeng Xin <address@hidden>
> > > CC: Keating Brian <address@hidden>
> > > CC: Ma Liang J <address@hidden>
> > > CC: Griffin John <address@hidden>
> > > CC: Hanweidong <address@hidden>
> > >
> > > Changes from v6:
> > > - add conformance clauses for virtio crypto device. [Michael]
> > > - drop VIRTIO_CRYPTO_S_STARTED. [Michael]
> > > - fix some characters problems. [Stefan]
> > > - add a MAC algorithm, named VIRTIO_CRYPTO_MAC_ZUC_EIA3. [Zeng Xin]
> > > - add the fourth return code, named VIRTIO_CRYPTO_OP_INVSESS used
> > > for invalid session id when executing crypto operations.
> > > - drop some gpu stuff forgot to delete. [Michael]
> > > - convert tab to space all over the content.
> > >
> > > Changes from v4:
> > > - introduce crypto services into virtio crypto device. The services
> > > currently defined are CIPHER, MAC, HASH, AEAD, KDF, ASYM, PRIMITIVE.
> > > - define a unified crypto request format that is consisted of
> > > general header + service specific request, Where 'general header' is
> > > for
> > all
> > > crypto request, 'service specific request' is composed of
> > > operation parameter + input data + output data in generally.
> > > operation parameter is algorithm-specific parameters,
> > > input data is the data should be operated ,
> > > output data is the "operation result + result buffer".
> > > - redefine the algorithms and structure based on above crypto services.
> > > - rearrange the title and subtitle
> > > - Only support CIPHER, MAC, HASH and AEAD crypto services, and Xin will
> > > focus KDF, ASYM and PRIMITIVE services.
> > > - Some other corresponding fixes.
> > > - Make a formal patch using tex type.
> > >
> > > This version is a big reconstruction based on Zeng, Xin' comments, thanks
> > > a
> > lot.
> > >
> > > Changes from v3:
> > > - Don't use enum is the spec but macros in specific structures. [Michael
> > > &
> > Stefan]
> > > - Add two complete structures for session creation and closing, so that
> > > the spec is clear on how to lay out the request. [Stefan]
> > > - Definite the crypto operation request with assigned structure, in this
> > > way,
> > > each data request only occupies *one entry* of the Vring descriptor
> > > table,
> > > which *improves* the *throughput* of data transferring.
> > >
> > > Changes from v2:
> > > - Reserve virtio device ID 20 for crypto device. [Cornelia]
> > > - Drop all feature bits, those capabilities are offered by the device
> > > all the
> > time. [Stefan & Cornelia]
> > > - Add a new section 1.4.2 for driver requirements. [Stefan]
> > > - Use definite type definition instead of enum type in some structure.
> > [Stefan]
> > > - Add virtio_crypto_cipher_alg definition. [Stefan]
> > > - Add a "Device requirements" section as using MUST. [Stefan]
> > > - Some grammar nits fixes and typo fixes. [Stefan & Cornelia]
> > > - Add one VIRTIO_CRYPTO_S_STARTED status for the driver as the flag of
> > virtio-crypto device started and can work now.
> > >
> > > Great thanks for Stefan and Cornelia!
> > >
> > > Changes from v1:
> > > - Drop the feature bit definition for each algorithm, and using config
> > > space
> > instead [Cornelia]
> > > - Add multiqueue support and add corresponding feature bit
> > > - Update Encryption process and header definition
> > > - Add session operation process and add corresponding header description
> > > - Other better description in order to fit for virtio spec [Michael]
> > > - Some other trivial fixes.
> > >
> > >
> > > Gonglei (2):
> > > virtio-crypto: Add virtio crypto device specification
> > > virtio-crypto: Add conformance clauses
> > >
> > > conformance.tex | 30 +++
> > > content.tex | 2 +
> > > virtio-crypto.tex | 793
> > ++++++++++++++++++++++++++++++++++++++++++++++++++++++
> > > 3 files changed, 825 insertions(+)
> > > create mode 100644 virtio-crypto.tex
> > >
> > > --
> > > 1.7.12.4
> > >
- Re: [Qemu-devel] [PATCH v6 1/2] virtio-crypto: Add virtio crypto device specification, (continued)
Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Michael S. Tsirkin, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Cornelia Huck, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Michael S. Tsirkin, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Cornelia Huck, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Cornelia Huck, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Michael S. Tsirkin, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Michael S. Tsirkin, 2016/08/04
Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification, Zeng, Xin, 2016/08/03
- Re: [Qemu-devel] [PATCH v6 0/2] virtio-crypto: virtio crypto device specification,
Michael S. Tsirkin <=