[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers |
Date: |
Mon, 17 Dec 2012 22:50:43 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/17.0 Thunderbird/17.0 |
Am 17.12.2012 15:43, schrieb Markus Armbruster:
> Alexander Graf <address@hidden> writes:
>
>> On 17.12.2012, at 15:05, Markus Armbruster wrote:
>>
>>> They complicate IDE data structures and keep getting in the way.
>>> Also, TRIM support (commit d353fb72) is broken for them, because
>>> ide_identify() accesses IDEDevice member conf, but IDEDevice exists
>>> only with qdevified controllers.
>>>
>>> The non-qdevified controllers are still there, but attempting to
>>> connect devices to them fails with "IDE controller not qdevified yet;
>>> drive <name> ignored".
>>>
>>> Affected machines:
>>>
>>> * g3beige's first IDE channel (MacIO)
>>> -hda, -hdb are on first channel, and no longer work
>>> -hdc, -hdd are on second channel, and still work
>>> * mac99's second and third IDE channel (MacIO)
>>> All four IDE drives no longer work
>>
>> Nack. This breaks the default targets of qemu-system-ppc and
>> qemu-system-ppc64.
>
> Please tell us how much more time you want to qdevify IDE for these
> targets. Thanks!
I believe I have a branch with macio QOM'ifications somewhere that I
could revive. Note that I know little about IDE or block layer and
mainly care about consistent infrastructure there; I vaguely remember
something about the mac's IDE channels being mixed together from two
devices unlike real hardware, guess I would be unable to fix that.
As for your question, 2013 and a gentle reminder to all involved would
be nice. :) In particular we have the Soft Freeze coming up shortly
after the holidays, so is this needed for 1.4 Soft Freeze or can it be
deferred to 1.5 or done during the 1.4 Soft Freeze?
If Aurélien (CC'ed) doesn't manage, I can look at r2d as well.
CC'ing Peter and Andrzej for the arm devices.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
- [Qemu-devel] [PATCH 02/10] ide: Move IDEDevice pointer from IDEBus to IDEState, (continued)
- [Qemu-devel] [PATCH 02/10] ide: Move IDEDevice pointer from IDEBus to IDEState, Markus Armbruster, 2012/12/17
- [Qemu-devel] [PATCH 06/10] ide: Drop redundant IDEState geometry members, Markus Armbruster, 2012/12/17
- [Qemu-devel] [PATCH 03/10] ide: Use IDEState member dev for "device connected" test, Markus Armbruster, 2012/12/17
- [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Markus Armbruster, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Alexander Graf, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Markus Armbruster, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Alexander Graf, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Markus Armbruster, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Alexander Graf, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Markus Armbruster, 2012/12/17
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers,
Andreas Färber <=
- Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Markus Armbruster, 2012/12/18
Re: [Qemu-devel] [PATCH 01/10] ide: Break all non-qdevified controllers, Peter Maydell, 2012/12/18
[Qemu-devel] [PATCH 07/10] ide: Drop redundant IDEState member version, Markus Armbruster, 2012/12/17
[Qemu-devel] [PATCH 04/10] ide: Don't block-align IDEState member smart_selftest_data, Markus Armbruster, 2012/12/17
[Qemu-devel] [PATCH 10/10] ide: Drop redundant IDEState member wwn, Markus Armbruster, 2012/12/17
[Qemu-devel] [PATCH 08/10] ide: Drop redundant IDEState member drive_serial_str, Markus Armbruster, 2012/12/17
[Qemu-devel] [PATCH 09/10] ide: Drop redundant IDEState member model, Markus Armbruster, 2012/12/17
[Qemu-devel] [PATCH 05/10] ide: Drop redundant IDEState member bs, Markus Armbruster, 2012/12/17