Skip to content

Commit 8179d35

Browse files
committed
chore: better panic messages
Add info about queue len and actual size to the message. Signed-off-by: Egor Lazarchuk <[email protected]>
1 parent 6b4e295 commit 8179d35

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

src/vmm/src/devices/virtio/queue.rs

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -333,7 +333,10 @@ impl Queue {
333333
// We are choosing to interrupt execution since this could be a potential malicious
334334
// driver scenario. This way we also eliminate the risk of repeatedly
335335
// logging and potentially clogging the microVM through the log system.
336-
panic!("The number of available virtio descriptors is greater than queue size!");
336+
panic!(
337+
"The number of available virtio descriptors {len} is greater than queue size: {}!",
338+
self.actual_size()
339+
);
337340
}
338341

339342
if len == 0 {
@@ -514,7 +517,11 @@ impl Queue {
514517
// driver scenario. This way we also eliminate the risk of
515518
// repeatedly logging and potentially clogging the microVM through
516519
// the log system.
517-
panic!("The number of available virtio descriptors is greater than queue size!");
520+
panic!(
521+
"The number of available virtio descriptors {len} is greater than queue size: \
522+
{}!",
523+
self.actual_size()
524+
);
518525
}
519526
return false;
520527
}

0 commit comments

Comments
 (0)