Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-06 Thread Kevin Wolf
Am 06.04.2017 um 10:48 hat Kevin Wolf geschrieben: > Am 05.04.2017 um 23:13 hat Paolo Bonzini geschrieben: > > On 05/04/2017 13:01, Kevin Wolf wrote: > > > Am 04.04.2017 um 17:09 hat Paolo Bonzini geschrieben: > > >> On 04/04/2017 16:53, Kevin Wolf wrote: > > The big question is how this fits

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-06 Thread Kevin Wolf
Am 05.04.2017 um 23:13 hat Paolo Bonzini geschrieben: > On 05/04/2017 13:01, Kevin Wolf wrote: > > Am 04.04.2017 um 17:09 hat Paolo Bonzini geschrieben: > >> On 04/04/2017 16:53, Kevin Wolf wrote: > The big question is how this fits into release management. We have > another important

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-05 Thread Paolo Bonzini
On 05/04/2017 13:01, Kevin Wolf wrote: > Am 04.04.2017 um 17:09 hat Paolo Bonzini geschrieben: >> On 04/04/2017 16:53, Kevin Wolf wrote: The big question is how this fits into release management. We have another important regression from the op blocker work and only a week to go

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-05 Thread Kevin Wolf
Am 04.04.2017 um 17:09 hat Paolo Bonzini geschrieben: > On 04/04/2017 16:53, Kevin Wolf wrote: > >> The big question is how this fits into release management. We have > >> another important regression from the op blocker work and only a week > >> to go before the last rc. Are we going to delay

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Paolo Bonzini
On 04/04/2017 16:53, Kevin Wolf wrote: >> The big question is how this fits into release management. We have >> another important regression from the op blocker work and only a week >> to go before the last rc. Are we going to delay 2.9 arbitrarily? Are >> we going to shorten the 2.10

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Kevin Wolf
Am 04.04.2017 um 16:04 hat Paolo Bonzini geschrieben: > On 04/04/2017 14:16, Kevin Wolf wrote: > > Just not requesting the > > write permission initially if runstate_check(RUN_STATE_INMIGRATE) is > > easy. But we need to find a place to actually request it later, after > > the mirror has

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Paolo Bonzini
On 04/04/2017 14:16, Kevin Wolf wrote: > Just not requesting the > write permission initially if runstate_check(RUN_STATE_INMIGRATE) is > easy. But we need to find a place to actually request it later, after > the mirror has completed, and before the VM is running. Isn't there already a

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Kevin Wolf
Am 04.04.2017 um 15:27 hat Peter Krempa geschrieben: > On Tue, Apr 04, 2017 at 15:19:02 +0200, Kevin Wolf wrote: > > Am 03.04.2017 um 14:51 hat Peter Krempa geschrieben: > > > On Mon, Apr 03, 2017 at 10:15:42 +0200, Kevin Wolf wrote: > > > > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > >

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Eric Blake
On 04/04/2017 07:16 AM, Kevin Wolf wrote: > > Now the big question is how to implement this. Just not requesting the > write permission initially if runstate_check(RUN_STATE_INMIGRATE) is > easy. But we need to find a place to actually request it later, after > the mirror has completed, and

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Peter Krempa
On Tue, Apr 04, 2017 at 15:19:02 +0200, Kevin Wolf wrote: > Am 03.04.2017 um 14:51 hat Peter Krempa geschrieben: > > On Mon, Apr 03, 2017 at 10:15:42 +0200, Kevin Wolf wrote: > > > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > > > On 31.03.2017 18:03, Ciprian Barbu wrote: [...] > > >

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Kevin Wolf
Am 03.04.2017 um 14:51 hat Peter Krempa geschrieben: > On Mon, Apr 03, 2017 at 10:15:42 +0200, Kevin Wolf wrote: > > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > > On 31.03.2017 18:03, Ciprian Barbu wrote: > > [...] > > > > So this doesn't work: > > > > > > $

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Kevin Wolf
Am 03.04.2017 um 15:50 hat Peter Krempa geschrieben: > On Mon, Apr 03, 2017 at 15:00:41 +0200, Kevin Wolf wrote: > > If I understand correctly, this is a case of incoming live migration, > > i.e. the virtio-blk device which is blocking the writes to the image > > doesn't really belong to a running

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Daniel P. Berrange
On Tue, Apr 04, 2017 at 01:00:08PM +0200, Paolo Bonzini wrote: > > > On 04/04/2017 10:17, ciprian.barbu wrote: > >> > >> but we do NOT make any guarantees of supporting > >> > >> new qemu, old libvirt > > > > Sounds reasonable enough, I guess we didn't look at it this way. > > Yes, but usually

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread Paolo Bonzini
On 04/04/2017 10:17, ciprian.barbu wrote: >> >> but we do NOT make any guarantees of supporting >> >> new qemu, old libvirt > > Sounds reasonable enough, I guess we didn't look at it this way. Yes, but usually it's "new qemu, very old libvirt", like several years old. I think this should be

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread ciprian . barbu
Hi, On 03.04.2017 22:44, Eric Blake wrote: On 04/03/2017 07:39 AM, Max Reitz wrote: As for just allowing the NBD server write access to the device... To me that appears pretty difficult from an implementation perspective. We assert that nobody can write without having requested write access

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-04 Thread ciprian . barbu
Hi, On 03.04.2017 21:52, Kashyap Chamarthy wrote: On Fri, Mar 31, 2017 at 05:49:52PM +, Ciprian Barbu wrote: Hi, [...] We also use libvirt v1.3.4, which might be a problem, but at least we want to understand if the commit in question introduced an obvious problem or if it's all in the

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Eric Blake
On 04/03/2017 08:00 AM, Kevin Wolf wrote: >> The question remains whether it is practical not to make an exception. >> As far as I know, libvirt is only guaranteed to support older qemu >> versions, not necessarily future ones. So we should be allowed to break >> existing use cases here until

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Eric Blake
On 04/03/2017 07:39 AM, Max Reitz wrote: >>> As for just allowing the NBD server write access to the device... To me >>> that appears pretty difficult from an implementation perspective. We >>> assert that nobody can write without having requested write access and >>> we make sure that nobody can

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Kashyap Chamarthy
On Fri, Mar 31, 2017 at 05:49:52PM +, Ciprian Barbu wrote: > Hi, [...] > We also use libvirt v1.3.4, which might be a problem, but at least we > want to understand if the commit in question introduced an obvious > problem or if it's all in the details. A tangential question -- Just curious,

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Peter Krempa
On Mon, Apr 03, 2017 at 15:00:41 +0200, Kevin Wolf wrote: > Am 03.04.2017 um 14:39 hat Max Reitz geschrieben: > > On 03.04.2017 10:15, Kevin Wolf wrote: > > > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > > > [...] > > > > >> So in theory all that's necessary is to set share-rw=on for

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Kevin Wolf
Am 03.04.2017 um 14:39 hat Max Reitz geschrieben: > On 03.04.2017 10:15, Kevin Wolf wrote: > > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > [...] > > >> So in theory all that's necessary is to set share-rw=on for the device > >> in the management layer. But I'm not sure whether that's

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Peter Krempa
On Mon, Apr 03, 2017 at 10:15:42 +0200, Kevin Wolf wrote: > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > > On 31.03.2017 18:03, Ciprian Barbu wrote: [...] > > So this doesn't work: > > > > $ x86_64-softmmu/qemu-system-x86_64 \ > > -blockdev node-name=image,driver=qcow2,\ > >

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Max Reitz
On 03.04.2017 10:15, Kevin Wolf wrote: > Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: [...] >> So in theory all that's necessary is to set share-rw=on for the device >> in the management layer. But I'm not sure whether that's practical. > > Yes, libvirt needs to provide this option if the

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-04-03 Thread Kevin Wolf
Am 31.03.2017 um 19:43 hat Max Reitz geschrieben: > On 31.03.2017 18:03, Ciprian Barbu wrote: > > Hello, > > > > Similar to the other thread about possible regression with rbd, there might > > be a regression with nbd. > > This time we are launching an instance from an image (not volume) and try

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Alexandru Avadanii
Hi, > -Original Message- > From: Max Reitz [mailto:mre...@redhat.com] > Sent: Friday, March 31, 2017 8:57 PM > To: Ciprian Barbu; qemu-devel@nongnu.org; Eric Blake; Alexandru Avadanii > Cc: Jeff Cody; Markus Armbruster; svc-armband; Kevin Wolf > Subject: Re: [Qemu-de

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Max Reitz
On 31.03.2017 19:49, Ciprian Barbu wrote: > Hi, > > Thank you for getting back! > > I'm trying to follow you, but I don't understand all the details. I would > like to ask this question though: > > What is the difference between v2.8.0 and this commit? With v2.8.0 the same > qemu command

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Ciprian Barbu
t;jc...@redhat.com>; Markus Armbruster <arm...@redhat.com>; svc-armband <armb...@enea.com>; Kevin Wolf <kw...@redhat.com> Subject: Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs On 31.03.2017 18:03, Ciprian Barbu wrote: > Hello, > > Similar to the other thread about pos

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Max Reitz
On 31.03.2017 18:03, Ciprian Barbu wrote: > Hello, > > Similar to the other thread about possible regression with rbd, there might > be a regression with nbd. > This time we are launching an instance from an image (not volume) and try to > live migrate it: > > nova live-migration > > The

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Ciprian Barbu
<ebl...@redhat.com>; Alexandru Avadanii <alexandru.avada...@enea.com> Cc: Jeff Cody <jc...@redhat.com>; Markus Armbruster <arm...@redhat.com>; svc-armband <armb...@enea.com> Subject: RE: [Qemu-devel] nbd: Possible regression in 2.9 RCs I suspect the culprit here is [1

Re: [Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Ciprian Barbu
exandru Avadanii <alexandru.avada...@enea.com> Cc: Jeff Cody <jc...@redhat.com>; Markus Armbruster <arm...@redhat.com>; svc-armband <armb...@enea.com> Subject: [Cosnos] [Qemu-devel] nbd: Possible regression in 2.9 RCs Hello, Similar to the other thread about possible regressio

[Qemu-devel] nbd: Possible regression in 2.9 RCs

2017-03-31 Thread Ciprian Barbu
Hello, Similar to the other thread about possible regression with rbd, there might be a regression with nbd. This time we are launching an instance from an image (not volume) and try to live migrate it: nova live-migration The nova-compute service complains with: 2017-03-31 15:32:56.179