Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 14 Dec 2008 10:27:26 +0100
From:      Niclas Zeising <niclas.zeising@gmail.com>
To:        Garrett Cooper <yanefbsd@gmail.com>
Cc:        svn-src-head@freebsd.org, Joseph Koshy <jkoshy@freebsd.org>, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r186076 - head/sys/amd64/amd64
Message-ID:  <4944D17E.9080900@gmail.com>
In-Reply-To: <7d6fde3d0812140108l4b0c75e1o229c667c0b86dd7e@mail.gmail.com>
References:  <200812140906.mBE96SEZ054272@svn.freebsd.org> <7d6fde3d0812140108l4b0c75e1o229c667c0b86dd7e@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
Garrett Cooper wrote:
> On Sun, Dec 14, 2008 at 1:06 AM, Joseph Koshy <jkoshy@freebsd.org> wrote:
>> Author: jkoshy
>> Date: Sun Dec 14 09:06:28 2008
>> New Revision: 186076
>> URL: http://svn.freebsd.org/changeset/base/186076
>>
>> Log:
>>  Bug fix: %ebx needs to be preserved in the user callchain capture
>>  path.
>>
>> Modified:
>>  head/sys/amd64/amd64/exception.S
>>
>> Modified: head/sys/amd64/amd64/exception.S
>> ==============================================================================
>> --- head/sys/amd64/amd64/exception.S    Sun Dec 14 06:40:04 2008        (r186075)
>> +++ head/sys/amd64/amd64/exception.S    Sun Dec 14 09:06:28 2008        (r186076)
>> @@ -459,9 +459,9 @@ nmi_calltrap:
>>         */
>>        movq    %rsp,%rsi       /* source stack pointer */
>>        movq    $TF_SIZE,%rcx
>> -       movq    PCPU(RSP0),%rbx
>> -       subq    %rcx,%rbx
>> -       movq    %rbx,%rdi       /* destination stack pointer */
>> +       movq    PCPU(RSP0),%rdx
>> +       subq    %rcx,%rdx
>> +       movq    %rdx,%rdi       /* destination stack pointer */
>>
>>        shrq    $3,%rcx         /* trap frame size in long words */
>>        cld
>> @@ -470,7 +470,7 @@ nmi_calltrap:
>>
>>        movl    %ss,%eax
>>        pushq   %rax            /* tf_ss */
>> -       pushq   %rbx            /* tf_rsp (on kernel stack) */
>> +       pushq   %rdx            /* tf_rsp (on kernel stack) */
                    ^^^^^^
>>        pushfq                  /* tf_rflags */
>>        movl    %cs,%eax
>>        pushq   %rax            /* tf_cs */
> 
> Stupid question -- where's the change? All I see are potentially
> whitespace modifications..
> -Garrett

%rbx is changed to a %rdx in some places, I hilighted one.
Regards!
//Niclas




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4944D17E.9080900>