[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtc
From: |
Amit Shah |
Subject: |
Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole |
Date: |
Fri, 4 Sep 2009 22:03:06 +0530 |
User-agent: |
Mutt/1.5.19 (2009-01-05) |
On (Fri) Sep 04 2009 [19:26:27], Blue Swirl wrote:
> On Fri, Sep 4, 2009 at 12:14 PM, Amit Shah<address@hidden> wrote:
> > Currently the VirtIOConsole struct is allocated from the call
> > to virtio_common_init, also doing an UP_CAST implicitly.
> >
> > The new multiport functionality will need a few arrays and
> > it's easier to move to the new VMState infrastructure by
> > keeping it all within one struct.
>
> > +VirtIOConsole virtconsole;
>
> IMHO this is going to wrong direction. What kind of code would need a
> static instance?
Adding multiple ports to the console device, we'll have to store an
array of ports here as well as config space. Both of these are
device-specific.
Amit
- [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Amit Shah, 2009/09/04
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Blue Swirl, 2009/09/04
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole,
Amit Shah <=
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Blue Swirl, 2009/09/04
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Amit Shah, 2009/09/04
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Blue Swirl, 2009/09/04
- Re: [Qemu-devel] [PATCH] virtio-console: Have a static instance of virtconsole, Amit Shah, 2009/09/04
- Message not available
- [Qemu-devel] Re: [PATCH] virtio-console: Have a static instance of virtconsole, Amit Shah, 2009/09/04
[Qemu-devel] Re: [PATCH] virtio-console: Have a static instance of virtconsole, Anthony Liguori, 2009/09/08