[xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[osstest service owner]

[-- Attachment #1: Type: text/plain, Size: 12316 bytes --]

branch xen-unstable
xenbranch xen-unstable
job test-amd64-amd64-i386-pvgrub
testid guest-start

Tree: linux git://xenbits.xen.org/linux-pvops.git
Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
  Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/


  commit f48b5449dabc770acdde6d25cfbd265cfb71034d
  Author: Wei Liu <wei.liu2@citrix.com>
  Date:   Thu Oct 12 20:19:07 2017 +0100
  
      tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
      
      The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
      builder is for libxl_dom() to translate the console and xenstore pfns back
      into useful values.  PV guest pfns are only interesting to the domain builder,
      and gfns are the address space used by all other hypercalls.
      
      Renaming the fields in xc_dom_image is deliberate, as it will cause
      out-of-tree users of the dombuilder to notice the different semantics.
      
      Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
      using gfns despite the existing variable names.
      
      Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
      Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
      Acked-by: Wei Liu <wei.liu2@citrix.com>
      Tested-by: Julien Grall <julien.grall@arm.com>
      Release-acked-by: Julien Grall <julien.grall@linaro.org>
      [ wei: fix stubdom build ]
      Signed-off-by: Wei Liu <wei.liu2@citrix.com>


For bisection revision-tuple graph see:
   http://logs.test-lab.xenproject.org/osstest/results/bisect/xen-unstable/test-amd64-amd64-i386-pvgrub.guest-start.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/test-amd64-amd64-i386-pvgrub.guest-start --summary-out=tmp/115612.bisection-summary --basis-template=115526 --blessings=real,real-bisect xen-unstable test-amd64-amd64-i386-pvgrub guest-start
Searching for failure / basis pass:
 115593 fail [host=merlot1] / 115526 [host=italia1] 115496 [host=nobling0] 115471 [host=godello0] 115464 [host=godello0] 115450 [host=fiano0] 115419 [host=huxelrebe0] 115401 [host=pinot1] 115378 [host=baroque1] 115362 [host=merlot0] 115345 [host=rimava1] 115331 [host=elbling1] 115314 [host=nocera1] 115295 [host=chardonnay1] 115272 [host=fiano1] 115235 [host=nobling1] 115195 [host=baroque0] 115174 [host=elbling0] 115163 [host=italia0] 115132 [host=huxelrebe1] 115087 [host=nocera0] 115037 [host=rimava0] 114972 [host=chardonnay0] 114836 [host=nobling0] 114808 ok.
Failure / basis pass flights: 115593 / 114808
(tree with no url: minios)
(tree with no url: ovmf)
(tree with no url: seabios)
Tree: linux git://xenbits.xen.org/linux-pvops.git
Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
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 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 e07ebab86595df29838f0960693df84f34528833
Basis pass 9d36d3eff2f85efad0a3b0c6031081654ae33928 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 8e77dabc58c4b6c747dfb4b948551147905a7840
Generating revisions with ./adhoc-revtuple-generator  git://xenbits.xen.org/linux-pvops.git#9d36d3eff2f85efad0a3b0c6031081654ae33928-5d7a76acad403638f635c918cc63d1d44ffa4065 git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860 git://xenbits.xen.org/qemu-xen-traditional.git#c8ea0457495342c417c3dc033bba25148b279f60-c8ea0457495342c417c3dc033bba25148b279f60 git://xenbits.xen.org/qemu-xen.git#5cd7ce5dde3f228b3b669ed9ca432f588947bd40-5cd7ce5dde3f228b3b669ed9ca432f588947bd40 git://xenbits.xen.org/xen.git#8e77dabc58c4b6c747dfb4b948551147905a7840-e07ebab86595df29838f0960693df84f34528833
Loaded 2001 nodes in revision graph
Searching for test results:
 114808 pass 9d36d3eff2f85efad0a3b0c6031081654ae33928 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 8e77dabc58c4b6c747dfb4b948551147905a7840
 114836 [host=nobling0]
 115037 [host=rimava0]
 114972 [host=chardonnay0]
 115087 [host=nocera0]
 115132 [host=huxelrebe1]
 115163 [host=italia0]
 115272 [host=fiano1]
 115174 [host=elbling0]
 115235 [host=nobling1]
 115195 [host=baroque0]
 115345 [host=rimava1]
 115295 [host=chardonnay1]
 115314 [host=nocera1]
 115331 [host=elbling1]
 115362 [host=merlot0]
 115378 [host=baroque1]
 115401 [host=pinot1]
 115419 [host=huxelrebe0]
 115450 [host=fiano0]
 115471 [host=godello0]
 115464 [host=godello0]
 115496 [host=nobling0]
 115594 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 1abaf4f1fe5e106b2578c1df3a13e77170a885cc
 115526 [host=italia1]
 115593 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 e07ebab86595df29838f0960693df84f34528833
 115603 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 86cf189a957129ea1ad6468fe9a0887b9e2819f3
 115584 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 e07ebab86595df29838f0960693df84f34528833
 115608 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 f48b5449dabc770acdde6d25cfbd265cfb71034d
 115612 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 f48b5449dabc770acdde6d25cfbd265cfb71034d
 115555 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 e07ebab86595df29838f0960693df84f34528833
 115590 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 87b0ae7e8277d2fa13486ce2e11a941e55f8df40
 115596 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 89d55473ed16543044a31d1e0d4660cf5a3f49df
 115579 pass 9d36d3eff2f85efad0a3b0c6031081654ae33928 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 8e77dabc58c4b6c747dfb4b948551147905a7840
 115591 pass 0c80bbb7681400a6ac802b918e5b77979320998a c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 8e77dabc58c4b6c747dfb4b948551147905a7840
 115604 fail 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 f48b5449dabc770acdde6d25cfbd265cfb71034d
 115598 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 b3b3de05a84d26509122b96f299db3f8196b3fbe
 115601 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 716273c94077e354194c7236623fe3dfa34572d3
 115607 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 86cf189a957129ea1ad6468fe9a0887b9e2819f3
 115611 pass 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 86cf189a957129ea1ad6468fe9a0887b9e2819f3
Searching for interesting versions
 Result found: flight 114808 (pass), for basis pass
 Result found: flight 115555 (fail), for basis failure
 Repro found: flight 115579 (pass), for basis pass
 Repro found: flight 115584 (fail), for basis failure
 0 revisions at 5d7a76acad403638f635c918cc63d1d44ffa4065 c530a75c1e6a472b0eb9558310b518f0dfcd8860 c8ea0457495342c417c3dc033bba25148b279f60 5cd7ce5dde3f228b3b669ed9ca432f588947bd40 86cf189a957129ea1ad6468fe9a0887b9e2819f3
No revisions left to test, checking graph state.
 Result found: flight 115603 (pass), for last pass
 Result found: flight 115604 (fail), for first failure
 Repro found: flight 115607 (pass), for last pass
 Repro found: flight 115608 (fail), for first failure
 Repro found: flight 115611 (pass), for last pass
 Repro found: flight 115612 (fail), for first failure

*** Found and reproduced problem changeset ***

  Bug is in tree:  xen git://xenbits.xen.org/xen.git
  Bug introduced:  f48b5449dabc770acdde6d25cfbd265cfb71034d
  Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
  Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/


  commit f48b5449dabc770acdde6d25cfbd265cfb71034d
  Author: Wei Liu <wei.liu2@citrix.com>
  Date:   Thu Oct 12 20:19:07 2017 +0100
  
      tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
      
      The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
      builder is for libxl_dom() to translate the console and xenstore pfns back
      into useful values.  PV guest pfns are only interesting to the domain builder,
      and gfns are the address space used by all other hypercalls.
      
      Renaming the fields in xc_dom_image is deliberate, as it will cause
      out-of-tree users of the dombuilder to notice the different semantics.
      
      Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
      using gfns despite the existing variable names.
      
      Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
      Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
      Acked-by: Wei Liu <wei.liu2@citrix.com>
      Tested-by: Julien Grall <julien.grall@arm.com>
      Release-acked-by: Julien Grall <julien.grall@linaro.org>
      [ wei: fix stubdom build ]
      Signed-off-by: Wei Liu <wei.liu2@citrix.com>

Revision graph left in /home/logs/results/bisect/xen-unstable/test-amd64-amd64-i386-pvgrub.guest-start.{dot,ps,png,html,svg}.
----------------------------------------
115612: tolerable ALL FAIL

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

Failures :-/ but no regressions.

Tests which did not succeed,
including tests which could not be run:
 test-amd64-amd64-i386-pvgrub 11 guest-start             fail baseline untested


jobs:
 test-amd64-amd64-i386-pvgrub                                 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



[-- Attachment #2: Type: text/plain, Size: 127 bytes --]

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Wei Liu]

On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
> branch xen-unstable
> xenbranch xen-unstable
> job test-amd64-amd64-i386-pvgrub
> testid guest-start
> 
> Tree: linux git://xenbits.xen.org/linux-pvops.git
> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
> 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
>   Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
>   Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
> 
> 
>   commit f48b5449dabc770acdde6d25cfbd265cfb71034d
>   Author: Wei Liu <wei.liu2@citrix.com>
>   Date:   Thu Oct 12 20:19:07 2017 +0100
>   
>       tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
>       
>       The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
>       builder is for libxl_dom() to translate the console and xenstore pfns back
>       into useful values.  PV guest pfns are only interesting to the domain builder,
>       and gfns are the address space used by all other hypercalls.
>       
>       Renaming the fields in xc_dom_image is deliberate, as it will cause
>       out-of-tree users of the dombuilder to notice the different semantics.
>       
>       Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
>       using gfns despite the existing variable names.
>       
>       Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>       Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
>       Acked-by: Wei Liu <wei.liu2@citrix.com>
>       Tested-by: Julien Grall <julien.grall@arm.com>
>       Release-acked-by: Julien Grall <julien.grall@linaro.org>
>       [ wei: fix stubdom build ]
>       Signed-off-by: Wei Liu <wei.liu2@citrix.com>

This has broken pvgrub. The problem is more than just the name of the
variables. I have reverted this and its successor patch.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Julien Grall]

Hi Wei,

On 06/11/17 14:55, Wei Liu wrote:
> On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
>> branch xen-unstable
>> xenbranch xen-unstable
>> job test-amd64-amd64-i386-pvgrub
>> testid guest-start
>>
>> Tree: linux git://xenbits.xen.org/linux-pvops.git
>> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
>> 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
>>    Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
>>    Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
>>
>>
>>    commit f48b5449dabc770acdde6d25cfbd265cfb71034d
>>    Author: Wei Liu <wei.liu2@citrix.com>
>>    Date:   Thu Oct 12 20:19:07 2017 +0100
>>    
>>        tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
>>        
>>        The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
>>        builder is for libxl_dom() to translate the console and xenstore pfns back
>>        into useful values.  PV guest pfns are only interesting to the domain builder,
>>        and gfns are the address space used by all other hypercalls.
>>        
>>        Renaming the fields in xc_dom_image is deliberate, as it will cause
>>        out-of-tree users of the dombuilder to notice the different semantics.
>>        
>>        Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
>>        using gfns despite the existing variable names.
>>        
>>        Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>>        Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
>>        Acked-by: Wei Liu <wei.liu2@citrix.com>
>>        Tested-by: Julien Grall <julien.grall@arm.com>
>>        Release-acked-by: Julien Grall <julien.grall@linaro.org>
>>        [ wei: fix stubdom build ]
>>        Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> 
> This has broken pvgrub. The problem is more than just the name of the
> variables. I have reverted this and its successor patch.

It looks like osstest is still broken after the patches you reverted 
(see [1] and [2]).

AFAICT, the only series between the two flights is the dombuilder, there 
are 2 patches not reverted.

Do you have an idea of what's going on?

Cheers,

[1] http://logs.test-lab.xenproject.org/osstest/logs/115624/
[2] 
https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg00391.html

Cheers,

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Wei Liu]

