qat: drop setupDeviceIDs()

setupDeviceIDs() is obsoleted and the preferred approach is driver_override
already implemented in qat-init.sh initcontainer.

The new_id mechanism was added way before we had the initcontainer support in place.
Furthermore, at least for vfio-pci we don't need it at all if the driver uses
ids=8086:<qat VF dev IDs>.

Drop write attemps to new_id in favor of the initcontainer functionality.

Signed-off-by: Mikko Ylinen <[email protected]>

Author: mythi

cmd/qat_plugin/dpdkdrv/dpdkdrv.go

@@ -44,7 +44,6 @@ const (
 	pciDriverDirectory = "/sys/bus/pci/drivers"
 	uioSuffix          = "uio"
 	iommuGroupSuffix   = "iommu_group"
-	vendorPrefix       = "8086 "
 	envVarPrefix       = "QAT"
 
 	igbUio  = "igb_uio"
@@ -187,31 +186,10 @@ func newDevicePlugin(pciDriverDir, pciDeviceDir string, maxDevices int, kernelVf
 	}
 }
 
-func (dp *DevicePlugin) setupDeviceIDs() error {
-	for devID, driver := range qatDeviceDriver {
-		for _, enabledDriver := range dp.kernelVfDrivers {
-			if driver != enabledDriver {
-				continue
-			}
-
-			err := writeToDriver(filepath.Join(dp.pciDriverDir, dp.dpdkDriver, "new_id"), vendorPrefix+devID)
-			if err != nil && !errors.Is(err, os.ErrExist) {
-				return errors.WithMessagef(err, "failed to set device ID %s for %s. Driver module not loaded?", devID, dp.dpdkDriver)
-			}
-		}
-	}
-
-	return nil
-}
-
 // Scan implements Scanner interface for vfio based QAT plugin.
 func (dp *DevicePlugin) Scan(notifier dpapi.Notifier) error {
 	defer dp.scanTicker.Stop()
 
-	if err := dp.setupDeviceIDs(); err != nil {
-		return err
-	}
-
 	for {
 		devTree, err := dp.scan()
 		if err != nil {