From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp.gentoo.org ([140.211.166.183]:54120 "EHLO smtp.gentoo.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752915AbcHQVfY (ORCPT ); Wed, 17 Aug 2016 17:35:24 -0400 Date: Wed, 17 Aug 2016 14:35:22 -0700 From: Mike Frysinger To: Jeff Layton Cc: "Michael Kerrisk (man-pages)" , libc-alpha@sourceware.org, linux-fsdevel@vger.kernel.org, Carlos O'Donell , Yuriy Kolerov Subject: Re: [glibc PATCH] fcntl: put F_OFD_* constants under #ifdef __USE_FILE_OFFSET64 Message-ID: <20160817213522.GG21655@vapier.lan> References: <1471445251-2450-1-git-send-email-jlayton@redhat.com> <20160817184333.GC21655@vapier.lan> <1471461304.3196.101.camel@redhat.com> <1471464343.3196.125.camel@redhat.com> <20160817203746.GF21655@vapier.lan> <1471467478.3196.143.camel@redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="O8XZ+2Hy8Kj8wLPZ" Content-Disposition: inline In-Reply-To: <1471467478.3196.143.camel@redhat.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: --O8XZ+2Hy8Kj8wLPZ Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 17 Aug 2016 16:57, Jeff Layton wrote: > On Wed, 2016-08-17 at 13:37 -0700, Mike Frysinger wrote: > > On 17 Aug 2016 16:05, Jeff Layton wrote: > > >=20 > > > The way it works now is that when you define _FILE_OFFSET_BITS=3D64 a= nd > > > call fcntl(fd, F_SETLK, fl) glibc swaps in a struct flock64 for your > > > struct flock, and F_SETLK64 for the F_SETLK. > >=20 > > does it ?=C2=A0=C2=A0doesn't seem like it does to me.=C2=A0=C2=A0here's= glibc's fcntl.c: > > io/fcntl.c - generic stub that sets ENOSYS > > sysdeps/unix/sysv/linux/fcntl.c - just calls syscall(fcntl) > > sysdeps/unix/sysv/linux/generic/wordsize-32/fcntl.c - just calls sysca= ll(fcntl64) > > sysdeps/unix/sysv/linux/i386/fcntl.c - same as above > > > >=20 >=20 > Ok, I was being a little cavalier with my description. This is what > really happens (from x86 struct flock definition): >=20 > struct flock > { > short int l_type; /* Type of lock: F_RDLCK, F_WRLCK, or F_UNLCK. */ > short int l_whence; /* Where `l_start' is relative to (like `lseek').= */ > #ifndef __USE_FILE_OFFSET64 > __off_t l_start; /* Offset where the lock begins. */ > __off_t l_len; /* Size of the locked area; zero means until EOF.= */ > #else > __off64_t l_start; /* Offset where the lock begins. */ > __off64_t l_len; /* Size of the locked area; zero means until EOF.= */ > #endif > __pid_t l_pid; /* Process holding the lock. */ > }; >=20 > So, l_start and l_len get redefined into larger sizes when LFS is > enabled. The F_GETLK/F_SETLK/F_SETLKW are also redefined to their *64 > equivalents in that case using the preprocessor. ah i forgot about that in the glibc header. so it's not as grimm as i was thinking, and explains how glibc has been providing the API so the user doesn't have to explicitly pick the 64-bit types. in order to do the same for OFD, we'd need to go the LFS route (i.e. add fcntl64 and transparent rewrites in glibc). or we can just run with your idea ... i'm warmer to it now that i see we only have to tell the user "enable LFS support" rather than "use the flock64 struct". -mike --O8XZ+2Hy8Kj8wLPZ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJXtNiaAAoJEEFjO5/oN/WByhYP/0CS1Pv9G7A3Vy6C/8QKaW02 V6Mbtk/nrNQTtDnKYNuylSRW94BNtBBWIhjTCSUA0eMDDhV+B1TOVo4WGtOrG3BN JrlBM2sKE+fKL6JvBz45lJEaSEru5r85YMms4VDhbKf9x2Bx6L9vBurxeFSYtcSf n4eGIetEEA3bww+cQSTGtinMBXltZ2uHzivuhe+t8j897gRZXBBu9M6g//bBotAM SCqlLOzp1AupF+dhA93Yl+KiT5ZrRFZqbuLus+hlBP39OBrbNVr1DtyYF5pdXndH gmkGfuk143wJlM22S2e7mP3+Mbux4kHy1prZMFyap71tQ4sv9RWGoQactcYe+ec4 35VUtJfQEwOyzSr/17WaXmj5pYNp6I0WDcFC2KU94DOGvUjgL/4LU7U2dJ7/aEPX 0KYknheK+IAP1exSYsiuSe5+TUT1VdbLgj3s8Sur6v1eo3QYursanDeJCDPqN/1x oJJG5SLqYJCqO2o7TTxmOTjCEt6YRa1WuWbsNrySQ9FbSH3coOVC3+zTi4NxGP/F ErSi3heyANWxEz1XTCjjkEZIaDEkLQkhSPRPCbdMdoT0y9pN3UQvT3TkyLAZ6XLA AeTTYosQeA12QaE2HeWhsVvSZCy8SxqGXRjfEkRbOjRcCyux9/Qwco9n1EhVeWnE iTKRGIlil9iOdq/tvlqb =R1dK -----END PGP SIGNATURE----- --O8XZ+2Hy8Kj8wLPZ--