On Tue, Nov 07, 2017 at 03:09:07PM +0000, Julien Grall wrote:
> Hi Wei,
> 
> On 06/11/17 14:55, Wei Liu wrote:
> > On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
> > > branch xen-unstable
> > > xenbranch xen-unstable
> > > job test-amd64-amd64-i386-pvgrub
> > > testid guest-start
> > > 
> > > Tree: linux git://xenbits.xen.org/linux-pvops.git
> > > Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
> > > 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
> > >    Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
> > >    Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
> > > 
> > > 
> > >    commit f48b5449dabc770acdde6d25cfbd265cfb71034d
> > >    Author: Wei Liu <wei.liu2@citrix.com>
> > >    Date:   Thu Oct 12 20:19:07 2017 +0100
> > >        tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
> > >        The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
> > >        builder is for libxl_dom() to translate the console and xenstore pfns back
> > >        into useful values.  PV guest pfns are only interesting to the domain builder,
> > >        and gfns are the address space used by all other hypercalls.
> > >        Renaming the fields in xc_dom_image is deliberate, as it will cause
> > >        out-of-tree users of the dombuilder to notice the different semantics.
> > >        Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
> > >        using gfns despite the existing variable names.
> > >        Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> > >        Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
> > >        Acked-by: Wei Liu <wei.liu2@citrix.com>
> > >        Tested-by: Julien Grall <julien.grall@arm.com>
> > >        Release-acked-by: Julien Grall <julien.grall@linaro.org>
> > >        [ wei: fix stubdom build ]
> > >        Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> > 
> > This has broken pvgrub. The problem is more than just the name of the
> > variables. I have reverted this and its successor patch.
> 
> It looks like osstest is still broken after the patches you reverted (see
> [1] and [2]).
> 
> AFAICT, the only series between the two flights is the dombuilder, there are
> 2 patches not reverted.
> 
> Do you have an idea of what's going on?
> 
> Cheers,
> 
> [1] http://logs.test-lab.xenproject.org/osstest/logs/115624/
> [2]
> https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg00391.html
> 

