| Message ID | 1417589535-15867-1-git-send-email-yamada.m@jp.panasonic.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
Masahiro, On Wed, 2014-12-03 at 15:52 +0900, Masahiro Yamada wrote: > The exported headers were moved to "uapi" directories. > We should check the existence of arch/*/include/uapi/asm/Kbuild. > > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com> > --- Does this patch fix any problems? If so, which? And why did no one notice these problems before? Perhaps the commit explanation could mention that. See, none of this is obvious, at least to me, and I don't think people reading the patch should be expected to figure this out themselves. > scripts/headers.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/headers.sh b/scripts/headers.sh > index 95ece06..b164336 100755 > --- a/scripts/headers.sh > +++ b/scripts/headers.sh > @@ -6,7 +6,7 @@ set -e > > do_command() > { > - if [ -f ${srctree}/arch/$2/include/asm/Kbuild ]; then > + if [ -f ${srctree}/arch/$2/include/uapi/asm/Kbuild ]; then > make ARCH=$2 KBUILD_HEADERS=$1 headers_$1 > else > printf "Ignoring arch: %s\n" ${arch} Thanks, Paul Bolle -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Paul, On Mon, 08 Dec 2014 22:58:33 +0100 Paul Bolle <pebolle@tiscali.nl> wrote: > Masahiro, > > On Wed, 2014-12-03 at 15:52 +0900, Masahiro Yamada wrote: > > The exported headers were moved to "uapi" directories. > > We should check the existence of arch/*/include/uapi/asm/Kbuild. > > > > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com> > > --- > > Does this patch fix any problems? If so, which? I am fixing a wrong code. Actually, the "um" architecture does not support headers install. arch/um/include/asm/Kbuild exists arch/um/include/uapi/asm/Kbuild does not exist Checking arch/*/include/asm/Kbuild does not work. Moreover, the top Makefile is checking arch/*/include/uapi/asm/Kbuild headers_install: __headers $(if $(wildcard $(srctree)/arch/$(hdr-arch)/include/uapi/asm/Kbuild),, \ $(error Headers not exportable for the $(SRCARCH) architecture)) > And why did no one > notice these problems before? Perhaps the commit explanation could > mention that. You will notice the code below: for arch in ${archs}; do case ${arch} in um) # no userspace export ;; "um" has already been omitted and it never reaches do_command(). That is why nobody has noticed this issue. Uh, OK, we do not need double-checking. Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Sam, IIRC, you mentioned scripts/headers.sh is pointless and should be removed from the code base. If so, this patch of mine has no point. Could you consider removing scripts/headers.sh ? Best Regards Masahiro Yamada On Wed, 3 Dec 2014 15:52:15 +0900 Masahiro Yamada <yamada.m@jp.panasonic.com> wrote: > The exported headers were moved to "uapi" directories. > We should check the existence of arch/*/include/uapi/asm/Kbuild. > > Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com> > --- > > scripts/headers.sh | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/scripts/headers.sh b/scripts/headers.sh > index 95ece06..b164336 100755 > --- a/scripts/headers.sh > +++ b/scripts/headers.sh > @@ -6,7 +6,7 @@ set -e > > do_command() > { > - if [ -f ${srctree}/arch/$2/include/asm/Kbuild ]; then > + if [ -f ${srctree}/arch/$2/include/uapi/asm/Kbuild ]; then > make ARCH=$2 KBUILD_HEADERS=$1 headers_$1 > else > printf "Ignoring arch: %s\n" ${arch} > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Jan 13, 2015 at 02:24:04PM +0900, Masahiro Yamada wrote: > Hi Sam, > > IIRC, you mentioned scripts/headers.sh is pointless > and should be removed from the code base. Took a quik look at it. It seems that there is use for it still. um require some special handling. And the HDR_ARCH_LIST feature is also implemented here. Both could be done in other ways and the shell script may be cleaned up a little. But we cannot get rid of it as I initally thought. Sam -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/scripts/headers.sh b/scripts/headers.sh index 95ece06..b164336 100755 --- a/scripts/headers.sh +++ b/scripts/headers.sh @@ -6,7 +6,7 @@ set -e do_command() { - if [ -f ${srctree}/arch/$2/include/asm/Kbuild ]; then + if [ -f ${srctree}/arch/$2/include/uapi/asm/Kbuild ]; then make ARCH=$2 KBUILD_HEADERS=$1 headers_$1 else printf "Ignoring arch: %s\n" ${arch}
The exported headers were moved to "uapi" directories. We should check the existence of arch/*/include/uapi/asm/Kbuild. Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com> --- scripts/headers.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)