[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [RFC 0/9] QOM: qomify -netdev
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [RFC 0/9] QOM: qomify -netdev |
Date: |
Mon, 26 Mar 2012 12:54:57 +0100 |
On Mon, Mar 26, 2012 at 6:40 AM, <address@hidden> wrote:
> From: Zhi Yong Wu <address@hidden>
>
> Sending the patchset is mainly intended to get some comments and void the
> wrong development direction.
>
> The patchset is used to qomify -netdev, but it introduce one infrastructure
> for host devices based on raw Class and Object, not qdev. So they are not
> related with DeviceClass and DeviceState.
>
> patch #1 introduce one new class and object for host devices.
A common infrastructure for host devices is useful. The Property
mechanism in hw/qdev-properties.c is especially good. I think host
devices will also need to 2-step creation process (init and realize).
I think the qdev-properties.c code can mostly be shared. It currently
uses the qdev DeviceState class but I think it only really access the
Property array.
Therefore it should be possible to really share a single copy of the
code. We don't need to duplicate Property for host devices, instead
we can extract it out of hw/.
Stefan
- [Qemu-devel] [RFC 2/9] net: introduce one net host device class, (continued)
- [Qemu-devel] [RFC 2/9] net: introduce one net host device class, zwu . kernel, 2012/03/26
- [Qemu-devel] [RFC 6/9] net: qomify -netdev user, zwu . kernel, 2012/03/26
- [Qemu-devel] [PATCH] net: qomify -netdev, zwu . kernel, 2012/03/26
- [Qemu-devel] [RFC 7/9] net: qomify -netdev socket, zwu . kernel, 2012/03/26
- [Qemu-devel] [RFC 8/9] net: qomify -netdev vde, zwu . kernel, 2012/03/26
- [Qemu-devel] [RFC 5/9] net: adjust dump init API, zwu . kernel, 2012/03/26
- [Qemu-devel] [RFC 9/9] net: qomify -netdev tap & -netdev bridge, zwu . kernel, 2012/03/26
- Re: [Qemu-devel] [RFC 0/9] QOM: qomify -netdev,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [RFC 0/9] QOM: qomify -netdev, Andreas Färber, 2012/03/26