Commit Graph
40 Commits
Author SHA1 Message Date
Keith Randall 7138967c19 windows: fix slicing of NTUnicodeString values
We were slicing using a count of bytes, not a count of uint16s.

Fixes golang/go#73460

Change-Id: If0fd19e795078c01fda5b976e3c34af115b25dcc
Reviewed-on: https://go-review.googlesource.com/c/sys/+/667235
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Junyang Shao <shaojunyang@google.com>
Reviewed-by: Quim Muntal <quimmuntal@gmail.com>
Auto-Submit: Keith Randall <khr@google.com>
Reviewed-by: Keith Randall <khr@google.com>
2025-04-22 08:26:09 -07:00
Dmitri Shuralyov cff53d5a33 unix: gofmt after CL 610296
Also remove a few unhelpful blank lines.

Change-Id: I4001a25ec409406855b7800afd88738c43f60b90
Reviewed-on: https://go-review.googlesource.com/c/sys/+/621921
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
2024-10-25 18:35:15 +00:00
Dmitri Shuralyov 68ed59b20b windows/svc: fix printf(var) mistake detected by latest printf checker
For golang/go#69267.

Change-Id: Ie240b5c826bb96c0e2021a7e99a3c0f973f0a0e1
Reviewed-on: https://go-review.googlesource.com/c/sys/+/610940
Reviewed-by: Tim King <taking@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2024-09-06 16:31:18 +00:00
Dmitri Shuralyov 1bfbee0e20 all: update go directive to 1.18
Done with:

go get go@1.18
go mod tidy
go fix ./...

Using go1.21.3.

Also update code generators to use only the new go:build lines,
not the old +build ones.

For golang/go#60268.

Change-Id: I6aabc42efb6ab3329981100e1db2263aac5e92a6
Reviewed-on: https://go-review.googlesource.com/c/sys/+/534222
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
2023-10-11 21:54:30 +00:00
Dmitri Shuralyov 81cb935067 unix, windows: use ^TestName$ regular pattern for invoking a single test
For reasons motivated in the commit message of CL 524948.

Change-Id: I0085411f30a77634962239ee36d15e7b6ef893ef
Reviewed-on: https://go-review.googlesource.com/c/sys/+/525618
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Ian Lance Taylor <iant@google.com>
2023-09-06 00:53:53 +00:00
Keith Randall 7861aae155 cpu: pretend AVX-512 is disabled on Darwin
Darwin doesn't save/restore the AVX-512 mask registers when
processing a signal. Go generates lots of signals, which cause
the mask registers to randomly get clobbered. Better to pretend
that AVX-512 isn't supported.

Update golang/go#49233

Change-Id: I0dda77a6f205e10eefccd82e6f4f0f184dae42d9
Reviewed-on: https://go-review.googlesource.com/c/sys/+/361255
Trust: Keith Randall <khr@golang.org>
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Martin Möhrmann <martin@golang.org>
2021-11-03 23:57:46 +00:00
Keith Randall 6f8348627a unix: fix darwin pipe implementation
The raw syscall returned the two pipes whereas the libc call
takes a pointer to a location to write the two pipes.
When we switched over from raw syscalls to libc calls, this
change in behavior was missed.

Fixes golang/go#43498

Change-Id: Icee2204dcb8be8fc94be0df106e1ff061cafa446
Reviewed-on: https://go-review.googlesource.com/c/sys/+/281432
Trust: Keith Randall <khr@golang.org>
Run-TryBot: Keith Randall <khr@golang.org>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
2021-01-04 20:47:34 +00:00
Sameer Ajmani 0ed46b70d9 unix: delete asm.s
This contains a stub for func "use", which was deleted.

See issue https://golang.org/issue/16607

Change-Id: I7165ca55ad956ac449e950f2be6f53cfeb8e6ffa
Reviewed-on: https://go-review.googlesource.com/36720
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
2017-02-13 21:28:12 +00:00
Sameer Ajmani aaabbdc969 x/sys/unix: remove the "use" function and its calls.
Update mksyscall*.pl to skip generating calls to "use".
Make build tag handling match what's used in the syscall package.

See issue https://golang.org/issue/16607
and CL https://golang.org/cl/36616

Change-Id: I2d2e823fe1846d2110c07eb1bf976852706ecf60
Reviewed-on: https://go-review.googlesource.com/36715
Run-TryBot: Sameer Ajmani <sameer@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
2017-02-10 16:30:38 +00:00
Rob Pike 3826714b1e x/sys/unix: allow nil on unix for all variants of Utimes
Fixes golang/go#11830.

