* [xen-unstable test] 10332: regressions - FAIL
@ 2011-12-05 4:11 xen.org
2011-12-05 9:25 ` Jan Beulich
0 siblings, 1 reply; 6+ messages in thread
From: xen.org @ 2011-12-05 4:11 UTC (permalink / raw)
To: xen-devel; +Cc: ian.jackson
flight 10332 xen-unstable real [real]
http://www.chiark.greenend.org.uk/~xensrcts/logs/10332/
Regressions :-(
Tests which did not succeed and are blocking:
test-amd64-i386-xl-winxpsp3-vcpus1 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xl-win7-amd64 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xend-winxpsp3 4 xen-install fail REGR. vs. 10201
test-amd64-i386-pv 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xl-credit2 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xl-multivcpu 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xl 4 xen-install fail REGR. vs. 10201
test-amd64-i386-pair 6 xen-install/dst_host fail REGR. vs. 10201
test-amd64-i386-pair 5 xen-install/src_host fail REGR. vs. 10201
build-i386 4 xen-build fail REGR. vs. 10201
build-i386-oldkern 4 xen-build fail REGR. vs. 10201
test-amd64-i386-rhel6hvm-intel 4 xen-install fail REGR. vs. 10201
test-amd64-i386-rhel6hvm-amd 4 xen-install fail REGR. vs. 10201
test-amd64-i386-win-vcpus1 4 xen-install fail REGR. vs. 10201
test-amd64-amd64-win 8 guest-saverestore fail REGR. vs. 10201
test-amd64-amd64-xl-win 8 guest-saverestore fail REGR. vs. 10201
test-amd64-i386-win 4 xen-install fail REGR. vs. 10201
test-amd64-i386-xl-win-vcpus1 4 xen-install fail REGR. vs. 10201
Tests which are failing intermittently (not blocking):
test-amd64-amd64-xl-sedf 13 guest-localmigrate.2 fail pass in 10317
test-amd64-amd64-xl-sedf 14 guest-localmigrate/x10 fail in 10317 pass in 10313
Tests which did not succeed, but are not blocking,
including regressions (tests previously passed) regarded as allowable:
test-amd64-amd64-xl-win7-amd64 7 windows-install fail never pass
test-i386-i386-xl-winxpsp3 1 xen-build-check(1) blocked n/a
test-amd64-amd64-xl-pcipt-intel 9 guest-start fail never pass
test-i386-i386-pv 1 xen-build-check(1) blocked n/a
test-i386-i386-xl 1 xen-build-check(1) blocked n/a
test-i386-i386-pair 1 xen-build-check(1) blocked n/a
test-amd64-amd64-xl-winxpsp3 7 windows-install fail never pass
test-i386-i386-win 1 xen-build-check(1) blocked n/a
test-i386-i386-xl-win 1 xen-build-check(1) blocked n/a
test-amd64-amd64-xl-sedf-pin 14 guest-localmigrate/x10 fail in 10317 blocked in 10201
version targeted for testing:
xen a4d7c27ec1f1
baseline version:
xen 62ff6a318c5d
------------------------------------------------------------
People who touched revisions under test:
Adin Scannell <adin@scannell.ca>
Andres Lagar-Cavilla <andres@lagarcavilla.org>
Anthony PERARD <anthony.perard@citrix.com>
Brendan Cully <brendan@cs.ubc.ca>
Daniel De Graaf <dgdegra@tycho.nsa.gov>
Ian Campbell <ian.campbell@citrix.com>
Ian Jackson <ian.jackson.citrix.com>
Ian Jackson <ian.jackson@eu.citrix.com>
Jan Beulich <jbeulich@novell.com>
Jan Beulich <jbeulich@suse.com>
Jean Guyader <jean.guyader@eu.citrix.com>
Jonathan Davies <jonathan.davies@citrix.com>
juergen.gross@ts.fujitsu.com
Keir Fraser <keir@xen.org>
Liu, Jinsong <jinsong.liu@intel.com>
Olaf Hering <olaf@aepfle.de>
Paul Durrant <paul.durrant@citrix.com>
Philipp Hahn <hahn@univention.de>
Shriram Rajagopalan <rshriram@cs.ubc.ca>
Stefano Stabellini <stefano.stabellini@eu.citrix.com>
Tim Deegan <tim@xen.org>
Wei Wang2 <wei.wang2@amd.com>
------------------------------------------------------------
jobs:
build-amd64 pass
build-i386 fail
build-amd64-oldkern pass
build-i386-oldkern fail
build-amd64-pvops pass
build-i386-pvops pass
test-amd64-amd64-xl pass
test-amd64-i386-xl fail
test-i386-i386-xl blocked
test-amd64-i386-rhel6hvm-amd fail
test-amd64-amd64-xl-win7-amd64 fail
test-amd64-i386-xl-win7-amd64 fail
test-amd64-i386-xl-credit2 fail
test-amd64-amd64-xl-pcipt-intel fail
test-amd64-i386-rhel6hvm-intel fail
test-amd64-i386-xl-multivcpu fail
test-amd64-amd64-pair pass
test-amd64-i386-pair fail
test-i386-i386-pair blocked
test-amd64-amd64-xl-sedf-pin pass
test-amd64-amd64-pv pass
test-amd64-i386-pv fail
test-i386-i386-pv blocked
test-amd64-amd64-xl-sedf fail
test-amd64-i386-win-vcpus1 fail
test-amd64-i386-xl-win-vcpus1 fail
test-amd64-i386-xl-winxpsp3-vcpus1 fail
test-amd64-amd64-win fail
test-amd64-i386-win fail
test-i386-i386-win blocked
test-amd64-amd64-xl-win fail
test-i386-i386-xl-win blocked
test-amd64-i386-xend-winxpsp3 fail
test-amd64-amd64-xl-winxpsp3 fail
test-i386-i386-xl-winxpsp3 blocked
------------------------------------------------------------
sg-report-flight on woking.cam.xci-test.com
logs: /home/xc_osstest/logs
images: /home/xc_osstest/images
Logs, config files, etc. are available at
http://www.chiark.greenend.org.uk/~xensrcts/logs
Test harness code can be found at
http://xenbits.xensource.com/gitweb?p=osstest.git;a=summary
Not pushing.
(No revision log; it would be 1262 lines long.)
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [xen-unstable test] 10332: regressions - FAIL
2011-12-05 4:11 [xen-unstable test] 10332: regressions - FAIL xen.org
@ 2011-12-05 9:25 ` Jan Beulich
2011-12-05 11:29 ` [PATCH] flask: Fix 32-bit compilation of label-pci tool Daniel De Graaf
2011-12-05 11:34 ` [PATCH v2] " Daniel De Graaf
0 siblings, 2 replies; 6+ messages in thread
From: Jan Beulich @ 2011-12-05 9:25 UTC (permalink / raw)
To: dgdegra; +Cc: xen-devel
>>> On 05.12.11 at 05:11, xen.org <ian.jackson@eu.citrix.com> wrote:
> build-i386 4 xen-build fail REGR. vs. 10201
cc1: warnings being treated as errors
label-pci.c: In function 'main':
label-pci.c:79: error: format '%lx' expects type 'long unsigned int *', but argument 3 has type 'uint64_t *'
label-pci.c:79: error: format '%lx' expects type 'long unsigned int *', but argument 4 has type 'uint64_t *'
label-pci.c:79: error: format '%lx' expects type 'long unsigned int *', but argument 5 has type 'uint64_t *'
label-pci.c:85: error: format '%lx' expects type 'long unsigned int', but argument 3 has type 'uint64_t'
label-pci.c:85: error: format '%lx' expects type 'long unsigned int', but argument 4 has type 'uint64_t'
label-pci.c:95: error: format '%lx' expects type 'long unsigned int', but argument 3 has type 'uint64_t'
label-pci.c:95: error: format '%lx' expects type 'long unsigned int', but argument 4 has type 'uint64_t'
label-pci.c:108: error: format '%ld' expects type 'long int *', but argument 3 has type 'uint64_t *'
label-pci.c:113: error: format '%ld' expects type 'long int', but argument 3 has type 'uint64_t'
make[5]: *** [label-pci.o] Error 1
This is due to entirely non-portable code in tools/flask/utils/label-pci.c: If
you use C99 types, you also need to use C99 format macros for calls to
the printf and scanf function families.
Jan
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH] flask: Fix 32-bit compilation of label-pci tool
2011-12-05 9:25 ` Jan Beulich
@ 2011-12-05 11:29 ` Daniel De Graaf
2011-12-05 11:34 ` [PATCH v2] " Daniel De Graaf
1 sibling, 0 replies; 6+ messages in thread
From: Daniel De Graaf @ 2011-12-05 11:29 UTC (permalink / raw)
To: Keir Fraser; +Cc: Daniel De Graaf, xen-devel, JBeulich
The 32-bit tools need to support 64-bit addresses, so use the correct
printf/scanf formats.
Signed-off-by: Daniel De Graaf <dgdegra@tycho.nsa.gov>
---
tools/flask/utils/label-pci.c | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/tools/flask/utils/label-pci.c b/tools/flask/utils/label-pci.c
index 839ad61..ca53ea8 100644
--- a/tools/flask/utils/label-pci.c
+++ b/tools/flask/utils/label-pci.c
@@ -15,6 +15,7 @@
#include <sys/stat.h>
#include <string.h>
#include <unistd.h>
+#include <inttypes.h>
#include <libflask.h>
/* Pulled from linux/include/linux/ioport.h */
@@ -76,22 +77,22 @@ int main (int argCnt, char *argv[])
goto done;
}
- while (fscanf(f, "0x%lx 0x%lx 0x%lx\n", &start, &end, &flags) == 3) {
+ while (fscanf(f, "0x%"PRIx64" 0x%"PRIx64" 0x%"PRIx64"\n", &start, &end, &flags) == 3) {
if (flags & IORESOURCE_IO) {
- // printf("Port %lx-%lx\n", start, end);
+ // printf("Port %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_ioport(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_ioport %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_ioport %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
} else if (flags & IORESOURCE_MEM) {
start >>= 12;
end >>= 12;
- // printf("IOMEM %lx-%lx\n", start, end);
+ // printf("IOMEM %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_iomem(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_iomem %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_iomem %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
@@ -105,11 +106,11 @@ int main (int argCnt, char *argv[])
if (!f)
goto done;
start = 0;
- fscanf(f, "%ld", &start);
+ fscanf(f, "%" PRIu64, &start);
if (start) {
ret = flask_add_pirq(xch, start, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_pirq %ld failed: %d\n",
+ fprintf(stderr, "flask_add_pirq %"PRIu64" failed: %d\n",
start, ret);
err = 2;
}
--
1.7.7.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* [PATCH v2] flask: Fix 32-bit compilation of label-pci tool
2011-12-05 9:25 ` Jan Beulich
2011-12-05 11:29 ` [PATCH] flask: Fix 32-bit compilation of label-pci tool Daniel De Graaf
@ 2011-12-05 11:34 ` Daniel De Graaf
2011-12-05 18:05 ` Olaf Hering
1 sibling, 1 reply; 6+ messages in thread
From: Daniel De Graaf @ 2011-12-05 11:34 UTC (permalink / raw)
To: Keir Fraser; +Cc: Daniel De Graaf, xen-devel, JBeulich
The 32-bit tools need to support 64-bit addresses, so use the correct
printf/scanf formats.
Signed-off-by: Daniel De Graaf <dgdegra@tycho.nsa.gov>
---
tools/flask/utils/label-pci.c | 15 ++++++++-------
1 files changed, 8 insertions(+), 7 deletions(-)
diff --git a/tools/flask/utils/label-pci.c b/tools/flask/utils/label-pci.c
index 839ad61..009a733 100644
--- a/tools/flask/utils/label-pci.c
+++ b/tools/flask/utils/label-pci.c
@@ -15,6 +15,7 @@
#include <sys/stat.h>
#include <string.h>
#include <unistd.h>
+#include <inttypes.h>
#include <libflask.h>
/* Pulled from linux/include/linux/ioport.h */
@@ -76,22 +77,22 @@ int main (int argCnt, char *argv[])
goto done;
}
- while (fscanf(f, "0x%lx 0x%lx 0x%lx\n", &start, &end, &flags) == 3) {
+ while (fscanf(f, "0x%"SCNx64" 0x%"SCNx64" 0x%"SCNx64"\n", &start, &end, &flags) == 3) {
if (flags & IORESOURCE_IO) {
- // printf("Port %lx-%lx\n", start, end);
+ // printf("Port %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_ioport(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_ioport %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_ioport %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
} else if (flags & IORESOURCE_MEM) {
start >>= 12;
end >>= 12;
- // printf("IOMEM %lx-%lx\n", start, end);
+ // printf("IOMEM %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_iomem(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_iomem %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_iomem %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
@@ -105,11 +106,11 @@ int main (int argCnt, char *argv[])
if (!f)
goto done;
start = 0;
- fscanf(f, "%ld", &start);
+ fscanf(f, "%" SCNu64, &start);
if (start) {
ret = flask_add_pirq(xch, start, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_pirq %ld failed: %d\n",
+ fprintf(stderr, "flask_add_pirq %"PRIu64" failed: %d\n",
start, ret);
err = 2;
}
--
1.7.7.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] flask: Fix 32-bit compilation of label-pci tool
2011-12-05 11:34 ` [PATCH v2] " Daniel De Graaf
@ 2011-12-05 18:05 ` Olaf Hering
2011-12-05 18:45 ` [PATCH v3] " Daniel De Graaf
0 siblings, 1 reply; 6+ messages in thread
From: Olaf Hering @ 2011-12-05 18:05 UTC (permalink / raw)
To: Daniel De Graaf; +Cc: xen-devel, Keir Fraser, JBeulich
On Mon, Dec 05, Daniel De Graaf wrote:
> + fscanf(f, "%" SCNu64, &start);
label-pci.c:108: error: ignoring return value of 'fscanf', declared with attribute warn_unused_result
This still leads to an error.
Olaf
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH v3] flask: Fix 32-bit compilation of label-pci tool
2011-12-05 18:05 ` Olaf Hering
@ 2011-12-05 18:45 ` Daniel De Graaf
0 siblings, 0 replies; 6+ messages in thread
From: Daniel De Graaf @ 2011-12-05 18:45 UTC (permalink / raw)
To: Keir Fraser, olaf; +Cc: Daniel De Graaf, xen-devel, JBeulich
The 32-bit tools need to support 64-bit addresses, so use the correct
printf/scanf formats. Also, some systems declare fscanf with attribute
warn_unused_result, so check the result instead of relying on the value
of start being unmodified across a failed call.
Signed-off-by: Daniel De Graaf <dgdegra@tycho.nsa.gov>
---
tools/flask/utils/label-pci.c | 17 +++++++++--------
1 files changed, 9 insertions(+), 8 deletions(-)
diff --git a/tools/flask/utils/label-pci.c b/tools/flask/utils/label-pci.c
index 839ad61..da0cb61 100644
--- a/tools/flask/utils/label-pci.c
+++ b/tools/flask/utils/label-pci.c
@@ -15,6 +15,7 @@
#include <sys/stat.h>
#include <string.h>
#include <unistd.h>
+#include <inttypes.h>
#include <libflask.h>
/* Pulled from linux/include/linux/ioport.h */
@@ -76,22 +77,22 @@ int main (int argCnt, char *argv[])
goto done;
}
- while (fscanf(f, "0x%lx 0x%lx 0x%lx\n", &start, &end, &flags) == 3) {
+ while (fscanf(f, "0x%"SCNx64" 0x%"SCNx64" 0x%"SCNx64"\n", &start, &end, &flags) == 3) {
if (flags & IORESOURCE_IO) {
- // printf("Port %lx-%lx\n", start, end);
+ // printf("Port %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_ioport(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_ioport %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_ioport %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
} else if (flags & IORESOURCE_MEM) {
start >>= 12;
end >>= 12;
- // printf("IOMEM %lx-%lx\n", start, end);
+ // printf("IOMEM %"PRIx64"-%"PRIx64"\n", start, end);
ret = flask_add_iomem(xch, start, end, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_iomem %lx-%lx failed: %d\n",
+ fprintf(stderr, "flask_add_iomem %"PRIx64"-%"PRIx64" failed: %d\n",
start, end, ret);
err = 2;
}
@@ -104,12 +105,12 @@ int main (int argCnt, char *argv[])
f = fopen(buf, "r");
if (!f)
goto done;
- start = 0;
- fscanf(f, "%ld", &start);
+ if (fscanf(f, "%" SCNu64, &start) != 1)
+ start = 0;
if (start) {
ret = flask_add_pirq(xch, start, argv[2]);
if (ret) {
- fprintf(stderr, "flask_add_pirq %ld failed: %d\n",
+ fprintf(stderr, "flask_add_pirq %"PRIu64" failed: %d\n",
start, ret);
err = 2;
}
--
1.7.7.3
^ permalink raw reply related [flat|nested] 6+ messages in thread
end of thread, other threads:[~2011-12-05 18:45 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-05 4:11 [xen-unstable test] 10332: regressions - FAIL xen.org
2011-12-05 9:25 ` Jan Beulich
2011-12-05 11:29 ` [PATCH] flask: Fix 32-bit compilation of label-pci tool Daniel De Graaf
2011-12-05 11:34 ` [PATCH v2] " Daniel De Graaf
2011-12-05 18:05 ` Olaf Hering
2011-12-05 18:45 ` [PATCH v3] " Daniel De Graaf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.