All of lore.kernel.org
 help / color / mirror / Atom feed
* [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-21  4:28 ` kbuild test robot
  0 siblings, 0 replies; 14+ messages in thread
From: kbuild test robot @ 2020-03-21  4:28 UTC (permalink / raw)
  To: Kristina Martsenko
  Cc: kbuild-all, Kees Cook, Catalin Marinas, Amit Daniel Kachhap,
	Vincenzo Frascino, linux-arm-kernel

[-- Attachment #1: Type: text/plain, Size: 1170 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
head:   3b446c7d27ddd06342901bb35211363f6944291a
commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
config: arm64-randconfig-a001-20200321 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
        # save the attached .config to linux build tree
        GCC_VERSION=9.2.0 make.cross ARCH=arm64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
--
>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 39526 bytes --]

[-- Attachment #3: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-21  4:28 ` kbuild test robot
  0 siblings, 0 replies; 14+ messages in thread
From: kbuild test robot @ 2020-03-21  4:28 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 1196 bytes --]

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
head:   3b446c7d27ddd06342901bb35211363f6944291a
commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
config: arm64-randconfig-a001-20200321 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
        # save the attached .config to linux build tree
        GCC_VERSION=9.2.0 make.cross ARCH=arm64 

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
--
>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 39526 bytes --]

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-21  4:28 ` kbuild test robot
@ 2020-03-23 11:48   ` Catalin Marinas
  -1 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-23 11:48 UTC (permalink / raw)
  To: Amit Daniel Kachhap, kbuild test robot
  Cc: kbuild-all, Kees Cook, Szabolcs Nagy, Kristina Martsenko,
	Vincenzo Frascino, linux-arm-kernel, daniel.kiss

On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
> head:   3b446c7d27ddd06342901bb35211363f6944291a
> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
> config: arm64-randconfig-a001-20200321 (attached as .config)
> compiler: aarch64-linux-gcc (GCC) 9.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>         # save the attached .config to linux build tree
>         GCC_VERSION=9.2.0 make.cross ARCH=arm64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
> >> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> --
> >> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

Just to make sure I understand the cause of this warning: gcc generates
a .note.gnu.property when PAC is enabled for the kernel but binutils
doesn't understand it. Is there a way to check for this via the Kconfig
options?

-- 
Catalin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-23 11:48   ` Catalin Marinas
  0 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-23 11:48 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 1426 bytes --]

On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
> head:   3b446c7d27ddd06342901bb35211363f6944291a
> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
> config: arm64-randconfig-a001-20200321 (attached as .config)
> compiler: aarch64-linux-gcc (GCC) 9.2.0
> reproduce:
>         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>         # save the attached .config to linux build tree
>         GCC_VERSION=9.2.0 make.cross ARCH=arm64 
> 
> If you fix the issue, kindly add following tag
> Reported-by: kbuild test robot <lkp@intel.com>
> 
> All warnings (new ones prefixed by >>):
> 
> >> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> --
> >> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

Just to make sure I understand the cause of this warning: gcc generates
a .note.gnu.property when PAC is enabled for the kernel but binutils
doesn't understand it. Is there a way to check for this via the Kconfig
options?

-- 
Catalin

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-23 11:48   ` Catalin Marinas
@ 2020-03-23 17:04     ` Daniel Kiss
  -1 siblings, 0 replies; 14+ messages in thread
From: Daniel Kiss @ 2020-03-23 17:04 UTC (permalink / raw)
  To: Catalin Marinas
  Cc: kbuild-all, Kees Cook, Szabolcs Nagy, Kristina Martsenko,
	Amit Kachhap, Vincenzo Frascino, linux-arm-kernel,
	kbuild test robot


> On 23 Mar 2020, at 12:48, Catalin Marinas <Catalin.Marinas@arm.com> wrote:
>
> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>> config: arm64-randconfig-a001-20200321 (attached as .config)
>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>> reproduce:
>>        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>        chmod +x ~/bin/make.cross
>>        git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>        # save the attached .config to linux build tree
>>        GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>
>> If you fix the issue, kindly add following tag
>> Reported-by: kbuild test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>> --
>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>
> Just to make sure I understand the cause of this warning: gcc generates
> a .note.gnu.property when PAC is enabled for the kernel but binutils
> doesn't understand it.
This is my understanding as well.

> Is there a way to check for this via the Kconfig
> options?
Maybe a version check on binutils could help here.

Cheers,
Daniel

> --
> Catalin

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-23 17:04     ` Daniel Kiss
  0 siblings, 0 replies; 14+ messages in thread
