Skip to content

Commit 39dbc8e

Browse files
committed
API tweaks, tests.
1 parent be024e2 commit 39dbc8e

File tree

3 files changed

+54
-1
lines changed

3 files changed

+54
-1
lines changed

Diff for: unix/mmap_unix_test.go

+23
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ package unix_test
99
import (
1010
"runtime"
1111
"testing"
12+
"unsafe"
1213

1314
"golang.org/x/sys/unix"
1415
)
@@ -49,3 +50,25 @@ func TestMmap(t *testing.T) {
4950
t.Fatalf("Munmap: %v", err)
5051
}
5152
}
53+
54+
func TestMmapPtr(t *testing.T) {
55+
mmapProt := unix.PROT_NONE
56+
mmapPtrProt := unix.PROT_READ | unix.PROT_WRITE
57+
b, err := unix.Mmap(-1, 0, 2*unix.Getpagesize(), mmapProt, unix.MAP_ANON|unix.MAP_PRIVATE)
58+
if err != nil {
59+
t.Fatalf("Mmap: %v", err)
60+
}
61+
if _, err := unix.MmapPtr(-1, 0, unsafe.Pointer(&b[0]), uintptr(unix.Getpagesize()),
62+
mmapPtrProt, unix.MAP_ANON|unix.MAP_PRIVATE|unix.MAP_FIXED); err != nil {
63+
t.Fatalf("MmapPtr: %v", err)
64+
}
65+
66+
b[0] = 42
67+
68+
if err := unix.MunmapPtr(unsafe.Pointer(&b[0]), uintptr(unix.Getpagesize())); err != nil {
69+
t.Fatalf("MunmapPtr: %v", err)
70+
}
71+
if err := unix.Munmap(b); err != nil {
72+
t.Fatalf("Munmap: %v", err)
73+
}
74+
}

Diff for: unix/mremap_test.go

+30
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ package unix_test
88

99
import (
1010
"testing"
11+
"unsafe"
1112

1213
"golang.org/x/sys/unix"
1314
)
@@ -44,3 +45,32 @@ func TestMremap(t *testing.T) {
4445
t.Fatalf("remapping to a fixed address; got %v, want %v", err, unix.EINVAL)
4546
}
4647
}
48+
49+
func TestMremapPtr(t *testing.T) {
50+
mmapProt := unix.PROT_NONE
51+
mmapPtrProt := unix.PROT_READ | unix.PROT_WRITE
52+
b, err := unix.Mmap(-1, 0, 2*unix.Getpagesize(), mmapProt, unix.MAP_ANON|unix.MAP_PRIVATE)
53+
if err != nil {
54+
t.Fatalf("Mmap: %v", err)
55+
}
56+
if _, err := unix.MmapPtr(-1, 0, unsafe.Pointer(&b[0]), uintptr(unix.Getpagesize()),
57+
mmapPtrProt, unix.MAP_ANON|unix.MAP_PRIVATE|unix.MAP_FIXED); err != nil {
58+
t.Fatalf("MmapPtr: %v", err)
59+
}
60+
61+
b[0] = 42
62+
63+
if _, err := unix.MremapPtr(
64+
unsafe.Pointer(&b[0]), uintptr(unix.Getpagesize()),
65+
unsafe.Pointer(&b[unix.Getpagesize()]), uintptr(unix.Getpagesize()),
66+
unix.MremapFixed|unix.MremapMaymove); err != nil {
67+
t.Fatalf("MremapPtr: %v", err)
68+
}
69+
if got := b[unix.Getpagesize()]; got != 42 {
70+
t.Errorf("got %d, want 42", got)
71+
}
72+
73+
if err := unix.Munmap(b); err != nil {
74+
t.Fatalf("Munmap: %v", err)
75+
}
76+
}

Diff for: unix/syscall_unix.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ func Munmap(b []byte) (err error) {
154154
return mapper.Munmap(b)
155155
}
156156

157-
func MmapPtr(addr unsafe.Pointer, length uintptr, prot int, flags int, fd int, offset int64) (ret unsafe.Pointer, err error) {
157+
func MmapPtr(fd int, offset int64, addr unsafe.Pointer, length uintptr, prot int, flags int) (ret unsafe.Pointer, err error) {
158158
xaddr, err := mapper.mmap(uintptr(addr), length, prot, flags, fd, offset)
159159
return unsafe.Pointer(xaddr), err
160160
}

0 commit comments

Comments
 (0)