]> xenbits.xensource.com Git - xen-guest-agent.git/commitdiff
pnet: reword cache as addresses_cache
authorYann Dirson <yann.dirson@vates.fr>
Thu, 23 Nov 2023 10:39:05 +0000 (11:39 +0100)
committerYann Dirson <yann.dirson@vates.fr>
Fri, 8 Dec 2023 09:59:46 +0000 (10:59 +0100)
We're going to have an interface cache too, let's be clear.

Signed-off-by: Yann Dirson <yann.dirson@vates.fr>
src/collector_net_pnet.rs

index 476a80142743edb02226cd4d1fbc2f9ef69acdc1..9c4f87f3eb150449ae540d370e13653b96c53da3 100644 (file)
@@ -28,12 +28,12 @@ impl InterfaceInfo {
 
 type AddressesState = HashMap<u32, InterfaceInfo>;
 pub struct NetworkSource {
-    cache: AddressesState,
+    addresses_cache: AddressesState,
 }
 
 impl NetworkSource {
     pub fn new() -> io::Result<NetworkSource> {
-        Ok(NetworkSource {cache: AddressesState::new()})
+        Ok(NetworkSource {addresses_cache: AddressesState::new()})
     }
 
     pub async fn collect_current(&mut self) -> Result<Vec<NetEvent>, Box<dyn Error>> {
@@ -56,7 +56,7 @@ impl NetworkSource {
     fn get_ifconfig_data(&mut self) -> io::Result<Vec<NetEvent>> {
         let network_interfaces = pnet_datalink::interfaces();
 
-        // get a full view of interfaces, diffable with the cache
+        // get a full view of interfaces, diffable with addresses_cache
         let mut network_view = AddressesState::new();
         for iface in network_interfaces.iter() {
             // KLUDGE: drop ":alias" suffix for Linux interface aliases
@@ -72,7 +72,7 @@ impl NetworkSource {
             }
         }
 
-        // diff cache and view
+        // diff addresses_cache and view
 
         // events to be returned
         let mut events = vec!();
@@ -80,7 +80,7 @@ impl NetworkSource {
         let empty_address_set: HashSet<Address> = HashSet::new();
 
         // disappearing addresses
-        for (cached_iface_index, cached_info) in self.cache.iter() {
+        for (cached_iface_index, cached_info) in self.addresses_cache.iter() {
             let iface = NetInterface { index: *cached_iface_index,
                                        name: cached_info.name.to_string(),
                                        toolstack_iface: ToolstackNetInterface::None,
@@ -107,7 +107,7 @@ impl NetworkSource {
                                        toolstack_iface: ToolstackNetInterface::None,
             };
             let cache_adresses =
-                if let Some(cache_info) = self.cache.get(iface_index) {
+                if let Some(cache_info) = self.addresses_cache.get(iface_index) {
                     &cache_info.addresses
                 } else {
                     &empty_address_set
@@ -124,7 +124,7 @@ impl NetworkSource {
         }
 
         // replace cache with view
-        self.cache = network_view; // FIXME expensive?
+        self.addresses_cache = network_view; // FIXME expensive?
 
         Ok(events)
     }