Re: [PATCH] x86/asm/entry/64: Clean up entry_64.S

From: Ingo Molnar
Date: Tue Jul 07 2015 - 03:02:04 EST



* Andy Lutomirski <luto@xxxxxxxxxxxxxx> wrote:

> On Mon, Jul 6, 2015 at 10:41 AM, Ingo Molnar <mingo@xxxxxxxxxx> wrote:
> >
> > * Sasha Levin <sasha.levin@xxxxxxxxxx> wrote:
> >
> >> On 07/06/2015 01:20 PM, Andy Lutomirski wrote:
> >> > On Mon, Jul 6, 2015 at 8:00 AM, Sasha Levin <sasha.levin@xxxxxxxxxx> 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/