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

[Xen-devel] [xen-unstable-smoke bisection] complete build-amd64



branch xen-unstable-smoke
xenbranch xen-unstable-smoke
job build-amd64
testid xen-build

Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
Tree: qemuu git://xenbits.xen.org/qemu-xen.git
Tree: xen git://xenbits.xen.org/xen.git

*** Found and reproduced problem changeset ***

  Bug is in tree:  xen git://xenbits.xen.org/xen.git
  Bug introduced:  e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
  Bug not present: 40dbe1818a6af630cc8595ea97b35f3a0d296606
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/128076/


  commit e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
  Author: Christopher Clark <christopher.clark6@xxxxxxxxxxxxxx>
  Date:   Tue Sep 25 16:30:32 2018 +0200
  
      fuzz, test x86_emulator: disable sse before including always_inline fns
      
      Workaround for compiler rejection of SSE-using always_inlines defined 
before
      SSE is disabled.
      
      Compiling with _FORTIFY_SOURCE or higher levels of optimization enabled
      will always_inline several library fns (memset, memcpy, ...)
      (with gcc 8.2.0 and glibc 2.28).
      
      In fuzz and x86_emulator test, the compiler is instructed not
      to generate SSE instructions via: #pragma GCC target("no-sse")
      because those registers are needed for use by the workload.
      
      The combination above causes compilation failure as the inline functions
      use those instructions. This is resolved by reordering the inclusion of
      <stdio.h> and <string.h> to after the pragma disabling SSE generation.
      
      It would be preferable to locate the no-sse pragma within x86-emulate.h 
at the
      top of the file, prior to including any other headers; unfortunately 
doing so
      before <stdlib.h> causes compilation failure due to declaration of 'atof' 
with:
        "SSE register return with SSE disabled".
      Fortunately there is no (known) current dependency on any always_inline
      SSE-inclined function declared in <stdlib.h> or any of its dependencies, 
so the
      pragma is therefore issued immediately after inclusion of <stdlib.h> with 
a
      comment introduced to explain its location there.
      
      Add compile-time checks for unwanted prior inclusion of <string.h> and
      <stdio.h>, which are the two headers that provide the library functions 
that
      are handled with wrappers and listed within "x86-emulate.h" as ones "we 
think
      might access any of the FPU state".
      * Use standard-defined "EOF" macro to detect prior <stdio.h> inclusion.
      * Use "_STRING_H" (non-standardized guard macro) as best-effort
        for detection of prior <string.h> inclusion. This is non-universally
        viable but will provide error output on common GLIBC systems, so
        provides some defensive coverage.
      
      Adds conditional #include <stdio.h> to x86-emulate.h because fwrite, 
printf,
      etc. are referenced when WRAP has been defined.
      
      Signed-off-by: Christopher Clark <christopher.clark6@xxxxxxxxxxxxxx>
      Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>


For bisection revision-tuple graph see:
   
http://logs.test-lab.xenproject.org/osstest/results/bisect/xen-unstable-smoke/build-amd64.xen-build.html
Revision IDs in each graph node refer, respectively, to the Trees above.

----------------------------------------
Running cs-bisection-step 
--graph-out=/home/logs/results/bisect/xen-unstable-smoke/build-amd64.xen-build 
--summary-out=tmp/128076.bisection-summary --basis-template=127928 
--blessings=real,real-bisect xen-unstable-smoke build-amd64 xen-build
Searching for failure / basis pass:
 128062 fail [host=baroque1] / 128052 [host=albana0] 128048 [host=godello1] 
128045 [host=godello0] 128038 ok.
Failure / basis pass flights: 128062 / 128038
(tree with no url: minios)
(tree with no url: ovmf)
(tree with no url: seabios)
Tree: qemu git://xenbits.xen.org/qemu-xen-traditional.git
Tree: qemuu git://xenbits.xen.org/qemu-xen.git
Tree: xen git://xenbits.xen.org/xen.git
Latest 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
6cf4d8d3aa2699ff1ffa9e56240a6d188f91938c
Basis pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
1bd9cc34e152addeacbbf44898125c7be00e7677
Generating revisions with ./adhoc-revtuple-generator  
git://xenbits.xen.org/qemu-xen-traditional.git#9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149-9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149
 
git://xenbits.xen.org/qemu-xen.git#de5b678ca4dcdfa83e322491d478d66df56c1986-de5b678ca4dcdfa83e322491d478d66df56c1986
 
git://xenbits.xen.org/xen.git#1bd9cc34e152addeacbbf44898125c7be00e7677-6cf4d8d3aa2699ff1ffa9e56240a6d188f91938c
Loaded 1001 nodes in revision graph
Searching for test results:
 128019 [host=italia0]
 128067 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
1bd9cc34e152addeacbbf44898125c7be00e7677
 128068 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
6cf4d8d3aa2699ff1ffa9e56240a6d188f91938c
 128048 [host=godello1]
 128025 [host=albana0]
 128069 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
40dbe1818a6af630cc8595ea97b35f3a0d296606
 128029 [host=italia0]
 128052 [host=albana0]
 128070 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
 128071 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
40dbe1818a6af630cc8595ea97b35f3a0d296606
 128038 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
