qemu-devel
[Top][All Lists]
Advanced

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

Re: [PATCH v12 03/11] qapi: Add builtin type time


From: Tao Xu
Subject: Re: [PATCH v12 03/11] qapi: Add builtin type time
Date: Tue, 15 Oct 2019 14:22:44 +0800
User-agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0

Hi Eric,

I am wondering if you could help to review this patch, 1/11, 2/11, 4/11. Thanks for your help.

Tao
On 9/20/2019 3:43 PM, Xu, Tao3 wrote:
Add optional builtin type time, fallback is uint64. This type use
qemu_strtotime_ps() for pre-converting time suffix to numbers.

Signed-off-by: Tao Xu <address@hidden>
---

No changes in v11 and v12.

New patch in v10.
---
  include/qapi/visitor-impl.h  |  4 ++++
  include/qapi/visitor.h       |  9 +++++++++
  qapi/opts-visitor.c          | 22 ++++++++++++++++++++++
  qapi/qapi-visit-core.c       | 12 ++++++++++++
  qapi/qobject-input-visitor.c | 18 ++++++++++++++++++
  qapi/trace-events            |  1 +
  scripts/qapi/common.py       |  2 ++
  7 files changed, 68 insertions(+)

diff --git a/include/qapi/visitor-impl.h b/include/qapi/visitor-impl.h
index 8ccb3b6c20..e0979563c7 100644
--- a/include/qapi/visitor-impl.h
+++ b/include/qapi/visitor-impl.h
@@ -88,6 +88,10 @@ struct Visitor
      void (*type_size)(Visitor *v, const char *name, uint64_t *obj,
                        Error **errp);
+ /* Optional; fallback is type_uint64() */
+    void (*type_time)(Visitor *v, const char *name, uint64_t *obj,
+                      Error **errp);
+
      /* Must be set */
      void (*type_bool)(Visitor *v, const char *name, bool *obj, Error **errp);
