From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Hutchings Subject: Re: [PATCH -V2] Generic name to handle and open by handle syscalls Date: Fri, 26 Mar 2010 00:42:01 +0000 Message-ID: <1269564121.8653.26.camel@localhost> References: <1268932144-14105-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-yWbw0NMviiWYP9IbhImC" Cc: hch@infradead.org, viro@zeniv.linux.org.uk, linux-fsdevel@vger.kernel.org, adilger@sun.com, corbet@lwn.net To: "Aneesh Kumar K.V" Return-path: Received: from shadbolt.e.decadent.org.uk ([88.96.1.126]:40086 "EHLO shadbolt.e.decadent.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751868Ab0CZAmR (ORCPT ); Thu, 25 Mar 2010 20:42:17 -0400 In-Reply-To: <1268932144-14105-1-git-send-email-aneesh.kumar@linux.vnet.ibm.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: --=-yWbw0NMviiWYP9IbhImC Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, 2010-03-18 at 22:39 +0530, Aneesh Kumar K.V wrote: > The below set of patches implement open by handle support using exportfs > operations. This allows user space application to map a file name to file= =20 > handle and later open the file using handle. This should be usable > for userspace NFS [1] and 9P server [2]. XFS already support this with th= e ioctls > XFS_IOC_PATH_TO_HANDLE and XFS_IOC_OPEN_BY_HANDLE. [...] I think this is quite a poor choice of name. A 'handle' is normally a capability and a counted reference to some resource. In a Linux context 'file handle' suggests to me a pointer to struct file. But as I understand it you are trying to provide an uncounted reference that is still subject to later permission checks. I would suggest something like 'file id', 'file number' (though that could be confused with file descriptor numbers) or 'file cookie'. Ben. --=20 Ben Hutchings Once a job is fouled up, anything done to improve it makes it worse. --=-yWbw0NMviiWYP9IbhImC Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iQIVAwUAS6wC1Oe/yOyVhhEJAQLedBAAvjLVhPPCwgghMTD+PWZBrkNNZ4ZSePCK 9MYw87wRzC9OzXKyZYoEQncgR8iF2qn9+koQQG2MqgtEaYcfAWSd/Wl9p40Hkvn9 9liUX+2CW/LKcKvuCGaG5QyhlFknNzhPKPiUU/rUuDX1EudGuHBPmTTZxudILuUx JiHcfJC1Jc60Ijo5O2lPMNpKrBJ3kGNF8H6rf3SvHqSAN4OxMQDcde5bnWYHl9Pl QrGYjidCztYqr7L9uXSJ04ze0bgjD4ns1UkOMoTEzq5TLDKdk1BpcYVcB3rJd0ro ModuTplUMDV9mgKl8JeFhhEtEZohSzuLC8HvKmhBktA8fQ7sxYM51kAmnGsqENfE Vlbh4GxlDI5OSBy1g/wt/MT8erEy20whNjpxHZEHcV/I98pUPfT0jRl1+OU20dRl WrutHiQKgiccpBNm/uMkAubdzyauzQz0DtOnqhGDgu6SQIaEZJgNvPafn2KylQo8 Rxu03r2U65OMiBP8EzlIk2ts94Id4LWmKfvmz8iNsR9vy6c/Cmj8Q2xe1aBvRD3r nVMwwISszWAt3s0WvUl1gnWsWlVkzb7ZB7QpvJk4qh/i1vx2oOK+hYGYbYMTLygE zCyeUZc6DcZUzF+O46YfWMSXeTzbDNOaS8J94N5LvRPu1L8fPoKB/plfY2bpVZCF VobCMuCzjK8= =vrzi -----END PGP SIGNATURE----- --=-yWbw0NMviiWYP9IbhImC--