From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55965) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1c4V2D-0007DM-Vq for qemu-devel@nongnu.org; Wed, 09 Nov 2016 10:43:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1c4V2A-0003sT-T6 for qemu-devel@nongnu.org; Wed, 09 Nov 2016 10:43:34 -0500 Received: from mx1.redhat.com ([209.132.183.28]:51156) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1c4V2A-0003sH-Nj for qemu-devel@nongnu.org; Wed, 09 Nov 2016 10:43:30 -0500 Date: Wed, 9 Nov 2016 17:43:24 +0200 From: "Michael S. Tsirkin" Message-ID: <20161109173948-mutt-send-email-mst@kernel.org> References: <1477632235-72496-1-git-send-email-arei.gonglei@huawei.com> <1477632235-72496-2-git-send-email-arei.gonglei@huawei.com> <989f16d4-1854-24b5-cf79-dbdd7f4255b0@linux.vnet.ibm.com> <33183CC9F5247A488A2544077AF19020B0421668@SZXEMA503-MBS.china.huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <33183CC9F5247A488A2544077AF19020B0421668@SZXEMA503-MBS.china.huawei.com> Subject: Re: [Qemu-devel] [PATCH v13 1/2] virtio-crypto: Add virtio crypto device specification List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Gonglei (Arei)" Cc: Halil Pasic , "qemu-devel@nongnu.org" , "virtio-dev@lists.oasis-open.org" , "Huangweidong (C)" , "john.griffin@intel.com" , Shiqing Fan , "Zhoujian (jay, Euler)" , "Varun.Sethi@freescale.com" , "denglingli@chinamobile.com" , "arei.gonglei@hotmail.com" , "Hanweidong (Randy)" , "agraf@suse.de" , "nmorey@kalray.eu" , "vincent.jardin@6wind.com" , "Ola.Liljedahl@arm.com" , Luonengjun , "xin.zeng@intel.com" , "Huangpeng (Peter)" , "liang.j.ma@intel.com" , "stefanha@redhat.com" , "cornelia.huck@de.ibm.com" , Jani Kokkonen , "brian.a.keating@intel.com" , Claudio Fontana , "mike.caraman@nxp.com" , "Wubin (H)" On Wed, Nov 09, 2016 at 01:11:20AM +0000, Gonglei (Arei) wrote: > Nope, Actually I kept those description here is because I wanted to represent each packet > Intuitionally, otherwise I don't know how to explain them only occupy one entry in desc table > by indirect table method. whether to use the indirect method should not be up to device. I don't see where does the spec require indirect method, ANY_LAYOUT is implied so spec should not dictate that IMO. > So I changed the code completely as Stefan's suggestion and > revised the spec a little. > > This just is a representative of the realization so that the people can easily understand what > the virtio crypto request's components. It isn't completely same with the code. > For virtio-scsi device, the struct virtio_scsi_req_cmd also used this way IIUR. > > Thanks, > -Gonglei