These APIs are redundant, their functionality can be replaced by existing library APIs. Removed them, to simplify the API set: tracecmd_create_init_fd() tracecmd_create_init_file() Signed-off-by: Tzvetomir Stoyanov (VMware) <tz.stoyanov@xxxxxxxxx> --- .../include/private/trace-cmd-private.h | 2 -- lib/trace-cmd/trace-output.c | 36 ------------------- tracecmd/trace-stream.c | 3 +- 3 files changed, 2 insertions(+), 39 deletions(-) diff --git a/lib/trace-cmd/include/private/trace-cmd-private.h b/lib/trace-cmd/include/private/trace-cmd-private.h index 79196147..069283c7 100644 --- a/lib/trace-cmd/include/private/trace-cmd-private.h +++ b/lib/trace-cmd/include/private/trace-cmd-private.h @@ -281,9 +281,7 @@ int tracecmd_output_write_headers(struct tracecmd_output *handle, struct tracecmd_output *tracecmd_output_create(const char *output_file); struct tracecmd_output *tracecmd_output_create_fd(int fd); struct tracecmd_output *tracecmd_create_file_latency(const char *output_file, int cpus); -struct tracecmd_output *tracecmd_create_init_fd(int fd); -struct tracecmd_output *tracecmd_create_init_file(const char *output_file); struct tracecmd_option *tracecmd_add_option(struct tracecmd_output *handle, unsigned short id, int size, const void *data); diff --git a/lib/trace-cmd/trace-output.c b/lib/trace-cmd/trace-output.c index 067deae8..a20e42df 100644 --- a/lib/trace-cmd/trace-output.c +++ b/lib/trace-cmd/trace-output.c @@ -1828,42 +1828,6 @@ struct tracecmd_output *tracecmd_output_create(const char *output_file) return out; } -struct tracecmd_output *tracecmd_create_init_fd(int fd) -{ - struct tracecmd_output *out; - - out = tracecmd_output_create_fd(fd); - if (!out) - return NULL; - if (tracecmd_output_write_init(out)) - goto error; - if (tracecmd_output_write_headers(out, NULL)) - goto error; - - return out; -error: - tracecmd_output_close(out); - return NULL; -} - -struct tracecmd_output *tracecmd_create_init_file(const char *output_file) -{ - struct tracecmd_output *handle; - int fd; - - fd = open(output_file, O_RDWR | O_CREAT | O_TRUNC | O_LARGEFILE, 0644); - if (fd < 0) - return NULL; - handle = tracecmd_create_init_fd(fd); - if (!handle) { - close(fd); - unlink(output_file); - return NULL; - } - - return handle; -} - /** * tracecmd_copy - copy the headers of one trace.dat file for another * @ihandle: input handle of the trace.dat file to copy diff --git a/tracecmd/trace-stream.c b/tracecmd/trace-stream.c index b47b208c..ee310f3d 100644 --- a/tracecmd/trace-stream.c +++ b/tracecmd/trace-stream.c @@ -43,11 +43,12 @@ trace_stream_init(struct buffer_instance *instance, int cpu, int fd, int cpus, tfd = fileno(fp); ofd = dup(tfd); - trace_output = tracecmd_create_init_fd(ofd); + trace_output = tracecmd_output_create_fd(ofd); if (!trace_output) { fclose(fp); return NULL; } + tracecmd_output_write_headers(trace_output, NULL); tracecmd_output_free(trace_output); } -- 2.33.1