[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev |
Date: |
Tue, 15 Feb 2011 11:07:13 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.1 (gnu/linux) |
Anthony Liguori <address@hidden> writes:
> On 02/12/2011 11:03 AM, Markus Armbruster wrote:
>> Blue Swirl<address@hidden> writes:
>>
>>
>>> Convert to qdev, also add a proper reset function.
[...]
>> Pointer properties are for dirty hacks only. Is there really no better
>> solution? Why does it have to be a property?
>>
>
> vmmouse is really just an extension to the PS2 Mouse. It's definitely
> not an ISA device.
>
> In terms of qdev enablement, I would just make it a boolean option to
> the PS2Mouse and not expose it as a top level device at all. It
> cannot exist without a PS2Mouse.
Which means making it a separate qdev is wrong. That wrongness gave
rise to the dirty pointer property. Pointer property serves as canary
again.
What now?
PS: Grumpy reviewer venting: review can keep such mistakes out of the
code, but it got committed less than two days after it was posted.
That, and the lack of proper reference headers bounced it several places
down my review queue.
- [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/03
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Markus Armbruster, 2011/02/12
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/12
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Anthony Liguori, 2011/02/13
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/15
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/15
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Markus Armbruster, 2011/02/16
- Re: [Qemu-devel] [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/17
- [Qemu-devel] Re: [PATCH 06/10] vmmouse: convert to qdev, Paolo Bonzini, 2011/02/18
- [Qemu-devel] Re: [PATCH 06/10] vmmouse: convert to qdev, Blue Swirl, 2011/02/18