summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlex Bennée2022-08-02 11:50:06 +0200
committerMichael S. Tsirkin2022-10-07 15:41:51 +0200
commitf48d994fb50c49093dd1dfe64c0d7314c6b62faf (patch)
tree8eb483ed52e1ed31bcdf501ca394dd218a14531e
parenttests/qtest: catch unhandled vhost-user messages (diff)
downloadqemu-f48d994fb50c49093dd1dfe64c0d7314c6b62faf.tar.gz
qemu-f48d994fb50c49093dd1dfe64c0d7314c6b62faf.tar.xz
qemu-f48d994fb50c49093dd1dfe64c0d7314c6b62faf.zip
tests/qtest: plain g_assert for VHOST_USER_F_PROTOCOL_FEATURES
checkpatch.pl warns that non-plain asserts should be avoided so convert the check to a plain g_assert. Signed-off-by: Alex Bennée <alex.bennee@linaro.org> Message-Id: <20220802095010.3330793-19-alex.bennee@linaro.org> Reviewed-by: Michael S. Tsirkin <mst@redhat.com> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
-rw-r--r--tests/qtest/vhost-user-test.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/tests/qtest/vhost-user-test.c b/tests/qtest/vhost-user-test.c
index 8d2d4ba535..a99f55ed84 100644
--- a/tests/qtest/vhost-user-test.c
+++ b/tests/qtest/vhost-user-test.c
@@ -985,8 +985,7 @@ static void test_multiqueue(void *obj, void *arg, QGuestAllocator *alloc)
static void vu_net_set_features(TestServer *s, CharBackend *chr,
VhostUserMsg *msg)
{
- g_assert_cmpint(msg->payload.u64 &
- (0x1ULL << VHOST_USER_F_PROTOCOL_FEATURES), !=, 0ULL);
+ g_assert(msg->payload.u64 & (0x1ULL << VHOST_USER_F_PROTOCOL_FEATURES));
if (s->test_flags == TEST_FLAGS_DISCONNECT) {
qemu_chr_fe_disconnect(chr);
s->test_flags = TEST_FLAGS_BAD;