Date: Fri, 24 Jan 2020 11:29:10 -0500 From: Rich Felker <dalias@...c.org> To: Florian Weimer <fweimer@...hat.com> Cc: musl@...ts.openwall.com Subject: Re: [PATCH] add statx On Fri, Jan 24, 2020 at 05:12:51PM +0100, Florian Weimer wrote: > * Rich Felker: > > > On Fri, Jan 24, 2020 at 04:27:47PM +0100, Florian Weimer wrote: > >> * Rich Felker: > >> > >> > This is under BSD||GNU (i.e. DEFAULT||ALL) rather than just under the > >> > latter. Is there a reason for that? Generally the extensions that are > >> > pretty clearly Linux-only, as opposed to something that other > >> > POSIX-based OS's are likely to adopt, are put under GNU/ALL to > >> > discourage their use without intent and to avoid namespace clutter. > >> > >> statx is not Linux-specific in glibc, but also available on Hurd. > > > > OK, well GNU/Linux-specific. :-) Some ppl find _GNU_SOURCE odd for > > stuff that comes from Linux not GNU, but in this case it seems pretty > > appropriate since GNU and Linux are the two systems doing it. > > Sorry for nit-picking, it's common to both GNU and Linux. Yes, I was just making a bad joke about "GNU/Linux", reusing it to mean "applies to both GNU and to Linux". Sorry for being unclear. > >> We received feedback that our headers are not useful due to the > >> __u64/uint64_t mismatch. > >> > >> <https://sourceware.org/bugzilla/show_bug.cgi?id=25292 > >> <https://sourceware.org/ml/libc-alpha/2019-12/msg00631.html> > > > > Uhg. That change seems unfortunate since it's incompatible with > > theoretical future archs having 128-bit long long -- an idea I'm not > > much a fan of, but don't actually want to preclude. Is there a reason > > to actually care about compatibility with the kernel types? > > Yes, printf format strings. 8-( Why not let ppl get the warnings and fix them by casting to an appropriate type to print. I prefer [u]intmax_t anyway to avoid the PRIu64 etc. mess that makes your format strings unreadable and that interferes with translation (*). > > It's not like both struct definitions can be included in the same > > source anyway; the tags would clash. Using the canonical uintN_t types > > makes more sense from an API perspective, I think; kernel assumptions > > about types should not leak into an API intended to be clean and > > shared with non-Linux implementations. > > I thought so too, which is why I used them. > > But it is fairly compelling to use the kernel types if the header is > available, so that we don't have to patch and rebuild glibc if someone > backports new statx features into the kernel. That's why I added the > __has_include kludge. I see. This is something of a tradeoff I guess; for musl users I'd expect folks to have libc headers newer than kernel headers in most cases since distros are slow to follow new kernels but fast to follow new libc, and you want compile-time support for future kernel features even if you don't have the kernel to use them yet. But on glibc-based dists it may be the other way around. Rich (*) GNU gettext has "SYSDEP" format strings that solve this problem but do so by requiring non-shareable string memory for all the affected strings. musl gettext does not support this, but is intended to be used with gettext-tiny's version of msgfmt that works out the combinatorics at .mo generation time and emits them all in shareable form. Still I think it's better to get rid of this practice and just use %jd etc. everywhere with suitable casts.
Powered by blists - more mailing lists
Confused about mailing lists and their use? Read about mailing lists on Wikipedia and check out these guidelines on proper formatting of your messages.