linux-kernel - Re: [PATCH] ia64: prefix header search path with $(srctree)/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAK7LNARf-P0Qjj2ScgQHk2yFw13h7zXk0VCB0MJe3W2ka3s40A@mail.gmail.com>
Date:   Mon, 18 Feb 2019 13:54:39 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Tony Luck <tony.luck@...el.com>, Fenghua Yu <fenghua.yu@...el.com>,
        linux-ia64@...r.kernel.org
Cc:     Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ia64: prefix header search path with $(srctree)/

On Fri, Jan 25, 2019 at 4:21 PM Masahiro Yamada
<yamada.masahiro@...ionext.com> wrote:
>
> Currently, the Kbuild core manipulates header search paths in a crazy
> way [1].
>
> To fix this mess, I want all Makefiles to add explicit $(srctree)/ to
> the search paths in the srctree. Some Makefiles are already written in
> that way, but not all. The goal of this work is to make the notation
> consistent, and finally get rid of the gross hacks.
>
> Having whitespaces after -I does not matter since commit 48f6e3cf5bc6
> ("kbuild: do not drop -I without parameter").
>
> I removed some header search paths because I was able to build ia64
> without them.
>
> [1]: https://patchwork.kernel.org/patch/9632347/
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@...ionext.com>
> ---


ia64 development is not very active these days.

I will wait for response from the maintainers.

If they are silent, I want to pick up this to my kbuild tree.


Thanks.


>  arch/ia64/sn/kernel/Makefile     | 2 +-
>  arch/ia64/sn/kernel/sn2/Makefile | 2 --
>  arch/ia64/sn/pci/Makefile        | 2 --
>  arch/ia64/sn/pci/pcibr/Makefile  | 2 +-
>  4 files changed, 2 insertions(+), 6 deletions(-)
>
> diff --git a/arch/ia64/sn/kernel/Makefile b/arch/ia64/sn/kernel/Makefile
> index d27df1d..9c349dd 100644
> --- a/arch/ia64/sn/kernel/Makefile
> +++ b/arch/ia64/sn/kernel/Makefile
> @@ -7,7 +7,7 @@
>  # Copyright (C) 1999,2001-2006,2008 Silicon Graphics, Inc.  All Rights Reserved.
>  #
>
> -ccflags-y := -Iarch/ia64/sn/include
> +ccflags-y := -I $(srctree)/arch/ia64/sn/include
>
>  obj-y                          += setup.o bte.o bte_error.o irq.o mca.o idle.o \
>                                    huberror.o io_acpi_init.o io_common.o \
> diff --git a/arch/ia64/sn/kernel/sn2/Makefile b/arch/ia64/sn/kernel/sn2/Makefile
> index 3d09108..170bde4 100644
> --- a/arch/ia64/sn/kernel/sn2/Makefile
> +++ b/arch/ia64/sn/kernel/sn2/Makefile
> @@ -9,7 +9,5 @@
>  # sn2 specific kernel files
>  #
>
> -ccflags-y := -Iarch/ia64/sn/include
> -
>  obj-y += cache.o io.o ptc_deadlock.o sn2_smp.o sn_proc_fs.o \
>          prominfo_proc.o timer.o timer_interrupt.o sn_hwperf.o
> diff --git a/arch/ia64/sn/pci/Makefile b/arch/ia64/sn/pci/Makefile
> index df2a901..321576b 100644
> --- a/arch/ia64/sn/pci/Makefile
> +++ b/arch/ia64/sn/pci/Makefile
> @@ -7,6 +7,4 @@
>  #
>  # Makefile for the sn pci general routines.
>
> -ccflags-y := -Iarch/ia64/sn/include
> -
>  obj-y := pci_dma.o tioca_provider.o tioce_provider.o pcibr/
> diff --git a/arch/ia64/sn/pci/pcibr/Makefile b/arch/ia64/sn/pci/pcibr/Makefile
> index 396bcae..712f6af 100644
> --- a/arch/ia64/sn/pci/pcibr/Makefile
> +++ b/arch/ia64/sn/pci/pcibr/Makefile
> @@ -7,7 +7,7 @@
>  #
>  # Makefile for the sn2 io routines.
>
> -ccflags-y := -Iarch/ia64/sn/include
> +ccflags-y := -I $(srctree)/arch/ia64/sn/include
>
>  obj-y                          +=  pcibr_dma.o pcibr_reg.o \
>                                     pcibr_ate.o pcibr_provider.o
> --
> 2.7.4
>


-- 
Best Regards
Masahiro Yamada