[libvirt PATCH] scripts: avoid error in DCO check on empty branches
by Daniel P. Berrangé
If the DCO check is run on an empty branch (ie one which has no commits
different from master), it throws an error due to trying to interpret
the empty string as a git commit SHA.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
---
scripts/require-dco.py | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/scripts/require-dco.py b/scripts/require-dco.py
index 9fe13823a9..ae94393319 100755
--- a/scripts/require-dco.py
+++ b/scripts/require-dco.py
@@ -46,7 +46,10 @@ print("\nChecking for 'Signed-off-by: NAME <EMAIL>' on all commits since %s...\n
log = subprocess.check_output(["git", "log", "--format=%H %s", ancestor + "..."],
universal_newlines=True)
-commits = [[c[0:40], c[41:]] for c in log.strip().split("\n")]
+if log == "":
+ commits = []
+else:
+ commits = [[c[0:40], c[41:]] for c in log.strip().split("\n")]
for sha, subject in commits:
--
2.24.1
4 years, 9 months
[libvirt PATCH] gitlab: don't run the DCO check on git master
by Daniel P. Berrangé
The "only: branches" does not in fact exclude "master", so it must be
excluded explicitly.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
---
Pushed as a build fix since the DCO check fails on master
.gitlab-ci.yml | 1 +
1 file changed, 1 insertion(+)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index d8b316d618..1e3675ace0 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -228,3 +228,4 @@ dco:
- branches
except:
- /^v.*-maint$/
+ - master
--
2.24.1
4 years, 9 months
[PATCH 1/1] qemu_domain.c: remove unneeded size check in NVDIMM alignment
by Daniel Henrique Barboza
Commit 5540acb9a2b added a minimum size verification for the target
size of ppc64 NVDIMMs but forgot to remove a MAX() size check that
was being used in earlier reviews of that commit. The size
verification makes this check unneeded since we're making sure
that guestArea will always be at least equal to ppc64AlignSize.
Fixes: 5540acb9a2bd33fb97a7446a610b602c9e8629a3
Signed-off-by: Daniel Henrique Barboza <danielhb413(a)gmail.com>
---
src/qemu/qemu_domain.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/qemu/qemu_domain.c b/src/qemu/qemu_domain.c
index 2c9fb47d17..2c52778515 100644
--- a/src/qemu/qemu_domain.c
+++ b/src/qemu/qemu_domain.c
@@ -12639,8 +12639,6 @@ qemuDomainNVDimmAlignSizePseries(virDomainDefPtr def,
}
guestArea = (guestArea/ppc64AlignSize) * ppc64AlignSize;
- guestArea = MAX(guestArea, ppc64AlignSize);
-
mem->size = guestArea + mem->labelsize;
return 0;
--
2.25.1
4 years, 9 months
[PATCH 0/2] Accept http cookies with quoted value
by Peter Krempa
The validator didn't accept the quoted version of the cookie value. Add
a test and fix the issue.
Peter Krempa (2):
qemuxml2xmltest: Wire up 'disk-network-http' case
virStorageSourceNetCookieValidate: Accept quoted cookie value
docs/schemas/domaincommon.rng | 2 +-
src/util/virstoragefile.c | 19 ++++-
.../disk-network-http.x86_64-latest.args | 4 +-
tests/qemuxml2argvdata/disk-network-http.xml | 4 +-
.../disk-network-http.x86_64-latest.xml | 71 +++++++++++++++++++
tests/qemuxml2xmltest.c | 2 +
6 files changed, 96 insertions(+), 6 deletions(-)
create mode 100644 tests/qemuxml2xmloutdata/disk-network-http.x86_64-latest.xml
--
2.24.1
4 years, 9 months
[PATCHv2 1/1] qemu_agent.c: Use g_autofree in qemu_agent.c
by Seeteena Thoufeek
This is the only instance of g_autofree change applicable for
qemu_agent.c
Signed-off-by: Seeteena Thoufeek <s1seetee(a)linux.vnet.ibm.com>
---
src/qemu/qemu_agent.c | 12 ++++--------
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/src/qemu/qemu_agent.c b/src/qemu/qemu_agent.c
index 95a7d9b..d7fcc86 100644
--- a/src/qemu/qemu_agent.c
+++ b/src/qemu/qemu_agent.c
@@ -332,9 +332,8 @@ static int qemuAgentIOProcessData(qemuAgentPtr agent,
size_t i = 0;
#if DEBUG_IO
# if DEBUG_RAW_IO
- char *str1 = qemuAgentEscapeNonPrintable(data);
+ g_autofree char *str1 = qemuAgentEscapeNonPrintable(data);
VIR_ERROR(_("[%s]"), str1);
- VIR_FREE(str1);
# else
VIR_DEBUG("Data %zu bytes [%s]", len, data);
# endif
@@ -377,12 +376,10 @@ qemuAgentIOProcess(qemuAgentPtr agent)
#if DEBUG_IO
# if DEBUG_RAW_IO
- char *str1 = qemuAgentEscapeNonPrintable(msg ? msg->txBuffer : "");
- char *str2 = qemuAgentEscapeNonPrintable(agent->buffer);
+ g_autofree char *str1 = qemuAgentEscapeNonPrintable(msg ? msg->txBuffer : "");
+ g_autofree char *str2 = qemuAgentEscapeNonPrintable(agent->buffer);
VIR_ERROR(_("Process %zu %p %p [[[%s]]][[[%s]]]"),
agent->bufferOffset, agent->msg, msg, str1, str2);
- VIR_FREE(str1);
- VIR_FREE(str2);
# else
VIR_DEBUG("Process %zu", agent->bufferOffset);
# endif
@@ -1083,7 +1080,7 @@ qemuAgentCommandFull(qemuAgentPtr agent,
{
int ret = -1;
qemuAgentMessage msg;
- char *cmdstr = NULL;
+ g_autofree char *cmdstr = NULL;
int await_event = agent->await_event;
*reply = NULL;
@@ -1133,7 +1130,6 @@ qemuAgentCommandFull(qemuAgentPtr agent,
ret = qemuAgentCheckError(cmd, *reply, report_unsupported);
cleanup:
- VIR_FREE(cmdstr);
VIR_FREE(msg.txBuffer);
agent->await_event = QEMU_AGENT_EVENT_NONE;
--
1.8.3.1
4 years, 9 months
[PATCH 0/5] Introduce getHost support for ARM cpus
by Zhenyu Zheng
Introduce getHost support for ARM cpus. First add some common
ARM server models to cpu_map, then add some helper methods to
parse and load them. The 'lscpu' command from 'util-linux'
provides very good data for cpu information, we can parse them
directly.
Signed-of-by: Zhenyu Zheng <zhengzhenyulixi(a)gmail.com>
Zhenyu Zheng (5):
cpu_map: Introduce arm CPU models
cpu: Introduce virCPUarmData to virCPUData
cpu: Introduce arm related structs
cpu: Add helper functions to parse vendor and model for ARM
cpu: Introduce getHost supoort for ARM
src/cpu/cpu.h | 2 +
src/cpu/cpu_arm.c | 412 +++++++++++++++++++++++++++++-
src/cpu/cpu_arm_data.h | 32 +++
src/cpu_map/Makefile.inc.am | 7 +
src/cpu_map/arm_Falkor.xml | 15 ++
src/cpu_map/arm_Kunpeng-920.xml | 23 ++
src/cpu_map/arm_ThunderX299xx.xml | 15 ++
src/cpu_map/arm_cortex-a53.xml | 15 ++
src/cpu_map/arm_cortex-a57.xml | 14 +
src/cpu_map/arm_cortex-a72.xml | 14 +
src/cpu_map/arm_vendors.xml | 14 +
src/cpu_map/index.xml | 16 ++
12 files changed, 577 insertions(+), 2 deletions(-)
create mode 100644 src/cpu/cpu_arm_data.h
create mode 100644 src/cpu_map/arm_Falkor.xml
create mode 100644 src/cpu_map/arm_Kunpeng-920.xml
create mode 100644 src/cpu_map/arm_ThunderX299xx.xml
create mode 100644 src/cpu_map/arm_cortex-a53.xml
create mode 100644 src/cpu_map/arm_cortex-a57.xml
create mode 100644 src/cpu_map/arm_cortex-a72.xml
create mode 100644 src/cpu_map/arm_vendors.xml
--
2.26.0.windows.1
4 years, 9 months
[PATCH 0/5] Introduce getHost support for ARM cpus
by Zhenyu Zheng
Introduce getHost support for ARM cpus. First add some common
ARM server models to cpu_map, then add some helper methods to
parse and load them. The 'lscpu' command from 'util-linux'
provides very good data for cpu information, we can parse them
directly.
Zhenyu Zheng (5):
cpu_map: Introduce arm CPU models
cpu: Introduce virCPUarmData to virCPUData
cpu: Introduce arm related structs
cpu: Add helper functions to parse vendor and model for ARM
cpu: Introduce getHost supoort for ARM
src/cpu/cpu.h | 2 +
src/cpu/cpu_arm.c | 412 +++++++++++++++++++++++++++++-
src/cpu/cpu_arm_data.h | 32 +++
src/cpu_map/Makefile.inc.am | 7 +
src/cpu_map/arm_Falkor.xml | 15 ++
src/cpu_map/arm_Kunpeng-920.xml | 23 ++
src/cpu_map/arm_ThunderX299xx.xml | 15 ++
src/cpu_map/arm_cortex-a53.xml | 15 ++
src/cpu_map/arm_cortex-a57.xml | 14 +
src/cpu_map/arm_cortex-a72.xml | 14 +
src/cpu_map/arm_vendors.xml | 14 +
src/cpu_map/index.xml | 16 ++
12 files changed, 577 insertions(+), 2 deletions(-)
create mode 100644 src/cpu/cpu_arm_data.h
create mode 100644 src/cpu_map/arm_Falkor.xml
create mode 100644 src/cpu_map/arm_Kunpeng-920.xml
create mode 100644 src/cpu_map/arm_ThunderX299xx.xml
create mode 100644 src/cpu_map/arm_cortex-a53.xml
create mode 100644 src/cpu_map/arm_cortex-a57.xml
create mode 100644 src/cpu_map/arm_cortex-a72.xml
create mode 100644 src/cpu_map/arm_vendors.xml
--
2.26.0.windows.1
4 years, 9 months
[PATCH 0/5] Introduce getHost support for ARM cpus
by KevinZ
Introduce getHost support for ARM cpus. First add some common
ARM server models to cpu_map, then add some helper methods to
parse and load them. The 'lscpu' command from 'util-linux'
provides very good data for cpu information, we can parse them
directly.
Zhenyu Zheng (5):
cpu_map: Introduce arm CPU models
cpu: Introduce virCPUarmData to virCPUData
cpu: Introduce arm related structs
cpu: Add helper functions to parse vendor and model for ARM
cpu: Introduce getHost supoort for ARM
src/cpu/cpu.h | 2 +
src/cpu/cpu_arm.c | 412 +++++++++++++++++++++++++++++-
src/cpu/cpu_arm_data.h | 32 +++
src/cpu_map/Makefile.inc.am | 7 +
src/cpu_map/arm_Falkor.xml | 15 ++
src/cpu_map/arm_Kunpeng-920.xml | 23 ++
src/cpu_map/arm_ThunderX299xx.xml | 15 ++
src/cpu_map/arm_cortex-a53.xml | 15 ++
src/cpu_map/arm_cortex-a57.xml | 14 +
src/cpu_map/arm_cortex-a72.xml | 14 +
src/cpu_map/arm_vendors.xml | 14 +
src/cpu_map/index.xml | 16 ++
12 files changed, 577 insertions(+), 2 deletions(-)
create mode 100644 src/cpu/cpu_arm_data.h
create mode 100644 src/cpu_map/arm_Falkor.xml
create mode 100644 src/cpu_map/arm_Kunpeng-920.xml
create mode 100644 src/cpu_map/arm_ThunderX299xx.xml
create mode 100644 src/cpu_map/arm_cortex-a53.xml
create mode 100644 src/cpu_map/arm_cortex-a57.xml
create mode 100644 src/cpu_map/arm_cortex-a72.xml
create mode 100644 src/cpu_map/arm_vendors.xml
--
2.26.0.windows.1
4 years, 9 months
[libvirt] [PATCH 0/3] qemu: support -overcommit cpu-pm=on|off
by Menno Lageman
QEMU introduced a CPU power management feature with commit 6f131f13e68d
("kvm: support -overcommit cpu-pm=on|off").
With this flag, kvm allows guest to control host CPU power state. This
increases latency for other processes using same host CPU in an
unpredictable way, but if decreases idle entry/exit times for the
running VCPU, so to use it QEMU needs a hint about whether host CPU is
overcommitted, hence the flag name.
This patch series adds a new kvm feature 'cpu-pm' for controlling
"-overcommit cpu-pm=[on|off]"
<features>
<kvm>
<cpu-pm state='on'/>
</kvm>
</features>
Menno Lageman (2):
qemu: introduce qemuBuildOvercommitCommandLine()
tests: add tests for cpu-pm feature
Wim ten Have (1):
qemu: add hypervisor feature cpu-pm support for kvm
docs/formatdomain.html.in | 7 ++++
docs/schemas/domaincommon.rng | 5 +++
src/conf/domain_conf.c | 4 ++
src/conf/domain_conf.h | 1 +
src/libvirt_private.syms | 1 +
src/qemu/qemu_command.c | 39 +++++++++++++++++--
src/qemu/qemu_domain.c | 25 ++++++++----
tests/qemuxml2argvdata/kvm-features-off.args | 2 +-
tests/qemuxml2argvdata/kvm-features-off.xml | 1 +
tests/qemuxml2argvdata/kvm-features.args | 2 +-
tests/qemuxml2argvdata/kvm-features.xml | 1 +
tests/qemuxml2argvtest.c | 4 +-
tests/qemuxml2xmloutdata/kvm-features-off.xml | 1 +
tests/qemuxml2xmloutdata/kvm-features.xml | 1 +
14 files changed, 79 insertions(+), 15 deletions(-)
--
2.21.0
4 years, 9 months
[PATCH] qemu_agent.c: Use g_autofree in qemu_agent.c
by Seeteena Thoufeek
This is the only instance of g_autofree change applicable for
qemu_agent.c
Signed-off-by: Seeteena Thoufeek <s1seetee(a)linux.vnet.ibm.com>
---
src/qemu/qemu_agent.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/qemu/qemu_agent.c b/src/qemu/qemu_agent.c
index 95a7d9b..c74bd0e 100644
--- a/src/qemu/qemu_agent.c
+++ b/src/qemu/qemu_agent.c
@@ -1083,7 +1083,7 @@ qemuAgentCommandFull(qemuAgentPtr agent,
{
int ret = -1;
qemuAgentMessage msg;
- char *cmdstr = NULL;
+ g_autofree char *cmdstr = NULL;
int await_event = agent->await_event;
*reply = NULL;
@@ -1133,7 +1133,6 @@ qemuAgentCommandFull(qemuAgentPtr agent,
ret = qemuAgentCheckError(cmd, *reply, report_unsupported);
cleanup:
- VIR_FREE(cmdstr);
VIR_FREE(msg.txBuffer);
agent->await_event = QEMU_AGENT_EVENT_NONE;
--
1.8.3.1
4 years, 9 months
