linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kees Cook <keescook@chromium.org>
To: Josh Poimboeuf <jpoimboe@redhat.com>
Cc: Randy Dunlap <rdunlap@infradead.org>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	Linux Next Mailing List <linux-next@vger.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Peter Zijlstra <peterz@infradead.org>
Subject: Re: linux-next: Tree for Apr 23 (objtool warnings)
Date: Thu, 23 Apr 2020 12:24:05 -0700	[thread overview]
Message-ID: <202004231222.8A28DE1@keescook> (raw)
In-Reply-To: <20200423152921.shhjokjka7pwuyqr@treble>

On Thu, Apr 23, 2020 at 10:29:21AM -0500, Josh Poimboeuf wrote:
> On Thu, Apr 23, 2020 at 07:59:31AM -0700, Randy Dunlap wrote:
> > On 4/23/20 12:41 AM, Stephen Rothwell wrote:
> > > Hi all,
> > > 
> > > Changes since 20200422:
> > > 
> > 
> > on x86_64:
> 
> More truncation of functions by CONFIG_USBAN_TRAP.  Kees?

Looks like another randconfig with CONFIG_UBSAN_ALIGNMENT=y

I will test this and see if my proposed Kconfig change solves it.

-Kees

> 
> > drivers/target/target_core_xcopy.o: warning: objtool: target_xcopy_do_work()+0x6dd: unreachable instruction
> 
> 0000000000000170 <target_xcopy_read_source>:
>      170:	55                   	push   %rbp
>      171:	48 89 e5             	mov    %rsp,%rbp
>      174:	48 81 ec 40 03 00 00 	sub    $0x340,%rsp
>      17b:	65 48 8b 0c 25 28 00 	mov    %gs:0x28,%rcx
>      182:	00 00 
>      184:	48 89 4d f8          	mov    %rcx,-0x8(%rbp)
>      188:	31 c9                	xor    %ecx,%ecx
>      18a:	83 e2 07             	and    $0x7,%edx
>      18d:	75 34                	jne    1c3 <target_xcopy_read_source+0x53>
>      18f:	48 89 f0             	mov    %rsi,%rax
>      192:	83 e0 07             	and    $0x7,%eax
>      195:	75 2c                	jne    1c3 <target_xcopy_read_source+0x53>
>      197:	48 8d 95 d0 fc ff ff 	lea    -0x330(%rbp),%rdx
>      19e:	b9 65 00 00 00       	mov    $0x65,%ecx
>      1a3:	48 c7 c6 00 00 00 00 	mov    $0x0,%rsi
> 			1a6: R_X86_64_32S	.rodata.str1.1
>      1aa:	48 89 d7             	mov    %rdx,%rdi
>      1ad:	48 c7 c2 00 00 00 00 	mov    $0x0,%rdx
> 			1b0: R_X86_64_32S	.bss+0x560
>      1b4:	f3 48 ab             	rep stos %rax,%es:(%rdi)
>      1b7:	48 8d bd 48 ff ff ff 	lea    -0xb8(%rbp),%rdi
>      1be:	e8 00 00 00 00       	callq  1c3 <target_xcopy_read_source+0x53>
> 			1bf: R_X86_64_PLT32	__init_swait_queue_head-0x4
>      1c3:	0f 0b                	ud2    
>      1c5:	66 66 2e 0f 1f 84 00 	data16 nopw %cs:0x0(%rax,%rax,1)
>      1cc:	00 00 00 00 
> 
> > net/mac80211/rx.o: warning: objtool: __ieee80211_check_fast_rx_iface()+0x79: unreachable instruction
> 
> 0000000000007be0 <ieee80211_check_fast_rx>:
>     7be0:	40 f6 c7 07          	test   $0x7,%dil
>     7be4:	75 44                	jne    7c2a <ieee80211_check_fast_rx+0x4a>
>     7be6:	48 8b 57 50          	mov    0x50(%rdi),%rdx
>     7bea:	48 89 d0             	mov    %rdx,%rax
>     7bed:	83 e0 07             	and    $0x7,%eax
>     7bf0:	75 3c                	jne    7c2e <ieee80211_check_fast_rx+0x4e>
>     7bf2:	55                   	push   %rbp
>     7bf3:	b9 06 00 00 00       	mov    $0x6,%ecx
>     7bf8:	48 89 e5             	mov    %rsp,%rbp
>     7bfb:	48 83 ec 38          	sub    $0x38,%rsp
>     7bff:	48 8d 75 d0          	lea    -0x30(%rbp),%rsi
>     7c03:	48 89 f7             	mov    %rsi,%rdi
>     7c06:	f3 48 ab             	rep stos %rax,%es:(%rdi)
>     7c09:	48 8b 82 d8 03 00 00 	mov    0x3d8(%rdx),%rax
>     7c10:	48 89 45 d0          	mov    %rax,-0x30(%rbp)
>     7c14:	8b 82 68 0f 00 00    	mov    0xf68(%rdx),%eax
>     7c1a:	89 45 d8             	mov    %eax,-0x28(%rbp)
>     7c1d:	48 c7 c0 00 00 00 00 	mov    $0x0,%rax
> 			7c20: R_X86_64_32S	rfc1042_header
>     7c24:	a8 03                	test   $0x3,%al
>     7c26:	75 04                	jne    7c2c <ieee80211_check_fast_rx+0x4c>
>     7c28:	0f 0b                	ud2    
>     7c2a:	0f 0b                	ud2    
>     7c2c:	0f 0b                	ud2    
>     7c2e:	0f 0b                	ud2    
> 
> -- 
> Josh
> 

-- 
Kees Cook

      reply	other threads:[~2020-04-23 19:24 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-23  7:41 linux-next: Tree for Apr 23 Stephen Rothwell
2020-04-23 14:50 ` linux-next: Tree for Apr 23 (firmware_loader) Randy Dunlap
2020-04-23 20:16   ` Luis Chamberlain
2020-04-23 14:59 ` linux-next: Tree for Apr 23 (objtool warnings) Randy Dunlap
2020-04-23 15:29   ` Josh Poimboeuf
2020-04-23 19:24     ` Kees Cook [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202004231222.8A28DE1@keescook \
    --to=keescook@chromium.org \
    --cc=jpoimboe@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-next@vger.kernel.org \
    --cc=peterz@infradead.org \
    --cc=rdunlap@infradead.org \
    --cc=sfr@canb.auug.org.au \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).