test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs.  115526
test-armhf-armhf-xl-vhd     15 guest-start/debian.repeat fail REGR. vs.  115526 

These aren't related to dombuilder at first glance.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Julien Grall]

Hi Wei,

On 07/11/17 15:13, Wei Liu wrote:
> On Tue, Nov 07, 2017 at 03:09:07PM +0000, Julien Grall wrote:
>> Hi Wei,
>>
>> On 06/11/17 14:55, Wei Liu wrote:
>>> On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
>>>> branch xen-unstable
>>>> xenbranch xen-unstable
>>>> job test-amd64-amd64-i386-pvgrub
>>>> testid guest-start
>>>>
>>>> Tree: linux git://xenbits.xen.org/linux-pvops.git
>>>> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
>>>> 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
>>>>     Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
>>>>     Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
>>>>
>>>>
>>>>     commit f48b5449dabc770acdde6d25cfbd265cfb71034d
>>>>     Author: Wei Liu <wei.liu2@citrix.com>
>>>>     Date:   Thu Oct 12 20:19:07 2017 +0100
>>>>         tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
>>>>         The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
>>>>         builder is for libxl_dom() to translate the console and xenstore pfns back
>>>>         into useful values.  PV guest pfns are only interesting to the domain builder,
>>>>         and gfns are the address space used by all other hypercalls.
>>>>         Renaming the fields in xc_dom_image is deliberate, as it will cause
>>>>         out-of-tree users of the dombuilder to notice the different semantics.
>>>>         Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
>>>>         using gfns despite the existing variable names.
>>>>         Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>>>>         Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
>>>>         Acked-by: Wei Liu <wei.liu2@citrix.com>
>>>>         Tested-by: Julien Grall <julien.grall@arm.com>
>>>>         Release-acked-by: Julien Grall <julien.grall@linaro.org>
>>>>         [ wei: fix stubdom build ]
>>>>         Signed-off-by: Wei Liu <wei.liu2@citrix.com>
>>>
>>> This has broken pvgrub. The problem is more than just the name of the
>>> variables. I have reverted this and its successor patch.
>>
>> It looks like osstest is still broken after the patches you reverted (see
>> [1] and [2]).
>>
>> AFAICT, the only series between the two flights is the dombuilder, there are
>> 2 patches not reverted.
>>
>> Do you have an idea of what's going on?
>>
>> Cheers,
>>
>> [1] http://logs.test-lab.xenproject.org/osstest/logs/115624/
>> [2]
>> https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg00391.html
>>
> 
> test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs.  115526
> test-armhf-armhf-xl-vhd     15 guest-start/debian.repeat fail REGR. vs.  115526