Change-Id: Ie6e9f82d05b7c04092b6ee6117238873b746380e
Reviewed-on: https://go-review.googlesource.com/12690
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
2015-07-27 05:51:57 +00:00
Rob Pike c1f6f8bcc5 go.sys/unix: use "use" in syscall interface
LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/144980043
2014-09-16 13:36:53 -07:00
Rob Pike c83757ae5b go.sys/unix/mkerrors.sh: restore missing include blocks
Dropped (or not moved) for some unknown reason. Restore them.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/140680044
2014-09-16 10:48:40 -07:00
Rob Pike 6d0305dc7d go.sys/unix: undo CL 119530044's effects in go.sys for BSD
That CL worked around a bug present in the
OS X Yosemite Public Beta versions 1 and 2.
Beta 3 (released today) has fixed the bug.

LGTM=bradfitz
R=golang-codereviews, bradfitz
CC=golang-codereviews
https://golang.org/cl/144010043
2014-09-16 10:39:57 -07:00
Rob Pike 417a323d1f go.sys/plan9: define "use" in plan9/syscall.go
LGTM=0intro
R=rsc, 0intro
CC=golang-codereviews
https://golang.org/cl/141870043
2014-09-09 11:25:43 -07:00
Rob Pike 477fd0357c go.sys/plan9: update to use "use"
LGTM=rsc, aram, 0intro
R=rsc, aram, 0intro
CC=golang-codereviews
https://golang.org/cl/138280043
2014-09-09 11:02:42 -07:00
Rob Pike fcfc1156f8 go.sys/plan9: delete Exec etc.
These are being deleted from go.sys because in general they can
only be implemented in close coordination with the runtime.

LGTM=0intro
R=golang-codereviews, 0intro
CC=golang-codereviews
https://golang.org/cl/132810043
2014-08-22 15:33:52 -07:00
Rob Pike 866b9d8a48 go.sys/unix: delete Fork and Exec etc.
These are being deleted from go.sys because in general they can
only be implemented in close coordination with the runtime.

LGTM=dave
R=rsc, dave, josharian
CC=golang-codereviews
https://golang.org/cl/129500043
2014-08-21 16:14:51 -07:00
Rob Pike 9782d98bb0 go.sys/windows: delete StartProcess and Exec
These are being deleted from go.sys because in general they can
only be implemented in close coordination with the runtime.

LGTM=alex.brainman
R=alex.brainman
CC=golang-codereviews
https://golang.org/cl/132010043
2014-08-21 16:07:11 -07:00
Rob Pike dc3c21c62b go.sys/unix: use syscall.Errno for errors
If we use a local type, it won't compare properly with errors from
the rest of the standard library. Errors are the one type from syscall
that propagates through the system, so it's important to have only
one type for them.

Ditto for syscall.Signal.

LGTM=dave
R=rsc, dave
CC=golang-codereviews
https://golang.org/cl/123490043
2014-08-15 09:57:24 -07:00
Rob Pike 96cf2ceb2c go.sys/unix: arm uses B not JMP
Fix build.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/123510043
2014-08-14 17:39:12 -07:00
Rob Pike aec3c6a331 go.sys/unix: just jump to syscall.Syscall etc.
Don't re-implement these functions.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/124390043
2014-08-14 12:30:17 -07:00
Rob Pike ba5045e1e0 go.sys/plan9: just jump to syscall.Syscall etc.
Don't re-implement these functions.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/124350045
2014-08-14 12:05:46 -07:00
Rob Pike 4faa336c56 go.sys/windows: use syscall.Errno for windows errors
If we use a local type, it won't compare properly with errors from
the rest of the standard library. Errors are the one type from syscall
that propagates through the system, so it's important to have only
one type for them.

mkerrors_windows.sh is gone, so:
 - rename zerrors_windows.go and delete its DO NOT EDIT mark
 - delete the contentless zerrors_windows_*.go files

LGTM=rsc
R=rsc
CC=alex.brainman, golang-codereviews
https://golang.org/cl/122600043
2014-08-14 10:28:19 -07:00
Rob Pike b87d025ac4 go.sys/plan9: use syscall.ErrorString for errors
If we use a local type, it won't compare properly with errors from
the rest of the standard library. Errors are the one type from syscall
that propagates through the system, so it's important to have only
one type for them.
Will apply the corresponding change to the other packages once
this is approved.

Also delete some deprecated code. This is a new package; no need
to carry the past.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/126250043
2014-08-14 09:29:35 -07:00
Rob Pike 0c07981137 go.sys/unix: add asm hooks for solaris to runtime-resident syscall help
also: race is supported on freebsd, so fix a couple of build tags

LGTM=minux
R=rsc, minux
CC=golang-codereviews
https://golang.org/cl/124330043
2014-08-13 20:54:29 -07:00
Rob Pike 73a4d37962 go.sys/windows: import syscall
Simple oversight.