From: Daniel Kiss @ 2020-03-23 17:04 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 1969 bytes --]


> On 23 Mar 2020, at 12:48, Catalin Marinas <Catalin.Marinas@arm.com> wrote:
>
> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>> config: arm64-randconfig-a001-20200321 (attached as .config)
>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>> reproduce:
>>        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>        chmod +x ~/bin/make.cross
>>        git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>        # save the attached .config to linux build tree
>>        GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>
>> If you fix the issue, kindly add following tag
>> Reported-by: kbuild test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>> --
>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>
> Just to make sure I understand the cause of this warning: gcc generates
> a .note.gnu.property when PAC is enabled for the kernel but binutils
> doesn't understand it.
This is my understanding as well.

> Is there a way to check for this via the Kconfig
> options?
Maybe a version check on binutils could help here.

Cheers,
Daniel

> --
> Catalin

IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-23 11:48   ` Catalin Marinas
@ 2020-03-23 17:06     ` Amit Kachhap
  -1 siblings, 0 replies; 14+ messages in thread
From: Amit Kachhap @ 2020-03-23 17:06 UTC (permalink / raw)
  To: Catalin Marinas, kbuild test robot
  Cc: kbuild-all, Kees Cook, Szabolcs Nagy, Kristina Martsenko,
	Vincenzo Frascino, linux-arm-kernel, daniel.kiss

Hi Catalin,

On 3/23/20 5:18 PM, Catalin Marinas wrote:
> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>> config: arm64-randconfig-a001-20200321 (attached as .config)
>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>> reproduce:
>>          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>          chmod +x ~/bin/make.cross
>>          git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>          # save the attached .config to linux build tree
>>          GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>
>> If you fix the issue, kindly add following tag
>> Reported-by: kbuild test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>> --
>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> 
> Just to make sure I understand the cause of this warning: gcc generates
> a .note.gnu.property when PAC is enabled for the kernel but binutils
> doesn't understand it. Is there a way to check for this via the Kconfig
> options?

Yes you are right. In this case the binutils version lesser than 2.33 do 
not recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.
I tried somethings like below but could not find a good way to generate
error via Kconfig for this warning.

~/linux$ echo "int main(void) {return 0;}" > tmp.c | aarch64-linux-gcc 
-Wall -Werror -nostdlib -mbranch-protection=pac-ret+leaf -e main tmp.c 
-o tmp

~0day/gcc-9.2.0-nolibc/aarch64-linux/bin/../lib/gcc/aarch64-linux/9.2.0/../../../../aarch64-linux/bin/ld: 
warning: /tmp/ccExLvun.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

~/linux$ echo $?
~/linux$ 0

However we can disable pauth in this case by checking linker version
and GNU_PROPERTY feature as,
(readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge 
233000000).

But I feel disabling pauth in this case seems too aggressive as the 
current way do not break any functionality even though it generates 
confusing warning messages.

