netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: David Miller <davem@davemloft.net>
To: hauke@hauke-m.de
Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org,
	jarod@redhat.com, jogo@openwrt.org, david.heidelberger@ixit.cz,
	maillist-linux@barfooze.de, mikko.rapeli@iki.fi,
	dwmw2@infradead.org
Subject: Re: [PATCH v2 0/3] uapi glibc compat: fix musl libc compatibility
Date: Thu, 20 Apr 2017 16:07:39 -0400 (EDT)	[thread overview]
Message-ID: <20170420.160739.1263793303398330801.davem@davemloft.net> (raw)
In-Reply-To: <20170418210036.26039-1-hauke@hauke-m.de>

From: Hauke Mehrtens <hauke@hauke-m.de>
Date: Tue, 18 Apr 2017 23:00:33 +0200

> The code from libc-compat.h depends on some glibc specific defines and 
> causes compile problems with the musl libc. These patches remove some 
> of the glibc dependencies. With these patches the LEDE (OpenWrt) base 
> user space applications can be build with unmodified kernel headers and 
> musl libc.
> 
> This was compile tested with the user space from LEDE (OpenWrt) with 
> musl 1.1.16, glibc 2.25 and uClibc-ng 1.0.22.
> 
> Changes since v1:
>  - fix typo in commit message
>  - combine two changes

I think I have to put the brakes on this patch series, after much
consideration.

It does not scale if we continually add a hodge-podge of different
ifdef tests to the UAPI headers in order to prevent mutliple
definitions.

We will add that IFF_ECHO ifdef for MUSL libc today, and for another
libc we will add another such hack.  And so on and so forth...

Instead, LIBC implementation must adopt the ifdef protections which
have standard names and are being adopted by GLIBC and hopefully
others.

So please instead adjust the musl headers so that they interact
properly with the framework we've designed specifically for this
purpose.

Thank you.

  parent reply	other threads:[~2017-04-20 20:07 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-18 21:00 [PATCH v2 0/3] uapi glibc compat: fix musl libc compatibility Hauke Mehrtens
2017-04-18 21:00 ` [PATCH v2 1/3] uapi glibc compat: add libc compat code when not build for kernel Hauke Mehrtens
2017-04-18 21:00 ` [PATCH v2 2/3] uapi glibc compat: fix build if libc defines IFF_ECHO Hauke Mehrtens
2017-04-18 21:00 ` [PATCH v2 3/3] uapi/if_ether.h: prevent redefinition of struct ethhdr Hauke Mehrtens
2017-04-20 20:07 ` David Miller [this message]
2017-04-20 20:14   ` [PATCH v2 0/3] uapi glibc compat: fix musl libc compatibility David Woodhouse
2017-04-20 20:36     ` David Miller
2017-04-21 13:14       ` Hauke Mehrtens
2017-04-21 13:17         ` David Woodhouse
2017-04-21 14:41         ` Rich Felker

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=20170420.160739.1263793303398330801.davem@davemloft.net \
    --to=davem@davemloft.net \
    --cc=david.heidelberger@ixit.cz \
    --cc=dwmw2@infradead.org \
    --cc=hauke@hauke-m.de \
    --cc=jarod@redhat.com \
    --cc=jogo@openwrt.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maillist-linux@barfooze.de \
    --cc=mikko.rapeli@iki.fi \
    --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).