Make-initrd development discussion
 help / color / mirror / Atom feed
From: Alexey Gladkov <gladkov.alexey@gmail.com>
To: make-initrd@lists.altlinux.org
Subject: [make-initrd] ueventd: Add a prefix to the logging functions to avoid name collisions
Date: Sat,  6 May 2023 21:45:01 +0200
Message-ID: <20230506194504.1471267-1-gladkov.alexey@gmail.com> (raw)
In-Reply-To: <cover.1683200226.git.gladkov.alexey@gmail.com>

Signed-off-by: Alexey Gladkov <gladkov.alexey@gmail.com>
---
 datasrc/ueventd/logging.c         | 10 +++---
 datasrc/ueventd/memory.c          |  6 ++--
 datasrc/ueventd/path.c            |  4 +--
 datasrc/ueventd/queue-processor.c | 28 +++++++--------
 datasrc/ueventd/ueventd.c         | 60 +++++++++++++++----------------
 datasrc/ueventd/ueventd.h         | 20 +++++------
 6 files changed, 64 insertions(+), 64 deletions(-)

diff --git a/datasrc/ueventd/logging.c b/datasrc/ueventd/logging.c
index 671f6814..3a04a5a9 100644
--- a/datasrc/ueventd/logging.c
+++ b/datasrc/ueventd/logging.c
@@ -12,7 +12,7 @@
 
 int log_priority = LOG_INFO;
 
-int logging_level(const char *name)
+int rd_logging_level(const char *name)
 {
 	if (!strcasecmp(name, "debug"))   return LOG_DEBUG;
 	if (!strcasecmp(name, "info"))    return LOG_INFO;
@@ -21,7 +21,7 @@ int logging_level(const char *name)
 	return log_priority;
 }
 
-void logging_init(int loglevel)
+void rd_logging_init(int loglevel)
 {
 	if (!getenv("UEVENTD_USE_STDERR")) {
 		char *rdlog = getenv("RDLOG");
@@ -32,7 +32,7 @@ void logging_init(int loglevel)
 
 		FILE *cons = fopen(logfile, "w+");
 		if (!cons)
-			fatal("open(%s): %m", logfile);
+			rd_fatal("open(%s): %m", logfile);
 
 		fclose(stderr);
 		stderr = cons;
@@ -41,11 +41,11 @@ void logging_init(int loglevel)
 	log_priority = loglevel;
 }
 
-void logging_close(void)
+void rd_logging_close(void)
 {
 }
 
-void message(int priority, const char *fmt, ...)
+void rd_message(int priority, const char *fmt, ...)
 {
 	va_list ap;
 	va_start(ap, fmt);
diff --git a/datasrc/ueventd/memory.c b/datasrc/ueventd/memory.c
index cb911d58..674928c0 100644
--- a/datasrc/ueventd/memory.c
+++ b/datasrc/ueventd/memory.c
@@ -14,8 +14,8 @@ void *xcalloc(size_t nmemb, size_t size)
 {
 	void *r = calloc(nmemb, size);
 	if (!r)
-		fatal("calloc: allocating %lu*%lu bytes: %m",
-		      (unsigned long) nmemb, (unsigned long) size);
+		rd_fatal("calloc: allocating %lu*%lu bytes: %m",
+		         (unsigned long) nmemb, (unsigned long) size);
 	return r;
 }
 
@@ -25,7 +25,7 @@ char *xasprintf(char **ptr, const char *fmt, ...)
 
 	va_start(arg, fmt);
 	if (vasprintf(ptr, fmt, arg) < 0)
-		fatal("vasprintf: %m");
+		rd_fatal("vasprintf: %m");
 	va_end(arg);
 
 	return *ptr;
diff --git a/datasrc/ueventd/path.c b/datasrc/ueventd/path.c
index 7880ad5c..835f4fbb 100644
--- a/datasrc/ueventd/path.c
+++ b/datasrc/ueventd/path.c
@@ -18,7 +18,7 @@ DIR *xopendir(const char *path)
 {
 	DIR *dir = opendir(path);
 	if (!dir)
-		fatal("opendir: %s: %m", path);
+		rd_fatal("opendir: %s: %m", path);
 	return dir;
 }
 
@@ -32,7 +32,7 @@ struct dirent *xreaddir(DIR *d, const char *path)
 	if (!ent) {
 		if (!errno)
 			return NULL;
-		fatal("readdir: %s: %m", path);
+		rd_fatal("readdir: %s: %m", path);
 	}
 	return ent;
 }
diff --git a/datasrc/ueventd/queue-processor.c b/datasrc/ueventd/queue-processor.c
index ab5e03e4..74e6dfc0 100644
--- a/datasrc/ueventd/queue-processor.c
+++ b/datasrc/ueventd/queue-processor.c
@@ -24,20 +24,20 @@ void event_handler(struct watch *queue, char *path)
 	pid_t pid = vfork();
 
 	if (pid < 0) {
-		err("fork: %m");
+		rd_err("fork: %m");
 
 	} else if (!pid) {
 		execve(argv[0], argv, environ);
-		fatal("exec: %s: %m", argv[0]);
+		rd_fatal("exec: %s: %m", argv[0]);
 	} else {
 		int status = 0;
 
 		if (waitpid_retry(pid, &status, 0) != pid || !WIFEXITED(status))
-			info("%s: session=%lu: %s failed",
-			     queue->q_name, session, handler_file);
+			rd_info("%s: session=%lu: %s failed",
+			        queue->q_name, session, handler_file);
 		else
-			info("%s: session=%lu: %s finished with return code %d",
-			     queue->q_name, session, handler_file, WEXITSTATUS(status));
+			rd_info("%s: session=%lu: %s finished with return code %d",
+			        queue->q_name, session, handler_file, WEXITSTATUS(status));
 	}
 }
 