The log for the xl-vhd contains ([1])

libxl: error: libxl_bootloader.c:283:bootloader_local_detached_cb: Domain 11:unable to detach locally attached disk
libxl: error: libxl_create.c:1246:domcreate_rebuild_done: Domain 11:cannot (re-)build domain: -3
libxl: debug: libxl_domain.c:1138:devices_destroy_cb: Domain 11:Forked pid 5103 for destroy of domain
libxl: debug: libxl_create.c:1683:do_domain_create: Domain 0:ao 0x5d6e8: inprogress: poller=0x56ad8, flags=i
libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5d6e8: complete, rc=-3
libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5d6e8: destroy
libxl: debug: libxl_domain.c:868:libxl_domain_destroy: Domain 11:ao 0x5a170: create: how=(nil) callback=(nil) poller=0x56ad8
libxl: error: libxl_domain.c:1000:libxl__destroy_domid: Domain 11:Non-existant domain
libxl: error: libxl_domain.c:959:domain_destroy_callback: Domain 11:Unable to destroy guest
libxl: error: libxl_domain.c:886:domain_destroy_cb: Domain 11:Destruction of domain failed
libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5a170: complete, rc=-21
libxl: debug: libxl_domain.c:877:libxl_domain_destroy: Domain 11:ao 0x5a170: inprogress: poller=0x56ad8, flags=ic
libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5a170: destroy

