Skip to content

Commit

Permalink
gofmt changes (new)
Browse files Browse the repository at this point in the history
Formatting in 'utils/machine/machine.go' and
'utils/sysfs/sysfs.go' was incorrect and causing ./build/presubmit.sh
script to fail in Travis giving below error:
The following files are not properly formatted:
$ ./build/presubmit.sh
+./build/check_gofmt.sh .
utils/machine/machine.go utils/sysfs/sysfs.go utils/machine/machine.go
utils/sysfs/sysfs.go

Fixed the formatting using below command:
gofmt -l -w -s utils/sysfs/sysfs.go
gofmt -l -w -s utils/machine/machine.go
  • Loading branch information
shahidhs-ibm committed Oct 19, 2015
1 parent be8de94 commit 305bb27
Show file tree
Hide file tree
Showing 2 changed files with 54 additions and 54 deletions.
76 changes: 38 additions & 38 deletions utils/machine/machine.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,11 +20,11 @@ import (
"regexp"
"strconv"
"strings"
// s390/s390x changes
"syscall"
"runtime"

// s390/s390x changes
"runtime"
"syscall"

"github.com/golang/glog"
info "github.com/google/cadvisor/info/v1"
"github.com/google/cadvisor/utils"
Expand All @@ -44,10 +44,10 @@ var swapCapacityRegexp = regexp.MustCompile("SwapTotal: *([0-9]+) kB")
// GetClockSpeed returns the CPU clock speed, given a []byte formatted as the /proc/cpuinfo file.
func GetClockSpeed(procInfo []byte) (uint64, error) {
// s390/s390x changes
if (true == isSystemZ()) {
if true == isSystemZ() {
return 0, nil
}

// First look through sys to find a max supported cpu frequency.
const maxFreqFile = "/sys/devices/system/cpu/cpu0/cpufreq/cpuinfo_max_freq"
if utils.FileExists(maxFreqFile) {
Expand Down Expand Up @@ -128,12 +128,12 @@ func parseCapacity(b []byte, r *regexp.Regexp) (int64, error) {

func GetTopology(sysFs sysfs.SysFs, cpuinfo string) ([]info.Node, int, error) {
nodes := []info.Node{}
// s390/s390x changes
if true == isSystemZ() {
return nodes, getNumCores(), nil
}

// s390/s390x changes
if true == isSystemZ() {
return nodes, getNumCores(), nil
}

numCores := 0
lastThread := -1
lastCore := -1
Expand Down Expand Up @@ -259,39 +259,39 @@ func addNode(nodes *[]info.Node, id int) (int, error) {

// s390/s390x changes
func getMachineArch() (string, error) {
uname := syscall.Utsname{}
err := syscall.Uname(&uname)
if err != nil {
return "", err
}
uname := syscall.Utsname{}
err := syscall.Uname(&uname)
if err != nil {
return "", err
}

var arch string
for _, val := range uname.Machine {
arch += string(int(val))
}
var arch string
for _, val := range uname.Machine {
arch += string(int(val))
}

return arch, nil
return arch, nil
}

// s390/s390x changes
func isSystemZ() bool {
arch, err := getMachineArch()
if err == nil {
if (true == strings.Contains(arch, "390")) {
return true
}
}
return false
arch, err := getMachineArch()
if err == nil {
if true == strings.Contains(arch, "390") {
return true
}
}
return false
}

// s390/s390x changes
func getNumCores() int {
maxProcs := runtime.GOMAXPROCS(0)
numCPU := runtime.NumCPU()
if maxProcs < numCPU {
return maxProcs
}
return numCPU
maxProcs := runtime.GOMAXPROCS(0)
numCPU := runtime.NumCPU()

if maxProcs < numCPU {
return maxProcs
}

return numCPU
}
32 changes: 16 additions & 16 deletions utils/sysfs/sysfs.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,11 +24,11 @@ import (
)

const (
blockDir = "/sys/block"
cacheDir = "/sys/devices/system/cpu/cpu"
netDir = "/sys/class/net"
dmiDir = "/sys/class/dmi"
ppcDevTree = "/proc/device-tree"
blockDir = "/sys/block"
cacheDir = "/sys/devices/system/cpu/cpu"
netDir = "/sys/class/net"
dmiDir = "/sys/class/dmi"
ppcDevTree = "/proc/device-tree"
s390xDevTree = "/etc" // s390/s390x changes
)

Expand Down Expand Up @@ -235,15 +235,15 @@ func (self *realSysFs) GetCacheInfo(id int, name string) (CacheInfo, error) {
}

func (self *realSysFs) GetSystemUUID() (string, error) {
if id, err := ioutil.ReadFile(path.Join(dmiDir, "id", "product_uuid")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(ppcDevTree, "system-id")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(ppcDevTree, "vm,uuid")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(s390xDevTree, "machine-id")); err == nil {
return strings.TrimSpace(string(id)), nil
} else {
return "", err
}
if id, err := ioutil.ReadFile(path.Join(dmiDir, "id", "product_uuid")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(ppcDevTree, "system-id")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(ppcDevTree, "vm,uuid")); err == nil {
return strings.TrimSpace(string(id)), nil
} else if id, err = ioutil.ReadFile(path.Join(s390xDevTree, "machine-id")); err == nil {
return strings.TrimSpace(string(id)), nil
} else {
return "", err
}
}

0 comments on commit 305bb27

Please sign in to comment.