code review 5519054: windows: use ArbitraryUserPointer as TLS slot

windows: use ArbitraryUserPointer as TLS slot

[vcc, 2012-01-06 08:40:54]

Hello hectorchu@gmail.com, alex.brainman@gmail.com (cc:
golang-dev@googlegroups.com),

I'd like you to review this change to
http://go.googlecode.com/hg/

[hector, 2012-01-06 08:48:06]

Can you confirm that it solves your ODBC problem Wei?

On 6 January 2012 08:40,  <vcc.163@gmail.com> wrote:
> Reviewers: hector, brainman,
>
> Message:
> Hello hectorchu@gmail.com, alex.brainman@gmail.com (cc:
> golang-dev@googlegroups.com),
>
> I'd like you to review this change to
> http://go.googlecode.com/hg/
>
>
> Description:
> windows: use ArbitraryUserPointer as TLS slot
>
> Please review this at http://codereview.appspot.com/5519054/
>
> Affected files:
>  M src/cmd/6l/pass.c
>  M src/cmd/8l/pass.c
>  M src/pkg/runtime/cgo/gcc_windows_386.c
>  M src/pkg/runtime/cgo/gcc_windows_amd64.c
>  M src/pkg/runtime/mkasmh.sh
>  M src/pkg/runtime/sys_windows_386.s
>  M src/pkg/runtime/sys_windows_amd64.s
>
>
> Index: src/cmd/6l/pass.c
> ===================================================================
> --- a/src/cmd/6l/pass.c
> +++ b/src/cmd/6l/pass.c
> @@ -276,7 +276,7 @@
>                        // Convert
>                        //   op   n(GS), reg
>                        // to
> -                       //   MOVL 0x58(GS), reg
> +                       //   MOVL 0x28(GS), reg
>                        //   op   n(reg), reg
>                        // The purpose of this patch is to fix some accesses
>                        // to extern register variables (TLS) on Windows, as
> @@ -291,7 +291,7 @@
>                                q->as = p->as;
>                                p->as = AMOVQ;
>                                p->from.type = D_INDIR+D_GS;
> -                               p->from.offset = 0x58;
> +                               p->from.offset = 0x28;
>                        }
>                }
>                if(HEADTYPE == Hlinux || HEADTYPE == Hfreebsd
> @@ -428,11 +428,11 @@
>                        p->from.offset = tlsoffset+0;
>                        p->to.type = D_CX;
>                        if(HEADTYPE == Hwindows) {
> -                               // movq %gs:0x58, %rcx
> +                               // movq %gs:0x28, %rcx
>                                // movq (%rcx), %rcx
>                                p->as = AMOVQ;
>                                p->from.type = D_INDIR+D_GS;
> -                               p->from.offset = 0x58;
> +                               p->from.offset = 0x28;
>                                p->to.type = D_CX;
>
>
> Index: src/cmd/8l/pass.c
> ===================================================================
> --- a/src/cmd/8l/pass.c
> +++ b/src/cmd/8l/pass.c
> @@ -259,7 +259,7 @@
>                                // Convert
>                                //   op   n(GS), reg
>                                // to
> -                               //   MOVL 0x2C(FS), reg
> +                               //   MOVL 0x14(FS), reg
>                                //   op   n(reg), reg
>                                // The purpose of this patch is to fix some
> accesses
>                                // to extern register variables (TLS) on
> Windows, as
> @@ -273,7 +273,7 @@
>                                        q->as = p->as;
>                                        p->as = AMOVL;
>                                        p->from.type = D_INDIR+D_FS;
> -                                       p->from.offset = 0x2C;
> +                                       p->from.offset = 0x14;
>                                }
>                        }
>                        if(HEADTYPE == Hlinux) {
> @@ -424,7 +424,7 @@
>                        case Hwindows:
>                                p->as = AMOVL;
>                                p->from.type = D_INDIR+D_FS;
> -                               p->from.offset = 0x2c;
> +                               p->from.offset = 0x14;
>                                p->to.type = D_CX;
>
>                                p = appendp(p);
> Index: src/pkg/runtime/cgo/gcc_windows_386.c
> ===================================================================
> --- a/src/pkg/runtime/cgo/gcc_windows_386.c
> +++ b/src/pkg/runtime/cgo/gcc_windows_386.c
> @@ -45,8 +45,8 @@
>         */
>        tls0 = (void*)LocalAlloc(LPTR, 32);
>        asm volatile (
> -               "movl %0, %%fs:0x2c\n"  // MOVL tls0, 0x2c(FS)
> -               "movl %%fs:0x2c, %%eax\n"       // MOVL 0x2c(FS), tmp
> +               "movl %0, %%fs:0x14\n"  // MOVL tls0, 0x2c(FS)
> +               "movl %%fs:0x14, %%eax\n"       // MOVL 0x2c(FS), tmp
>                "movl %1, 0(%%eax)\n"   // MOVL g, 0(FS)
>                "movl %2, 4(%%eax)\n"   // MOVL m, 4(FS)
>                :: "r"(tls0), "r"(ts.g), "r"(ts.m) : "%eax"
> Index: src/pkg/runtime/cgo/gcc_windows_amd64.c
> ===================================================================
> --- a/src/pkg/runtime/cgo/gcc_windows_amd64.c
> +++ b/src/pkg/runtime/cgo/gcc_windows_amd64.c
> @@ -45,8 +45,8 @@
>         */
>        tls0 = (void*)LocalAlloc(LPTR, 64);
>        asm volatile (
> -         "movq %0, %%gs:0x58\n"        // MOVL tls0, 0x58(GS)
> -         "movq %%gs:0x58, %%rax\n" // MOVQ 0x58(GS), tmp
> +         "movq %0, %%gs:0x28\n"        // MOVL tls0, 0x58(GS)
> +         "movq %%gs:0x28, %%rax\n" // MOVQ 0x58(GS), tmp
>          "movq %1, 0(%%rax)\n" // MOVQ g, 0(GS)
>          "movq %2, 8(%%rax)\n" // MOVQ m, 8(GS)
>          :: "r"(tls0), "r"(ts.g), "r"(ts.m) : "%rax"
> Index: src/pkg/runtime/mkasmh.sh
> ===================================================================
> --- a/src/pkg/runtime/mkasmh.sh
> +++ b/src/pkg/runtime/mkasmh.sh
> @@ -38,7 +38,7 @@
>        #       ../../libcgo/darwin_386.c:/^threadentry
>        case "$GOOS" in
>        windows)
> -               echo '#define   get_tls(r)      MOVL 0x2c(FS), r'
> +               echo '#define   get_tls(r)      MOVL 0x14(FS), r'
>                echo '#define   g(r)    0(r)'
>                echo '#define   m(r)    4(r)'
>                ;;
> @@ -82,7 +82,7 @@
>  amd64)
>        case "$GOOS" in
>        windows)
> -               echo '#define   get_tls(r) MOVQ 0x58(GS), r'
> +               echo '#define   get_tls(r) MOVQ 0x28(GS), r'
>                echo '#define   g(r) 0(r)'
>                echo '#define   m(r) 8(r)'
>                ;;
> Index: src/pkg/runtime/sys_windows_386.s
> ===================================================================
> --- a/src/pkg/runtime/sys_windows_386.s
> +++ b/src/pkg/runtime/sys_windows_386.s
> @@ -98,7 +98,7 @@
>        PUSHL   BX
>        PUSHL   SI
>        PUSHL   DI
> -       PUSHL   0x2c(FS)
> +       PUSHL   0x14(FS)
>        MOVL    SP, DX
>
>        // setup dummy m, g
> @@ -108,7 +108,7 @@
>        CALL    runtime·memclr(SB)      // smashes AX,BX,CX
>
>        LEAL    m_tls(SP), CX
> -       MOVL    CX, 0x2c(FS)
> +       MOVL    CX, 0x14(FS)
>        MOVL    SP, m(CX)
>        MOVL    SP, BX
>        SUBL    $g_end, SP              // space for G
> @@ -129,7 +129,7 @@
>        get_tls(CX)
>        MOVL    g(CX), CX
>        MOVL    g_stackbase(CX), SP
> -       POPL    0x2c(FS)
> +       POPL    0x14(FS)
>        POPL    DI
>        POPL    SI
>        POPL    BX
> @@ -208,7 +208,7 @@
>
>        // Set up tls.
>        LEAL    m_tls(CX), SI
> -       MOVL    SI, 0x2c(FS)
> +       MOVL    SI, 0x14(FS)
>        MOVL    CX, m(SI)
>        MOVL    DX, g(SI)
>
> @@ -246,5 +246,5 @@
>  // setldt(int entry, int address, int limit)
>  TEXT runtime·setldt(SB),7,$0
>        MOVL    address+4(FP), CX
> -       MOVL    CX, 0x2c(FS)
> +       MOVL    CX, 0x14(FS)
>        RET
> Index: src/pkg/runtime/sys_windows_amd64.s
> ===================================================================
> --- a/src/pkg/runtime/sys_windows_amd64.s
> +++ b/src/pkg/runtime/sys_windows_amd64.s
> @@ -121,7 +121,7 @@
>        PUSHQ   BX
>        PUSHQ   SI
>        PUSHQ   DI
> -       PUSHQ   0x58(GS)
> +       PUSHQ   0x28(GS)
>        MOVQ    SP, DX
>
>        // setup dummy m, g
> @@ -131,7 +131,7 @@
>        CALL    runtime·memclr(SB)      // smashes AX,BX,CX
>
>        LEAQ    m_tls(SP), CX
> -       MOVQ    CX, 0x58(GS)
> +       MOVQ    CX, 0x28(GS)
>        MOVQ    SP, m(CX)
>        MOVQ    SP, BX
>        SUBQ    $g_end, SP              // space for G
> @@ -152,7 +152,7 @@
>        get_tls(CX)
>        MOVQ    g(CX), CX
>        MOVQ    g_stackbase(CX), SP
> -       POPQ    0x58(GS)
> +       POPQ    0x28(GS)
>        POPQ    DI
>        POPQ    SI
>        POPQ    BX
> @@ -254,7 +254,7 @@
>
>        // Set up tls.
>        LEAQ    m_tls(CX), SI
> -       MOVQ    SI, 0x58(GS)
> +       MOVQ    SI, 0x28(GS)
>        MOVQ    CX, m(SI)
>        MOVQ    DX, g(SI)
>
> @@ -276,5 +276,5 @@
>  // set tls base to DI
>  TEXT runtime·settls(SB),7,$0
>        CALL    runtime·setstacklimits(SB)
> -       MOVQ    DI, 0x58(GS)
> +       MOVQ    DI, 0x28(GS)
>        RET
>
>