Cheers,
Amit
> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-23 17:06     ` Amit Kachhap
  0 siblings, 0 replies; 14+ messages in thread
From: Amit Kachhap @ 2020-03-23 17:06 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2509 bytes --]

Hi Catalin,

On 3/23/20 5:18 PM, Catalin Marinas wrote:
> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>> config: arm64-randconfig-a001-20200321 (attached as .config)
>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>> reproduce:
>>          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>          chmod +x ~/bin/make.cross
>>          git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>          # save the attached .config to linux build tree
>>          GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>
>> If you fix the issue, kindly add following tag
>> Reported-by: kbuild test robot <lkp@intel.com>
>>
>> All warnings (new ones prefixed by >>):
>>
>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>> --
>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> 
> Just to make sure I understand the cause of this warning: gcc generates
> a .note.gnu.property when PAC is enabled for the kernel but binutils
> doesn't understand it. Is there a way to check for this via the Kconfig
> options?

Yes you are right. In this case the binutils version lesser than 2.33 do 
not recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.
I tried somethings like below but could not find a good way to generate
error via Kconfig for this warning.

~/linux$ echo "int main(void) {return 0;}" > tmp.c | aarch64-linux-gcc 
-Wall -Werror -nostdlib -mbranch-protection=pac-ret+leaf -e main tmp.c 
-o tmp

~0day/gcc-9.2.0-nolibc/aarch64-linux/bin/../lib/gcc/aarch64-linux/9.2.0/../../../../aarch64-linux/bin/ld: 
warning: /tmp/ccExLvun.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000

~/linux$ echo $?
~/linux$ 0

However we can disable pauth in this case by checking linker version
and GNU_PROPERTY feature as,
(readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge 
233000000).

But I feel disabling pauth in this case seems too aggressive as the 
current way do not break any functionality even though it generates 
confusing warning messages.

Cheers,
Amit
> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-23 17:06     ` Amit Kachhap
@ 2020-03-24 16:15       ` Catalin Marinas
  -1 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-24 16:15 UTC (permalink / raw)
  To: Amit Kachhap
  Cc: kbuild-all, kbuild test robot, Szabolcs Nagy, daniel.kiss,
	Kristina Martsenko, Vincenzo Frascino, linux-arm-kernel,
	Kees Cook

On Mon, Mar 23, 2020 at 10:36:15PM +0530, Amit Kachhap wrote:
> On 3/23/20 5:18 PM, Catalin Marinas wrote:
> > On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
> > > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
> > > head:   3b446c7d27ddd06342901bb35211363f6944291a
> > > commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
> > > config: arm64-randconfig-a001-20200321 (attached as .config)
> > > compiler: aarch64-linux-gcc (GCC) 9.2.0
> > > reproduce:
> > >          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > >          chmod +x ~/bin/make.cross
> > >          git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
> > >          # save the attached .config to linux build tree
> > >          GCC_VERSION=9.2.0 make.cross ARCH=arm64
> > > 
> > > If you fix the issue, kindly add following tag
> > > Reported-by: kbuild test robot <lkp@intel.com>
> > > 
> > > All warnings (new ones prefixed by >>):
> > > 
> > > > > aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> > > --
> > > > > aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> > 
> > Just to make sure I understand the cause of this warning: gcc generates
> > a .note.gnu.property when PAC is enabled for the kernel but binutils
> > doesn't understand it. Is there a way to check for this via the Kconfig
> > options?
> 
> Yes you are right. In this case the binutils version lesser than 2.33 do not
> recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.

Are there gcc versions supporting PAC but not generating the
.note.gnu.property?

> However we can disable pauth in this case by checking linker version
> and GNU_PROPERTY feature as,
> (readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge
> 233000000).
> 
> But I feel disabling pauth in this case seems too aggressive as the current
> way do not break any functionality even though it generates confusing
> warning messages.

If there isn't a way to silence the linker or objdump, could we align
the compiler version we know that generates this note with a minimum
binutils? Something like:

	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)

-- 
Catalin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-24 16:15       ` Catalin Marinas
  0 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-24 16:15 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2479 bytes --]

On Mon, Mar 23, 2020 at 10:36:15PM +0530, Amit Kachhap wrote:
> On 3/23/20 5:18 PM, Catalin Marinas wrote:
> > On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
> > > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
> > > head:   3b446c7d27ddd06342901bb35211363f6944291a
> > > commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
> > > config: arm64-randconfig-a001-20200321 (attached as .config)
> > > compiler: aarch64-linux-gcc (GCC) 9.2.0
> > > reproduce:
> > >          wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> > >          chmod +x ~/bin/make.cross
> > >          git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
> > >          # save the attached .config to linux build tree
> > >          GCC_VERSION=9.2.0 make.cross ARCH=arm64
> > > 
> > > If you fix the issue, kindly add following tag
> > > Reported-by: kbuild test robot <lkp@intel.com>
> > > 
> > > All warnings (new ones prefixed by >>):
> > > 
> > > > > aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> > > --
> > > > > aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
> > 
> > Just to make sure I understand the cause of this warning: gcc generates
> > a .note.gnu.property when PAC is enabled for the kernel but binutils
> > doesn't understand it. Is there a way to check for this via the Kconfig
> > options?
> 
> Yes you are right. In this case the binutils version lesser than 2.33 do not
> recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.

Are there gcc versions supporting PAC but not generating the
.note.gnu.property?

> However we can disable pauth in this case by checking linker version
> and GNU_PROPERTY feature as,
> (readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge
> 233000000).
> 
> But I feel disabling pauth in this case seems too aggressive as the current
> way do not break any functionality even though it generates confusing
> warning messages.

If there isn't a way to silence the linker or objdump, could we align
the compiler version we know that generates this note with a minimum
binutils? Something like:

	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)

