Messages in this thread | | | From | Masahiro Yamada <> | Date | Sun, 7 Jul 2019 15:41:20 +0900 | Subject | Re: [PATCH] nilfs2: do not use unexported cpu_to_le32()/le32_to_cpu() in uapi header |
| |
Konishi-san,
On Sun, Jul 7, 2019 at 3:11 PM Ryusuke Konishi <konishi.ryusuke@lab.ntt.co.jp> wrote: > > Yamada-san, > > I'm sorry for not being able to follow. > > Could you tell me how did you find the build error? > Still I cannot reproduce the error in my environment.
"You can compile UAPI headers in kernel-space" does not necessarily mean "you can compile them in user-space".
To make sure UAPI headers can be included from libc etc. you must compile them after 'make headers_install', but people often miss to test that.
To automate this testing, I will merge the following for 5.3-rc1: https://patchwork.kernel.org/patch/11024715/
Once it lands in upstream, all the new breakage will be caught by 0-day bot.
Currently, nilfs2_ondisk.h is excluded from the test coverage since it is broken.
Thanks. Masahiro Yamada
> Or, if it's right that we should not use > cpu_to_le{16,32,64}/le{16,32,64}_to_cpu() in UAPI header files, > I will acknowledge the change and modify nilfs utilities to comply > with the guideline. > > Thanks, > Ryusuke Konishi > > 2019年7月7日(日) 13:31 Masahiro Yamada <yamada.masahiro@socionext.com>: > > > > Hi Andrew, > > > > On Sun, Jul 7, 2019 at 12:38 PM Andrew Morton <akpm@linux-foundation.org> wrote: > > > > > > > > Seems fairly serious. I'm thinking this needs a cc:stable? > > > > > > Ah, yes. > > > > > > Cc: linux-stable <stable@vger.kernel.org> # 4.9 > > Fixes: e63e88bc53ba ("nilfs2: move ioctl interface and disk layout to > > uapi separately") > > > > > > Thanks. > > > > -- > > Best Regards > > Masahiro Yamada
-- Best Regards Masahiro Yamada
| |