External/cloud-hypervisor
1
0
Fork 0
mirror of https://github.com/cloud-hypervisor/cloud-hypervisor.git synced 2024-12-22 13:45:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

net_util: Remove unnecessary return value from CtrlQueue::process()

Browse Source 
Since the code has been adapted to support VIRTIO_F_EVENT_IDX we use
Queue::needs_notification() to determine whether to signal the guest so
it is no longer necessary to check if there are any used descriptors. If
the feature is not negotiated then Queue::needs_notification() will
return true triggering an interrupt of the guest. Theoretically this
could be a spurious interrupt of the guest if there were no used used
descriptors but this is unlikely as we only generate used descriptors
for the control queue as a result of an interrupt of the VMM by the
guest.

Signed-off-by: Rob Bradford <robert.bradford@intel.com>
This commit is contained in:
Rob Bradford 2022-03-15 15:30:18 +00:00
parent aa3ef70ea6
commit b9aeaf6634
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
net_util/src/ctrl_queue.rs
View File

@ -60,7 +60,7 @@ impl CtrlQueue {
&mut self, &mut self,
queue: &mut Queue<GuestMemoryAtomic<GuestMemoryMmap>>, queue: &mut Queue<GuestMemoryAtomic<GuestMemoryMmap>>,
access_platform: Option<&Arc<dyn AccessPlatform>>, access_platform: Option<&Arc<dyn AccessPlatform>>,
) -> Result<bool> { ) -> Result<()> {
let mut used_desc_heads = Vec::new(); let mut used_desc_heads = Vec::new();
loop { loop {
for mut desc_chain in queue.iter().map_err(Error::QueueIterator)? { for mut desc_chain in queue.iter().map_err(Error::QueueIterator)? {
@ -156,7 +156,7 @@ impl CtrlQueue {
} }
} }
Ok(!used_desc_heads.is_empty()) Ok(())
} }
} }