Skip to content

Commit 93eded0

Browse files
Clément Chigotbradfitz
Clément Chigot
authored andcommitted
internal/syscall/unix, net: improve interface_aix.go
This commit improves the interface_aix.go file, based on feedbacks about CL 138724. To retrieve MTU, ioctl is needed. It's implemented inside internal/syscall/unix. Change-Id: Ic583d26b93935a32a5f1eb5a2170b86e80a4a85e Reviewed-on: https://go-review.googlesource.com/c/142157 Run-TryBot: Brad Fitzpatrick <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Brad Fitzpatrick <[email protected]>
1 parent 13d5cd7 commit 93eded0

File tree

4 files changed

+62
-7
lines changed

4 files changed

+62
-7
lines changed

src/go/build/deps_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -317,7 +317,7 @@ var pkgDeps = map[string][]string{
317317
"net": {
318318
"L0", "CGO",
319319
"context", "math/rand", "os", "reflect", "sort", "syscall", "time",
320-
"internal/nettrace", "internal/poll",
320+
"internal/nettrace", "internal/poll", "internal/syscall/unix",
321321
"internal/syscall/windows", "internal/singleflight", "internal/race",
322322
"golang_org/x/net/dns/dnsmessage", "golang_org/x/net/lif", "golang_org/x/net/route",
323323
},
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
// Copyright 2018 The Go Authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style
3+
// license that can be found in the LICENSE file.
4+
5+
#include "textflag.h"
6+
7+
//
8+
// System calls for aix/ppc64 are implemented in syscall/syscall_aix.go
9+
//
10+
11+
TEXT ·syscall6(SB),NOSPLIT,$0
12+
JMP syscall·syscall6(SB)
Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
// Copyright 2018 The Go Authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style
3+
// license that can be found in the LICENSE file.
4+
5+
package unix
6+
7+
import (
8+
"syscall"
9+
"unsafe"
10+
)
11+
12+
//go:cgo_import_dynamic libc_ioctl ioctl "libc.a/shr_64.o"
13+
//go:linkname libc_ioctl libc_ioctl
14+
var libc_ioctl uintptr
15+
16+
// Implemented in syscall/syscall_aix.go.
17+
func syscall6(trap, nargs, a1, a2, a3, a4, a5, a6 uintptr) (r1, r2 uintptr, err syscall.Errno)
18+
19+
func Ioctl(fd int, cmd int, args uintptr) (err error) {
20+
_, _, e1 := syscall6(uintptr(unsafe.Pointer(&libc_ioctl)), 3, uintptr(fd), uintptr(cmd), uintptr(args), 0, 0, 0)
21+
if e1 != 0 {
22+
err = e1
23+
}
24+
return
25+
}

src/net/interface_aix.go

Lines changed: 24 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,12 @@
55
package net
66

77
import (
8-
//"os"
8+
"internal/syscall/unix"
99
"syscall"
1010
"unsafe"
1111
)
1212

13-
type RawSockaddrDatalink struct {
13+
type rawSockaddrDatalink struct {
1414
Len uint8
1515
Family uint8
1616
Index uint16
@@ -21,6 +21,11 @@ type RawSockaddrDatalink struct {
2121
Data [120]byte
2222
}
2323

24+
type ifreq struct {
25+
Name [16]uint8
26+
Ifru [16]byte
27+
}
28+
2429
const _KINFO_RT_IFLIST = (0x1 << 8) | 3 | (1 << 30)
2530

2631
const _RTAX_NETMASK = 2
@@ -30,12 +35,12 @@ const _RTAX_MAX = 8
3035
func getIfList() ([]byte, error) {
3136
needed, err := syscall.Getkerninfo(_KINFO_RT_IFLIST, 0, 0, 0)
3237
if err != nil {
33-
return nil, nil // XXX
38+
return nil, err
3439
}
3540
tab := make([]byte, needed)
3641
_, err = syscall.Getkerninfo(_KINFO_RT_IFLIST, uintptr(unsafe.Pointer(&tab[0])), uintptr(unsafe.Pointer(&needed)), 0)
3742
if err != nil {
38-
return nil, nil // XXX
43+
return nil, err
3944
}
4045
return tab[:needed], nil
4146
}
@@ -57,12 +62,25 @@ func interfaceTable(ifindex int) ([]Interface, error) {
5762
}
5863
if ifm.Type == syscall.RTM_IFINFO {
5964
if ifindex == 0 || ifindex == int(ifm.Index) {
60-
sdl := (*RawSockaddrDatalink)(unsafe.Pointer(&tab[syscall.SizeofIfMsghdr]))
65+
sdl := (*rawSockaddrDatalink)(unsafe.Pointer(&tab[syscall.SizeofIfMsghdr]))
6166

6267
ifi := &Interface{Index: int(ifm.Index), Flags: linkFlags(ifm.Flags)}
6368
ifi.Name = string(sdl.Data[:sdl.Nlen])
6469
ifi.HardwareAddr = sdl.Data[sdl.Nlen : sdl.Nlen+sdl.Alen]
65-
/* XXX MTU? */
70+
71+
// Retrieve MTU
72+
ifr := &ifreq{}
73+
copy(ifr.Name[:], ifi.Name)
74+
sock, err := syscall.Socket(syscall.AF_INET, syscall.SOCK_DGRAM, 0)
75+
if err != nil {
76+
return nil, err
77+
}
78+
err = unix.Ioctl(sock, syscall.SIOCGIFMTU, uintptr(unsafe.Pointer(ifr)))
79+
if err != nil {
80+
return nil, err
81+
}
82+
ifi.MTU = int(ifr.Ifru[0])<<24 | int(ifr.Ifru[1])<<16 | int(ifr.Ifru[2])<<8 | int(ifr.Ifru[3])
83+
6684
ift = append(ift, *ifi)
6785
if ifindex == int(ifm.Index) {
6886
break

0 commit comments

Comments
 (0)