qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH for 2.6 1/1] qemu-char: append opt to stop trunc


From: Denis V. Lunev
Subject: Re: [Qemu-devel] [PATCH for 2.6 1/1] qemu-char: append opt to stop truncation of serial file
Date: Thu, 3 Dec 2015 09:32:54 +0300
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0

On 12/03/2015 09:24 AM, Denis V. Lunev wrote:
From: Olga Krishtal <address@hidden>

Our QA teams wants to preserve serial output of the guest in between QEMU
runs to perform post-analysis.

By default this behavior is off (file is truncated each time QEMU is started
or device is plugged).

Signed-off-by: Olga Krishtal <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
CC: Eric Blake <address@hidden>
CC: Markus Armbruster <address@hidden>
CC: Paolo Bonzini <address@hidden>
---
  qapi-schema.json |  7 ++++++-
  qemu-char.c      | 12 +++++++++++-
  2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/qapi-schema.json b/qapi-schema.json
index 8b1a423..802c138 100644
--- a/qapi-schema.json
+++ b/qapi-schema.json
@@ -3020,11 +3020,16 @@
  #
  # @in:  #optional The name of the input file
  # @out: The name of the output file
+# @append: #optional Open the file in append mode to preserve the content in
+#          between QEMU runs.
+#
+#          Since: 2.6
  #
  # Since: 1.4
  ##
  { 'struct': 'ChardevFile', 'data': { '*in' : 'str',
-                                   'out' : 'str' } }
+                                   'out' : 'str',
+                                   '*append': 'bool' } }
##
  # @ChardevHostdev:
diff --git a/qemu-char.c b/qemu-char.c
index 5448b0f..58454e2 100644
--- a/qemu-char.c
+++ b/qemu-char.c
@@ -3479,6 +3479,7 @@ static void qemu_chr_parse_file_out(QemuOpts *opts, 
ChardevBackend *backend,
      }
      backend->u.file = g_new0(ChardevFile, 1);
      backend->u.file->out = g_strdup(path);
+    backend->u.file->append = qemu_opt_get_bool(opts, "append", false);
  }
static void qemu_chr_parse_stdio(QemuOpts *opts, ChardevBackend *backend,
@@ -4036,6 +4037,9 @@ QemuOptsList qemu_chardev_opts = {
          },{
              .name = "chardev",
              .type = QEMU_OPT_STRING,
+        },{
+            .name = "append",
+            .type = QEMU_OPT_BOOL,
          },
          { /* end of list */ }
      },
@@ -4096,7 +4100,13 @@ static CharDriverState *qmp_chardev_open_file(const char 
*id,
      ChardevFile *file = backend->u.file;
      int flags, in = -1, out;
- flags = O_WRONLY | O_TRUNC | O_CREAT | O_BINARY;
+    flags = O_WRONLY | O_CREAT | O_BINARY;
+    if (file->append) {
+        flags |= O_APPEND;
+    } else {
+        flags |= O_TRUNC;
+    }
+
      out = qmp_chardev_open_file_source(file->out, flags, errp);
      if (out < 0) {
          return NULL;
pls disregard. Originator e-mails are wrong. Sorry :(



reply via email to

[Prev in Thread] Current Thread [Next in Thread]