summaryrefslogtreecommitdiff
path: root/src/os
Commit message (Collapse)AuthorAgeFilesLines
* all: power64 is now ppc64Russ Cox2014-12-051-3/+3
| | | | | | | | | Fixes issue 8654. LGTM=austin R=austin CC=golang-codereviews https://codereview.appspot.com/180600043
* [dev.cc] all: merge dev.power64 (7667e41f3ced) into dev.ccRuss Cox2014-11-144-2/+5
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is to reduce the delta between dev.cc and dev.garbage to just garbage collector changes. These are the files that had merge conflicts and have been edited by hand: malloc.go mem_linux.go mgc.go os1_linux.go proc1.go panic1.go runtime1.go LGTM=austin R=austin CC=golang-codereviews https://codereview.appspot.com/174180043
| * os: document that users of Fd should keep f aliveRuss Cox2014-11-063-1/+4
| | | | | | | | | | | | | | | | | | Fixes issue 9046. LGTM=r R=r CC=golang-codereviews https://codereview.appspot.com/162680043
| * os/exec: tell lsof not to blockKeith Randall2014-11-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | For some reason lsof is now hanging on my workstation without the -b (avoid blocking in the kernel) option. Adding -b makes the test pass and shouldn't hurt. I don't know how recent the -b option is. If the builders are ok with it, it's probably ok. LGTM=rsc R=golang-codereviews, bradfitz, rsc CC=golang-codereviews https://codereview.appspot.com/166220043
* | [dev.power64] all: merge default (dd5014ed9b01) into dev.power64Russ Cox2014-10-291-1/+7
|\ \ | |/ |/| | | | | | | | | | | | | Still passes on amd64. LGTM=austin R=austin CC=golang-codereviews https://codereview.appspot.com/165110043
| * [dev.power64] all: merge default into dev.power64Austin Clements2014-10-221-1/+7
| |\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This brings dev.power64 up-to-date with the current tip of default. go_bootstrap is still panicking with a bad defer when initializing the runtime (even on amd64). LGTM=rsc R=rsc CC=golang-codereviews https://codereview.appspot.com/152570049
| | * build: merge the great pkg/ rename into dev.power64Austin Clements2014-10-221-1/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | This also removes pkg/runtime/traceback_lr.c, which was ported to Go in an earlier commit and then moved to runtime/traceback.go. Reviewer: rsc@golang.org rsc: LGTM
* | | os: fix write on Plan 9David du Colombier2014-10-281-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In CL 160670043 the write function was changed so a zero-length write is now allowed. This leads the ExampleWriter_Init test to fail. The reason is that Plan 9 preserves message boundaries, while the os library expects systems that don't preserve them. We have to ignore zero-length writes so they will never turn into EOF. This issue was previously discussed in CL 7406046. LGTM=bradfitz R=rsc, bradfitz CC=golang-codereviews https://codereview.appspot.com/163510043
* | | os: fix buildRuss Cox2014-10-281-1/+1
| | | | | | | | | | | | | | | | | | TBR=crawshaw CC=golang-codereviews https://codereview.appspot.com/162390043
* | | os: do not assume syscall i/o funcs return n=0 on errorRuss Cox2014-10-286-16/+22
|/ / | | | | | | | | | | | | | | | | Fixes issue 9007. LGTM=iant, r R=r, iant CC=golang-codereviews https://codereview.appspot.com/160670043
* | os/exec: fix number of expected file descriptors on Plan 9David du Colombier2014-10-201-9/+1
| | | | | | | | | | | | | | | | | | | | Since CL 104570043 and 112720043, we are using the nsec system call instead of /dev/bintime on Plan 9. LGTM=rsc R=rsc CC=aram, golang-codereviews https://codereview.appspot.com/155590043
* | os/exec: document that Stdin goroutine must finish in WaitRuss Cox2014-10-151-2/+9
| | | | | | | | | | | | | | | | | | Fixes issue 7990. LGTM=iant, bradfitz R=bradfitz, iant, robryk CC=golang-codereviews https://codereview.appspot.com/156220043
* | os: handle 'no parent' error as IsNotExist on Plan 9David du Colombier2014-10-131-1/+2
| | | | | | | | | | | | | | | | | | | | | | This error is returned by lib9p when removing a file without parent. It should fix TestRemoveAllRace when running on ramfs. LGTM=bradfitz, aram R=rsc, bradfitz, aram CC=golang-codereviews, mischief https://codereview.appspot.com/153410044
* | os, syscall: test Chtimes on directories, fix on WindowsRuss Cox2014-10-061-9/+38
| | | | | | | | | | | | | | | | | | Fixes issue 8090. LGTM=alex.brainman R=alex.brainman CC=golang-codereviews https://codereview.appspot.com/154020043
* | os: recomment MkdirAllRuss Cox2014-10-061-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | The internal comments are not completely precise about what is going on, and they are causing confusion. Fixes issue 8283. LGTM=r R=r CC=golang-codereviews https://codereview.appspot.com/151460043
* | os: make Process.Signal 'process finished' error consistent on UnixRuss Cox2014-10-062-3/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While we're here, fix the implementation of Release on both Unix and Windows: Release is supposed to make Signal an error. While we're here, make sure we never Signal pid 0. (Don't try this at home.) Fixes issue 7658. LGTM=r R=golang-codereviews, r CC=golang-codereviews, iant https://codereview.appspot.com/152240043
* | os, syscall: add UnsetenvBrad Fitzpatrick2014-10-012-0/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | Also address a TODO, making Clearenv pass through to cgo. Based largely on Minux's earlier https://codereview.appspot.com/82040044 Fixes Issue 6423 LGTM=iant, alex.brainman, r, rsc R=rsc, iant, r, alex.brainman CC=golang-codereviews https://codereview.appspot.com/148370043
* | os: fix Args setup on WindowsBrad Fitzpatrick2014-09-241-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Should fix the Windows build. Untested. on Windows, args are made by src/os/exec_windows.go, not package runtime. runtime?goargs has if(Windows) return; The two init funcs in pkg os were conflicting, with the second overwriting Args back to an empty slice. LGTM=rsc R=rsc CC=golang-codereviews https://codereview.appspot.com/143540044 Committer: Russ Cox <rsc@golang.org>
* | cmd/cc, cmd/ld, runtime: disallow conservative data/bss objectsRuss Cox2014-09-241-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In linker, refuse to write conservative (array of pointers) as the garbage collection type for any variable in the data/bss GC program. In the linker, attach the Go type to an already-read C declaration during dedup. This gives us Go types for C globals for free as long as the cmd/dist-generated Go code contains the declaration. (Most runtime C declarations have a corresponding Go declaration. Both are bss declarations and so the linker dedups them.) In cmd/dist, add a few more C files to the auto-Go-declaration list in order to get Go type information for the C declarations into the linker. In C compiler, mark all non-pointer-containing global declarations and all string data as NOPTR. This allows them to exist in C files without any corresponding Go declaration. Count C function pointers as "non-pointer-containing", since we have no heap-allocated C functions. In runtime, add NOPTR to the remaining pointer-containing declarations, none of which refer to Go heap objects. In runtime, also move os.Args and syscall.envs data into runtime-owned variables. Otherwise, in programs that do not import os or syscall, the runtime variables named os.Args and syscall.envs will be missing type information. I believe that this CL eliminates the final source of conservative GC scanning in non-SWIG Go programs, and therefore... Fixes issue 909. LGTM=iant R=iant CC=golang-codereviews https://codereview.appspot.com/149770043
* | os: add a comment inside RemoveAllBrad Fitzpatrick2014-09-231-0/+2
| | | | | | | | | | | | | | LGTM=r R=r CC=golang-codereviews https://codereview.appspot.com/149950043
* | os: fix another case where RemoveAll should return nilBrad Fitzpatrick2014-09-231-0/+3
| | | | | | | | | | | | | | | | | | | | | | This hopefully fixes issue 8793. Fixes Issue 8793 LGTM=iant R=rsc, iant CC=golang-codereviews https://codereview.appspot.com/150860046
* | os: handle 'has been removed' error as IsNotExist on Plan 9David du Colombier2014-09-181-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | It fixes the TestRemoveAllRace test introduced in CL 145860043. LGTM=bradfitz R=rsc, bradfitz CC=golang-codereviews https://codereview.appspot.com/147820044 Committer: Russ Cox <rsc@golang.org>
* | os: skip TestRemoveAllRace on WindowsRuss Cox2014-09-181-0/+8
| | | | | | | | | | | | | | | | | | | | It's just fundamentally incompatible with Windows' pickiness about removing things that are in use. TBR=brainman CC=golang-codereviews https://codereview.appspot.com/142270043
* | os: avoid error result when dir is removed out from under RemoveAllRuss Cox2014-09-182-1/+46
| | | | | | | | | | | | | | | | | | Fixes issue 7776. LGTM=bradfitz R=golang-codereviews, bradfitz CC=golang-codereviews, r https://codereview.appspot.com/145860043
* | net: separate NaCl dependent placeholders from BSD'sMikio Hara2014-09-181-2/+3
| | | | | | | | | | | | | | | | | | To clarify the dependency of NaCl platform. LGTM=adg R=golang-codereviews, adg CC=golang-codereviews https://codereview.appspot.com/143830044
* | os/signal: increase timeout from 10ms to 100msRuss Cox2014-09-161-3/+3
|/ | | | | | | | | | Might make test less flaky. Fixes issue 8682. LGTM=bradfitz R=bradfitz CC=golang-codereviews https://codereview.appspot.com/143160043
* build: move package sources from src/pkg to srcRuss Cox2014-09-0883-0/+9855
Preparation was in CL 134570043. This CL contains only the effect of 'hg mv src/pkg/* src'. For more about the move, see golang.org/s/go14nopkg.
View Lorry Controller Status