mirrors/crosvm
1
0
Fork 0
mirror of https://chromium.googlesource.com/crosvm/crosvm synced 2024-11-25 05:03:05 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

devices: virtio: vhost-user: add stop failure logs.

Browse source 
In a recent CL, we found that errors from stopping a queue were being
swallowed by the vhost-user middleware. This CL adds some logging to
cover this case.

BUG=b:308890529
TEST=used new logging to debug an issue.

Change-Id: I83a89e542d79ae0fcc4ff430c90edb5fc4335497
Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/5054420
Commit-Queue: Noah Gold <nkgold@google.com>
Reviewed-by: Keiichi Watanabe <keiichiw@chromium.org>
Reviewed-by: Daniel Verkamp <dverkamp@chromium.org>
This commit is contained in:
Noah Gold 2023-11-22 21:58:14 -08:00 committed by crosvm LUCI
parent fda3cfc61f
commit 4a994d90ba
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
devices/src/virtio/vhost/user/device/handler.rs
View file

@ -721,7 +721,10 @@ impl VhostUserSlaveReqHandlerMut for DeviceRequestHandler {
{
match self.backend.stop_queue(index) {
Ok(queue) => vring.paused_queue = Some(queue),
Err(e) => return Err(VhostError::StopQueueError(e)),
Err(e) => {
error!("failed to stop queue index {}: {:#}", index, e);
return Err(VhostError::StopQueueError(e));
}
}
}
self.backend