Commit 79decce3 authored by Greg Kurz's avatar Greg Kurz
Browse files

9pfs: drop excessive error message from virtfs_reset() 



The virtfs_reset() function is called either when the virtio-9p device
gets reset, or when the client starts a new 9P session. In both cases,
if it finds fids from a previous session, the following is printed in
the monitor:

9pfs:virtfs_reset: One or more uncluncked fids found during reset

For example, if a linux guest with a mounted 9P share is reset from the
monitor with system_reset, the message will be printed. This is excessive
since these fids are now clunked and the state is clean.

Signed-off-by: default avatarGreg Kurz <groug@kaod.org>
Reviewed-by: default avatarEric Blake <eblake@redhat.com>
parent 49dd946b

hw/9pfs/9p.c

+1 −6
Original line number Diff line number Diff line
@@ -535,7 +535,7 @@ static int coroutine_fn v9fs_mark_fids_unreclaim(V9fsPDU *pdu, V9fsPath *path) 
static void coroutine_fn virtfs_reset(V9fsPDU *pdu)

{

    V9fsState *s = pdu->s;

    V9fsFidState *fidp = NULL;

    V9fsFidState *fidp;



    /* Free all fids */

    while (s->fid_list) {
@@ -548,11 +548,6 @@ static void coroutine_fn virtfs_reset(V9fsPDU *pdu) 
            free_fid(pdu, fidp);

        }

    }

    if (fidp) {

        /* One or more unclunked fids found... */

        error_report("9pfs:%s: One or more uncluncked fids "

                     "found during reset", __func__);

    }

}



#define P9_QID_TYPE_DIR         0x80