[vcc, 2012-01-06 12:59:28]

在 2012年1月6日星期五，Hector Chu <hectorchu@gmail.com> 写道：
> Can you confirm that it solves your ODBC problem Wei?

yes

>
> On 6 January 2012 08:40,  <vcc.163@gmail.com> wrote:
>> Reviewers: hector, brainman,
>>
>> Message:
>> Hello hectorchu@gmail.com, alex.brainman@gmail.com (cc:
>> golang-dev@googlegroups.com),
>>
>> I'd like you to review this change to
>> http://go.googlecode.com/hg/
>>
>>
>> Description:
>> windows: use ArbitraryUserPointer as TLS slot
>>
>> Please review this at http://codereview.appspot.com/5519054/
>>
>> Affected files:
>>  M src/cmd/6l/pass.c
>>  M src/cmd/8l/pass.c
>>  M src/pkg/runtime/cgo/gcc_windows_386.c
>>  M src/pkg/runtime/cgo/gcc_windows_amd64.c
>>  M src/pkg/runtime/mkasmh.sh
>>  M src/pkg/runtime/sys_windows_386.s
>>  M src/pkg/runtime/sys_windows_amd64.s
>>
>>
>> Index: src/cmd/6l/pass.c
>> ===================================================================
>> --- a/src/cmd/6l/pass.c
>> +++ b/src/cmd/6l/pass.c
>> @@ -276,7 +276,7 @@
>>                        // Convert
>>                        //   op   n(GS), reg
>>                        // to
>> -                       //   MOVL 0x58(GS), reg
>> +                       //   MOVL 0x28(GS), reg
>>                        //   op   n(reg), reg
>>                        // The purpose of this patch is to fix some
accesses
>>                        // to extern register variables (TLS) on Windows,
as
>> @@ -291,7 +291,7 @@
>>                                q->as = p->as;
>>                                p->as = AMOVQ;
>>                                p->from.type = D_INDIR+D_GS;
>> -                               p->from.offset = 0x58;
>> +                               p->from.offset = 0x28;
>>                        }
>>                }
>>                if(HEADTYPE == Hlinux || HEADTYPE == Hfreebsd
>> @@ -428,11 +428,11 @@
>>                        p->from.offset = tlsoffset+0;
>>                        p->to.type = D_CX;
>>                        if(HEADTYPE == Hwindows) {
>> -                               // movq %gs:0x58, %rcx
>> +                               // movq %gs:0x28, %rcx
>>                                // movq (%rcx), %rcx
>>                                p->as = AMOVQ;
>>                                p->from.type = D_INDIR+D_GS;
>> -                               p->from.offset = 0x58;
>> +                               p->from.offset = 0x28;
>>

[hector, 2012-01-06 16:19:07]

LGTM

[brainman, 2012-01-09 00:22:28]

LGTM

[brainman, 2012-01-09 00:23:17]

*** Submitted as http://code.google.com/p/go/source/detail?r=003182d3b2cd ***

windows: use ArbitraryUserPointer as TLS slot

R=hectorchu, alex.brainman
CC=golang-dev
http://codereview.appspot.com/5519054

Committer: Alex Brainman <alex.brainman@gmail.com>