1bd9cc34e152addeacbbf44898125c7be00e7677
 128072 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
 128057 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
0348184dd2ee43d343623d3db647af3e635fd0c3
 128045 [host=godello0]
 128061 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
1bd9cc34e152addeacbbf44898125c7be00e7677
 128063 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
0348184dd2ee43d343623d3db647af3e635fd0c3
 128073 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
40dbe1818a6af630cc8595ea97b35f3a0d296606
 128064 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
e70934f44ad63e5433756f785ab4bcb316f860f2
 128062 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
6cf4d8d3aa2699ff1ffa9e56240a6d188f91938c
 128066 pass 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
7cbcd02ff302ea558d870ac7b21d30343b695af9
 128076 fail 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
Searching for interesting versions
 Result found: flight 128038 (pass), for basis pass
 Result found: flight 128062 (fail), for basis failure
 Repro found: flight 128067 (pass), for basis pass
 Repro found: flight 128068 (fail), for basis failure
 0 revisions at 9c0eed618f37dd5b4a57c8b3fbc48ef8913e3149 
de5b678ca4dcdfa83e322491d478d66df56c1986 
40dbe1818a6af630cc8595ea97b35f3a0d296606
No revisions left to test, checking graph state.
 Result found: flight 128069 (pass), for last pass
 Result found: flight 128070 (fail), for first failure
 Repro found: flight 128071 (pass), for last pass
 Repro found: flight 128072 (fail), for first failure
 Repro found: flight 128073 (pass), for last pass
 Repro found: flight 128076 (fail), for first failure

*** Found and reproduced problem changeset ***

  Bug is in tree:  xen git://xenbits.xen.org/xen.git
  Bug introduced:  e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
  Bug not present: 40dbe1818a6af630cc8595ea97b35f3a0d296606
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/128076/


  commit e8dfbc2962365ffa3d7ddcacaa5baaf4ed24b2af
  Author: Christopher Clark <christopher.clark6@xxxxxxxxxxxxxx>
  Date:   Tue Sep 25 16:30:32 2018 +0200
  
      fuzz, test x86_emulator: disable sse before including always_inline fns
      
      Workaround for compiler rejection of SSE-using always_inlines defined 
before
      SSE is disabled.
      
      Compiling with _FORTIFY_SOURCE or higher levels of optimization enabled
      will always_inline several library fns (memset, memcpy, ...)
      (with gcc 8.2.0 and glibc 2.28).
      
      In fuzz and x86_emulator test, the compiler is instructed not
      to generate SSE instructions via: #pragma GCC target("no-sse")
      because those registers are needed for use by the workload.
      
      The combination above causes compilation failure as the inline functions
      use those instructions. This is resolved by reordering the inclusion of
      <stdio.h> and <string.h> to after the pragma disabling SSE generation.
      
      It would be preferable to locate the no-sse pragma within x86-emulate.h 
at the
      top of the file, prior to including any other headers; unfortunately 
doing so
      before <stdlib.h> causes compilation failure due to declaration of 'atof' 
with:
        "SSE register return with SSE disabled".
      Fortunately there is no (known) current dependency on any always_inline
      SSE-inclined function declared in <stdlib.h> or any of its dependencies, 
so the
      pragma is therefore issued immediately after inclusion of <stdlib.h> with 
a
      comment introduced to explain its location there.
      
      Add compile-time checks for unwanted prior inclusion of <string.h> and
      <stdio.h>, which are the two headers that provide the library functions 
that
      are handled with wrappers and listed within "x86-emulate.h" as ones "we 
think
      might access any of the FPU state".
      * Use standard-defined "EOF" macro to detect prior <stdio.h> inclusion.
      * Use "_STRING_H" (non-standardized guard macro) as best-effort
        for detection of prior <string.h> inclusion. This is non-universally
        viable but will provide error output on common GLIBC systems, so
        provides some defensive coverage.
      
      Adds conditional #include <stdio.h> to x86-emulate.h because fwrite, 
printf,
      etc. are referenced when WRAP has been defined.
      
      Signed-off-by: Christopher Clark <christopher.clark6@xxxxxxxxxxxxxx>
      Reviewed-by: Jan Beulich <jbeulich@xxxxxxxx>

Revision graph left in 
/home/logs/results/bisect/xen-unstable-smoke/build-amd64.xen-build.{dot,ps,png,html,svg}.
----------------------------------------
128076: tolerable ALL FAIL

flight 128076 xen-unstable-smoke real-bisect [real]
http://logs.test-lab.xenproject.org/osstest/logs/128076/

Failures :-/ but no regressions.

Tests which did not succeed,
including tests which could not be run:
 build-amd64                   6 xen-build               fail baseline untested


jobs:
 build-amd64                                                  fail    


------------------------------------------------------------
sg-report-flight on osstest.test-lab.xenproject.org
logs: /home/logs/logs
images: /home/logs/images

Logs, config files, etc. are available at
    http://logs.test-lab.xenproject.org/osstest/logs

Explanation of these reports, and of osstest in general, is at
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README.email;hb=master
    http://xenbits.xen.org/gitweb/?p=osstest.git;a=blob;f=README;hb=master

Test harness code can be found at
    http://xenbits.xen.org/gitweb?p=osstest.git;a=summary


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.