[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v19 0/2] virtio-crypto: virtio crypto device spe
From: |
Longpeng (Mike) |
Subject: |
Re: [Qemu-devel] [PATCH v19 0/2] virtio-crypto: virtio crypto device specification |
Date: |
Fri, 8 Sep 2017 14:32:32 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20120327 Thunderbird/11.0.1 |
On 2017/9/8 12:31, Michael S. Tsirkin wrote:
> On Fri, Sep 08, 2017 at 12:22:01PM +0800, Longpeng (Mike) wrote:
>>
>>
>> On 2017/9/8 11:49, Michael S. Tsirkin wrote:
>>
>>> On Fri, Sep 01, 2017 at 08:47:28AM +0800, Longpeng (Mike) wrote:
>>>> Ping...
>>>>
>>>> Stefan, Halil, do you have any suggestion ?
>>>
>>> I do not see this patchset in the virtio list archives.
>>> It could be that youa re posting from an email that is
>>> not a subscriber.
>>>
>>> Please subscribe and repost.
>>>
>>
>>
>> OK, I'll subscribe and repost after I complete the implementation based on
>> the
>> v19 :)
>
> To clarify does not existing device in qemu work according to
> this spec? We need to make the spec compatible with what's
> out there in the field somehow (feature bit?).
Yep, the spec is compatible with the existing device in QEMU by a feature
bit(VIRTIO_CRYPTO_F_MUX_MODE).
>
>>> We'll then have to allow up to 2 weeks for comments.
>>>
>>> Halil, I hope this timing works for you.
>>>
>>>> --
>>>> Regards,
>>>> Longpeng(Mike)
>>>>
>>>> On 2017/8/26 15:53, Longpeng(Mike) wrote:
>>>>
>>>>> Hi guys,
>>>>>
>>>>> I'll work on the virtio-crypto spec with Gonglei together, Because He is
>>>>> so busy on the inner production project.
>>>>>
>>>>> ---
>>>>> v19 -> v18:
>>>>> - fix some typos and grammar fixes [Stefan, Halil]
>>>>> - rename VIRTIO_CRYPTO_F_STATELESS_MODE to VIRTIO_CRYPTO_F_MUX_MODE
>>>>> - describe the VIRTIO_CRYPTO_STATUS in detial. [Halil]
>>>>> - refactor and redescribe the controlq/dataq request's format
>>>>> of mux mode. [Halil]
>>>>> - other small fixes. [Halil]
>>>>>
>>>>> v18 -> v17:
>>>>> - fix many English grammar problems suggested by Stefan, Thanks a lot!
>>>>>
>>>>> v17 -> v16:
>>>>> - Some grammar fixes [Stefan, Halil, Michael]
>>>>> - add a section named "Supported crypto services" in order to explain bit
>>>>> numbers and valuse clearly. [Halil, Cornelia]
>>>>> - avoid word reptition [Halil]
>>>>> - rename non-session mode to stateless mode [Halil]
>>>>> - change descriptions for all elements in struct virtio_crypto_config
>>>>> [Halil]
>>>>> - add Halil as a reviewer in the ackonwledgement part, thanks for his
>>>>> work.
>>>>> - other fixes here and there.
>>>>>
>>>>> Changes since v15:
>>>>> - use feature bits for non-session mode in order to keep compatibility
>>>>> with
>>>>> pre-existing code. [Halil & Michael]
>>>>> - introduce VIRTIO_CRYPTO_F_ NON_SESSION_MODE feature bit to control all
>>>>> other
>>>>> non-session mode feature bits.
>>>>> - fix some typos. [Stefan]
>>>>> - introduce struct virtio_crypto_op_data_req_mux to support both session
>>>>> and non-session based crypto operations and keep compatibility with
>>>>> pre-existing code.
>>>>>
>>>>> Changes since v14:
>>>>> - drop VIRTIO_CRYPTO_S_STARTED status [Halil & Cornelia]
>>>>> - correct a sentence about dataqueue and controlq in the first
>>>>> paragraph.
>>>>> [Halil]
>>>>> - change a MAY to MUST about max_dataqueues. [Halil]
>>>>> - add non-session mode support
>>>>> a) add four features for different crypto services to identify
>>>>> wheather
>>>>> support session mode.
>>>>> b) rewrite some
>>>>>
>>>>> For pervious versions of virtio crypto spec, Pls see:
>>>>>
>>>>> [v18]:
>>>>> https://www.mail-archive.com/address@hidden/msg444897.html
>>>>>
>>>>> [v14]:
>>>>> https://lists.gnu.org/archive/html/qemu-devel/2016-11/msg02212.html
>>>>>
>>>>> [v13]:
>>>>> https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg07348.html
>>>>>
>>>>> For more information, please see:
>>>>> http://qemu-project.org/Features/VirtioCrypto
>>>>>
>>>>> ---
>>>>> Gonglei (2):
>>>>> virtio-crypto: Add virtio crypto device specification
>>>>> virtio-crypto: Add conformance clauses
>>>>>
>>>>> acknowledgements.tex | 3 +
>>>>> conformance.tex | 29 +
>>>>> content.tex | 2 +
>>>>> virtio-crypto.tex | 1470
>>>>> ++++++++++++++++++++++++++++++++++++++++++++++++++
>>>>> 4 files changed, 1504 insertions(+)
>>>>> create mode 100644 virtio-crypto.tex
>>>>>
>>>>
>>>>
>>>> --
>>>> Regards,
>>>> Longpeng(Mike)
>>>
>>> .
>>>
>>
>>
>> --
>> Regards,
>> Longpeng(Mike)
>
> .
>
--
Regards,
Longpeng(Mike)