From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43)
	id 1MWF78-000310-V0
	for qemu-devel@nongnu.org; Wed, 29 Jul 2009 15:47:02 -0400
Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43)
	id 1MWF74-0002wo-Bf
	for qemu-devel@nongnu.org; Wed, 29 Jul 2009 15:47:02 -0400
Received: from [199.232.76.173] (port=56609 helo=monty-python.gnu.org)
	by lists.gnu.org with esmtp (Exim 4.43) id 1MWF74-0002wd-7x
	for qemu-devel@nongnu.org; Wed, 29 Jul 2009 15:46:58 -0400
Received: from mx2.redhat.com ([66.187.237.31]:56861)
	by monty-python.gnu.org with esmtp (Exim 4.60)
	(envelope-from <quintela@redhat.com>) id 1MWF73-0002VM-N9
	for qemu-devel@nongnu.org; Wed, 29 Jul 2009 15:46:58 -0400
From: Juan Quintela <quintela@redhat.com>
In-Reply-To: <4A70A400.2070703@codemonkey.ws> (Anthony Liguori's message of
	"Wed\, 29 Jul 2009 14\:33\:20 -0500")
References: <20090729120719.GN30449@redhat.com>
	<20090729135057.GY30449@redhat.com>
	<5b31733c0907290735u5dd86d86i293f34df4ce09d10@mail.gmail.com>
	<4A70A400.2070703@codemonkey.ws>
Date: Wed, 29 Jul 2009 21:44:57 +0200
Message-ID: <m3vdlb2s06.fsf@neno.mitica>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Subject: [Qemu-devel] Re: [PATCH v2] make windows notice media change
List-Id: qemu-devel.nongnu.org
List-Unsubscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.gnu.org/pipermail/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <http://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Anthony Liguori <anthony@codemonkey.ws>
Cc: Filip Navara <filip.navara@gmail.com>, qemu-devel@nongnu.org, Gleb Natapov <gleb@redhat.com>

Anthony Liguori <anthony@codemonkey.ws> wrote:
> Filip Navara wrote:
>> On Wed, Jul 29, 2009 at 3:50 PM, Gleb Natapov<gleb@redhat.com> wrote:
>> [snip]
>>   
>>> @@ -2898,6 +2901,7 @@ static void ide_load(QEMUFile* f, IDEState *s)
>>>
>>>     qemu_get_8s(f, &s->sense_key);
>>>     qemu_get_8s(f, &s->asc);
>>> +    qemu_get_8s(f, &s->cdrom_changed);
>>>     /* XXX: if a transfer is pending, we do not save it yet */
>>>  }
>>>
>>> @@ -3219,7 +3223,7 @@ static int pci_ide_load(QEMUFile* f, void *opaque, int version_id)
>>>     PCIIDEState *d = opaque;
>>>     int ret, i;
>>>
>>> -    if (version_id != 2)
>>> +    if (version_id != 3)
>>>         return -EINVAL;
>>>     ret = pci_device_load(&d->dev, f);
>>>     if (ret < 0)
>>>     
>>
>> Why shouldn't we support loading version 2 snapshots? Afterall that's
>> why we had the versioning in the first place.
>>   
>
> What value do you give cdrom_changed?
>
> If you give it zero, and you happened to be in a state where the cdrom
> was in the process of being changed when you migrated, this bug would
> surface again.
>
> In this case, it's better to refuse the old (broken) version.

You didn't lost anything from the old version, it already have the bug.
Value that you give it: zero.  If it was in the middle of the operation,
it will give the same bug that when it was in the older host, and when
it tells "retry", it will see the new behaviour from this on.

Later, Juan.


> Regards,
>
> Anthony Liguori