Re: [PATCH 5/7] src: Replace virAtomicIntInc() with g_atomic_int_add()

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

 



On Sat, Feb 01, 2020 at 07:33:47AM +0100, Michal Privoznik wrote:
Signed-off-by: Michal Privoznik <mprivozn@xxxxxxxxxx>
---
src/libxl/libxl_domain.c          |  2 +-
src/libxl/libxl_driver.c          |  2 +-
src/lxc/lxc_process.c             |  4 ++--
src/nwfilter/nwfilter_dhcpsnoop.c | 10 +++++-----
src/qemu/qemu_process.c           |  4 ++--
src/test/test_driver.c            |  2 +-
src/util/viratomic.h              | 11 -----------
src/util/virobject.c              |  4 ++--
src/util/virprocess.c             |  2 +-
9 files changed, 15 insertions(+), 26 deletions(-)


diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 4e159157d7..ba157c0686 100644
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -695,7 +695,7 @@ testDomainStartState(testDriverPtr privconn,
    int ret = -1;

    virDomainObjSetState(dom, VIR_DOMAIN_RUNNING, reason);
-    dom->def->id = virAtomicIntInc(&privconn->nextDomID);
+    dom->def->id = g_atomic_int_add(&privconn->nextDomID, 1);

    if (virDomainObjSetDefTransient(privconn->xmlopt,
                                    dom, NULL) < 0) {

This belongs in patch 1/7

To the rest:
Reviewed-by: Ján Tomko <jtomko@xxxxxxxxxx>

Jano

Attachment: signature.asc
Description: PGP signature


[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]

  Powered by Linux