@@ -47,27 +47,27 @@ void move_files(char *srcdir, char *dstdir)
 	int srcfd, dstfd;
 
 	if ((srcfd = open(srcdir, O_RDONLY | O_CLOEXEC)) < 0)
-		fatal("open: %s: %m", srcdir);
+		rd_fatal("open: %s: %m", srcdir);
 
 	errno = 0;
 	if ((dstfd = open(dstdir, O_RDONLY | O_CLOEXEC)) < 0) {
 		if (errno == ENOENT) {
 			if (mkdir(dstdir, 0755) < 0)
-				fatal("mkdir: %s: %m", dstdir);
+				rd_fatal("mkdir: %s: %m", dstdir);
 			dstfd = open(dstdir, O_RDONLY | O_CLOEXEC);
 		}
 		if (dstfd < 0)
-			fatal("open: %s: %m", dstdir);
+			rd_fatal("open: %s: %m", dstdir);
 	}
 
 	DIR *d = fdopendir(srcfd);
 	if (!d)
-		fatal("fdopendir: %m");
+		rd_fatal("fdopendir: %m");
 
 	while ((ent = xreaddir(d, srcdir)) != NULL) {
 		if (ent->d_name[0] != '.' && ent->d_type == DT_REG &&
 		    renameat(srcfd, ent->d_name, dstfd, ent->d_name) < 0)
-			fatal("rename `%s/%s' -> `%s/%s': %m", srcdir, ent->d_name, dstdir, ent->d_name);
+			rd_fatal("rename `%s/%s' -> `%s/%s': %m", srcdir, ent->d_name, dstdir, ent->d_name);
 	}
 
 	closedir(d);
@@ -81,14 +81,14 @@ void signal_unhandled_events(struct watch *queue)
 	len = write_loop(queue->q_parentfd,
 	                 (char *) &queue->q_watchfd, sizeof(queue->q_watchfd));
 	if (len < 0)
-		err("write(pipe): %m");
+		rd_err("write(pipe): %m");
 
