From: Ian Campbell <ian.campbell@citrix.com>
To: "osstest service owner" <osstest-admin@xenproject.org>,
xen-devel@lists.xensource.com,
"Marek Marczykowski-Górecki" <marmarek@invisiblethingslab.com>,
"David Vrabel" <david.vrabel@citrix.com>
Subject: Re: [linux-3.14 bisection] complete test-amd64-i386-xl-qcow2
Date: Tue, 1 Sep 2015 10:19:58 +0100 [thread overview]
Message-ID: <1441099198.27618.13.camel@citrix.com> (raw)
In-Reply-To: <E1ZUgtn-0001q8-QQ@osstest.test-lab.xenproject.org>
On Wed, 2015-08-26 at 20:02 +0000, osstest service owner wrote:
> branch xen-unstable
> xen branch xen-unstable
> job test-amd64-i386-xl-qcow2
> test guest-saverestore
>
> Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
> Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
> Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git
> Tree: xen git://xenbits.xen.org/xen.git
>
> *** Found and reproduced problem changeset ***
>
> Bug is in tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> Bug introduced: 9e6c072a69d87100808d16279d60e9f857291340
> Bug not present: b60d3eee854b881b3f7a478c8b622cbef72d4c4e
>
>
> commit 9e6c072a69d87100808d16279d60e9f857291340
> Author: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
> Date: Fri Jun 26 03:28:24 2015 +0200
>
> xen/gntdevt: Fix race condition in gntdev_release()
I'm not sure what to make of this.
The qcow2 test is one of the only ones I'd expect to be exercising gntdev
(most tests use LVM+blkback), which explains why this particular commit is
apparently seeing issues due to this particular change.
From these ...
http://logs.test-lab.xenproject.org/osstest/results/history/test-amd64-i386-xl-qcow2/linux-3.14.html
http://logs.test-lab.xenproject.org/osstest/results/history/test-amd64-i386-xl-qcow2/linux-3.16.html
http://logs.test-lab.xenproject.org/osstest/results/history/test-amd64-i386-xl-qcow2/linux-3.18.html
http://logs.test-lab.xenproject.org/osstest/results/history/test-amd64-i386-xl-qcow2/linux-4.1.html
http://logs.test-lab.xenproject.org/osstest/results/history/test-amd64-i386-xl-qcow2/linux-linus.html
... it seems only 3.14 is hit, I wonder why?
Maybe the other stable trees haven't gotten this yet (or osstest hasn't
gotten to a revision with it) but linux-linus certainly will and it appears
to be fine there. Perhaps a mis-applied/behaving backport or a missing
associated patch?
Flight 60952 on linux-linus was on huxelrebe0, contained 30b03d05e0746 and
passed. That is the same h/w as huxelrebe1 which is failing (sticky) on
3.14. Also 60785 on linux-4.1 was on huxelrebe1 itself and contains the
backport, and was fine. So I conclude it is probably not a h/w specific
issue.
Ian.
>
> commit 30b03d05e07467b8c6ec683ea96b5bffcbcd3931 upstream.
>
> While gntdev_release() is called the MMU notifier is still registered
> and can traverse priv->maps list even if no pages are mapped (which is
> the case -- gntdev_release() is called after all). But
> gntdev_release() will clear that list, so make sure that only one of
> those things happens at the same time.
>
> Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
> Signed-off-by: David Vrabel <david.vrabel@citrix.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
>
> For bisection revision-tuple graph see:
> http://logs.test-lab.xenproject.org/osstest/results/bisect/linux-3.14/test-amd64-i386-xl-qcow2.guest-saverestore.html
> Revision IDs in each graph node refer, respectively, to the Trees above.
>
> ----------------------------------------
> Searching for failure / basis pass:
> 60823 fail [host=huxelrebe1] / 60666 [host=chardonnay1] 60655 [host=elbling1] 60549 [host=chardonnay0] template as basis? using template as basis.
> Failure / basis pass flights: 60823 / 60666
> (tree with no url: ovmf)
> (tree with no url: seabios)
> Tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> Tree: linuxfirmware git://xenbits.xen.org/osstest/linux-firmware.git
> Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
> Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git
> Tree: xen git://xenbits.xen.org/xen.git
> Latest 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> Basis pass 9b8b905951bde404f20a7bd4b37a5134f3484569 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 201eac83831d94ba2e9a63a7eed4c128633fafb1
> Generating revisions with ./adhoc-revtuple-generator git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git#9b8b905951bde404f20a7bd4b37a5134f3484569-318ff69ca4c275bae4b875b87df5bdbd7988486a git://xenbits.xen.org/osstest/linux-firmware.git#c530a75c1e6a472b0eb9558310b518f0dfcd8860-c530a75c1e6a472b0eb9558310b518f0dfcd8860 git://xenbits.xen.org/staging/qemu-xen-unstable.git#7f057440b31da38196e3398fd1b618fc36ad97d6-7f057440b31da38196e3398fd1b618fc36ad97d6 git://xenbits.xen.org/staging/qemu-upstream-unstable.git#bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa-bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa git://xenbits.xen.org/xen.git#201eac83831d94ba2e9a63a7eed4c128633fafb1-145a8004a7d659668d5a3b0ad9868d7678b24822
> + exec
> + sh -xe
> + cd /home/osstest/repos/linux-stable
> + git remote set-url origin git://cache:9419/git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> + git fetch -p origin +refs/heads/*:refs/remotes/origin/*
> + exec
> + sh -xe
> + cd /home/osstest/repos/xen
> + git remote set-url origin git://cache:9419/git://xenbits.xen.org/xen.git
> + git fetch -p origin +refs/heads/*:refs/remotes/origin/*
> + exec
> + sh -xe
> + cd /home/osstest/repos/linux-stable
> + git remote set-url origin git://cache:9419/git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> + git fetch -p origin +refs/heads/*:refs/remotes/origin/*
> + exec
> + sh -xe
> + cd /home/osstest/repos/xen
> + git remote set-url origin git://cache:9419/git://xenbits.xen.org/xen.git
> + git fetch -p origin +refs/heads/*:refs/remotes/origin/*
> Loaded 2001 nodes in revision graph
> Searching for test results:
> 60655 [host=elbling1]
> 60666 [host=chardonnay1]
> 60821 pass 9b8b905951bde404f20a7bd4b37a5134f3484569 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 201eac83831d94ba2e9a63a7eed4c128633fafb1
> 60747 fail 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60787 fail 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60823 fail 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60824 fail 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60850 pass 9b8b905951bde404f20a7bd4b37a5134f3484569 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 201eac83831d94ba2e9a63a7eed4c128633fafb1
> 60852 fail 318ff69ca4c275bae4b875b87df5bdbd7988486a c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60885 fail 9e6c072a69d87100808d16279d60e9f857291340 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60854 fail 6221fbc5ac6bf4a0d21ad5881e31daa9700c7a88 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60855 pass d68c869854ff29a7baa1355470caaf6c999d2008 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60857 pass adbbaa36dd55ff0bde07391d898779760b5206df c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60887 pass b60d3eee854b881b3f7a478c8b622cbef72d4c4e c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60891 fail 9e6c072a69d87100808d16279d60e9f857291340 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60892 pass b60d3eee854b881b3f7a478c8b622cbef72d4c4e c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60893 fail 9e6c072a69d87100808d16279d60e9f857291340 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60859 pass 166b8915aad6f7db7446b74f38a8c3a45626c815 c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> 60883 pass b60d3eee854b881b3f7a478c8b622cbef72d4c4e c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> Searching for interesting versions
> Result found: flight 60821 (pass), for basis pass
> Result found: flight 60823 (fail), for basis failure
> Repro found: flight 60850 (pass), for basis pass
> Repro found: flight 60852 (fail), for basis failure
> 0 revisions at b60d3eee854b881b3f7a478c8b622cbef72d4c4e c530a75c1e6a472b0eb9558310b518f0dfcd8860 7f057440b31da38196e3398fd1b618fc36ad97d6 bcf35eec0b621c46dbf0aeb40c6bc06b5d3981aa 145a8004a7d659668d5a3b0ad9868d7678b24822
> No revisions left to test, checking graph state.
> Result found: flight 60883 (pass), for last pass
> Result found: flight 60885 (fail), for first failure
> Repro found: flight 60887 (pass), for last pass
> Repro found: flight 60891 (fail), for first failure
> Repro found: flight 60892 (pass), for last pass
> Repro found: flight 60893 (fail), for first failure
>
> *** Found and reproduced problem changeset ***
>
> Bug is in tree: linux git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> Bug introduced: 9e6c072a69d87100808d16279d60e9f857291340
> Bug not present: b60d3eee854b881b3f7a478c8b622cbef72d4c4e
>
> + exec
> + sh -xe
> + cd /home/osstest/repos/linux-stable
> + git remote set-url origin git://cache:9419/git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable.git
> + git fetch -p origin +refs/heads/*:refs/remotes/origin/*
>
> commit 9e6c072a69d87100808d16279d60e9f857291340
> Author: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
> Date: Fri Jun 26 03:28:24 2015 +0200
>
> xen/gntdevt: Fix race condition in gntdev_release()
>
> commit 30b03d05e07467b8c6ec683ea96b5bffcbcd3931 upstream.
>
> While gntdev_release() is called the MMU notifier is still registered
> and can traverse priv->maps list even if no pages are mapped (which is
> the case -- gntdev_release() is called after all). But
> gntdev_release() will clear that list, so make sure that only one of
> those things happens at the same time.
>
> Signed-off-by: Marek Marczykowski-Górecki <marmarek@invisiblethingslab.com>
> Signed-off-by: David Vrabel <david.vrabel@citrix.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> Revision graph left in /home/logs/results/bisect/linux-3.14/test-amd64-i386-xl-qcow2.guest-saverestore.{dot,ps,png,html}.
> ----------------------------------------
> 60893: tolerable ALL FAIL
>
> flight 60893 linux-3.14 real-bisect [real]
> http://logs.test-lab.xenproject.org/osstest/logs/60893/
>
> Failures :-/ but no regressions.
>
> Tests which did not succeed,
> including tests which could not be run:
> test-amd64-i386-xl-qcow2 13 guest-saverestore fail baseline untested
>
>
> jobs:
> test-amd64-i386-xl-qcow2 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@lists.xen.org
http://lists.xen.org/xen-devel
next prev parent reply other threads:[~2015-09-01 9:19 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-26 20:02 [linux-3.14 bisection] complete test-amd64-i386-xl-qcow2 osstest service owner
2015-09-01 9:19 ` Ian Campbell [this message]
2015-09-01 9:38 ` Fabio Fantoni
2015-09-01 10:10 ` Ian Campbell
2015-09-01 9:57 ` Ian Jackson
2015-09-01 10:05 ` Ian Campbell
2015-09-02 9:03 ` Ian Campbell
2015-09-02 9:18 ` Ian Campbell
2015-09-02 9:18 ` Ian Campbell
2015-09-03 11:05 ` Luis Henriques
2015-09-03 11:16 ` [Xen-devel] " David Vrabel
2015-09-03 12:21 ` Luis Henriques
2015-09-03 12:21 ` Luis Henriques
2015-09-11 15:10 ` Ian Campbell
2015-09-11 15:10 ` Ian Campbell
2015-09-11 15:51 ` Greg Kroah-Hartman
2015-09-11 15:55 ` Ian Campbell
2015-09-26 17:30 ` Greg Kroah-Hartman
2015-09-28 9:27 ` Ian Campbell
2015-09-28 9:43 ` Ian Campbell
2015-10-08 22:14 ` Ben Hutchings
2015-10-09 9:15 ` Ian Campbell
2015-10-09 9:15 ` Ian Campbell
2015-10-12 10:11 ` [Xen-devel] " David Vrabel
2015-10-12 10:11 ` David Vrabel
2015-10-12 12:56 ` Ben Hutchings
2015-10-12 13:01 ` David Vrabel
2015-10-12 13:01 ` David Vrabel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1441099198.27618.13.camel@citrix.com \
--to=ian.campbell@citrix.com \
--cc=david.vrabel@citrix.com \
--cc=marmarek@invisiblethingslab.com \
--cc=osstest-admin@xenproject.org \
--cc=xen-devel@lists.xensource.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.