From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932390AbdCFVl5 (ORCPT ); Mon, 6 Mar 2017 16:41:57 -0500 Received: from userp1040.oracle.com ([156.151.31.81]:31082 "EHLO userp1040.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752726AbdCFVln (ORCPT ); Mon, 6 Mar 2017 16:41:43 -0500 Subject: Re: [PATCH 2/7] xen: introduce the header file for the Xen 9pfs transport protocol To: Stefano Stabellini References: <1488830488-18506-1-git-send-email-sstabellini@kernel.org> <1488830488-18506-2-git-send-email-sstabellini@kernel.org> <20ed41c0-f275-58b6-229f-07f78adf1927@oracle.com> Cc: xen-devel@lists.xenproject.org, linux-kernel@vger.kernel.org, Stefano Stabellini , konrad.wilk@oracle.com, jgross@suse.com From: Boris Ostrovsky Message-ID: <3c6439e1-5331-d7dc-92e8-6665942dfd6d@oracle.com> Date: Mon, 6 Mar 2017 16:42:13 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit X-Source-IP: aserv0022.oracle.com [141.146.126.234] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/06/2017 04:36 PM, Stefano Stabellini wrote: > On Mon, 6 Mar 2017, Boris Ostrovsky wrote: >>> + uint32_t size; >>> + uint8_t id; >>> + uint16_t tag; >> I realize that this is in the spec now and it's probably too late to ask >> this question but wouldn't it be better if id and tag were swapped? No >> need to pack and potentially faster access to tag. > I cannot do anything about it: that struct is defined by the 9pfs > specification (not the Xen spec, the general 9pfs spec). See: Oh, I thought it was Xen-specific (because it's described in 9pfs.markdown). Nevermind then. -boris > > https://www.usenix.org/legacy/event/usenix05/tech/freenix/full_papers/hensbergen/hensbergen.pdf > >>> +} __attribute__((packed)); >>> + >>> +#define XEN_9PFS_RING_ORDER 6 >>> +#define XEN_9PFS_RING_SIZE XEN_FLEX_RING_SIZE(XEN_9PFS_RING_ORDER) >>> +DEFINE_XEN_FLEX_RING_AND_INTF(xen_9pfs); >>> + >>> +#endif From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Ostrovsky Subject: Re: [PATCH 2/7] xen: introduce the header file for the Xen 9pfs transport protocol Date: Mon, 6 Mar 2017 16:42:13 -0500 Message-ID: <3c6439e1-5331-d7dc-92e8-6665942dfd6d@oracle.com> References: <1488830488-18506-1-git-send-email-sstabellini@kernel.org> <1488830488-18506-2-git-send-email-sstabellini@kernel.org> <20ed41c0-f275-58b6-229f-07f78adf1927@oracle.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Received: from mail6.bemta5.messagelabs.com ([195.245.231.135]) by lists.xenproject.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cl0NN-0004sa-4C for xen-devel@lists.xenproject.org; Mon, 06 Mar 2017 21:41:05 +0000 In-Reply-To: List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xen.org Sender: "Xen-devel" To: Stefano Stabellini Cc: Stefano Stabellini , xen-devel@lists.xenproject.org, jgross@suse.com, linux-kernel@vger.kernel.org List-Id: xen-devel@lists.xenproject.org T24gMDMvMDYvMjAxNyAwNDozNiBQTSwgU3RlZmFubyBTdGFiZWxsaW5pIHdyb3RlOgo+IE9uIE1v biwgNiBNYXIgMjAxNywgQm9yaXMgT3N0cm92c2t5IHdyb3RlOgo+Pj4gKwl1aW50MzJfdCBzaXpl Owo+Pj4gKwl1aW50OF90IGlkOwo+Pj4gKwl1aW50MTZfdCB0YWc7Cj4+IEkgcmVhbGl6ZSB0aGF0 IHRoaXMgaXMgaW4gdGhlIHNwZWMgbm93IGFuZCBpdCdzIHByb2JhYmx5IHRvbyBsYXRlIHRvIGFz awo+PiB0aGlzIHF1ZXN0aW9uIGJ1dCB3b3VsZG4ndCBpdCBiZSBiZXR0ZXIgaWYgaWQgYW5kIHRh ZyB3ZXJlIHN3YXBwZWQ/IE5vCj4+IG5lZWQgdG8gcGFjayBhbmQgcG90ZW50aWFsbHkgZmFzdGVy IGFjY2VzcyB0byB0YWcuCj4gSSBjYW5ub3QgZG8gYW55dGhpbmcgYWJvdXQgaXQ6IHRoYXQgc3Ry dWN0IGlzIGRlZmluZWQgYnkgdGhlIDlwZnMKPiBzcGVjaWZpY2F0aW9uIChub3QgdGhlIFhlbiBz cGVjLCB0aGUgZ2VuZXJhbCA5cGZzIHNwZWMpLiBTZWU6CgpPaCwgSSB0aG91Z2h0IGl0IHdhcyBY ZW4tc3BlY2lmaWMgKGJlY2F1c2UgaXQncyBkZXNjcmliZWQgaW4KOXBmcy5tYXJrZG93bikuIE5l dmVybWluZCB0aGVuLgoKLWJvcmlzCgo+Cj4gaHR0cHM6Ly93d3cudXNlbml4Lm9yZy9sZWdhY3kv ZXZlbnQvdXNlbml4MDUvdGVjaC9mcmVlbml4L2Z1bGxfcGFwZXJzL2hlbnNiZXJnZW4vaGVuc2Jl cmdlbi5wZGYKPgo+Pj4gK30gX19hdHRyaWJ1dGVfXygocGFja2VkKSk7Cj4+PiArCj4+PiArI2Rl ZmluZSBYRU5fOVBGU19SSU5HX09SREVSIDYKPj4+ICsjZGVmaW5lIFhFTl85UEZTX1JJTkdfU0la RSAgWEVOX0ZMRVhfUklOR19TSVpFKFhFTl85UEZTX1JJTkdfT1JERVIpCj4+PiArREVGSU5FX1hF Tl9GTEVYX1JJTkdfQU5EX0lOVEYoeGVuXzlwZnMpOwo+Pj4gKwo+Pj4gKyNlbmRpZgoKCl9fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fClhlbi1kZXZlbCBtYWls aW5nIGxpc3QKWGVuLWRldmVsQGxpc3RzLnhlbi5vcmcKaHR0cHM6Ly9saXN0cy54ZW4ub3JnL3hl bi1kZXZlbAo=