Groups > linux.kernel > #1361987

Re: [PATCH] Remove v850 from linux/elf-em.h

From	David Daney <ddaney.cavm@gmail.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] Remove v850 from linux/elf-em.h
Date	2016-03-21 17:40 +0100
Message-ID	<rfb22-8iP-27@gated-at.bofh.it> (permalink)
References	(2 earlier) <rddUl-5zP-3@gated-at.bofh.it> <rdOhc-4TJ-13@gated-at.bofh.it> <rdSuu-7Hu-11@gated-at.bofh.it> <re6H8-3uY-9@gated-at.bofh.it> <redIC-6en-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On 03/18/2016 06:15 PM, Rob Landley wrote:
> On 03/18/2016 12:46 PM, David Daney wrote:
>> I am not going to comment on it any more, but [commenting more]
>
> Yes you are. (And did then too.)
>
>> On 03/17/2016 07:32 PM, Rob Landley wrote:
>> [...]
>>>
>>> As I explained last email, userspace uses the libc header, not the linux
>>> header,
>>
>> The fallacy in this argument is the assertion that we know what
>> userspace does.
>
> Userspace programs that did that already broke on earlier symbol removals.
>
>> Userspace could easily do:
>>
>> #include <linux/elf-em.h>
>> .
>>     case SYMBOL_YOU_WANT_TO_REMOVE:
>>
>>   ¡BOOM!  it is broken.
>
> So you're assuming I don't know how headers get used by userspace.

Yes, exactly.  Don't feel bad about it though, because nobody else knows 
either.

> That's nice. Clearly, I never would have thought of that.
>
> Once again, "As I explained last email", symbols have been removed from
> this particular header before.

Since you know, a priori, that symbols in that file are never used by 
userspace, why not send a patch that moves it to include/linux/elf-em.h 
so that userspace doesn't see it?  Then you could remove as many symbols 
as you like.

David Daney

[...]

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

Re: [PATCH] Remove v850 from linux/elf-em.h David Daney <ddaney.cavm@gmail.com> - 2016-03-17 23:10 +0100
  Re: [PATCH] Remove v850 from linux/elf-em.h Rob Landley <rob@landley.net> - 2016-03-18 03:40 +0100
    Re: [PATCH] Remove v850 from linux/elf-em.h David Daney <ddaney.cavm@gmail.com> - 2016-03-18 18:50 +0100
      Re: [PATCH] Remove v850 from linux/elf-em.h Rob Landley <rob@landley.net> - 2016-03-19 02:20 +0100
        Re: [PATCH] Remove v850 from linux/elf-em.h David Daney <ddaney.cavm@gmail.com> - 2016-03-21 17:40 +0100

csiph-web