Merge pull request intel#1048 from bart0sh/PR142-get-rid-of-sysfsDir

Get rid of unused sysfsDir parameter

Author: mythi

cmd/dsa_plugin/dsa_plugin.go

@@ -27,8 +27,6 @@ import (
 const (
 // Device plugin settings.
 namespace = "dsa.intel.com"
-// SysFS directory.
-sysfsDir = "/sys/bus/dsa/devices"
 // Device directories.
 devDir = "/dev/dsa"
 // Glob pattern for the state sysfs entry.
@@ -46,7 +44,7 @@ func main() {
 os.Exit(1)
 }
 
-plugin := idxd.NewDevicePlugin(sysfsDir, statePattern, devDir, sharedDevNum)
+plugin := idxd.NewDevicePlugin(statePattern, devDir, sharedDevNum)
 if plugin == nil {
 klog.Fatal("Cannot create device plugin, please check above error messages.")
 }

cmd/iaa_plugin/iaa_plugin.go

@@ -27,8 +27,6 @@ import (
 const (
 // Device plugin settings.
 namespace = "iaa.intel.com"
-// SysFS directory.
-sysfsDir = "/sys/bus/iax/devices"
 // Device directories.
 devDir = "/dev/iax"
 // Glob pattern for the state sysfs entry.
@@ -46,7 +44,7 @@ func main() {
 os.Exit(1)
 }
 
-plugin := idxd.NewDevicePlugin(sysfsDir, statePattern, devDir, sharedDevNum)
+plugin := idxd.NewDevicePlugin(statePattern, devDir, sharedDevNum)
 if plugin == nil {
 klog.Fatal("Cannot create device plugin, please check above error messages.")
 }

pkg/idxd/plugin.go

@@ -45,17 +45,15 @@ type DevicePlugin struct {
 scanTicker *time.Ticker
 scanDone chan bool
 getDevNodes getDevNodesFunc
-sysfsDir string
 statePattern string
 devDir string
 charDevDir string
 sharedDevNum int
 }
 
 // NewDevicePlugin creates DevicePlugin.
-func NewDevicePlugin(sysfsDir, statePattern, devDir string, sharedDevNum int) *DevicePlugin {
+func NewDevicePlugin(statePattern, devDir string, sharedDevNum int) *DevicePlugin {
 return &DevicePlugin{
-sysfsDir: sysfsDir,
 statePattern: statePattern,
 devDir: devDir,
 charDevDir: charDevDir,

pkg/idxd/plugin_test.go

@@ -220,7 +220,7 @@ func genTest(sysfs, statePattern string, tc testCase) func(t *testing.T) {
 }
 }
 
-plugin := NewDevicePlugin(sysfs, statePattern, "", tc.sharedDevNum)
+plugin := NewDevicePlugin(statePattern, "", tc.sharedDevNum)
 plugin.getDevNodes = getFakeDevNodes
 
 notifier := &fakeNotifier{