It is in guest repeat and has succeed few times before.

Looking at the success/failure ([2]), the same configuration passed on the Arndale
(see 115580) but fails reliably on the cubietruck.

My guess would be the disk is not detached by the previous guest in time.
Now the question is why? I am not familiar with this area, any ideas? 

> 
> These aren't related to dombuilder at first glance.
> 

Cheers,

[1] http://logs.test-lab.xenproject.org/osstest/logs/115624/test-armhf-armhf-xl-vhd/15.ts-repeat-test.log
[2] http://logs.test-lab.xenproject.org/osstest/results/history/test-armhf-armhf-xl-vhd/xen-unstable

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Wei Liu]

On Tue, Nov 07, 2017 at 05:24:32PM +0000, Julien Grall wrote:
> Hi Wei,
> 
> On 07/11/17 15:13, Wei Liu wrote:
> > On Tue, Nov 07, 2017 at 03:09:07PM +0000, Julien Grall wrote:
> >> Hi Wei,
> >>
> >> On 06/11/17 14:55, Wei Liu wrote:
> >>> On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
> >>>> branch xen-unstable
> >>>> xenbranch xen-unstable
> >>>> job test-amd64-amd64-i386-pvgrub
> >>>> testid guest-start
> >>>>
> >>>> Tree: linux git://xenbits.xen.org/linux-pvops.git
> >>>> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
> >>>> 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
> >>>>     Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
> >>>>     Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
> >>>>
> >>>>
> >>>>     commit f48b5449dabc770acdde6d25cfbd265cfb71034d
> >>>>     Author: Wei Liu <wei.liu2@citrix.com>
> >>>>     Date:   Thu Oct 12 20:19:07 2017 +0100
> >>>>         tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
> >>>>         The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
> >>>>         builder is for libxl_dom() to translate the console and xenstore pfns back
> >>>>         into useful values.  PV guest pfns are only interesting to the domain builder,
> >>>>         and gfns are the address space used by all other hypercalls.
> >>>>         Renaming the fields in xc_dom_image is deliberate, as it will cause
> >>>>         out-of-tree users of the dombuilder to notice the different semantics.
> >>>>         Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
> >>>>         using gfns despite the existing variable names.
> >>>>         Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
> >>>>         Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
> >>>>         Acked-by: Wei Liu <wei.liu2@citrix.com>
> >>>>         Tested-by: Julien Grall <julien.grall@arm.com>
> >>>>         Release-acked-by: Julien Grall <julien.grall@linaro.org>
> >>>>         [ wei: fix stubdom build ]
> >>>>         Signed-off-by: Wei Liu <wei.liu2@citrix.com>
> >>>
> >>> This has broken pvgrub. The problem is more than just the name of the
> >>> variables. I have reverted this and its successor patch.
> >>
> >> It looks like osstest is still broken after the patches you reverted (see
> >> [1] and [2]).
> >>
> >> AFAICT, the only series between the two flights is the dombuilder, there are
> >> 2 patches not reverted.
> >>
> >> Do you have an idea of what's going on?
> >>
> >> Cheers,
> >>
> >> [1] http://logs.test-lab.xenproject.org/osstest/logs/115624/
> >> [2]
> >> https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg00391.html
> >>
> > 
> > test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs.  115526
> > test-armhf-armhf-xl-vhd     15 guest-start/debian.repeat fail REGR. vs.  115526
> 
> The log for the xl-vhd contains ([1])
> 
> libxl: error: libxl_bootloader.c:283:bootloader_local_detached_cb: Domain 11:unable to detach locally attached disk
> libxl: error: libxl_create.c:1246:domcreate_rebuild_done: Domain 11:cannot (re-)build domain: -3
> libxl: debug: libxl_domain.c:1138:devices_destroy_cb: Domain 11:Forked pid 5103 for destroy of domain
> libxl: debug: libxl_create.c:1683:do_domain_create: Domain 0:ao 0x5d6e8: inprogress: poller=0x56ad8, flags=i
> libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5d6e8: complete, rc=-3
> libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5d6e8: destroy
> libxl: debug: libxl_domain.c:868:libxl_domain_destroy: Domain 11:ao 0x5a170: create: how=(nil) callback=(nil) poller=0x56ad8
> libxl: error: libxl_domain.c:1000:libxl__destroy_domid: Domain 11:Non-existant domain
> libxl: error: libxl_domain.c:959:domain_destroy_callback: Domain 11:Unable to destroy guest
> libxl: error: libxl_domain.c:886:domain_destroy_cb: Domain 11:Destruction of domain failed
> libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5a170: complete, rc=-21
> libxl: debug: libxl_domain.c:877:libxl_domain_destroy: Domain 11:ao 0x5a170: inprogress: poller=0x56ad8, flags=ic
> libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5a170: destroy
> 
> It is in guest repeat and has succeed few times before.
> 
> Looking at the success/failure ([2]), the same configuration passed on the Arndale
> (see 115580) but fails reliably on the cubietruck.
> 

