[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command
From: |
Gleb Natapov |
Subject: |
Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command |
Date: |
Wed, 30 Dec 2009 17:53:27 +0200 |
On Wed, Dec 30, 2009 at 05:04:08PM +0200, Avi Kivity wrote:
> On 12/30/2009 01:50 PM, Gleb Natapov wrote:
> >Knowing ioapic configuration is very useful for the poor soles
> >how need to debug guest occasionally.
> >+
> >+void do_info_ioapic(Monitor *mon, QObject **ret_data)
> >+{
> >+ int i;
> >+ QList *list;
> >+
> >+ *ret_data = NULL;
> >+
> >+ if (!ioapic)
> >+ return;
> >+
> >+ list = qlist_new();
> >+
> >+ for (i = 0; i< IOAPIC_NUM_PINS; i++) {
> >+ QObject *obj;
>
>
> This assumes there is only one ioapic. While I don't think there's
> a good reason to add more (the one we have is causing sufficient
> trouble), I suggest returning an array of ioapics (or include
> gsibase: [{ ioapic: object, gsibase: 0 }, { ioapic: object, gsibase:
> 24 }].
>
For the case of multiple ioapics I thought about extending the command
to get ioapic id as a parameter when time comes. gsibase is not know to
ioapic itself, so this info does not belong here.
--
Gleb.
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, (continued)
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Gleb Natapov, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Anthony Liguori, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Gleb Natapov, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Anthony Liguori, 2009/12/31
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Gleb Natapov, 2009/12/31
Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Luiz Capitulino, 2009/12/30
[Qemu-devel] Re: [PATCHv2] add "info ioapic" monitor command, Luiz Capitulino, 2009/12/30
Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Avi Kivity, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command,
Gleb Natapov <=
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Avi Kivity, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Anthony Liguori, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Gleb Natapov, 2009/12/30
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Anthony Liguori, 2009/12/31
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Gleb Natapov, 2009/12/31
- Re: [Qemu-devel] [PATCHv2] add "info ioapic" monitor command, Anthony Liguori, 2009/12/31