From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:37647) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1br5ts-00058u-A1 for qemu-devel@nongnu.org; Mon, 03 Oct 2016 12:15:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1br5tn-0005d4-HX for qemu-devel@nongnu.org; Mon, 03 Oct 2016 12:15:31 -0400 Received: from mx1.redhat.com ([209.132.183.28]:60498) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1br5tn-0005cf-7T for qemu-devel@nongnu.org; Mon, 03 Oct 2016 12:15:27 -0400 Date: Mon, 3 Oct 2016 17:15:18 +0100 From: "Daniel P. Berrange" Message-ID: <20161003161518.GM13491@redhat.com> Reply-To: "Daniel P. Berrange" References: <1475051152-400276-1-git-send-email-arei.gonglei@huawei.com> <1475051152-400276-2-git-send-email-arei.gonglei@huawei.com> <20161003161048.GU10245@stefanha-x1.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20161003161048.GU10245@stefanha-x1.localdomain> Subject: Re: [Qemu-devel] [PATCH v4 01/13] cryptodev: introduce cryptodev backend interface List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Hajnoczi Cc: Gonglei , qemu-devel@nongnu.org, virtio-dev@lists.oasis-open.org, luonengjun@huawei.com, mst@redhat.com, pbonzini@redhat.com, weidong.huang@huawei.com, wu.wubin@huawei.com, mike.caraman@nxp.com, agraf@suse.de, xin.zeng@intel.com, claudio.fontana@huawei.com, nmorey@kalray.eu, vincent.jardin@6wind.com, jianjay.zhou@huawei.com, hanweidong@huawei.com, peter.huangpeng@huawei.com On Mon, Oct 03, 2016 at 05:10:48PM +0100, Stefan Hajnoczi wrote: > On Wed, Sep 28, 2016 at 04:25:40PM +0800, Gonglei wrote: > > diff --git a/backends/cryptodev.c b/backends/cryptodev.c > > new file mode 100644 > > index 0000000..a15904b > > --- /dev/null > > +++ b/backends/cryptodev.c > > @@ -0,0 +1,175 @@ > > +/* > > + * QEMU Crypto Device Implement > > s/Implement/Implementation/ > > > diff --git a/include/sysemu/cryptodev.h b/include/sysemu/cryptodev.h > > new file mode 100644 > > index 0000000..cc3c3be > > --- /dev/null > > +++ b/include/sysemu/cryptodev.h > > @@ -0,0 +1,145 @@ > > +/* > > + * QEMU Crypto Device Implement > > s/Implement/Implementation/ > > > + * > > + * Copyright (c) 2016 HUAWEI TECHNOLOGIES CO., LTD. > > + * > > + * Authors: > > + * Gonglei > > + * > > + * This library is free software; you can redistribute it and/or > > + * modify it under the terms of the GNU Lesser General Public > > + * License as published by the Free Software Foundation; either > > + * version 2 of the License, or (at your option) any later version. > > + * > > + * This library is distributed in the hope that it will be useful, > > + * but WITHOUT ANY WARRANTY; without even the implied warranty of > > + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU > > + * Lesser General Public License for more details. > > + * > > + * You should have received a copy of the GNU Lesser General Public > > + * License along with this library; if not, see . > > + * > > + */ > > +#ifndef QCRYPTO_CRYPTODEV_H > > +#define QCRYPTO_CRYPTODEV_H > > + > > +#include "qom/object.h" > > +#include "qemu-common.h" > > + > > +/** > > + * QCryptoCryptoDevBackend: > > + * > > + * The QCryptoCryptoDevBackend object is an interface > > + * for different cryptodev backends, which provides crypto > > + * operation wrapper. > > I suggest calling it CryptoDevBackend since that's shorter and doesn't > repeat any information. I'm not sure why "QCrypto" is necessary. I suggested that naming when we had it under the crypto/ directory, since that's the standard for code there. We've moved this into the backends/ directory now, so we don't need that name prefix anymore. Regards, Daniel -- |: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :| |: http://libvirt.org -o- http://virt-manager.org :| |: http://entangle-photo.org -o- http://search.cpan.org/~danberr/ :|