There are many value_set_##x helper for integer, but only for integer.
This patch adds value_set_string() helper to help following commits
create string fields.

Signed-off-by: Wang Nan <wangn...@huawei.com>
Cc: Arnaldo Carvalho de Melo <a...@redhat.com>
Cc: Jiri Olsa <jo...@redhat.com>
---
 tools/perf/util/data-convert-bt.c | 30 ++++++++++++++++++++++++++++++
 1 file changed, 30 insertions(+)

diff --git a/tools/perf/util/data-convert-bt.c 
b/tools/perf/util/data-convert-bt.c
index 4b59879..7908278 100644
--- a/tools/perf/util/data-convert-bt.c
+++ b/tools/perf/util/data-convert-bt.c
@@ -140,6 +140,36 @@ FUNC_VALUE_SET(s64)
 FUNC_VALUE_SET(u64)
 __FUNC_VALUE_SET(u64_hex, u64)
 
+static int string_set_value(struct bt_ctf_field *field, const char *string);
+static __maybe_unused int
+value_set_string(struct ctf_writer *cw, struct bt_ctf_event *event,
+                const char *name, const char *string)
+{
+       struct bt_ctf_field_type *type = cw->data.string;
+       struct bt_ctf_field *field;
+       int ret = 0;
+
+       field = bt_ctf_field_create(type);
+       if (!field) {
+               pr_err("failed to create a field %s\n", name);
+               return -1;
+       }
+
+       ret = string_set_value(field, string);
+       if (ret) {
+               pr_err("failed to set value %s\n", name);
+               goto err_put_field;
+       }
+
+       ret = bt_ctf_event_set_payload(event, name, field);
+       if (ret)
+               pr_err("failed to set payload %s\n", name);
+
+err_put_field:
+       bt_ctf_field_put(field);
+       return ret;
+}
+
 static struct bt_ctf_field_type*
 get_tracepoint_field_type(struct ctf_writer *cw, struct format_field *field)
 {
-- 
1.8.3.4

Reply via email to