linux-kernel - Re: [PATCH v3 1/3] kbuild: Add _

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]

Date:   Mon, 19 Jun 2017 11:00:42 +0900
From:   Masahiro Yamada <yamada.masahiro@...ionext.com>
To:     Matthias Kaehlcke <mka@...omium.org>
Cc:     kbuild-all@...org, Thomas Gleixner <tglx@...utronix.de>,
        Ingo Molnar <mingo@...hat.com>,
        "H . Peter Anvin" <hpa@...or.com>,
        "H . J . Lu" <hjl.tools@...il.com>,
        David Woodhouse <dwmw2@...radead.org>,
        Michal Marek <mmarek@...e.com>, X86 ML <x86@...nel.org>,
        Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Michael Davidson <md@...gle.com>,
        Greg Hackmann <ghackmann@...gle.com>,
        Nick Desaulniers <ndesaulniers@...gle.com>,
        Stephen Hines <srhines@...gle.com>,
        Kees Cook <keescook@...omium.org>,
        Arnd Bergmann <arnd@...db.de>,
        Bernhard Rosenkränzer 
        <Bernhard.Rosenkranzer@...aro.org>,
        Peter Foley <pefoley2@...oley.com>,
        Behan Webster <behanw@...verseincode.com>,
        Douglas Anderson <dianders@...omium.org>,
        kbuild test robot <lkp@...el.com>
Subject: Re: [PATCH v3 1/3] kbuild: Add __cc-option macro

Hi Matthias,


Sorry, the code I suggested introduced a slight behavior change.

With this patch, cc-option adds a space before the returned option.


Please see line 46 of arch/sh/Makefile.

ifeq ($(cflags-y),)


Now cflags-y contains a space, so
this conditional is now evaluated as false,
which was actually true without the patch.
This is the cause of the build error.


Could you remove whitespaces before $(1) and $(2) in cc-option?

Also, please fix hostcc-option as well.

Thanks!






2017-06-18 15:58 GMT+09:00 kbuild test robot <lkp@...el.com>:
> Hi Matthias,
>
> [auto build test ERROR on tip/auto-latest]
> [also build test ERROR on v4.12-rc5 next-20170616]
> [if your patch is applied to the wrong git tree, please drop us a note to help improve the system]
>
> url:    https://github.com/0day-ci/linux/commits/Matthias-Kaehlcke/kbuild-Add-__cc-option-macro/20170616-082252
> config: sh-sh7710voipgw_defconfig (attached as .config)
> compiler: sh4-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
> reproduce:
>         wget https://raw.githubusercontent.com/01org/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         # save the attached .config to linux build tree
>         make.cross ARCH=sh
>
> All errors (new ones prefixed by >>):
>
>    {standard input}: Assembler messages:
>>> {standard input}:672: Error: invalid operands for opcode
>    {standard input}:673: Error: opcode not valid for this cpu variant
>    {standard input}:674: Error: opcode not valid for this cpu variant
>    {standard input}:675: Error: invalid operands for opcode
>    {standard input}:1900: Error: invalid operands for opcode
>    {standard input}:1901: Error: opcode not valid for this cpu variant
>    {standard input}:1902: Error: opcode not valid for this cpu variant
>    {standard input}:1903: Error: invalid operands for opcode
>    {standard input}:2067: Error: invalid operands for opcode
>    {standard input}:2068: Error: opcode not valid for this cpu variant
>    {standard input}:2069: Error: opcode not valid for this cpu variant
>    {standard input}:2070: Error: invalid operands for opcode
>    {standard input}:2570: Error: invalid operands for opcode
>    {standard input}:2571: Error: opcode not valid for this cpu variant
>    {standard input}:2572: Error: opcode not valid for this cpu variant
>    {standard input}:2573: Error: invalid operands for opcode
>
> ---
> 0-DAY kernel test infrastructure                Open Source Technology Center
> https://lists.01.org/pipermail/kbuild-all                   Intel Corporation



-- 
Best Regards
Masahiro Yamada