-	info("%s: session=%lu: retry with the events remaining in the queue", queue->q_name, session);
+	rd_info("%s: session=%lu: retry with the events remaining in the queue", queue->q_name, session);
 }
 
 void process_events(struct watch *queue)
 {
-	info("%s: session=%lu: processing events", queue->q_name, session);
+	rd_info("%s: session=%lu: processing events", queue->q_name, session);
 
 	char *numenv = NULL;
 
diff --git a/datasrc/ueventd/ueventd.c b/datasrc/ueventd/ueventd.c
index f5923367..f4b89cb9 100644
--- a/datasrc/ueventd/ueventd.c
+++ b/datasrc/ueventd/ueventd.c
@@ -85,7 +85,7 @@ int add_queue_dir(int inotifyfd, const char *path, uint32_t mask)
 		if (errno == EEXIST) {
 			return -128;
 		}
-		err("inotify failed to watch: %s: %m", path);
+		rd_err("inotify failed to watch: %s: %m", path);
 	}
 	return ret;
 }
@@ -103,7 +103,7 @@ int watch_path(int inotifyfd, const char *dir, const char *name, uint32_t mask,
 		return (wfd == -128 ? 0 : wfd);
 
 	if (stat(path, &st) < 0) {
-		err("stat: %s: %m", path);
+		rd_err("stat: %s: %m", path);
 		goto fail;
 	}
 
@@ -127,7 +127,7 @@ int watch_path(int inotifyfd, const char *dir, const char *name, uint32_t mask,
 
 	watch_list = new;
 
-	info("watch path: %s", path);
+	rd_info("watch path: %s", path);
 	free(path);
 	return 0;
 fail:
@@ -216,7 +216,7 @@ void watch_queues(int inotifyfd)
 {
 	DIR *dir = opendir(filter_dir);
 	if (!dir)
-		fatal("opendir: %s: %m", filter_dir);
+		rd_fatal("opendir: %s: %m", filter_dir);
 
 	struct dirent *ent;
 
@@ -225,7 +225,7 @@ void watch_queues(int inotifyfd)
 			continue;
 
 		if (watch_path(inotifyfd, filter_dir, ent->d_name, EV_QUEUE_MASK, F_QUEUE_DIR) < 0)
-			fatal("unable to start watching: %s/%s", filter_dir, ent->d_name);
+			rd_fatal("unable to start watching: %s/%s", filter_dir, ent->d_name);
 	}
 	closedir(dir);
 }
@@ -238,7 +238,7 @@ int process_signal_events(int signfd)
 
 	size = read_retry(signfd, &fdsi, sizeof(fdsi));
 	if (size != sizeof(fdsi)) {
-		err("unable to read signal info");
+		rd_err("unable to read signal info");
 		return 0;
 	}
 
@@ -248,7 +248,7 @@ int process_signal_events(int signfd)
 
 			if (pid <= 0) {
 				if (pid < 0 && errno != ECHILD) {
-					err("waitpid: %m");
+					rd_err("waitpid: %m");
 					return -1;
 				}
 				break;
@@ -262,7 +262,7 @@ int process_signal_events(int signfd)
 		return 0;
 	}
 
-	info("got signal %d, exit", fdsi.ssi_signo);
+	rd_info("got signal %d, exit", fdsi.ssi_signo);
 	return -1;
 }
 
@@ -279,10 +279,10 @@ void setup_signal_fd(struct fd_handler *el)
 	sigaddset(&mask, SIGCHLD);
 
 	if ((el->fd = signalfd(-1, &mask, SFD_NONBLOCK | SFD_CLOEXEC)) < 0)
-		fatal("signalfd: %m");
+		rd_fatal("signalfd: %m");
 
 	if (sigprocmask(SIG_BLOCK, &mask, NULL) < 0)
-		fatal("sigprocmask: %m");
+		rd_fatal("sigprocmask: %m");
 
 	el->fd_handler = process_signal_events;
 }
@@ -297,7 +297,7 @@ int process_inotify_events(int inotifyfd)
 		if (len < 0) {
 			if (errno == EAGAIN)
 				break;
-			fatal("read: %m");
+			rd_fatal("read: %m");
 		} else if (!len) {
 			break;
 		}
