Date: Tue, 7 Jul 2015 09:01:49 +0200
From: Ingo Molnar <mingo@kernel.org>
To: Andy Lutomirski <luto@amacapital.net>
Cc: Sasha Levin <sasha.levin@oracle.com>, Denys Vlasenko <dvlasenk@redhat.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Brian Gerst <brgerst@gmail.com>, Peter Zijlstra <peterz@infradead.org>,
        Borislav Petkov <bp@alien8.de>, "H. Peter Anvin" <hpa@zytor.com>,
        Linus Torvalds <torvalds@linux-foundation.org>,
        Oleg Nesterov <oleg@redhat.com>, Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH] x86/asm/entry/64: Clean up entry_64.S
Message-ID: <20150707070149.GA9784@gmail.com>
References: <1433752501-15901-1-git-send-email-mingo@kernel.org>
 <1433752501-15901-5-git-send-email-mingo@kernel.org>
 <5575954F.5070408@redhat.com>
 <20150608185106.GA28747@gmail.com>
 <559A9821.4010600@oracle.com>
 <CALCETrUNurOoDE7TXb3-Xnw-gx559c_q-Fe3qY6KyMCfdL2UDA@mail.gmail.com>
 <559ABC24.8040300@oracle.com>
 <20150706174144.GA30462@gmail.com>
 <CALCETrWjsq_j4AOmpKGUFSe9u0bPcVwwszvhcHjqjDRUtn_M6w@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CALCETrWjsq_j4AOmpKGUFSe9u0bPcVwwszvhcHjqjDRUtn_M6w@mail.gmail.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2243
Lines: 54


* Andy Lutomirski <luto@amacapital.net> wrote:

> On Mon, Jul 6, 2015 at 10:41 AM, Ingo Molnar <mingo@kernel.org> wrote:
> >
> > * Sasha Levin <sasha.levin@oracle.com> wrote:
> >
> >> On 07/06/2015 01:20 PM, Andy Lutomirski wrote:
> >> > On Mon, Jul 6, 2015 at 8:00 AM, Sasha Levin <sasha.levin@oracle.com> wrote:
> >> >> >
> >> >> > --- entry.before.o.cmd  2015-07-06 10:48:32.110189938 -0400
> >> >> > +++ entry.after.o.cmd   2015-07-06 10:48:23.509645442 -0400
> >> >> > @@ -1,5 +1,5 @@
> >> >> >
> >> >> > -entry.before.o:     file format elf64-x86-64
> >> >> > +entry.after.o:     file format elf64-x86-64
> >> >> >
> >> >> >
> >> >> >  Disassembly of section .entry.text:
> >> >> > @@ -3961,8 +3961,8 @@
> >> >> >    3b:  09 02                   or     %eax,(%rdx)
> >> >> >         ...
> >> >> >                         3d: R_X86_64_64 .entry.text
> >> >> > -  45:  03 3a                   add    (%rdx),%edi
> >> >> > -  47:  01 3d 03 d6 00 c8       add    %edi,-0x37ff29fd(%rip)        # ffffffffc800d650 <ignore_sysret+0xffffffffc800b1f0>
> >> >> > +  45:  03 33                   add    (%rbx),%esi
> >> >> > +  47:  01 3d 03 d5 00 c8       add    %edi,-0x37ff2afd(%rip)        # ffffffffc800d550 <ignore_sysret+0xffffffffc800b0f0>
> >> > What exactly are you doing to generate this diff?  This all looks really weird.
> >> >
> >> >> >    4d:  44 91                   rex.R xchg %eax,%ecx
> >> >> >    4f:  93                      xchg   %eax,%ebx
> >> >> >    50:  2f                      (bad)
> >> > For example: what on earth is the asm above?
> >>
> >> objdump...
> >
> > Oh, so I'm using 'objdump -d' to compare - but you probably used 'objdump
> > --disassemble-all', to disassemble .data sections as well?
> >
> 
> I can reproduce the difference now.  Give me a few minutes to see if I
> can figure out what's causing it.

Yeah, so I'm using -d instead of --disassemble-all to not have false positives on 
.data details such as line number sensitive debug info.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/