The same test failed on Arndale as well in 115314 and 115526, with the
same error messages.

http://logs.test-lab.xenproject.org/osstest/logs/115526/test-armhf-armhf-xl-vhd/16.ts-guest-start.log

So the failure isn't related to Andrew's series.

> My guess would be the disk is not detached by the previous guest in time.
> Now the question is why? I am not familiar with this area, any ideas? 
> 

I don't have immediate idea either. I've set up a repro flight so that
we can have something to play with.

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Re: [xen-unstable bisection] complete test-amd64-amd64-i386-pvgrub

[Julien Grall]

Hi Wei,

On 08/11/17 11:36, Wei Liu wrote:
> On Tue, Nov 07, 2017 at 05:24:32PM +0000, Julien Grall wrote:
>> Hi Wei,
>>
>> On 07/11/17 15:13, Wei Liu wrote:
>>> On Tue, Nov 07, 2017 at 03:09:07PM +0000, Julien Grall wrote:
>>>> Hi Wei,
>>>>
>>>> On 06/11/17 14:55, Wei Liu wrote:
>>>>> On Mon, Nov 06, 2017 at 01:47:56PM +0000, osstest service owner wrote:
>>>>>> branch xen-unstable
>>>>>> xenbranch xen-unstable
>>>>>> job test-amd64-amd64-i386-pvgrub
>>>>>> testid guest-start
>>>>>>
>>>>>> Tree: linux git://xenbits.xen.org/linux-pvops.git
>>>>>> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
>>>>>> 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:  f48b5449dabc770acdde6d25cfbd265cfb71034d
>>>>>>      Bug not present: 86cf189a957129ea1ad6468fe9a0887b9e2819f3
>>>>>>      Last fail repro: http://logs.test-lab.xenproject.org/osstest/logs/115612/
>>>>>>
>>>>>>
>>>>>>      commit f48b5449dabc770acdde6d25cfbd265cfb71034d
>>>>>>      Author: Wei Liu <wei.liu2@citrix.com>
>>>>>>      Date:   Thu Oct 12 20:19:07 2017 +0100
>>>>>>          tools/dombuilder: Switch to using gfn terminology for console and xenstore rings
>>>>>>          The sole use of xc_dom_translated() and xc_dom_p2m() outside of the domain
>>>>>>          builder is for libxl_dom() to translate the console and xenstore pfns back
>>>>>>          into useful values.  PV guest pfns are only interesting to the domain builder,
>>>>>>          and gfns are the address space used by all other hypercalls.
>>>>>>          Renaming the fields in xc_dom_image is deliberate, as it will cause
>>>>>>          out-of-tree users of the dombuilder to notice the different semantics.
>>>>>>          Correct the terminology throughout xc_dom_gnttab{_hvm,}_seed(), which are all
>>>>>>          using gfns despite the existing variable names.
>>>>>>          Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
>>>>>>          Reviewed-by: Roger Pau Monn?? <roger.pau@citrix.com>
>>>>>>          Acked-by: Wei Liu <wei.liu2@citrix.com>
>>>>>>          Tested-by: Julien Grall <julien.grall@arm.com>
>>>>>>          Release-acked-by: Julien Grall <julien.grall@linaro.org>
>>>>>>          [ wei: fix stubdom build ]
>>>>>>          Signed-off-by: Wei Liu <wei.liu2@citrix.com>
>>>>>
>>>>> This has broken pvgrub. The problem is more than just the name of the
>>>>> variables. I have reverted this and its successor patch.
>>>>
>>>> It looks like osstest is still broken after the patches you reverted (see
>>>> [1] and [2]).
>>>>
>>>> AFAICT, the only series between the two flights is the dombuilder, there are
>>>> 2 patches not reverted.
>>>>
>>>> Do you have an idea of what's going on?
>>>>
>>>> Cheers,
>>>>
>>>> [1] http://logs.test-lab.xenproject.org/osstest/logs/115624/
>>>> [2]
>>>> https://lists.xenproject.org/archives/html/xen-devel/2017-11/msg00391.html
>>>>
>>>
>>> test-amd64-amd64-xl-qemut-win7-amd64 16 guest-localmigrate/x10 fail REGR. vs.  115526

