From: "Cédric Le Goater" <clg@kaod.org>
To: David Gibson <david@gibson.dropbear.id.au>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, Greg Kurz <groug@kaod.org>
Subject: Re: [Qemu-devel] [PATCH v3 1/3] spapr: introduce a fixed IRQ number space
Date: Mon, 2 Jul 2018 12:03:48 +0200 [thread overview]
Message-ID: <eb1ad8ca-89f7-b059-3a3c-412375871113@kaod.org> (raw)
In-Reply-To: <20180619140729.21949-2-clg@kaod.org>
> --- a/hw/ppc/spapr_vio.c
> +++ b/hw/ppc/spapr_vio.c
> @@ -436,6 +436,9 @@ static void spapr_vio_busdev_reset(DeviceState *qdev)
> }
> }
>
> +/* TODO : poor VIO device indexing ... */
> +static uint32_t vio_index;
I think we could also use (dev->reg & 0xff) as an index for
the VIO devices.
The unit address of the virtual IOA is simply allocated using
an increment of bus->next_reg, next_reg being initialized at
0x71000000.
I did not see any restrictions in the PAPR specs or in QEMU
that would break the above.
C.
> static void spapr_vio_busdev_realize(DeviceState *qdev, Error **errp)
> {
> sPAPRMachineState *spapr = SPAPR_MACHINE(qdev_get_machine());
> @@ -476,10 +479,18 @@ static void spapr_vio_busdev_realize(DeviceState *qdev, Error **errp)
> }
>
> if (!dev->irq) {
> - dev->irq = spapr_irq_findone(spapr, &local_err);
> - if (local_err) {
> - error_propagate(errp, local_err);
> - return;
> + if (SPAPR_MACHINE_GET_CLASS(spapr)->legacy_irq_allocation) {
> + dev->irq = spapr_irq_findone(spapr, &local_err);
> + if (local_err) {
> + error_propagate(errp, local_err);
> + return;
> + }
> + } else {
> + dev->irq = SPAPR_IRQ_VIO + vio_index++;
> + if (dev->irq == SPAPR_IRQ_PCI_LSI) {
> + error_setg(errp, "Too many VIO devices");
> + return;
> + }
next prev parent reply other threads:[~2018-07-02 10:04 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-19 14:07 [Qemu-devel] [PATCH v3 0/3] spapr: introduce a fixed IRQ number space and an IRQ controller backend Cédric Le Goater
2018-06-19 14:07 ` [Qemu-devel] [PATCH v3 1/3] spapr: introduce a fixed IRQ number space Cédric Le Goater
2018-07-02 10:03 ` Cédric Le Goater [this message]
2018-07-02 11:11 ` Cédric Le Goater
2018-07-03 15:19 ` Cédric Le Goater
2018-07-04 8:13 ` Greg Kurz
2018-07-06 5:44 ` David Gibson
2018-07-06 6:29 ` Cédric Le Goater
2018-07-06 7:40 ` Cédric Le Goater
2018-07-06 7:46 ` [Qemu-devel] [Qemu-ppc] " Cédric Le Goater
2018-06-19 14:07 ` [Qemu-devel] [PATCH v3 2/3] spapr: introduce a IRQ controller backend to the machine Cédric Le Goater
2018-06-19 14:07 ` [Qemu-devel] [PATCH v3 3/3] spapr: increase the size of the IRQ number space Cédric Le Goater
2018-06-19 15:16 ` [Qemu-devel] [PATCH v3 3/3 bonus] spapr: remove the XICS header from the machine and device models Cédric Le Goater
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=eb1ad8ca-89f7-b059-3a3c-412375871113@kaod.org \
--to=clg@kaod.org \
--cc=david@gibson.dropbear.id.au \
--cc=groug@kaod.org \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.