[PATCH 0/4] Introduce shared daemon startup code

Rafael Fonseca posted 4 patches 4 years, 1 month ago
Test syntax-check failed
Failed in applying to current master (apply log)
src/libvirt_private.syms   |   6 +
src/locking/lock_daemon.c  | 251 ++++-------------------------------
src/logging/log_daemon.c   | 239 ++++-----------------------------
src/remote/remote_daemon.c | 263 +++----------------------------------
src/util/Makefile.inc.am   |   2 +
src/util/virdaemon.c       | 255 +++++++++++++++++++++++++++++++++++
src/util/virdaemon.h       |  74 +++++++++++
7 files changed, 410 insertions(+), 680 deletions(-)
create mode 100644 src/util/virdaemon.c
create mode 100644 src/util/virdaemon.h
[PATCH 0/4] Introduce shared daemon startup code
Posted by Rafael Fonseca 4 years, 1 month ago
Libvirt ships several daemons (libvirtd, virtlogd, virtlockd) that all
have similar code around general daemon startup. Right now the code is
duplicated for each daemon, but it could be shared, like in a new file
src/util/virdaemon.c.

Rafael Fonseca (4):
  util: introduce shared daemon startup code
  locking: use shared daemon startup code
  logging: use shared daemon startup code
  remote: use shared daemon startup code

 src/libvirt_private.syms   |   6 +
 src/locking/lock_daemon.c  | 251 ++++-------------------------------
 src/logging/log_daemon.c   | 239 ++++-----------------------------
 src/remote/remote_daemon.c | 263 +++----------------------------------
 src/util/Makefile.inc.am   |   2 +
 src/util/virdaemon.c       | 255 +++++++++++++++++++++++++++++++++++
 src/util/virdaemon.h       |  74 +++++++++++
 7 files changed, 410 insertions(+), 680 deletions(-)
 create mode 100644 src/util/virdaemon.c
 create mode 100644 src/util/virdaemon.h

-- 
2.25.1


Re: [PATCH 0/4] Introduce shared daemon startup code
Posted by Michal Prívozník 4 years, 1 month ago
On 26. 3. 2020 16:17, Rafael Fonseca wrote:
> Libvirt ships several daemons (libvirtd, virtlogd, virtlockd) that all
> have similar code around general daemon startup. Right now the code is
> duplicated for each daemon, but it could be shared, like in a new file
> src/util/virdaemon.c.
> 
> Rafael Fonseca (4):
>   util: introduce shared daemon startup code
>   locking: use shared daemon startup code
>   logging: use shared daemon startup code
>   remote: use shared daemon startup code
> 
>  src/libvirt_private.syms   |   6 +
>  src/locking/lock_daemon.c  | 251 ++++-------------------------------
>  src/logging/log_daemon.c   | 239 ++++-----------------------------
>  src/remote/remote_daemon.c | 263 +++----------------------------------
>  src/util/Makefile.inc.am   |   2 +
>  src/util/virdaemon.c       | 255 +++++++++++++++++++++++++++++++++++
>  src/util/virdaemon.h       |  74 +++++++++++
>  7 files changed, 410 insertions(+), 680 deletions(-)
>  create mode 100644 src/util/virdaemon.c
>  create mode 100644 src/util/virdaemon.h
> 

I'm fixing all the small nits I've rasied and pushing.

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal