Initialize host boot time global var upfront

[libvirt] [PATCH v2 0/2] Initialize host boot time global var upfront

Michal Privoznik posted 2 patches 4 years, 4 months ago

Download series mbox

Test syntax-check failed

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

src/libvirt_private.syms   |  1 +
src/remote/remote_daemon.c |  9 +++++++++
src/util/virhostuptime.c   | 13 ++++++++++++-
src/util/virhostuptime.h   |  3 +++
4 files changed, 25 insertions(+), 1 deletion(-)

Expand all Fold all

[libvirt] [PATCH v2 0/2] Initialize host boot time global var upfront

Posted by Michal Privoznik 4 years, 4 months ago

This is a v2 for:

https://www.redhat.com/archives/libvir-list/2019-December/msg01045.html

While technically v1 was fixed I agree with Cole's suggestion, so I'm
discarding v1 and sending another version which implements his
suggestion.

Michal Prívozník (2):
  virhostuptime: Introduce virHostBootTimeInit()
  remote_daemon: Initialize host boot time global variable

 src/libvirt_private.syms   |  1 +
 src/remote/remote_daemon.c |  9 +++++++++
 src/util/virhostuptime.c   | 13 ++++++++++++-
 src/util/virhostuptime.h   |  3 +++
 4 files changed, 25 insertions(+), 1 deletion(-)

-- 
2.24.1

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

Re: [libvirt] [PATCH v2 0/2] Initialize host boot time global var upfront

Posted by Cole Robinson 4 years, 3 months ago

On 12/19/19 6:24 AM, Michal Privoznik wrote:
> This is a v2 for:
> 
> https://www.redhat.com/archives/libvir-list/2019-December/msg01045.html
> 
> While technically v1 was fixed I agree with Cole's suggestion, so I'm
> discarding v1 and sending another version which implements his
> suggestion.
> 
> Michal Prívozník (2):
>   virhostuptime: Introduce virHostBootTimeInit()
>   remote_daemon: Initialize host boot time global variable
> 
>  src/libvirt_private.syms   |  1 +
>  src/remote/remote_daemon.c |  9 +++++++++
>  src/util/virhostuptime.c   | 13 ++++++++++++-
>  src/util/virhostuptime.h   |  3 +++
>  4 files changed, 25 insertions(+), 1 deletion(-)
> 

With the fix Dan pointed out:

Reviewed-by: Cole Robinson <crobinso@redhat.com>

- Cole

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