[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI suppor
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [Qemu-ppc] [PATCH 16/19] openpic: add Shared MSI support |
Date: |
Tue, 11 Dec 2012 09:10:14 +0100 |
On 11.12.2012, at 01:36, Scott Wood <address@hidden> wrote:
> On 12/08/2012 07:44:39 AM, Alexander Graf wrote:
>> The OpenPIC allows MSI access through shared MSI registers. Implement
>> them for the MPC8544 MPIC, so we can support MSIs.
>> Signed-off-by: Alexander Graf <address@hidden>
>> ---
>> hw/openpic.c | 150
>> ++++++++++++++++++++++++++++++++++++++++++++++++++--------
>> 1 files changed, 130 insertions(+), 20 deletions(-)
>> diff --git a/hw/openpic.c b/hw/openpic.c
>> index f2f152f..f71d668 100644
>> --- a/hw/openpic.c
>> +++ b/hw/openpic.c
>> @@ -38,6 +38,7 @@
>> #include "pci.h"
>> #include "openpic.h"
>> #include "sysbus.h"
>> +#include "msi.h"
>> //#define DEBUG_OPENPIC
>> @@ -52,6 +53,7 @@
>> #define MAX_TMR 4
>> #define VECTOR_BITS 8
>> #define MAX_IPI 4
>> +#define MAX_MSI 8
>> #define VID 0x03 /* MPIC version ID */
>> /* OpenPIC capability flags */
>> @@ -62,6 +64,8 @@
>> #define OPENPIC_GLB_REG_SIZE 0x10F0
>> #define OPENPIC_TMR_REG_START 0x10F0
>> #define OPENPIC_TMR_REG_SIZE 0x220
>> +#define OPENPIC_MSI_REG_START 0x1600
>> +#define OPENPIC_MSI_REG_SIZE 0x200
>> #define OPENPIC_SRC_REG_START 0x10000
>> #define OPENPIC_SRC_REG_SIZE (MAX_IRQ * 0x20)
>> #define OPENPIC_CPU_REG_START 0x20000
>> @@ -126,6 +130,12 @@
>> #define IDR_P1_SHIFT 1
>> #define IDR_P0_SHIFT 0
>> +#define MSIIR_OFFSET 0x140
>> +#define MSIIR_SRS_SHIFT 29
>> +#define MSIIR_SRS_MASK (0x7 << MSIIR_SRS_SHIFT)
>> +#define MSIIR_IBS_SHIFT 24
>> +#define MSIIR_IBS_MASK (0x1f << MSIIR_IBS_SHIFT)
>
> FWIW, if you want to model newer MPICs such as on p4080, they have multiple
> banks of MSIs, so you may not want to hardcode one bank.
The OpenPIC code was suffering a lot from attempts to generalize different
implementations without implementing them.
If we want to add support for p4080 MPICs later, we add a new model to the
emulation and make the nr of msi banks a parameter, like the patch set does for
all the other raven/mpc8544 differences. That way we don't get into the current
mess of a halfway accurate emulation unless we really want it.
Alex
- [Qemu-devel] [PATCH 05/19] openpic: Convert subregions to memory api, (continued)
- [Qemu-devel] [PATCH 05/19] openpic: Convert subregions to memory api, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 11/19] openpic: convert simple reg operations to builtin bitops, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 18/19] PPC: e500: Declare pci bridge as bridge, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 17/19] PPC: e500: Add MSI support, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 01/19] openpic: Remove unused code, Alexander Graf, 2012/12/08
- [Qemu-devel] [PATCH 16/19] openpic: add Shared MSI support, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 12/19] openpic: rename openpic_t to OpenPICState, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 08/19] openpic: combine openpic and mpic reset functions, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 09/19] openpic: unify memory api subregions, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 06/19] openpic: combine mpic and openpic irq raise functions, Alexander Graf, 2012/12/08
[Qemu-devel] [PATCH 02/19] mpic: Unify numbering scheme, Alexander Graf, 2012/12/08