Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1642150 > unrolled thread

Re: [PATCH v11 11/12] uapi: export all arch specifics directories

Started byDan Williams <dan.j.williams@intel.com>
First post2017-05-16 03:10 +0200
Last post2017-05-16 07:00 +0200
Articles 3 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v11 11/12] uapi: export all arch specifics directories Dan Williams <dan.j.williams@intel.com> - 2017-05-16 03:10 +0200
    Re: [PATCH v11 11/12] uapi: export all arch specifics directories Dan Williams <dan.j.williams@intel.com> - 2017-05-16 03:20 +0200
      Re: [PATCH v11 11/12] uapi: export all arch specifics directories Masahiro Yamada <yamada.masahiro@socionext.com> - 2017-05-16 07:00 +0200

#1642150 — Re: [PATCH v11 11/12] uapi: export all arch specifics directories

FromDan Williams <dan.j.williams@intel.com>
Date2017-05-16 03:10 +0200
SubjectRe: [PATCH v11 11/12] uapi: export all arch specifics directories
Message-ID<tHz9T-7ej-3@gated-at.bofh.it>
On Mon, Mar 27, 2017 at 5:20 AM, Nicolas Dichtel
<nicolas.dichtel@6wind.com> wrote:
> This patch removes the need of subdir-y. Now all files/directories under
> arch/<arch>/include/uapi/ are exported.
>
> The only change for userland is the layout of the command 'make
> headers_install_all': directories asm-<arch> are replaced by arch-<arch>/.
> Those new directories contains all files/directories of the specified arch.
>
> Note that only cris and tile have more directories than only asm:
>  - arch-v[10|32] for cris;
>  - arch for tile.
>
> Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>

Starting with commit 61562f981e92 "uapi: export all arch specifics
directories", a "make INSTALL_HDR_PATH=$root/usr headers_install"
operation will delete standard glibc headers and others in
$root/usr/include.

Here's the snippet from the build process where it deletes system
headers with this commit:

  CHK     include/generated/uapi/linux/version.h
  INSTALL include/linux/nfsd (5 files)
  INSTALL include/linux (482 files)
  REMOVE  libgen.h resolv.h paths.h syscall.h spawn.h unistd.h ar.h
setjmp.h termio.h wchar.h semaphore.h fcntl.h fpu_control.h glob.h
termios.h nss.h fmtmsg.h shadow.h crypt.h aio.h grp.h uchar.h
libudev.h argz.h utime.h complex.h obstack.h argp.h math.h sched.h
xlocale.h netdb.h link.h mqueue.h ulimit.h regex.h fstab.h sgtty.h
iconv.h dirent.h inttypes.h pty.h locale.h utmp.h pthread.h gconv.h
lastlog.h assert.h endian.h stdio_ext.h signal.h search.h ctype.h
execinfo.h values.h re_comp.h libio.h ustat.h printf.h getopt.h time.h
stab.h stdlib.h gshadow.h mntent.h stdc-predef.h libkmod.h memory.h
nl_types.h mcheck.h errno.h poll.h fnmatch.h limits.h ucontext.h
libintl.h envz.h stdio.h wait.h monetary.h _G_config.h dlfcn.h
wordexp.h string.h pwd.h tar.h proc_service.h tgmath.h byteswap.h
elf.h cpio.h aliases.h wctype.h ftw.h ieee754.h err.h langinfo.h fts.h
features.h gnu-versions.h malloc.h regexp.h stdint.h ifaddrs.h
syslog.h error.h a.out.h utmpx.h sysexits.h fenv.h ttyent.h alloca.h
thread_db.h strings.h

[toc] | [next] | [standalone]


#1642163

FromDan Williams <dan.j.williams@intel.com>
Date2017-05-16 03:20 +0200
Message-ID<tHzjB-7hu-29@gated-at.bofh.it>
In reply to#1642150
On Mon, May 15, 2017 at 6:02 PM, Dan Williams <dan.j.williams@intel.com> wrote:
> On Mon, Mar 27, 2017 at 5:20 AM, Nicolas Dichtel
> <nicolas.dichtel@6wind.com> wrote:
>> This patch removes the need of subdir-y. Now all files/directories under
>> arch/<arch>/include/uapi/ are exported.
>>
>> The only change for userland is the layout of the command 'make
>> headers_install_all': directories asm-<arch> are replaced by arch-<arch>/.
>> Those new directories contains all files/directories of the specified arch.
>>
>> Note that only cris and tile have more directories than only asm:
>>  - arch-v[10|32] for cris;
>>  - arch for tile.
>>
>> Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
>
> Starting with commit 61562f981e92 "uapi: export all arch specifics
> directories", a "make INSTALL_HDR_PATH=$root/usr headers_install"
> operation will delete standard glibc headers and others in
> $root/usr/include.

Reverting commit 61562f981e92 fixes the problem for me.

[toc] | [prev] | [next] | [standalone]


#1642204

FromMasahiro Yamada <yamada.masahiro@socionext.com>
Date2017-05-16 07:00 +0200
Message-ID<tHCKu-Wk-1@gated-at.bofh.it>
In reply to#1642163
Hi Dan,


2017-05-16 10:15 GMT+09:00 Dan Williams <dan.j.williams@intel.com>:
> On Mon, May 15, 2017 at 6:02 PM, Dan Williams <dan.j.williams@intel.com> wrote:
>> On Mon, Mar 27, 2017 at 5:20 AM, Nicolas Dichtel
>> <nicolas.dichtel@6wind.com> wrote:
>>> This patch removes the need of subdir-y. Now all files/directories under
>>> arch/<arch>/include/uapi/ are exported.
>>>
>>> The only change for userland is the layout of the command 'make
>>> headers_install_all': directories asm-<arch> are replaced by arch-<arch>/.
>>> Those new directories contains all files/directories of the specified arch.
>>>
>>> Note that only cris and tile have more directories than only asm:
>>>  - arch-v[10|32] for cris;
>>>  - arch for tile.
>>>
>>> Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
>>
>> Starting with commit 61562f981e92 "uapi: export all arch specifics
>> directories", a "make INSTALL_HDR_PATH=$root/usr headers_install"
>> operation will delete standard glibc headers and others in
>> $root/usr/include.
>
> Reverting commit 61562f981e92 fixes the problem for me.

Thank you for your bug report.

Could you check this?
https://patchwork.kernel.org/patch/9728323/



-- 
Best Regards
Masahiro Yamada

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web