From: Lucas De Marchi <lucas.demarchi@profusion.mobi>
Date: Tue, 6 Nov 2012 19:08:43 +0000 (-0200)
Subject: tools: share function to convert prio to string
X-Git-Tag: v11~8
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=84341fbe013bca49038174b03e95c2943a8e184c;p=thirdparty%2Fkmod.git

tools: share function to convert prio to string

No change is expected in the final binary since right now only an inline
function is shared. Later we expect to share more code.
---

diff --git a/Makefile.am b/Makefile.am
index e4c1a83f..d49f3604 100644
--- a/Makefile.am
+++ b/Makefile.am
@@ -108,7 +108,7 @@ noinst_SCRIPTS = tools/insmod tools/rmmod tools/lsmod \
 tools_kmod_SOURCES = tools/kmod.c tools/kmod.h tools/lsmod.c \
 		     tools/rmmod.c tools/insmod.c \
 		     tools/modinfo.c tools/modprobe.c \
-		     tools/depmod.c
+		     tools/depmod.c tools/log.h
 tools_kmod_LDADD = libkmod/libkmod-util.la \
 		   libkmod/libkmod.la
 
diff --git a/tools/depmod.c b/tools/depmod.c
index 8fd0f125..5d94c17e 100644
--- a/tools/depmod.c
+++ b/tools/depmod.c
@@ -26,7 +26,6 @@
 #include <getopt.h>
 #include <errno.h>
 #include <string.h>
-#include <syslog.h>
 #include <limits.h>
 #include <dirent.h>
 #include <sys/utsname.h>
diff --git a/tools/kmod.h b/tools/kmod.h
index 7bf6c438..ceff8610 100644
--- a/tools/kmod.h
+++ b/tools/kmod.h
@@ -38,3 +38,5 @@ extern const struct kmod_cmd kmod_cmd_list;
 
 /* kmod.c */
 extern const char *binname;
+
+#include "log.h"
diff --git a/tools/log.h b/tools/log.h
new file mode 100644
index 00000000..3527bd91
--- /dev/null
+++ b/tools/log.h
@@ -0,0 +1,58 @@
+/*
+ * kmod - log infrastructure
+ *
+ * Copyright (C) 2012  ProFUSION embedded systems
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program.  If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <stdio.h>
+#include <syslog.h>
+
+#include "kmod.h"
+
+_always_inline_ const char *prio_to_str(int prio);
+
+
+_always_inline_ const char *prio_to_str(int prio)
+{
+	const char *prioname;
+	char buf[32];
+
+	switch (prio) {
+	case LOG_CRIT:
+		prioname = "FATAL";
+		break;
+	case LOG_ERR:
+		prioname = "ERROR";
+		break;
+	case LOG_WARNING:
+		prioname = "WARNING";
+		break;
+	case LOG_NOTICE:
+		prioname = "NOTICE";
+		break;
+	case LOG_INFO:
+		prioname = "INFO";
+		break;
+	case LOG_DEBUG:
+		prioname = "DEBUG";
+		break;
+	default:
+		snprintf(buf, sizeof(buf), "LOG-%03d", prio);
+		prioname = buf;
+	}
+
+	return prioname;
+}
diff --git a/tools/modprobe.c b/tools/modprobe.c
index 7c8e2afe..4cd66459 100644
--- a/tools/modprobe.c
+++ b/tools/modprobe.c
@@ -154,38 +154,6 @@ static inline void _show(const char *fmt, ...)
 	va_end(args);
 }
 
-static _always_inline_ const char *prio_to_str(int prio)
-{
-	const char *prioname;
-	char buf[32];
-
-	switch (prio) {
-	case LOG_CRIT:
-		prioname = "FATAL";
-		break;
-	case LOG_ERR:
-		prioname = "ERROR";
-		break;
-	case LOG_WARNING:
-		prioname = "WARNING";
-		break;
-	case LOG_NOTICE:
-		prioname = "NOTICE";
-		break;
-	case LOG_INFO:
-		prioname = "INFO";
-		break;
-	case LOG_DEBUG:
-		prioname = "DEBUG";
-		break;
-	default:
-		snprintf(buf, sizeof(buf), "LOG-%03d", prio);
-		prioname = buf;
-	}
-
-	return prioname;
-}
-
 static void log_modprobe(void *data, int priority, const char *file, int line,
 			const char *fn, const char *format, va_list args)
 {
diff --git a/tools/rmmod.c b/tools/rmmod.c
index 1983f70f..9d095157 100644
--- a/tools/rmmod.c
+++ b/tools/rmmod.c
@@ -62,38 +62,6 @@ static void help(void)
 		binname);
 }
 
-static _always_inline_ const char *prio_to_str(int prio)
-{
-	const char *prioname;
-	char buf[32];
-
-	switch (prio) {
-	case LOG_CRIT:
-		prioname = "FATAL";
-		break;
-	case LOG_ERR:
-		prioname = "ERROR";
-		break;
-	case LOG_WARNING:
-		prioname = "WARNING";
-		break;
-	case LOG_NOTICE:
-		prioname = "NOTICE";
-		break;
-	case LOG_INFO:
-		prioname = "INFO";
-		break;
-	case LOG_DEBUG:
-		prioname = "DEBUG";
-		break;
-	default:
-		snprintf(buf, sizeof(buf), "LOG-%03d", prio);
-		prioname = buf;
-	}
-
-	return prioname;
-}
-
 static void log_rmmod(void *data, int priority, const char *file, int line,
 			const char *fn, const char *format, va_list args)
 {