test: dm: virtio: Test notify before del_vqs

The virtqueue is passed to virtio_notify() so move the virtqueue
deletion to the end of the test when it's no longer needed. This wasn't
causing any problems because the sandbox virtio transport driver doesn't
do anything for notifications, but it could cause problems if things
change and it was a bad example.

Signed-off-by: Andrew Scull <ascull@google.com>
Reviewed-by: Simon Glass <sjg@chromium.org>

diff --git a/test/dm/virtio.c b/test/dm/virtio.c
index aa4e3d778e8bea930f22408d693b6dd5f69a8446..ff1dea323cf1785dd2aeb6a7715920f43b422df9 100644 (file)
--- a/test/dm/virtio.c
+++ b/test/dm/virtio.c
@@ -80,8 +80,8 @@ static int dm_test_virtio_all_ops(struct unit_test_state *uts)
        ut_asserteq_64(BIT_ULL(VIRTIO_F_VERSION_1), features);
        ut_assertok(virtio_set_features(dev));
        ut_assertok(virtio_find_vqs(dev, nvqs, vqs));
-       ut_assertok(virtio_del_vqs(dev));
        ut_assertok(virtio_notify(dev, vqs[0]));
+       ut_assertok(virtio_del_vqs(dev));
 
        return 0;
 }