diff options
| author | Greg Kurz <groug@kaod.org> | 2016-11-01 12:00:40 +0100 |
|---|---|---|
| committer | Greg Kurz <groug@kaod.org> | 2016-11-01 12:03:03 +0100 |
| commit | 79decce35b4d769fa878b048ab1a7b3e9045c9c6 (patch) | |
| tree | 1fe84a9e2c5c246a7048233c6a47f656f783fef9 | |
| parent | 49dd946bb5419681c8668b09a6d10f42bc707b78 (diff) | |
| download | focaccia-qemu-79decce35b4d769fa878b048ab1a7b3e9045c9c6.tar.gz focaccia-qemu-79decce35b4d769fa878b048ab1a7b3e9045c9c6.zip | |
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: Greg Kurz <groug@kaod.org> Reviewed-by: Eric Blake <eblake@redhat.com>
| -rw-r--r-- | hw/9pfs/9p.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/hw/9pfs/9p.c b/hw/9pfs/9p.c index 1050b89ec7..aea7e9d392 100644 --- a/hw/9pfs/9p.c +++ b/hw/9pfs/9p.c @@ -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 |