Reduce the logged output amount for VFs

Signed-off-by: Guvenc Gulce <[email protected]>

Author: guvenc

feos/utils/src/network/utils.rs

@@ -16,7 +16,6 @@ pub const INTERFACE_NAME: &str = "eth0";
 pub async fn configure_sriov(num_vfs: u32) -> Result<(), String> {
     let base_path = format!("/sys/class/net/{INTERFACE_NAME}/device");
 
-    // Disable driver autoprobe for new VFs to prevent mlx5_core from claiming them.
     let autoprobe_path = format!("{base_path}/sriov_drivers_autoprobe");
     info!("Disabling sriov_drivers_autoprobe at {autoprobe_path}");
     let mut autoprobe_file = OpenOptions::new()
@@ -29,7 +28,6 @@ pub async fn configure_sriov(num_vfs: u32) -> Result<(), String> {
         .await
         .map_err(|e| format!("Failed to disable autoprobe: {e}"))?;
 
-    // Reset VFs to 0 before creating new ones.
     let numvfs_path = format!("{base_path}/sriov_numvfs");
     let mut numvfs_file = OpenOptions::new()
         .write(true)
@@ -42,15 +40,14 @@ pub async fn configure_sriov(num_vfs: u32) -> Result<(), String> {
         .write_all(b"0\n")
         .await
         .map_err(|e| format!("Failed to write 0 to sriov_numvfs: {e}"))?;
-    sleep(Duration::from_secs(1)).await; // Give time for VFs to be removed.
+    sleep(Duration::from_secs(1)).await;
 
-    // Create the new VFs.
     info!("Creating {num_vfs} sriov virtual functions for {INTERFACE_NAME}");
     numvfs_file
         .write_all(format!("{num_vfs}\n").as_bytes())
         .await
         .map_err(|e| format!("Failed to write to sriov_numvfs: {e}"))?;
-    sleep(Duration::from_secs(2)).await; // Give time for VFs to be created.
+    sleep(Duration::from_secs(2)).await;
 
     let device_path = read_link(&base_path).await.map_err(|e| e.to_string())?;
     let pci_address = device_path
@@ -73,10 +70,8 @@ pub async fn configure_sriov(num_vfs: u32) -> Result<(), String> {
         .map(format_pci_address)
         .collect();
 
-    // Bind each newly created VF to the vfio-pci driver.
     for vf_pci in virtual_funcs.iter() {
         if let Err(e) = bind_vf_to_vfio(vf_pci).await {
-            // With autoprobe disabled, this should be reliable. A failure is a hard error.
             return Err(format!("Failed to bind VF {vf_pci} to vfio-pci: {e}"));
         }
     }
@@ -122,19 +117,13 @@ fn format_pci_address(address: (u16, u8, u8, u8)) -> String {
 }
 
 async fn bind_vf_to_vfio(pci_address: &str) -> Result<(), io::Error> {
-    info!("Binding {pci_address} to vfio-pci using driver_override");
-
-    // Set the driver_override to vfio-pci for this specific device.
     let override_path = format!("/sys/bus/pci/devices/{pci_address}/driver_override");
     let mut override_file = OpenOptions::new().write(true).open(&override_path).await?;
     override_file.write_all(b"vfio-pci").await?;
-    info!("Set driver_override for {pci_address}");
 
-    // Trigger the binding action. The kernel will see the override and use vfio-pci.
     let bind_path = "/sys/bus/pci/drivers/vfio-pci/bind";
     let mut bind_file = OpenOptions::new().write(true).open(bind_path).await?;
     bind_file.write_all(pci_address.as_bytes()).await?;
-    info!("Successfully triggered bind for {pci_address}");
 
     Ok(())
 }