[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 0/2] Block patches
From: |
Stefan Hajnoczi |
Subject: |
Re: [PULL 0/2] Block patches |
Date: |
Thu, 25 Mar 2021 17:40:45 +0000 |
On Thu, Mar 25, 2021 at 04:36:43PM +0000, Peter Maydell wrote:
> On Thu, 25 Mar 2021 at 16:28, Stefan Hajnoczi <stefanha@redhat.com> wrote:
> >
> > On Thu, Mar 25, 2021 at 01:17:50PM +0300, Vladimir Sementsov-Ogievskiy
> > wrote:
> > > Thanks! My path modifies migration/block-dirty-bitmap.c. qsd-jobs runs
> > > block-commit and block-stream jobs and don't start any kind of migration
> > > or snapshot or savevm, so it seems impossible that qsd-jobs runs the code
> > > touched by my patch..
> >
> > Confirmed. The failure is not related to this pull request.
> >
> > I reproduced the same s390x host failure on commit
> > 9e2e9fe3df9f539f8b6941ceb96d25355fdae47e (HEAD -> master, tag:
> > v6.0.0-rc0, origin/master, origin/HEAD):
> >
> > qsd-jobs fail [11:04:58] [11:04:58] 0.1s
> > output mismatch (see qsd-jobs.out.bad)
> > --- /root/qemu/tests/qemu-iotests/tests/qsd-jobs.out
> > +++ qsd-jobs.out.bad
> > @@ -9,11 +9,11 @@
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "created", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "job0"}}
> > {"return": {}}
> > +{"return": {}}
> > +{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "paused", "id": "job0"}}
> > +{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "running", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "ready", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "BLOCK_JOB_READY", "data": {"device": "job0", "len": 0, "offset":
> > 0, "speed": 0, "type": "commit"}}
> > -{"return": {}}
> > -{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "standby", "id": "job0"}}
> > -{"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "ready", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "waiting", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "JOB_STATUS_CHANGE", "data": {"status": "pending", "id": "job0"}}
> > {"timestamp": {"seconds": TIMESTAMP, "microseconds": TIMESTAMP},
> > "event": "BLOCK_JOB_COMPLETED", "data": {"device": "job0", "len": 0,
> > "offset": 0, "speed": 0, "type": "commit"}}
> >
> > Peter: Please merge this pull request since the failure is pre-existing.
>
> OK, I'll run the pullreq through merge testing again.
>
> Could somebody on the block team look at this s390 host intermittent ?
I need to look into more s390x stuff on Monday so I'd be happy to dig
deeper then.
Stefan
signature.asc
Description: PGP signature
- [PULL 0/2] Block patches, Stefan Hajnoczi, 2021/03/24
- [PULL 1/2] migration/block-dirty-bitmap: make incoming disabled bitmaps busy, Stefan Hajnoczi, 2021/03/24
- [PULL 2/2] migrate-bitmaps-postcopy-test: check that we can't remove in-flight bitmaps, Stefan Hajnoczi, 2021/03/24
- Re: [PULL 0/2] Block patches, Peter Maydell, 2021/03/24
- Re: [PULL 0/2] Block patches, Vladimir Sementsov-Ogievskiy, 2021/03/24
- Re: [PULL 0/2] Block patches, Peter Maydell, 2021/03/24
- Re: [PULL 0/2] Block patches, Stefan Hajnoczi, 2021/03/25
- Re: [PULL 0/2] Block patches, Vladimir Sementsov-Ogievskiy, 2021/03/25
- Re: [PULL 0/2] Block patches, Stefan Hajnoczi, 2021/03/25
- Re: [PULL 0/2] Block patches, Peter Maydell, 2021/03/25
- Re: [PULL 0/2] Block patches,
Stefan Hajnoczi <=
Re: [PULL 0/2] Block patches, Peter Maydell, 2021/03/26