diff --git a/include/qapi/visitor.h b/include/qapi/visitor.h
index 5b2ed3f202..4c3198b1c5 100644
--- a/include/qapi/visitor.h
+++ b/include/qapi/visitor.h
@@ -554,6 +554,15 @@ void visit_type_int64(Visitor *v, const char *name, 
int64_t *obj,
  void visit_type_size(Visitor *v, const char *name, uint64_t *obj,
                       Error **errp);
+/*
+ * Visit a uint64_t value.
+ * Like visit_type_uint64(), except that some visitors may choose to
+ * recognize numbers with timeunit suffix, such as "ps", "ns", "us"
+ * "ms" and "s".
+ */
+void visit_type_time(Visitor *v, const char *name, uint64_t *obj,
+                     Error **errp);
+
  /*
   * Visit a boolean value.
   *
diff --git a/qapi/opts-visitor.c b/qapi/opts-visitor.c
index 324b197495..d73b2e51a0 100644
--- a/qapi/opts-visitor.c
+++ b/qapi/opts-visitor.c
@@ -508,6 +508,27 @@ opts_type_size(Visitor *v, const char *name, uint64_t 
*obj, Error **errp)
      processed(ov, name);
  }
+static void
+opts_type_time(Visitor *v, const char *name, uint64_t *obj, Error **errp)
+{
+    OptsVisitor *ov = to_ov(v);
+    const QemuOpt *opt;
+    int err;
+
+    opt = lookup_scalar(ov, name, errp);
+    if (!opt) {
+        return;
+    }
+
+    err = qemu_strtotime_ps(opt->str ? opt->str : "", NULL, obj);
+    if (err < 0) {
+        error_setg(errp, QERR_INVALID_PARAMETER_VALUE, opt->name,
+                   "a time value");
+        return;
+    }
+
+    processed(ov, name);
+}
static void
  opts_optional(Visitor *v, const char *name, bool *present)
@@ -555,6 +576,7 @@ opts_visitor_new(const QemuOpts *opts)
      ov->visitor.type_int64  = &opts_type_int64;
      ov->visitor.type_uint64 = &opts_type_uint64;
      ov->visitor.type_size   = &opts_type_size;
+    ov->visitor.type_time   = &opts_type_time;
      ov->visitor.type_bool   = &opts_type_bool;
      ov->visitor.type_str    = &opts_type_str;
diff --git a/qapi/qapi-visit-core.c b/qapi/qapi-visit-core.c
index 5365561b07..ac8896455c 100644
--- a/qapi/qapi-visit-core.c
+++ b/qapi/qapi-visit-core.c
@@ -277,6 +277,18 @@ void visit_type_size(Visitor *v, const char *name, 
uint64_t *obj,
      }
  }
+void visit_type_time(Visitor *v, const char *name, uint64_t *obj,
+                     Error **errp)
+{
+    assert(obj);
+    trace_visit_type_time(v, name, obj);
+    if (v->type_time) {
+        v->type_time(v, name, obj, errp);
+    } else {
+        v->type_uint64(v, name, obj, errp);
+    }
+}
+
  void visit_type_bool(Visitor *v, const char *name, bool *obj, Error **errp)
  {
      assert(obj);
diff --git a/qapi/qobject-input-visitor.c b/qapi/qobject-input-visitor.c
index 32236cbcb1..9b66941d8a 100644
--- a/qapi/qobject-input-visitor.c
+++ b/qapi/qobject-input-visitor.c
@@ -627,6 +627,23 @@ static void qobject_input_type_size_keyval(Visitor *v, 
const char *name,
      }
  }
+static void qobject_input_type_time_keyval(Visitor *v, const char *name,
+                                           uint64_t *obj, Error **errp)
+{
+    QObjectInputVisitor *qiv = to_qiv(v);
+    const char *str = qobject_input_get_keyval(qiv, name, errp);
+
+    if (!str) {
+        return;
+    }
+
+    if (qemu_strtotime_ps(str, NULL, obj) < 0) {
+        /* TODO report -ERANGE more nicely */
+        error_setg(errp, QERR_INVALID_PARAMETER_VALUE,
+                   full_name(qiv, name), "time");
+    }
+}
+
  static void qobject_input_optional(Visitor *v, const char *name, bool 
*present)
  {
      QObjectInputVisitor *qiv = to_qiv(v);
@@ -708,6 +725,7 @@ Visitor *qobject_input_visitor_new_keyval(QObject *obj)
      v->visitor.type_any = qobject_input_type_any;
      v->visitor.type_null = qobject_input_type_null;
      v->visitor.type_size = qobject_input_type_size_keyval;
+    v->visitor.type_time = qobject_input_type_time_keyval;
      v->keyval = true;
return &v->visitor;
diff --git a/qapi/trace-events b/qapi/trace-events
index 5eb4afa110..c4605a7ccc 100644
--- a/qapi/trace-events
+++ b/qapi/trace-events
@@ -29,6 +29,7 @@ visit_type_int16(void *v, const char *name, int16_t *obj) "v=%p 
name=%s obj=%p"
  visit_type_int32(void *v, const char *name, int32_t *obj) "v=%p name=%s 
obj=%p"
  visit_type_int64(void *v, const char *name, int64_t *obj) "v=%p name=%s 
obj=%p"
  visit_type_size(void *v, const char *name, uint64_t *obj) "v=%p name=%s 
obj=%p"
+visit_type_time(void *v, const char *name, uint64_t *obj) "v=%p name=%s obj=%p"
  visit_type_bool(void *v, const char *name, bool *obj) "v=%p name=%s obj=%p"
  visit_type_str(void *v, const char *name, char **obj) "v=%p name=%s obj=%p"
  visit_type_number(void *v, const char *name, void *obj) "v=%p name=%s obj=%p"
diff --git a/scripts/qapi/common.py b/scripts/qapi/common.py
index d61bfdc526..3a6f108794 100644
--- a/scripts/qapi/common.py
+++ b/scripts/qapi/common.py
@@ -35,6 +35,7 @@ builtin_types = {
      'uint32':   'QTYPE_QNUM',
      'uint64':   'QTYPE_QNUM',
      'size':     'QTYPE_QNUM',
+    'time':     'QTYPE_QNUM',
      'any':      None,           # any QType possible, actually
      'QType':    'QTYPE_QSTRING',
  }
@@ -1834,6 +1835,7 @@ class QAPISchema(object):
                    ('uint32', 'int',     'uint32_t'),
                    ('uint64', 'int',     'uint64_t'),
                    ('size',   'int',     'uint64_t'),
+                  ('time',   'int',     'uint64_t'),
                    ('bool',   'boolean', 'bool'),
                    ('any',    'value',   'QObject' + pointer_suffix),
                    ('null',   'null',    'QNull' + pointer_suffix)]:





reply via email to

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