Fix the linting errors

pkg/nvpci/bytes/bytes.go

@@ -21,10 +21,12 @@ import (
 	"unsafe"
 )
 
+// Raw returns just the bytes without any assumptions about layout
 type Raw interface {
 	Raw() *[]byte
 }
 
+// Reader used to read various data sizes in the byte array
 type Reader interface {
 	Read8(pos int) uint8
 	Read16(pos int) uint16
@@ -33,6 +35,7 @@ type Reader interface {
 	Len() int
 }
 
+// Writer used to write various sizes of data in the byte array
 type Writer interface {
 	Write8(pos int, value uint8)
 	Write16(pos int, value uint16)
@@ -41,6 +44,7 @@ type Writer interface {
 	Len() int
 }
 
+// Bytes object for manipulating arbitrary byte arrays
 type Bytes interface {
 	Raw
 	Reader
@@ -66,22 +70,25 @@ func init() {
 	}
 }
 
+// New raw bytearray
 func New(data *[]byte) Bytes {
 	return (*native)(data)
 }
 
+// NewLittleEndian little endian ordering of bytes
 func NewLittleEndian(data *[]byte) Bytes {
 	if nativeByteOrder == binary.LittleEndian {
 		return (*native)(data)
-	} else {
-		return (*swapbo)(data)
 	}
+
+	return (*swapbo)(data)
 }
 
+// NewBigEndian big endian ordering of bytes
 func NewBigEndian(data *[]byte) Bytes {
 	if nativeByteOrder == binary.BigEndian {
 		return (*native)(data)
-	} else {
-		return (*swapbo)(data)
 	}
+
+	return (*swapbo)(data)
 }

pkg/nvpci/config.go

@@ -29,10 +29,12 @@ const (
 	pciCapabilityListPointer = 0x34
 )
 
+// ConfigSpace PCI configuration space (standard extended) file path
 type ConfigSpace struct {
 	Path string
 }
 
+// ConfigSpaceIO Interface for reading and writing raw and preconfigured values
 type ConfigSpaceIO interface {
 	bytes.Bytes
 	GetVendorID() uint16
@@ -44,15 +46,18 @@ type configSpaceIO struct {
 	bytes.Bytes
 }
 
+// PCIStandardCapability standard PCI config space
 type PCIStandardCapability struct {
 	bytes.Bytes
 }
 
+// PCIExtendedCapability extended PCI config space
 type PCIExtendedCapability struct {
 	bytes.Bytes
 	Version uint8
 }
 
+// PCICapabilities combines the standard and extended config space
 type PCICapabilities struct {
 	Standard map[uint8]*PCIStandardCapability
 	Extended map[uint16]*PCIExtendedCapability

pkg/nvpci/mmio/mmio.go

@@ -25,6 +25,7 @@ import (
 	"gitlab.com/nvidia/cloud-native/go-nvlib/pkg/nvpci/bytes"
 )
 
+// Mmio memory map a region
 type Mmio interface {
 	bytes.Raw
 	bytes.Reader
@@ -41,25 +42,25 @@ type mmio struct {
 }
 
 func open(path string, offset int, size int, flags int) (Mmio, error) {
-	var mmap_flags int
+	var mmapFlags int
 	switch flags {
 	case os.O_RDONLY:
-		mmap_flags = syscall.PROT_READ
+		mmapFlags = syscall.PROT_READ
 	case os.O_RDWR:
-		mmap_flags = syscall.PROT_READ | syscall.PROT_WRITE
+		mmapFlags = syscall.PROT_READ | syscall.PROT_WRITE
 	default:
-		return nil, fmt.Errorf("invalid flags: %v\n", flags)
+		return nil, fmt.Errorf("invalid flags: %v", flags)
 	}
 
 	file, err := os.OpenFile(path, flags, 0)
 	if err != nil {
-		return nil, fmt.Errorf("failed to open file: %v\n", err)
+		return nil, fmt.Errorf("failed to open file: %v", err)
 	}
 	defer file.Close()
 
 	fi, err := file.Stat()
 	if err != nil {
-		return nil, fmt.Errorf("failed to get file info: %v\n", err)
+		return nil, fmt.Errorf("failed to get file info: %v", err)
 	}
 
 	if size > int(fi.Size()) {
@@ -74,19 +75,21 @@ func open(path string, offset int, size int, flags int) (Mmio, error) {
 		int(file.Fd()),
 		int64(offset),
 		size,
-		mmap_flags,
+		mmapFlags,
 		syscall.MAP_SHARED)
 	if err != nil {
-		return nil, fmt.Errorf("failed to mmap file: %v\n", err)
+		return nil, fmt.Errorf("failed to mmap file: %v", err)
 	}
 
 	return &mmio{bytes.New(&mmap)}, nil
 }
 
+// OpenRO open region readonly
 func OpenRO(path string, offset int, size int) (Mmio, error) {
 	return open(path, offset, size, os.O_RDONLY)
 }
 
+// OpenRW open region read write
 func OpenRW(path string, offset int, size int) (Mmio, error) {
 	return open(path, offset, size, os.O_RDWR)
 }
@@ -106,7 +109,7 @@ func (m *mmio) BigEndian() Mmio {
 func (m *mmio) Close() error {
 	err := syscall.Munmap(*m.Bytes.Raw())
 	if err != nil {
-		return fmt.Errorf("failed to munmap file: %v\n", err)
+		return fmt.Errorf("failed to munmap file: %v", err)
 	}
 	return nil
 }

pkg/nvpci/mmio/mock.go

@@ -48,10 +48,12 @@ func mockOpen(source *[]byte, offset int, size int, rw bool) (Mmio, error) {
 	return m, nil
 }
 
+// MockOpenRO open read only
 func MockOpenRO(source *[]byte, offset int, size int) (Mmio, error) {
 	return mockOpen(source, offset, size, false)
 }
 
+// MockOpenRW open read write
 func MockOpenRW(source *[]byte, offset int, size int) (Mmio, error) {
 	return mockOpen(source, offset, size, true)
 }

pkg/nvpci/mock.go

@@ -25,12 +25,14 @@ import (
 	"gitlab.com/nvidia/cloud-native/go-nvlib/pkg/nvpci/bytes"
 )
 
+// MockNvpci mock pci device
 type MockNvpci struct {
 	*nvpci
 }
 
 var _ Interface = (*MockNvpci)(nil)
 
+// NewMockNvpci create new mock PCI and remove old devices
 func NewMockNvpci() (mock *MockNvpci, rerr error) {
 	rootDir, err := ioutil.TempDir("", "")
 	if err != nil {
@@ -49,10 +51,12 @@ func NewMockNvpci() (mock *MockNvpci, rerr error) {
 	return mock, nil
 }
 
+// Cleanup remove the mocked PCI devices root folder
 func (m *MockNvpci) Cleanup() {
 	os.RemoveAll(m.pciDevicesRoot)
 }
 
+// AddMockA100 Create an A100 like GPU mock device
 func (m *MockNvpci) AddMockA100(address string, numaNode int) error {
 	deviceDir := filepath.Join(m.pciDevicesRoot, address)
 	err := os.MkdirAll(deviceDir, 0755)
@@ -111,6 +115,9 @@ func (m *MockNvpci) AddMockA100(address string, numaNode int) error {
 
 	bar0 := []uint64{0x00000000c2000000, 0x00000000c2ffffff, 0x0000000000040200}
 	resource, err := os.Create(filepath.Join(deviceDir, "resource"))
+	if err != nil {
+		return err
+	}
 	_, err = resource.WriteString(fmt.Sprintf("0x%x 0x%x 0x%x", bar0[0], bar0[1], bar0[2]))
 	if err != nil {
 		return err

pkg/nvpci/nvpci.go

@@ -66,30 +66,34 @@ type NvidiaPCIDevice struct {
 	Resources map[int]*MemoryResource
 }
 
+// IsVGAController if class == 0x300
 func (d *NvidiaPCIDevice) IsVGAController() bool {
 	return d.Class == pciVgaControllerClass
 }
 
+// Is3DController if class == 0x302
 func (d *NvidiaPCIDevice) Is3DController() bool {
 	return d.Class == pci3dControllerClass
 }
 
+// IsNVSwitch if classe == 0x068
 func (d *NvidiaPCIDevice) IsNVSwitch() bool {
 	return d.Class == pciNvSwitchClass
 }
 
+// IsGPU either VGA for older cards or 3D for newer
 func (d *NvidiaPCIDevice) IsGPU() bool {
 	return d.IsVGAController() || d.Is3DController()
 }
 
+// IsResetAvailable some devices can be reset without rebooting,
+// check if applicable
 func (d *NvidiaPCIDevice) IsResetAvailable() bool {
 	_, err := os.Stat(path.Join(d.Path, "reset"))
-	if err != nil {
-		return false
-	}
-	return true
+	return err == nil
 }
 
+// Reset perform a reset to apply a new configuration at HW level
 func (d *NvidiaPCIDevice) Reset() error {
 	err := ioutil.WriteFile(path.Join(d.Path, "reset"), []byte("1"), 0)
 	if err != nil {
@@ -98,6 +102,7 @@ func (d *NvidiaPCIDevice) Reset() error {
 	return nil
 }
 
+// New interface that allows us to get a list of all NVIDIA PCI devices
 func New() Interface {
 	return &nvpci{pciDevicesRoot}
 }
@@ -202,7 +207,7 @@ func (p *nvpci) GetAllDevices() ([]*NvidiaPCIDevice, error) {
 		nvdevices = append(nvdevices, nvdevice)
 	}
 
-	addressToId := func(address string) uint64 {
+	addressToID := func(address string) uint64 {
 		address = strings.ReplaceAll(address, ":", "")
 		address = strings.ReplaceAll(address, ".", "")
 		id, _ := strconv.ParseUint(address, 16, 64)
@@ -210,7 +215,7 @@ func (p *nvpci) GetAllDevices() ([]*NvidiaPCIDevice, error) {
 	}
 
 	sort.Slice(nvdevices, func(i, j int) bool {
-		return addressToId(nvdevices[i].Address) < addressToId(nvdevices[j].Address)
+		return addressToID(nvdevices[i].Address) < addressToID(nvdevices[j].Address)
 	})
 
 	return nvdevices, nil

pkg/nvpci/resources.go

@@ -28,6 +28,7 @@ const (
 	pmcBigEndian      = 0x01000001
 )
 
+// MemoryResource represents a mmio region
 type MemoryResource struct {
 	Start uintptr
 	End   uintptr
@@ -35,10 +36,11 @@ type MemoryResource struct {
 	Path  string
 }
 
+// Open read write mmio region
 func (mr *MemoryResource) Open() (mmio.Mmio, error) {
 	rw, err := mmio.OpenRW(mr.Path, 0, int(mr.End-mr.Start+1))
 	if err != nil {
-		return nil, fmt.Errorf("failed to open file for mmio: %v\n", err)
+		return nil, fmt.Errorf("failed to open file for mmio: %v", err)
 	}
 	switch rw.Read32(pmcEndianRegister) {
 	case pmcBigEndian:
@@ -46,13 +48,14 @@ func (mr *MemoryResource) Open() (mmio.Mmio, error) {
 	case pmcLittleEndian:
 		return rw.LittleEndian(), nil
 	}
-	return nil, fmt.Errorf("unknown endianness for mmio: %v\n", err)
+	return nil, fmt.Errorf("unknown endianness for mmio: %v", err)
 }
 
+// OpenReadOnly read only mmio region
 func (mr *MemoryResource) OpenReadOnly() (mmio.Mmio, error) {
 	ro, err := mmio.OpenRO(mr.Path, 0, int(mr.End-mr.Start+1))
 	if err != nil {
-		return nil, fmt.Errorf("failed to open file for mmio: %v\n", err)
+		return nil, fmt.Errorf("failed to open file for mmio: %v", err)
 	}
 	switch ro.Read32(pmcEndianRegister) {
 	case pmcBigEndian:
@@ -60,5 +63,5 @@ func (mr *MemoryResource) OpenReadOnly() (mmio.Mmio, error) {
 	case pmcLittleEndian:
 		return ro.LittleEndian(), nil
 	}
-	return nil, fmt.Errorf("unknown endianness for mmio: %v\n", err)
+	return nil, fmt.Errorf("unknown endianness for mmio: %v", err)
 }