diff options
author | Russ Cox <rsc@golang.org> | 2019-05-08 13:16:05 -0400 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2019-05-09 21:13:19 +0000 |
commit | fe67ce2efb243b9289e81a61a5ccae79b75048f2 (patch) | |
tree | 88d9ceb8abb44da267f99e7e78b5532473002557 /src/cmd/vet | |
parent | 6ed2ec4aa5ea6c34b1e3d42b7995c81ab74a27c4 (diff) | |
download | go-fe67ce2efb243b9289e81a61a5ccae79b75048f2.tar.gz go-fe67ce2efb243b9289e81a61a5ccae79b75048f2.zip |
runtime: fix vet complaints for linux/amd64
Working toward making the tree vet-safe instead of having
so many exceptions in cmd/vet/all/whitelist.
This CL makes "GOOS=linux GOARCH=amd64 go vet -unsafeptr=false runtime" happy,
while keeping "GO_BUILDER_NAME=misc-vetall go tool dist test" happy too.
For #31916.
Change-Id: I4ca1acb02f4666b102d25fcc55fac96b8f80379a
Reviewed-on: https://go-review.googlesource.com/c/go/+/176100
Run-TryBot: Russ Cox <rsc@golang.org>
Reviewed-by: Austin Clements <austin@google.com>
Diffstat (limited to 'src/cmd/vet')
-rw-r--r-- | src/cmd/vet/all/whitelist/amd64.txt | 15 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/android_amd64.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/darwin_amd64.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/dragonfly_amd64.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/freebsd_amd64.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/illumos_amd64.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/linux_amd64.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/nacl_amd64p32.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/netbsd_386.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/netbsd_amd64.txt | 5 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/openbsd_386.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/openbsd_amd64.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/plan9_amd64.txt | 3 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/solaris_amd64.txt | 1 | ||||
-rw-r--r-- | src/cmd/vet/all/whitelist/windows_amd64.txt | 1 |
15 files changed, 0 insertions, 45 deletions
diff --git a/src/cmd/vet/all/whitelist/amd64.txt b/src/cmd/vet/all/whitelist/amd64.txt deleted file mode 100644 index 5ce30cc6c2..0000000000 --- a/src/cmd/vet/all/whitelist/amd64.txt +++ /dev/null @@ -1,15 +0,0 @@ -// amd64-specific vet whitelist. See readme.txt for details. - -// False positives. - -// reflect trampolines intentionally omit arg size. Same for morestack. -runtime/asm_amd64.s: [amd64] morestack: use of 8(SP) points beyond argument frame -runtime/asm_amd64.s: [amd64] morestack: use of 16(SP) points beyond argument frame -runtime/asm_amd64.s: [amd64] morestack: use of 8(SP) points beyond argument frame - -// Intentionally missing declarations. These are special assembly routines. -// Some are jumped into from other routines, with values in specific registers. -// duff* have direct calls from the compiler. -// Others use the platform ABI. -// There is no sensible corresponding Go prototype. -runtime/asm_amd64.s: [amd64] aeshashbody: function aeshashbody missing Go declaration diff --git a/src/cmd/vet/all/whitelist/android_amd64.txt b/src/cmd/vet/all/whitelist/android_amd64.txt deleted file mode 100644 index 90dabb0209..0000000000 --- a/src/cmd/vet/all/whitelist/android_amd64.txt +++ /dev/null @@ -1,3 +0,0 @@ -// android/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_linux_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/darwin_amd64.txt b/src/cmd/vet/all/whitelist/darwin_amd64.txt deleted file mode 100644 index fcdacb2dc1..0000000000 --- a/src/cmd/vet/all/whitelist/darwin_amd64.txt +++ /dev/null @@ -1,3 +0,0 @@ -// darwin/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_darwin_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/dragonfly_amd64.txt b/src/cmd/vet/all/whitelist/dragonfly_amd64.txt index 6c44159793..0db5503ef0 100644 --- a/src/cmd/vet/all/whitelist/dragonfly_amd64.txt +++ b/src/cmd/vet/all/whitelist/dragonfly_amd64.txt @@ -1,6 +1,5 @@ // dragonfly/amd64-specific vet whitelist. See readme.txt for details. -runtime/sys_dragonfly_amd64.s: [amd64] settls: function settls missing Go declaration syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 8(SP) should be num+0(FP) syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 16(SP) should be a1+8(FP) diff --git a/src/cmd/vet/all/whitelist/freebsd_amd64.txt b/src/cmd/vet/all/whitelist/freebsd_amd64.txt index a910f48ca5..356be022cc 100644 --- a/src/cmd/vet/all/whitelist/freebsd_amd64.txt +++ b/src/cmd/vet/all/whitelist/freebsd_amd64.txt @@ -1,6 +1,5 @@ // freebsd/amd64-specific vet whitelist. See readme.txt for details. -runtime/sys_freebsd_amd64.s: [amd64] settls: function settls missing Go declaration syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 8(SP) should be num+0(FP) syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 16(SP) should be a1+8(FP) syscall/asm9_unix2_amd64.s: [amd64] Syscall9: 24(SP) should be a2+16(FP) diff --git a/src/cmd/vet/all/whitelist/illumos_amd64.txt b/src/cmd/vet/all/whitelist/illumos_amd64.txt index 4525c03cd7..30f3912e3e 100644 --- a/src/cmd/vet/all/whitelist/illumos_amd64.txt +++ b/src/cmd/vet/all/whitelist/illumos_amd64.txt @@ -1,6 +1,5 @@ // illumos/amd64-specific vet whitelist. See readme.txt for details. -runtime/sys_solaris_amd64.s: [amd64] settls: function settls missing Go declaration runtime/sys_solaris_amd64.s: [amd64] pipe1: function pipe1 missing Go declaration runtime/sys_solaris_amd64.s: [amd64] asmsysvicall6: function asmsysvicall6 missing Go declaration runtime/sys_solaris_amd64.s: [amd64] usleep2: function usleep2 missing Go declaration diff --git a/src/cmd/vet/all/whitelist/linux_amd64.txt b/src/cmd/vet/all/whitelist/linux_amd64.txt deleted file mode 100644 index 2f4a933553..0000000000 --- a/src/cmd/vet/all/whitelist/linux_amd64.txt +++ /dev/null @@ -1,3 +0,0 @@ -// linux/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_linux_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/nacl_amd64p32.txt b/src/cmd/vet/all/whitelist/nacl_amd64p32.txt index b3d6aa40a1..fb95a360a6 100644 --- a/src/cmd/vet/all/whitelist/nacl_amd64p32.txt +++ b/src/cmd/vet/all/whitelist/nacl_amd64p32.txt @@ -10,7 +10,6 @@ runtime/sys_nacl_amd64p32.s: [amd64p32] sigtramp: unknown variable ctxt runtime/sys_nacl_amd64p32.s: [amd64p32] sigtramp: unknown variable ctxt runtime/sys_nacl_amd64p32.s: [amd64p32] nacl_sysinfo: function nacl_sysinfo missing Go declaration runtime/sys_nacl_amd64p32.s: [amd64p32] nacl_clock_gettime: function nacl_clock_gettime missing Go declaration -runtime/sys_nacl_amd64p32.s: [amd64p32] settls: function settls missing Go declaration // Clearer using FP than SP, but that requires named offsets. runtime/asm_amd64p32.s: [amd64p32] rt0_go: unknown variable argc diff --git a/src/cmd/vet/all/whitelist/netbsd_386.txt b/src/cmd/vet/all/whitelist/netbsd_386.txt index a85c6e0f78..608021cae4 100644 --- a/src/cmd/vet/all/whitelist/netbsd_386.txt +++ b/src/cmd/vet/all/whitelist/netbsd_386.txt @@ -1,8 +1,5 @@ // netbsd/386-specific vet whitelist. See readme.txt for details. -runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] settls: function settls missing Go declaration -runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] sigreturn_tramp: function sigreturn_tramp missing Go declaration - runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 140(SP) points beyond argument frame runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 4(SP) points beyond argument frame runtime/sys_netbsd_386.s: [386] sigreturn_tramp: use of 4(SP) points beyond argument frame diff --git a/src/cmd/vet/all/whitelist/netbsd_amd64.txt b/src/cmd/vet/all/whitelist/netbsd_amd64.txt deleted file mode 100644 index 23b6a8c0ad..0000000000 --- a/src/cmd/vet/all/whitelist/netbsd_amd64.txt +++ /dev/null @@ -1,5 +0,0 @@ -// netbsd/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_netbsd_ARCHSUFF.s: [GOARCH] sigreturn_tramp: function sigreturn_tramp missing Go declaration - -runtime/sys_netbsd_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/openbsd_386.txt b/src/cmd/vet/all/whitelist/openbsd_386.txt index 66172ff008..bfbf4d9c56 100644 --- a/src/cmd/vet/all/whitelist/openbsd_386.txt +++ b/src/cmd/vet/all/whitelist/openbsd_386.txt @@ -3,7 +3,6 @@ runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable signo runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable info runtime/sys_openbsd_386.s: [386] sigtramp: unknown variable context -runtime/sys_openbsd_386.s: [386] settls: function settls missing Go declaration syscall/asm_unix_386.s: [386] Syscall: 8(SP) should be a1+4(FP) syscall/asm_unix_386.s: [386] Syscall: 4(SP) should be trap+0(FP) syscall/asm_unix_386.s: [386] Syscall6: 8(SP) should be a1+4(FP) diff --git a/src/cmd/vet/all/whitelist/openbsd_amd64.txt b/src/cmd/vet/all/whitelist/openbsd_amd64.txt deleted file mode 100644 index 433f62ca07..0000000000 --- a/src/cmd/vet/all/whitelist/openbsd_amd64.txt +++ /dev/null @@ -1,3 +0,0 @@ -// openbsd/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_openbsd_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/plan9_amd64.txt b/src/cmd/vet/all/whitelist/plan9_amd64.txt deleted file mode 100644 index 1390475c77..0000000000 --- a/src/cmd/vet/all/whitelist/plan9_amd64.txt +++ /dev/null @@ -1,3 +0,0 @@ -// plan9/amd64-specific vet whitelist. See readme.txt for details. - -runtime/sys_plan9_amd64.s: [amd64] settls: function settls missing Go declaration diff --git a/src/cmd/vet/all/whitelist/solaris_amd64.txt b/src/cmd/vet/all/whitelist/solaris_amd64.txt index 26a9da4271..19f66f7588 100644 --- a/src/cmd/vet/all/whitelist/solaris_amd64.txt +++ b/src/cmd/vet/all/whitelist/solaris_amd64.txt @@ -1,6 +1,5 @@ // solaris/amd64-specific vet whitelist. See readme.txt for details. -runtime/sys_solaris_amd64.s: [amd64] settls: function settls missing Go declaration runtime/sys_solaris_amd64.s: [amd64] pipe1: function pipe1 missing Go declaration runtime/sys_solaris_amd64.s: [amd64] asmsysvicall6: function asmsysvicall6 missing Go declaration runtime/sys_solaris_amd64.s: [amd64] usleep2: function usleep2 missing Go declaration diff --git a/src/cmd/vet/all/whitelist/windows_amd64.txt b/src/cmd/vet/all/whitelist/windows_amd64.txt index 4e6da405e3..348afd152e 100644 --- a/src/cmd/vet/all/whitelist/windows_amd64.txt +++ b/src/cmd/vet/all/whitelist/windows_amd64.txt @@ -3,4 +3,3 @@ runtime/sys_windows_amd64.s: [amd64] ctrlhandler: RET without writing to 4-byte ret+8(FP) runtime/sys_windows_amd64.s: [amd64] callbackasm1: function callbackasm1 missing Go declaration runtime/sys_windows_amd64.s: [amd64] tstart_stdcall: RET without writing to 4-byte ret+8(FP) -runtime/sys_windows_amd64.s: [amd64] settls: function settls missing Go declaration |