From: Mike Frysinger <vapier@gentoo.org>
To: "H. Peter Anvin" <hpa@zytor.com>
Cc: David Miller <davem@davemloft.net>,
netdev@vger.kernel.org, klibc@zytor.com
Subject: Re: [klibc] [patch] import socket defines
Date: Fri, 11 Jan 2008 02:57:41 -0500 [thread overview]
Message-ID: <200801110257.42272.vapier@gentoo.org> (raw)
In-Reply-To: <4787164F.9030805@zytor.com>
[-- Attachment #1: Type: text/plain, Size: 2485 bytes --]
On Friday 11 January 2008, H. Peter Anvin wrote:
> Mike Frysinger wrote:
> > oh, sorry, i see what you mean. i was thinking in terms of crap removed
> > (as that's what i'm after), not crap added (which is what Peter is
> > after). i hadnt noticed that. i dont know if it'll break glibc (and
> > really, any other sane libc). if that is the case, then i think klibc
> > here is the 2nd class citizen to everyone else.
>
> I don't really understand why you insist on using such inflammatory
> language;
it's true. it is much easier to adapt klibc on the fly than every one else.
klibc is also of significant less importance in the larger open source
landscape than glibc or other libc's. this isnt really inflammatory so much
as fact.
> all this stuff is ABI constants, and the only reason glibc
> doesn't use them is that glibc prefers to use enums over #defines.
a proper libc defines things in their headers according to the POSIX specs
rather than relying on others to do it for them. if you want to argue about
linux-specific ABI pieces being exported, then you probably have a valid
point, but socket.h is hardly that. imo klibc is being lazy. you disagree.
here we are.
> Right now, glibc is special-cased. glibc also tends to be very
> deliberate about its kernel header inclusions. It wants a subset of the
> available defines, so it can include a subset header.
>
> The reverse is definitely possible too -- all other users (kernel,
> newlib, dietlibc, uclibc, and klibc) can change and leave the current
> state for glibc.
that list is inaccurate. newlib is generally not used under linux, but of the
few headers it does use, socket.h is not one of them.
i dont use dietlibc myself as ive found it generally not suitable for much,
but a quick look through their source code shows linux/socket.h not in use at
all.
uClibc is the exact reason i started this thread in the first place. we have
__GLIBC__ define in place to account for broken code and it's because of
things like linux/socket.h that we cannot yet remove it.
glibc has not used the typedefs/defines in question in ages, so they're not
affected by the things not being available.
that leaves klibc.
> We can special-case the kernel in the above case, but that would involve
> some additional ugliness.
so if the only consumer is klibc and you're against adding these things to it,
special case it for __KLIBC__.
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 835 bytes --]
next prev parent reply other threads:[~2008-01-11 7:57 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200801012029.21432.vapier@gentoo.org>
[not found] ` <477AF86E.1080309@zytor.com>
[not found] ` <200801020830.43449.vapier@gentoo.org>
2008-01-02 18:09 ` [klibc] [patch] import socket defines H. Peter Anvin
2008-01-11 6:16 ` David Miller
2008-01-11 6:23 ` Mike Frysinger
2008-01-11 6:41 ` H. Peter Anvin
2008-01-11 6:47 ` David Miller
2008-01-11 7:02 ` H. Peter Anvin
2008-01-11 7:07 ` Mike Frysinger
2008-01-11 7:10 ` H. Peter Anvin
2008-01-11 7:42 ` H. Peter Anvin
2008-01-11 7:57 ` Mike Frysinger [this message]
2008-01-11 8:00 ` H. Peter Anvin
2008-01-11 9:02 ` Mike Frysinger
2008-01-11 9:28 ` Mike Frysinger
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200801110257.42272.vapier@gentoo.org \
--to=vapier@gentoo.org \
--cc=davem@davemloft.net \
--cc=hpa@zytor.com \
--cc=klibc@zytor.com \
--cc=netdev@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).