virNetDaemonCallInhibit: check for logind availability

[libvirt] [PATCH 0/3] virNetDaemonCallInhibit: check for logind availability

Ján Tomko posted 3 patches 4 years, 8 months ago

Download series mbox

Test syntax-check passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1565708788.git.jtomko@redhat.com

src/libvirt_private.syms  |  2 ++
src/rpc/virnetdaemon.c    |  3 +++
src/util/virsystemd.c     | 37 ++++++++++++++++++++++++++++++++-----
src/util/virsystemd.h     |  2 ++
src/util/virsystemdpriv.h |  1 +
tests/virsystemdtest.c    |  3 +++
6 files changed, 43 insertions(+), 5 deletions(-)

Expand all Fold all

[libvirt] [PATCH 0/3] virNetDaemonCallInhibit: check for logind availability

Posted by Ján Tomko 4 years, 8 months ago

* get rid of a pointless error logged when system D-Bus is not available
* do not send messages on systems without logind
* remove four extra dbus calls on libvirtd startup when checking for
  logind availability

Ján Tomko (3):
  util: introduce virSystemdHasLogind
  util: cache the result of whether logind is available
  rpc: make virNetDaemonCallInhibit a no-op with no logind

 src/libvirt_private.syms  |  2 ++
 src/rpc/virnetdaemon.c    |  3 +++
 src/util/virsystemd.c     | 37 ++++++++++++++++++++++++++++++++-----
 src/util/virsystemd.h     |  2 ++
 src/util/virsystemdpriv.h |  1 +
 tests/virsystemdtest.c    |  3 +++
 6 files changed, 43 insertions(+), 5 deletions(-)

-- 
2.19.2

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

Re: [libvirt] [PATCH 0/3] virNetDaemonCallInhibit: check for logind availability

Posted by Michal Privoznik 4 years, 8 months ago

On 8/13/19 5:08 PM, Ján Tomko wrote:
> * get rid of a pointless error logged when system D-Bus is not available
> * do not send messages on systems without logind
> * remove four extra dbus calls on libvirtd startup when checking for
>    logind availability
> 
> Ján Tomko (3):
>    util: introduce virSystemdHasLogind
>    util: cache the result of whether logind is available
>    rpc: make virNetDaemonCallInhibit a no-op with no logind
> 
>   src/libvirt_private.syms  |  2 ++
>   src/rpc/virnetdaemon.c    |  3 +++
>   src/util/virsystemd.c     | 37 ++++++++++++++++++++++++++++++++-----
>   src/util/virsystemd.h     |  2 ++
>   src/util/virsystemdpriv.h |  1 +
>   tests/virsystemdtest.c    |  3 +++
>   6 files changed, 43 insertions(+), 5 deletions(-)
> 

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

Michal

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