-- 
Catalin

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-24 16:15       ` Catalin Marinas
@ 2020-03-24 16:32         ` Amit Kachhap
  -1 siblings, 0 replies; 14+ messages in thread
From: Amit Kachhap @ 2020-03-24 16:32 UTC (permalink / raw)
  To: Catalin Marinas
  Cc: kbuild-all, kbuild test robot, Szabolcs Nagy, daniel.kiss,
	Kristina Martsenko, Vincenzo Frascino, linux-arm-kernel,
	Kees Cook



On 3/24/20 9:45 PM, Catalin Marinas wrote:
> On Mon, Mar 23, 2020 at 10:36:15PM +0530, Amit Kachhap wrote:
>> On 3/23/20 5:18 PM, Catalin Marinas wrote:
>>> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>>>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>>>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>>>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>>>> config: arm64-randconfig-a001-20200321 (attached as .config)
>>>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>>>> reproduce:
>>>>           wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>>           chmod +x ~/bin/make.cross
>>>>           git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>>>           # save the attached .config to linux build tree
>>>>           GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>>>
>>>> If you fix the issue, kindly add following tag
>>>> Reported-by: kbuild test robot <lkp@intel.com>
>>>>
>>>> All warnings (new ones prefixed by >>):
>>>>
>>>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>>>> --
>>>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>>>
>>> Just to make sure I understand the cause of this warning: gcc generates
>>> a .note.gnu.property when PAC is enabled for the kernel but binutils
>>> doesn't understand it. Is there a way to check for this via the Kconfig
>>> options?
>>
>> Yes you are right. In this case the binutils version lesser than 2.33 do not
>> recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.
> 
> Are there gcc versions supporting PAC but not generating the
> .note.gnu.property?

Yes GCC older than GCC9 supports PAC without the note (I will find out 
the exact GCC version).

> 
>> However we can disable pauth in this case by checking linker version
>> and GNU_PROPERTY feature as,
>> (readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge
>> 233000000).
>>
>> But I feel disabling pauth in this case seems too aggressive as the current
>> way do not break any functionality even though it generates confusing
>> warning messages.
> 
> If there isn't a way to silence the linker or objdump, could we align
> the compiler version we know that generates this note with a minimum
> binutils? Something like:
> 
> 	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)

Yes this is possible.

> 

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-24 16:32         ` Amit Kachhap
  0 siblings, 0 replies; 14+ messages in thread
From: Amit Kachhap @ 2020-03-24 16:32 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 2653 bytes --]



On 3/24/20 9:45 PM, Catalin Marinas wrote:
> On Mon, Mar 23, 2020 at 10:36:15PM +0530, Amit Kachhap wrote:
>> On 3/23/20 5:18 PM, Catalin Marinas wrote:
>>> On Sat, Mar 21, 2020 at 12:28:06PM +0800, kbuild test robot wrote:
>>>> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/kernel-ptrauth
>>>> head:   3b446c7d27ddd06342901bb35211363f6944291a
>>>> commit: 74afda4016a7437e6e425c3370e4b93b47be8ddf [16/18] arm64: compile the kernel with ptrauth return address signing
>>>> config: arm64-randconfig-a001-20200321 (attached as .config)
>>>> compiler: aarch64-linux-gcc (GCC) 9.2.0
>>>> reproduce:
>>>>           wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
>>>>           chmod +x ~/bin/make.cross
>>>>           git checkout 74afda4016a7437e6e425c3370e4b93b47be8ddf
>>>>           # save the attached .config to linux build tree
>>>>           GCC_VERSION=9.2.0 make.cross ARCH=arm64
>>>>
>>>> If you fix the issue, kindly add following tag
>>>> Reported-by: kbuild test robot <lkp@intel.com>
>>>>
>>>> All warnings (new ones prefixed by >>):
>>>>
>>>>>> aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>>>> --
>>>>>> aarch64-linux-objdump: warning: drivers/gpio/gpio-bt8xx.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
>>>
>>> Just to make sure I understand the cause of this warning: gcc generates
>>> a .note.gnu.property when PAC is enabled for the kernel but binutils
>>> doesn't understand it. Is there a way to check for this via the Kconfig
>>> options?
>>
>> Yes you are right. In this case the binutils version lesser than 2.33 do not
>> recognize the GNU_PROPERTY_AARCH64_FEATURE_1_PAC property.
> 
> Are there gcc versions supporting PAC but not generating the
> .note.gnu.property?

Yes GCC older than GCC9 supports PAC without the note (I will find out 
the exact GCC version).

> 
>> However we can disable pauth in this case by checking linker version
>> and GNU_PROPERTY feature as,
>> (readelf -S --wide tmp.o | grep .note.gnu.property) && (ld-ifversion -ge
>> 233000000).
>>
>> But I feel disabling pauth in this case seems too aggressive as the current
>> way do not break any functionality even though it generates confusing
>> warning messages.
> 
> If there isn't a way to silence the linker or objdump, could we align
> the compiler version we know that generates this note with a minimum
> binutils? Something like:
> 
> 	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)

Yes this is possible.

> 

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
  2020-03-24 16:32         ` Amit Kachhap
