[Qemu-devel] [PATCH] Convert fprintf() to error_request(): virtio-9p

[Sanchit Garg]

Signed-off-by: Sanchit Garg <address@hidden>
---
 hw/virtio-9p.c |   21 +++++++++++----------
 1 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/hw/virtio-9p.c b/hw/virtio-9p.c
index 7c59988..d64d05a 100644
--- a/hw/virtio-9p.c
+++ b/hw/virtio-9p.c
@@ -18,6 +18,7 @@
 #include "fsdev/qemu-fsdev.h"
 #include "virtio-9p-debug.h"
 #include "virtio-9p-xattr.h"
+#include "qemu-error.h"
 
 int debug_9p_pdu;
 
@@ -388,8 +389,8 @@ v9fs_string_alloc_printf(char **strp, const char *fmt, 
va_list ap)
             len += 1;
             break;
         default:
-            fprintf(stderr,
-                   "v9fs_string_alloc_printf:Incorrect format %c", *iter);
+            error_report(
+           "v9fs_string_alloc_printf:Incorrect format %c", *iter);
             return -1;
         }
         iter++;
@@ -3677,15 +3678,15 @@ VirtIODevice *virtio_9p_init(DeviceState *dev, V9fsConf 
*conf)
 
     if (!fse) {
         /* We don't have a fsdev identified by fsdev_id */
-        fprintf(stderr, "Virtio-9p device couldn't find fsdev with the "
-                "id = %s\n", conf->fsdev_id ? conf->fsdev_id : "NULL");
+        error_report("Virtio-9p device couldn't find fsdev with the "
+                "id = %s", conf->fsdev_id ? conf->fsdev_id : "NULL");
         exit(1);
     }
 
     if (!fse->path || !conf->tag) {
         /* we haven't specified a mount_tag or the path */
-        fprintf(stderr, "fsdev with id %s needs path "
-                "and Virtio-9p device needs mount_tag arguments\n",
+        error_report("fsdev with id %s needs path "
+                "and Virtio-9p device needs mount_tag arguments",
                 conf->fsdev_id);
         exit(1);
     }
@@ -3707,19 +3708,19 @@ VirtIODevice *virtio_9p_init(DeviceState *dev, V9fsConf 
*conf)
         s->ctx.fs_sm = SM_NONE;
         s->ctx.xops = none_xattr_ops;
     } else {
-        fprintf(stderr, "Default to security_model=none. You may want"
+        error_report("Default to security_model=none. You may want"
                 " enable advanced security model using "
                 "security option:\n\t security_model=passthrough \n\t "
-                "security_model=mapped\n");
+                "security_model=mapped");
         s->ctx.fs_sm = SM_NONE;
         s->ctx.xops = none_xattr_ops;
     }
 
     if (lstat(fse->path, &stat)) {
-        fprintf(stderr, "share path %s does not exist\n", fse->path);
+        error_report("share path %s does not exist", fse->path);
         exit(1);
     } else if (!S_ISDIR(stat.st_mode)) {
-        fprintf(stderr, "share path %s is not a directory \n", fse->path);
+        error_report("share path %s is not a directory", fse->path);
         exit(1);
     }