Hi Ola,
> -----Original Message----- > From: Ola Liljedahl [mailto:ola.liljed...@arm.com] > Sent: Tuesday, September 13, 2016 7:53 PM > To: Paolo Bonzini; Daniel P. Berrange; Gonglei (Arei) > Cc: qemu-devel@nongnu.org; virtio-...@lists.oasis-open.org; Huangpeng > (Peter); Luonengjun; m...@redhat.com; stefa...@redhat.com; Huangweidong > (C); mike.cara...@nxp.com; ag...@suse.de; xin.z...@intel.com; Claudio > Fontana; nmo...@kalray.eu; vincent.jar...@6wind.com > Subject: Re: [virtio-dev] Re: [PATCH v2 02/15] crypto: introduce crypto queue > handler > > > > > > On 13/09/2016, 12:58, "virtio-...@lists.oasis-open.org on behalf of Paolo > Bonzini" <virtio-...@lists.oasis-open.org on behalf of > pbonz...@redhat.com> wrote: > > > > > > >On 13/09/2016 11:20, Daniel P. Berrange wrote: > >>> > +typedef struct CryptoPacket CryptoPacket; > >>> > +typedef struct CryptoQueue CryptoQueue; > >>> > +typedef struct CryptoPacketBuf CryptoPacketBuf; > >>> > + > >>> > +typedef void (CryptoPacketSent) (CryptoClientState *, int); > >> As previously, I'd expect naming of > >> > >> QCryptoCryptodevPacket > >> QCryptoCryptodevPacketBuf > >> QCryptoCryptodevQueue > >> > > > >Gonglei, > > > >you are copying a lot of code from network backends. > > > >I am not sure why you would need a queue for virtio-crypto rather than a > >direct connection between frontend and backend (and the backend would be > >QEMU crypto APIs, like Daniel suggested). > > What about backends implemented directly in HW? Bypass the middle man. > Make crypto offload meaningful for small size blocks. > As I said in other reply, I only provide interfaces, you can realize them according to different backend cryptodevs. Regards, -Gonglei > < Ola > > > > >Paolo > > > >--------------------------------------------------------------------- > >To unsubscribe, e-mail: virtio-dev-unsubscr...@lists.oasis-open.org > >For additional commands, e-mail: virtio-dev-h...@lists.oasis-open.org > > > > IMPORTANT NOTICE: The contents of this email and any attachments are > confidential and may also be privileged. If you are not the intended > recipient, > please notify the sender immediately and do not disclose the contents to any > other person, use it for any purpose, or store or copy the information in any > medium. Thank you.