Re: [PATCH 0/3] Couple of build fixes after latest cpu changes

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

 



Michal Privoznik <mprivozn@xxxxxxxxxx> [2019-04-15, 06:50AM +0200]:
> I've successfully tested these on mingw, arm and x86_64.

Fixes S390 as well, thanks.

> Almost trivial, but not pushed yet. I'll wait for review.

I got another incidential build error that doesn't seem related though:

	../../examples/domtop/domtop.c: In function 'print_cpu_usage':
	../../examples/domtop/domtop.c:242:28: error: 'pos' may be used uninitialized in this function [-Werror=maybe-uninitialized]
			 usage = (now_params[pos].value.ul - then_params[pos].value.ul) /
								^
	cc1: all warnings being treated as errors

GCC is 8.2.1. Here's a patch:

	diff --git a/examples/domtop/domtop.c b/examples/domtop/domtop.c
	index c9b51aed..e463e287 100644
	--- a/examples/domtop/domtop.c
	+++ b/examples/domtop/domtop.c
	@@ -206,7 +206,7 @@ print_cpu_usage(size_t cpu,
		 }

		 for (i = 0; i < ncpus; i++) {
	-        size_t pos;
	+        size_t pos = 0;
			 double usage;

			 /* check if the vCPU is in the maps */

Attachment: signature.asc
Description: PGP signature

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

  Powered by Linux