Commit 6e37f458 authored by Greg Kurz's avatar Greg Kurz
Browse files

tests: virtio-9p: improve error reporting 



Signed-off-by: default avatarGreg Kurz <groug@kaod.org>
parent a1bf8b74

tests/virtio-9p-test.c

+19 −5
Original line number Diff line number Diff line
@@ -236,6 +236,16 @@ static void v9fs_req_send(P9Req *req) 
    req->t_off = 0;

}



static const char *rmessage_name(uint8_t id)

{

    return

        id == P9_RLERROR ? "RLERROR" :

        id == P9_RVERSION ? "RVERSION" :

        id == P9_RATTACH ? "RATTACH" :

        id == P9_RWALK ? "RWALK" :

        "<unknown>";

}



static void v9fs_req_recv(P9Req *req, uint8_t id)

{

    QVirtIO9P *v9p = req->v9p;
@@ -258,11 +268,15 @@ static void v9fs_req_recv(P9Req *req, uint8_t id) 
    g_assert_cmpint(hdr.size, <=, P9_MAX_SIZE);

    g_assert_cmpint(hdr.tag, ==, req->tag);



    if (hdr.id != id && hdr.id == P9_RLERROR) {

    if (hdr.id != id) {

        g_printerr("Received response %d (%s) instead of %d (%s)\n",

                   hdr.id, rmessage_name(hdr.id), id, rmessage_name(id));



        if (hdr.id == P9_RLERROR) {

            uint32_t err;

            v9fs_uint32_read(req, &err);

        g_printerr("Received Rlerror (%d) instead of Response %d\n", err, id);

        g_assert_not_reached();

            g_printerr("Rlerror has errno %d (%s)\n", err, strerror(err));

        }

    }

    g_assert_cmpint(hdr.id, ==, id);

}