summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorQuentin Monnet <quentin.monnet@netronome.com>2017-11-03 13:59:07 -0700
committerDavid S. Miller <davem@davemloft.net>2017-11-05 22:28:51 +0900
commit0b1c27db12fd338ed912fec18f5cc02d7bd4e54e (patch)
tree0dd8caba4edcc1dee3ba8c30b520d57ace8ced45 /tools
parent8a3b718ac2c29abcba8e12636710cff3cee8c01b (diff)
downloadop-kernel-dev-0b1c27db12fd338ed912fec18f5cc02d7bd4e54e.zip
op-kernel-dev-0b1c27db12fd338ed912fec18f5cc02d7bd4e54e.tar.gz
tools: bpftool: move p_err() and p_info() from main.h to common.c
The two functions were declared as static inline in a header file. There is no particular reason why they should be inlined, they just happened to remain in the same header file when they were turned from macros to functions in a precious commit. Make them non-inlined functions and move them to common.c file instead. Suggested-by: Joe Perches <joe@perches.com> Signed-off-by: Quentin Monnet <quentin.monnet@netronome.com> Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com> Acked-by: Alexei Starovoitov <ast@kernel.org> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'tools')
-rw-r--r--tools/bpf/bpftool/common.c31
-rw-r--r--tools/bpf/bpftool/main.h34
2 files changed, 34 insertions, 31 deletions
diff --git a/tools/bpf/bpftool/common.c b/tools/bpf/bpftool/common.c
index f028826..aa70170 100644
--- a/tools/bpf/bpftool/common.c
+++ b/tools/bpf/bpftool/common.c
@@ -50,6 +50,37 @@
#include "main.h"
+void p_err(const char *fmt, ...)
+{
+ va_list ap;
+
+ va_start(ap, fmt);
+ if (json_output) {
+ jsonw_start_object(json_wtr);
+ jsonw_name(json_wtr, "error");
+ jsonw_vprintf_enquote(json_wtr, fmt, ap);
+ jsonw_end_object(json_wtr);
+ } else {
+ fprintf(stderr, "Error: ");
+ vfprintf(stderr, fmt, ap);
+ fprintf(stderr, "\n");
+ }
+ va_end(ap);
+}
+
+void p_info(const char *fmt, ...)
+{
+ va_list ap;
+
+ if (json_output)
+ return;
+
+ va_start(ap, fmt);
+ vfprintf(stderr, fmt, ap);
+ fprintf(stderr, "\n");
+ va_end(ap);
+}
+
static bool is_bpffs(char *path)
{
struct statfs st_fs;
diff --git a/tools/bpf/bpftool/main.h b/tools/bpf/bpftool/main.h
index d315d01..ff5ad05 100644
--- a/tools/bpf/bpftool/main.h
+++ b/tools/bpf/bpftool/main.h
@@ -71,6 +71,9 @@ extern const char *bin_name;
extern json_writer_t *json_wtr;
extern bool json_output;
+void p_err(const char *fmt, ...);
+void p_info(const char *fmt, ...);
+
bool is_prefix(const char *pfx, const char *str);
void fprint_hex(FILE *f, void *arg, unsigned int n, const char *sep);
void usage(void) __attribute__((noreturn));
@@ -97,35 +100,4 @@ int prog_parse_fd(int *argc, char ***argv);
void disasm_print_insn(unsigned char *image, ssize_t len, int opcodes);
void print_hex_data_json(uint8_t *data, size_t len);
-static inline void p_err(const char *fmt, ...)
-{
- va_list ap;
-
- va_start(ap, fmt);
- if (json_output) {
- jsonw_start_object(json_wtr);
- jsonw_name(json_wtr, "error");
- jsonw_vprintf_enquote(json_wtr, fmt, ap);
- jsonw_end_object(json_wtr);
- } else {
- fprintf(stderr, "Error: ");
- vfprintf(stderr, fmt, ap);
- fprintf(stderr, "\n");
- }
- va_end(ap);
-}
-
-static inline void p_info(const char *fmt, ...)
-{
- va_list ap;
-
- if (json_output)
- return;
-
- va_start(ap, fmt);
- vfprintf(stderr, fmt, ap);
- fprintf(stderr, "\n");
- va_end(ap);
-}
-
#endif
OpenPOWER on IntegriCloud