LGTM=alex.brainman
R=alex.brainman
CC=golang-codereviews, rsc
https://golang.org/cl/121660043
2014-08-13 17:45:19 -07:00
Rob Pike 9828c4dfac go.sys/unix: fix creds_test.go
For this particular system call, which uses the net package,
the expected error is syscall.EPERM not unix.EPERM.

With this change, go test passes on my linux.

LGTM=iant, rsc
R=rsc, iant
CC=golang-codereviews
https://golang.org/cl/125270043
2014-08-13 21:07:22 +00:00
Rob Pike 7118195da3 go.sys/windows: grab Syscall interface from the standard syscall package
Also provide stubs to call loadlibrary and getprocaddress, which are implemented
for package syscall by runtime/syscall_windows.goc.
Should get the windows go.sys build closer to green.

LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://golang.org/cl/126170043
2014-08-13 14:08:39 -07:00
Rob Pike 9e072b55ca go.sys: add solaris build tags
They were missed yesterday when adding the tags for all the other unix variants.

LGTM=aram
R=rsc, aram
CC=golang-codereviews
https://golang.org/cl/126150043
2014-08-13 10:02:35 -07:00
Rob Pike 11f12da052 go.sys/unix: implement the environment functions by wrapping syscall
The environment is global state that is owned by the standard syscall package.
With this change, go test passes on darwin in the unix directory.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/129900043
2014-08-13 09:00:28 -07:00
Rob Pike 279b3782ad go.sys: fix imports to cgc/p/go.sys, add build tags
Approach but probably not achieve a green build.

LGTM=adg
R=golang-codereviews, adg
CC=golang-codereviews
https://golang.org/cl/126120043
2014-08-12 22:59:00 -07:00
Rob Pike 55e2a23348 go.sys/unix: delete nacl
It's a peculiar environment that probably doesn't belong here.
We can bring it back easily if we need it.

LGTM=dave, rsc
R=rsc, dave
CC=golang-codereviews
https://golang.org/cl/128110043
2014-08-12 17:00:33 -07:00
Rob Pike 8768103c90 go.sys/all: update #includes to new location
Darwin/amd64 now builds. No others tested yet.
mkall.sh also runs correctly. The updates, which are
substantial, are not yet applied to avoid unnecessary churn.
Restore mkerrors.sh, which is needed in the unix directory.

LGTM=rsc
R=rsc
CC=golang-codereviews
https://golang.org/cl/122410044
2014-08-12 14:05:03 -07:00
Rob Pike 8442dd2c63 go.sys: update package names
Semi-automatic migration from package syscall to package {plan9,windows,unix}.
No builds attempted yet, but this gets a lot of noise behind us so subsequent
CLs will be more concise and easier to follow.
Subsequent CLs will have semantic content.

LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://golang.org/cl/121520043
2014-08-11 15:58:26 -07:00
Rob Pike 20acc5cf31 go.sys: copy files from syscall package to go.sys/{plan9,windows,unix}
This CL copies to each package of go.sys the files from syscall it will need.
Different directories have different files, but these:
        mkall.sh
        str.go
        syscall.go
        mksyscall.pl
        race.go
        race0.go
        syscall_test.go
are copied to all three.
No changes yet, these are just copies. They are not ready to use yet:
package names are wrong, for starters. But this clean copy will make
it easier to follow the changes as the packages are enabled.

LGTM=rsc
R=golang-codereviews, rsc
CC=golang-codereviews
https://golang.org/cl/126960043
2014-08-11 14:48:46 -07:00
Rob Pike 830b7a2031 go.sys: create new repository
See http://golang.org/s/go1.4-syscall for background.

LGTM=iant
R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/123130043
2014-08-08 14:33:53 -07:00
Rob Pike 1fabe7702b .hgignore: add comment setting policy
R=golang-codereviews, iant
CC=golang-codereviews
https://golang.org/cl/44810044
2013-12-20 16:28:21 -08:00
Rob Pike 3ead892b8d go.empty: restore the README
I edited the wrong README when cloning the empty repo to create the tools one.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/9486043
2013-05-17 12:11:15 -07:00
Rob Pike f4917bf174 go.tools: create subrepository
This subrepo is the ultimate home for some of the experimental components
being developed in go.exp now, and also existing pieces such as go vet.

R=golang-dev, bradfitz
CC=golang-dev
https://golang.org/cl/9481043
2013-05-17 11:52:26 -07:00
Rob Pike b38d45f7ff LICENSE: add
R=golang-dev, dsymonds
CC=golang-dev
https://golang.org/cl/5845057
2012-03-17 15:20:58 +11:00