@@ -315,7 +315,7 @@ int process_inotify_events(int inotifyfd)
 				continue;
 
 			if (event->mask & IN_DELETE_SELF) {
-				info("unwatch path: %s", p->q_name);
+				rd_info("unwatch path: %s", p->q_name);
 				unwatch_path(inotifyfd, p->q_ino);
 				continue;
 			}
@@ -328,9 +328,9 @@ int process_inotify_events(int inotifyfd)
 
 			if (p->q_flags & F_PAUSE_DIR) {
 				if (event->mask & IN_CREATE)
-					info("%s: queue paused", event->name);
+					rd_info("%s: queue paused", event->name);
 				else if (event->mask & IN_DELETE)
-					info("%s: queue unpaused", event->name);
+					rd_info("%s: queue unpaused", event->name);
 				apply_pause();
 				continue;
 			}
@@ -345,7 +345,7 @@ int process_inotify_events(int inotifyfd)
 void setup_inotify_fd(struct fd_handler *el)
 {
 	if ((el->fd = inotify_init1(IN_NONBLOCK | IN_CLOEXEC)) < 0)
-		fatal("inotify_init1: %m");
+		rd_fatal("inotify_init1: %m");
 
 	el->fd_handler = process_inotify_events;
 }
@@ -363,7 +363,7 @@ int process_pipefd_events(int readfd)
 		if (len < 0) {
 			if (errno == EAGAIN)
 				break;
-			fatal("read(pipe): %m");
+			rd_fatal("read(pipe): %m");
 		} else if (!len) {
 			break;
 		}
