|
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [PATCH v1 14/27] xen/riscv: introduce per-vCPU IMSIC state
Each vCPU interacting with the IMSIC requires state to track the
associated guest interrupt file and its backing context.
Introduce a per-vCPU structure to hold IMSIC-related state, including
the guest interrupt file identifier and the CPU providing the backing
VS-file. Access to the guest file identifier is protected by a lock.
Initialize this structure during vCPU setup and store it in arch_vcpu.
The initial state marks the VS-file as software-backed until it becomes
associated with a physical CPU.
Add helpers to retrieve and update the guest interrupt file identifier.
Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
---
xen/arch/riscv/imsic.c | 42 +++++++++++++++++++++++++++++
xen/arch/riscv/include/asm/domain.h | 2 ++
xen/arch/riscv/include/asm/imsic.h | 17 ++++++++++++
3 files changed, 61 insertions(+)
diff --git a/xen/arch/riscv/imsic.c b/xen/arch/riscv/imsic.c
index 0956b187705f..bbadbdf352a1 100644
--- a/xen/arch/riscv/imsic.c
+++ b/xen/arch/riscv/imsic.c
@@ -59,6 +59,29 @@ do { \
csr_clear(CSR_SIREG, v); \
} while (0)
+unsigned int vcpu_guest_file_id(const struct vcpu *v)
+{
+ struct imsic_state *imsic_state = v->arch.imsic_state;
+ unsigned long flags;
+ unsigned int vsfile_id;
+
+ read_lock_irqsave(&imsic_state->vsfile_lock, flags);
+ vsfile_id = imsic_state->guest_file_id;
+ read_unlock_irqrestore(&imsic_state->vsfile_lock, flags);
+
+ return vsfile_id;
+}
+
+void imsic_set_guest_file_id(const struct vcpu *v, unsigned int guest_file_id)
+{
+ struct imsic_state *imsic_state = v->arch.imsic_state;
+ unsigned long flags;
+
+ write_lock_irqsave(&imsic_state->vsfile_lock, flags);
+ imsic_state->guest_file_id = guest_file_id;
+ write_unlock_irqrestore(&imsic_state->vsfile_lock, flags);
+}
+
void __init imsic_ids_local_delivery(bool enable)
{
if ( enable )
@@ -315,6 +338,25 @@ static int imsic_parse_node(const struct dt_device_node
*node,
return 0;
}
+int __init vcpu_imsic_init(struct vcpu *v)
+{
+ struct imsic_state *imsic_state;
+
+ /* Allocate IMSIC context */
+ imsic_state = xvzalloc(struct imsic_state);
+ if ( !imsic_state )
+ return -ENOMEM;
+
+ v->arch.imsic_state = imsic_state;
+
+ /* Setup IMSIC context */
+ rwlock_init(&imsic_state->vsfile_lock);
+
+ imsic_state->guest_file_id = imsic_state->vsfile_pcpu = NR_CPUS;
+
+ return 0;
+}
+
/*
* Initialize the imsic_cfg structure based on the IMSIC DT node.
*
diff --git a/xen/arch/riscv/include/asm/domain.h
b/xen/arch/riscv/include/asm/domain.h
index 506365f199c7..bdb1ffd748c9 100644
--- a/xen/arch/riscv/include/asm/domain.h
+++ b/xen/arch/riscv/include/asm/domain.h
@@ -52,6 +52,8 @@ struct arch_vcpu {
struct vtimer vtimer;
+ struct imsic_state *imsic_state;
+
register_t hcounteren;
register_t hedeleg;
register_t hideleg;
diff --git a/xen/arch/riscv/include/asm/imsic.h
b/xen/arch/riscv/include/asm/imsic.h
index a63d56fbd5d9..13a563dce066 100644
--- a/xen/arch/riscv/include/asm/imsic.h
+++ b/xen/arch/riscv/include/asm/imsic.h
@@ -11,6 +11,7 @@
#ifndef ASM_RISCV_IMSIC_H
#define ASM_RISCV_IMSIC_H
+#include <xen/rwlock.h>
#include <xen/spinlock.h>
#include <xen/stdbool.h>
#include <xen/types.h>
@@ -64,8 +65,20 @@ struct imsic_config {
spinlock_t lock;
};
+struct imsic_state {
+ /* IMSIC VS-file */
+ rwlock_t vsfile_lock;
+ unsigned int guest_file_id;
+ /*
+ * (vsfile_pcpu >= 0) => h/w IMSIC VS-file
+ * (vsfile_pcpu == NR_CPUS) => s/w IMSIC SW-file
+ */
+ unsigned long vsfile_pcpu;
+};
+
struct dt_device_node;
struct kernel_info;
+struct vcpu;
int imsic_init(const struct dt_device_node *node);
@@ -78,4 +91,8 @@ void imsic_ids_local_delivery(bool enable);
int imsic_make_dt_node(const struct kernel_info *kinfo);
+int vcpu_imsic_init(struct vcpu *v);
+unsigned int vcpu_guest_file_id(const struct vcpu *v);
+void imsic_set_guest_file_id(const struct vcpu *v, unsigned int guest_file_id);
+
#endif /* ASM_RISCV_IMSIC_H */
--
2.53.0
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |