Re: [PATCH v2] systemd: directly notify systemd instead of using sd_notify

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 2016-06-06 at 16:21 +0100, Daniel P. Berrange wrote:
> The sd_notify method is used to tell systemd when libvirtd
> has finished starting up. All it does is send a datagram
> containing the string parameter to systemd on a UNIX socket
> named in the NOTIFY_SOCKET environment variable. Rather than
> pulling in the systemd libraries for this, just code the
> notification directly in libvirt as this is a stable ABI
> from systemd's POV which explicitly allows independant
> implementations:
> 
> See "Reimplementable Independently" column in the
> "$NOTIFY_SOCKET Daemon Notifications" row:
> 
> https://www.freedesktop.org/wiki/Software/systemd/InterfacePortabilityAndStabilityChart/
> 
> Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx>
> ---
>  configure.ac              |  2 --
>  libvirt.spec.in           | 12 -----------
>  m4/virt-systemd-daemon.m4 | 34 ------------------------------
>  src/Makefile.am           |  4 ++--
>  src/util/virsystemd.c     | 53 ++++++++++++++++++++++++++++++++++++++++++-----
>  5 files changed, 50 insertions(+), 55 deletions(-)
>  delete mode 100644 m4/virt-systemd-daemon.m4

As Cole noted in response to my previous attempt, this

  Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1314881

Please include said information in the commit message
before pushing.

-- 
Andrea Bolognani
Software Engineer - Virtualization Team

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]