@@ -383,7 +383,7 @@ int process_pipefd_events(int readfd)
 void setup_pipe_fd(struct fd_handler *el)
 {
 	if (pipe2(pipefd, O_NONBLOCK | O_CLOEXEC) < 0)
-		fatal("pipe2: %m");
+		rd_fatal("pipe2: %m");
 
 	el->fd = pipefd[PIPE_READ];
 	el->fd_handler = process_pipefd_events;
@@ -395,13 +395,13 @@ int setup_epoll_fd(struct fd_handler list[FD_MAX])
 	struct epoll_event ev = { .events = EPOLLIN };
 
 	if ((epollfd = epoll_create1(EPOLL_CLOEXEC)) < 0)
-		fatal("epoll_create1: %m");
+		rd_fatal("epoll_create1: %m");
 
 	for (int i = 0; i < FD_MAX; i++) {
 		ev.data.fd = list[i].fd;
 
 		if (epoll_ctl(epollfd, EPOLL_CTL_ADD, list[i].fd, &ev) < 0)
-			fatal("epoll_ctl: %m");
+			rd_fatal("epoll_ctl: %m");
 	}
 
 	return epollfd;
@@ -415,14 +415,14 @@ pid_t spawn_worker(int epollfd, struct watch *queue)
 
 	pid = fork();
 	if (pid < 0) {
-		err("fork: %m");
+		rd_err("fork: %m");
 		return 0;
 	} else if (pid > 0) {
 		return pid;
 	}
 
 	if (prctl(PR_SET_PDEATHSIG, SIGKILL) < 0)
-		fatal("prctl(PR_SET_PDEATHSIG): %m");
+		rd_fatal("prctl(PR_SET_PDEATHSIG): %m");
 
 	for (int i = 0; i < FD_MAX; i++) {
 		if (fd_list[i].fd >= 0)
@@ -434,7 +434,7 @@ pid_t spawn_worker(int epollfd, struct watch *queue)
 	sigemptyset(&mask);
 
 	if (sigprocmask(SIG_SETMASK, &mask, NULL) < 0)
-		fatal("sigprocmask: %m");
+		rd_fatal("sigprocmask: %m");
 
 	process_events(queue);
 
@@ -445,7 +445,7 @@ char *get_param_dir(const char *name)
 {
 	char *value = getenv(name);
 	if (!value)
-		fatal("please set `%s' env variable", name);
+		rd_fatal("please set `%s' env variable", name);
 	return value;
 }
 
@@ -463,7 +463,7 @@ int main(int argc, char **argv)
 				exit(0);
 				break;
 			case 'l':
-				loglevel = logging_level(optarg);
+				loglevel = rd_logging_level(optarg);
 				break;
 			default:
 				exit(1);
@@ -472,11 +472,11 @@ int main(int argc, char **argv)
 	}
 
 	if (optind == argc)
-		fatal("specify handler program");
+		rd_fatal("specify handler program");
 
-	logging_init(loglevel);
+	rd_logging_init(loglevel);
 
-	info("starting server ...");
+	rd_info("starting server ...");
 
 	filter_dir = get_param_dir("filterdir");
 	uevent_dir = get_param_dir("ueventdir");
@@ -507,7 +507,7 @@ int main(int argc, char **argv)
 		if (fdcount < 0) {
 			if (errno == EINTR)
 				continue;
-			fatal("epoll_wait: %m");
+			rd_fatal("epoll_wait: %m");
 		}
 
 		for (i = 0; i < fdcount; i++) {
@@ -540,12 +540,12 @@ done:
 	for (i = 0; i < FD_MAX; i++) {
 		if (fd_list[i].fd >= 0) {
 			if (epoll_ctl(epollfd, EPOLL_CTL_DEL, fd_list[i].fd, NULL) < 0)
-				err("epoll_ctl: %m");
+				rd_err("epoll_ctl: %m");
 			close(fd_list[i].fd);
 		}
 	}
 	close(epollfd);
-	logging_close();
+	rd_logging_close();
 
 	return EXIT_SUCCESS;
 }
diff --git a/datasrc/ueventd/ueventd.h b/datasrc/ueventd/ueventd.h
index a5e50379..e7ee1a8b 100644
--- a/datasrc/ueventd/ueventd.h
+++ b/datasrc/ueventd/ueventd.h
@@ -56,21 +56,21 @@ extern  int is_dot_dir(struct dirent *ent) __attribute__((nonnull(1)));
 #include <syslog.h>
 #include <stdlib.h>
 
-extern void logging_init(int level);
-extern void logging_close(void);
-extern int logging_level(const char *lvl)               __attribute__((nonnull(1)));
-extern void message(int priority, const char *fmt, ...) __attribute__((format(printf, 2, 3)));
+extern void rd_logging_init(int level);
+extern void rd_logging_close(void);
+extern int rd_logging_level(const char *lvl)               __attribute__((nonnull(1)));
+extern void rd_message(int priority, const char *fmt, ...) __attribute__((format(printf, 2, 3)));
 
-#define __message(level, format, arg...) message(level, format, ##arg)
+#define __rd_message(level, format, arg...) rd_message(level, format, ##arg)
 
-#define fatal(format, arg...)                       \
+#define rd_fatal(format, arg...)                    \
 	do {                                        \
-		__message(LOG_CRIT, "%s:%d: " format, __FILE__, __LINE__, ##arg); \
+		rd_message(LOG_CRIT, "%s:%d: " format, __FILE__, __LINE__, ##arg); \
 		_exit(EXIT_FAILURE);                \
 	} while (0)
 
-#define err(format, arg...)  __message(LOG_ERR,   format, ##arg)
-#define info(format, arg...) __message(LOG_INFO,  format, ##arg)
-#define dbg(format, arg...)  __message(LOG_DEBUG, format, ##arg)
+#define rd_err(format, arg...)  __rd_message(LOG_ERR,   format, ##arg)
+#define rd_info(format, arg...) __rd_message(LOG_INFO,  format, ##arg)
+#define rd_dbg(format, arg...)  __rd_message(LOG_DEBUG, format, ##arg)
 
 #endif /* __UEVENTD_H__ */
-- 
2.33.7



  parent reply	other threads:[~2023-05-06 19:45 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-04 13:42 [make-initrd] [PATCH 0/3] Reimplement ueventd Alexey Gladkov
2023-05-04 13:42 ` [make-initrd] [PATCH 1/3] " Alexey Gladkov
2023-05-05  3:08   ` Leonid Krivoshein
2023-05-05 17:02     ` Alexey Gladkov
2023-05-05  4:03   ` Leonid Krivoshein
2023-05-05 17:16     ` Alexey Gladkov
2023-05-05  5:21   ` Leonid Krivoshein
2023-05-05 17:24     ` Alexey Gladkov
2023-05-14 20:12   ` Leonid Krivoshein
2023-05-14 20:45     ` Alexey Gladkov
2023-05-14 20:57   ` Leonid Krivoshein
2023-05-15  8:52     ` Alexey Gladkov
2023-05-14 23:08   ` Leonid Krivoshein
2023-05-15  9:05     ` Alexey Gladkov
2023-05-15  0:47   ` Leonid Krivoshein
2023-05-15  9:12     ` Alexey Gladkov
2023-05-15  4:38   ` Leonid Krivoshein
2023-05-15 10:43     ` Alexey Gladkov
2023-05-18  6:39         ` Leonid Krivoshein
2023-05-18  7:05           ` Alexey Gladkov
2023-05-20  9:00   ` Leonid Krivoshein
2023-05-20 13:18     ` Alexey Gladkov
2023-05-20 15:17       ` Vladimir D. Seleznev
2023-05-20 17:23       ` Leonid Krivoshein
2023-05-20 18:51         ` Alexey Gladkov
2023-05-21  8:53         ` [make-initrd] [PATCH] ueventd: Don't use a epoll timeout when it's not needed Alexey Gladkov
2023-05-22  4:46           ` Leonid Krivoshein
2023-05-22  7:54             ` Alexey Gladkov
2023-05-22  9:19               ` Alexey Gladkov
2023-05-22  7:57             ` [make-initrd] [PATCH 1/2] ueventd: Fix memory leak Alexey Gladkov
2023-05-22  7:57             ` [make-initrd] [PATCH 2/2] ueventd: Change interface rd_asprintf_or_die Alexey Gladkov
2023-05-22  9:36               ` [make-initrd] [PATCH v2] " Alexey Gladkov
2023-05-20 16:37     ` [make-initrd] [PATCH 1/3] ueventd: Simplify call of the queue handler Alexey Gladkov
2023-05-20 16:37     ` [make-initrd] [PATCH 2/3] ueventd: Rename fd_list to fd_handler_list Alexey Gladkov
2023-05-20 16:37     ` [make-initrd] [PATCH 3/3] ueventd: Drop obsolete declarations Alexey Gladkov
2023-05-04 13:42 ` [make-initrd] [PATCH 2/3] Replace polld by uevent queue Alexey Gladkov
2023-05-04 13:42 ` [make-initrd] [PATCH 3/3] feature/kickstart: Reset rootdelay timer after kickstart Alexey Gladkov
2023-05-06 19:45 ` Alexey Gladkov [this message]
2023-05-06 19:45 ` [make-initrd] ueventd: Allow to configure the log destination Alexey Gladkov
2023-05-06 19:45 ` [make-initrd] ueventd: Pass the program name when initializing the logger Alexey Gladkov
2023-05-06 19:45 ` [make-initrd] ueventd: Rewrite logging function to make it more atomic Alexey Gladkov
2023-05-07 12:48 ` [make-initrd] [PATCH 0/3] Reimplement ueventd Alexey Gladkov
2023-05-13 11:50 ` Alexey Gladkov
2023-05-14 20:15   ` Leonid Krivoshein
2023-05-14 20:49     ` Alexey Gladkov

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230506194504.1471267-1-gladkov.alexey@gmail.com \
    --to=gladkov.alexey@gmail.com \
    --cc=make-initrd@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Make-initrd development discussion

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/make-initrd/0 make-initrd/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 make-initrd make-initrd/ http://lore.altlinux.org/make-initrd \
		make-initrd@lists.altlinux.org make-initrd@lists.altlinux.ru make-initrd@lists.altlinux.com
	public-inbox-index make-initrd

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.make-initrd


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git