Looking at the osstest result today, this one seem to be intermittent as 
it passed during the night but failed this morning.

>>> test-armhf-armhf-xl-vhd     15 guest-start/debian.repeat fail REGR. vs.  115526
>>
>> The log for the xl-vhd contains ([1])
>>
>> libxl: error: libxl_bootloader.c:283:bootloader_local_detached_cb: Domain 11:unable to detach locally attached disk
>> libxl: error: libxl_create.c:1246:domcreate_rebuild_done: Domain 11:cannot (re-)build domain: -3
>> libxl: debug: libxl_domain.c:1138:devices_destroy_cb: Domain 11:Forked pid 5103 for destroy of domain
>> libxl: debug: libxl_create.c:1683:do_domain_create: Domain 0:ao 0x5d6e8: inprogress: poller=0x56ad8, flags=i
>> libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5d6e8: complete, rc=-3
>> libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5d6e8: destroy
>> libxl: debug: libxl_domain.c:868:libxl_domain_destroy: Domain 11:ao 0x5a170: create: how=(nil) callback=(nil) poller=0x56ad8
>> libxl: error: libxl_domain.c:1000:libxl__destroy_domid: Domain 11:Non-existant domain
>> libxl: error: libxl_domain.c:959:domain_destroy_callback: Domain 11:Unable to destroy guest
>> libxl: error: libxl_domain.c:886:domain_destroy_cb: Domain 11:Destruction of domain failed
>> libxl: debug: libxl_event.c:1869:libxl__ao_complete: ao 0x5a170: complete, rc=-21
>> libxl: debug: libxl_domain.c:877:libxl_domain_destroy: Domain 11:ao 0x5a170: inprogress: poller=0x56ad8, flags=ic
>> libxl: debug: libxl_event.c:1838:libxl__ao__destroy: ao 0x5a170: destroy
>>
>> It is in guest repeat and has succeed few times before.
>>
>> Looking at the success/failure ([2]), the same configuration passed on the Arndale
>> (see 115580) but fails reliably on the cubietruck.
>>
> 
> The same test failed on Arndale as well in 115314 and 115526, with the
> same error messages.
> 
> http://logs.test-lab.xenproject.org/osstest/logs/115526/test-armhf-armhf-xl-vhd/16.ts-guest-start.log
> 
> So the failure isn't related to Andrew's series.
> 
>> My guess would be the disk is not detached by the previous guest in time.
>> Now the question is why? I am not familiar with this area, any ideas?
>>
> 
> I don't have immediate idea either. I've set up a repro flight so that
> we can have something to play with.

Thanks! Let me know when it is ready.

Cheers,

-- 
Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel