da4680ce3a
Currently we track in the TableDesc and CmdQDesc structs the state of the GITS_BASER<n> and GITS_CBASER Valid bits. However we aren't very consistent abut checking the valid field: we test it in update_cte() and update_dte(), but not anywhere else we look things up in tables. The GIC specification says that it is UNPREDICTABLE if a guest fails to set any of these Valid bits before enabling the ITS via GITS_CTLR.Enabled. So we can choose to handle Valid == 0 as equivalent to a zero-length table. This is in fact how we're already catching this case in most of the table-access paths: when Valid is 0 we leave the num_entries fields in TableDesc or CmdQDesc set to zero, and then the out-of-bounds check "index >= num_entries" that we have to do anyway before doing any of these table lookups will always be true, catching the no-valid-table case without any extra code. So we can remove the checks on the valid field from update_cte() and update_dte(): since these happen after the bounds check there was never any case when the test could fail. That means the valid fields would be entirely unused, so just remove them. Signed-off-by: Peter Maydell <peter.maydell@linaro.org> Reviewed-by: Richard Henderson <richard.henderson@linaro.org> Message-id: 20220201193207.2771604-11-peter.maydell@linaro.org
108 lines
2.7 KiB
C
108 lines
2.7 KiB
C
/*
|
|
* ITS support for ARM GICv3
|
|
*
|
|
* Copyright (c) 2015 Samsung Electronics Co., Ltd.
|
|
* Written by Pavel Fedin
|
|
*
|
|
* This program is free software; you can redistribute it and/or modify
|
|
* it under the terms of the GNU General Public License as published by
|
|
* the Free Software Foundation, either version 2 of the License, or
|
|
* (at your option) any later version.
|
|
*
|
|
* This program is distributed in the hope that it will be useful,
|
|
* but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
|
|
* GNU General Public License for more details.
|
|
*
|
|
* You should have received a copy of the GNU General Public License along
|
|
* with this program; if not, see <http://www.gnu.org/licenses/>.
|
|
*/
|
|
|
|
#ifndef QEMU_ARM_GICV3_ITS_COMMON_H
|
|
#define QEMU_ARM_GICV3_ITS_COMMON_H
|
|
|
|
#include "hw/sysbus.h"
|
|
#include "hw/intc/arm_gicv3_common.h"
|
|
#include "qom/object.h"
|
|
|
|
#define TYPE_ARM_GICV3_ITS "arm-gicv3-its"
|
|
|
|
#define ITS_CONTROL_SIZE 0x10000
|
|
#define ITS_TRANS_SIZE 0x10000
|
|
#define ITS_SIZE (ITS_CONTROL_SIZE + ITS_TRANS_SIZE)
|
|
|
|
#define GITS_CTLR 0x0
|
|
#define GITS_IIDR 0x4
|
|
#define GITS_TYPER 0x8
|
|
#define GITS_CBASER 0x80
|
|
#define GITS_CWRITER 0x88
|
|
#define GITS_CREADR 0x90
|
|
#define GITS_BASER 0x100
|
|
|
|
#define GITS_TRANSLATER 0x0040
|
|
|
|
typedef struct {
|
|
bool indirect;
|
|
uint16_t entry_sz;
|
|
uint32_t page_sz;
|
|
uint32_t num_entries;
|
|
uint64_t base_addr;
|
|
} TableDesc;
|
|
|
|
typedef struct {
|
|
uint32_t num_entries;
|
|
uint64_t base_addr;
|
|
} CmdQDesc;
|
|
|
|
struct GICv3ITSState {
|
|
SysBusDevice parent_obj;
|
|
|
|
MemoryRegion iomem_main;
|
|
MemoryRegion iomem_its_cntrl;
|
|
MemoryRegion iomem_its_translation;
|
|
|
|
GICv3State *gicv3;
|
|
|
|
int dev_fd; /* kvm device fd if backed by kvm vgic support */
|
|
uint64_t gits_translater_gpa;
|
|
bool translater_gpa_known;
|
|
|
|
/* Registers */
|
|
uint32_t ctlr;
|
|
uint32_t iidr;
|
|
uint64_t typer;
|
|
uint64_t cbaser;
|
|
uint64_t cwriter;
|
|
uint64_t creadr;
|
|
uint64_t baser[8];
|
|
|
|
TableDesc dt;
|
|
TableDesc ct;
|
|
CmdQDesc cq;
|
|
|
|
Error *migration_blocker;
|
|
};
|
|
|
|
typedef struct GICv3ITSState GICv3ITSState;
|
|
|
|
void gicv3_its_init_mmio(GICv3ITSState *s, const MemoryRegionOps *ops,
|
|
const MemoryRegionOps *tops);
|
|
|
|
#define TYPE_ARM_GICV3_ITS_COMMON "arm-gicv3-its-common"
|
|
typedef struct GICv3ITSCommonClass GICv3ITSCommonClass;
|
|
DECLARE_OBJ_CHECKERS(GICv3ITSState, GICv3ITSCommonClass,
|
|
ARM_GICV3_ITS_COMMON, TYPE_ARM_GICV3_ITS_COMMON)
|
|
|
|
struct GICv3ITSCommonClass {
|
|
/*< private >*/
|
|
SysBusDeviceClass parent_class;
|
|
/*< public >*/
|
|
|
|
int (*send_msi)(GICv3ITSState *s, uint32_t data, uint16_t devid);
|
|
void (*pre_save)(GICv3ITSState *s);
|
|
void (*post_load)(GICv3ITSState *s);
|
|
};
|
|
|
|
|
|
#endif
|