@ 2020-03-24 17:00           ` Catalin Marinas
  -1 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-24 17:00 UTC (permalink / raw)
  To: Amit Kachhap
  Cc: kbuild-all, kbuild test robot, Szabolcs Nagy, daniel.kiss,
	Kristina Martsenko, Vincenzo Frascino, linux-arm-kernel,
	Kees Cook

On Tue, Mar 24, 2020 at 10:02:16PM +0530, Amit Kachhap wrote:
> On 3/24/20 9:45 PM, Catalin Marinas wrote:
> > > On 3/23/20 5:18 PM, Catalin Marinas wrote:
> > > > Just to make sure I understand the cause of this warning: gcc generates
> > > > a .note.gnu.property when PAC is enabled for the kernel but binutils
> > > > doesn't understand it. Is there a way to check for this via the Kconfig
> > > > options?
[...]
> > If there isn't a way to silence the linker or objdump, could we align
> > the compiler version we know that generates this note with a minimum
> > binutils? Something like:
> > 
> > 	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)
> 
> Yes this is possible.

Note that LD_VERSION does not exist for kconfig. Please put together a
couple of patches here (and include clang versions as well) on top of
the arm64 for-next/kernel-ptrauth branch. They are not critical, we can
merge them after 5.7-rc1.

Thanks.

-- 
Catalin

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 14+ messages in thread

* Re: [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000
@ 2020-03-24 17:00           ` Catalin Marinas
  0 siblings, 0 replies; 14+ messages in thread
From: Catalin Marinas @ 2020-03-24 17:00 UTC (permalink / raw)
  To: kbuild-all

[-- Attachment #1: Type: text/plain, Size: 970 bytes --]

On Tue, Mar 24, 2020 at 10:02:16PM +0530, Amit Kachhap wrote:
> On 3/24/20 9:45 PM, Catalin Marinas wrote:
> > > On 3/23/20 5:18 PM, Catalin Marinas wrote:
> > > > Just to make sure I understand the cause of this warning: gcc generates
> > > > a .note.gnu.property when PAC is enabled for the kernel but binutils
> > > > doesn't understand it. Is there a way to check for this via the Kconfig
> > > > options?
[...]
> > If there isn't a way to silence the linker or objdump, could we align
> > the compiler version we know that generates this note with a minimum
> > binutils? Something like:
> > 
> > 	depends on (GCC_VERSION < X) || (LD_VERSION >= 233...)
> 
> Yes this is possible.

Note that LD_VERSION does not exist for kconfig. Please put together a
couple of patches here (and include clang versions as well) on top of
the arm64 for-next/kernel-ptrauth branch. They are not critical, we can
merge them after 5.7-rc1.

Thanks.

-- 
Catalin

^ permalink raw reply	[flat|nested] 14+ messages in thread

end of thread, other threads:[~2020-03-24 17:00 UTC | newest]

Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-03-21  4:28 [arm64:for-next/kernel-ptrauth 16/18] aarch64-linux-objdump: warning: fs/squashfs/decompressor_multi.o: unsupported GNU_PROPERTY_TYPE (5) type: 0xc0000000 kbuild test robot
2020-03-21  4:28 ` kbuild test robot
2020-03-23 11:48 ` Catalin Marinas
2020-03-23 11:48   ` Catalin Marinas
2020-03-23 17:04   ` Daniel Kiss
2020-03-23 17:04     ` Daniel Kiss
2020-03-23 17:06   ` Amit Kachhap
2020-03-23 17:06     ` Amit Kachhap
2020-03-24 16:15     ` Catalin Marinas
2020-03-24 16:15       ` Catalin Marinas
2020-03-24 16:32       ` Amit Kachhap
2020-03-24 16:32         ` Amit Kachhap
2020-03-24 17:00         ` Catalin Marinas
2020-03-24 17:00           ` Catalin Marinas

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.