Skip to content

Commit 9f9e3eb

Browse files
Andrew Jonesavpatel
authored andcommitted
RISC-V: KVM: Convert extension_disabled[] to ext_status[]
Change the boolean extension_disabled[] array to an array of enums, ext_status[]. For now, the enum only has two states, which correspond to the previous boolean states, so this patch has no intended functional change. The next patch will add another state, expanding the purpose of ext_status[]. Signed-off-by: Andrew Jones <ajones@ventanamicro.com> Reviewed-by: Anup Patel <anup@brainfault.org> Signed-off-by: Anup Patel <anup@brainfault.org>
1 parent ae328da commit 9f9e3eb

2 files changed

Lines changed: 12 additions & 4 deletions

File tree

arch/riscv/include/asm/kvm_vcpu_sbi.h

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,14 @@
1414
#define KVM_SBI_VERSION_MAJOR 1
1515
#define KVM_SBI_VERSION_MINOR 0
1616

17+
enum kvm_riscv_sbi_ext_status {
18+
KVM_RISCV_SBI_EXT_AVAILABLE,
19+
KVM_RISCV_SBI_EXT_UNAVAILABLE,
20+
};
21+
1722
struct kvm_vcpu_sbi_context {
1823
int return_handled;
19-
bool extension_disabled[KVM_RISCV_SBI_EXT_MAX];
24+
enum kvm_riscv_sbi_ext_status ext_status[KVM_RISCV_SBI_EXT_MAX];
2025
};
2126

2227
struct kvm_vcpu_sbi_return {

arch/riscv/kvm/vcpu_sbi.c

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,8 @@ static int riscv_vcpu_set_sbi_ext_single(struct kvm_vcpu *vcpu,
155155
if (!sext)
156156
return -ENOENT;
157157

158-
scontext->extension_disabled[sext->ext_idx] = !reg_val;
158+
scontext->ext_status[sext->ext_idx] = reg_val ?
159+
KVM_RISCV_SBI_EXT_AVAILABLE : KVM_RISCV_SBI_EXT_UNAVAILABLE;
159160

160161
return 0;
161162
}
@@ -180,7 +181,8 @@ static int riscv_vcpu_get_sbi_ext_single(struct kvm_vcpu *vcpu,
180181
if (!sext)
181182
return -ENOENT;
182183

183-
*reg_val = !scontext->extension_disabled[sext->ext_idx];
184+
*reg_val = scontext->ext_status[sext->ext_idx] ==
185+
KVM_RISCV_SBI_EXT_AVAILABLE;
184186

185187
return 0;
186188
}
@@ -316,7 +318,8 @@ const struct kvm_vcpu_sbi_extension *kvm_vcpu_sbi_find_ext(
316318
if (sext->ext_ptr->extid_start <= extid &&
317319
sext->ext_ptr->extid_end >= extid) {
318320
if (sext->ext_idx < KVM_RISCV_SBI_EXT_MAX &&
319-
scontext->extension_disabled[sext->ext_idx])
321+
scontext->ext_status[sext->ext_idx] ==
322+
KVM_RISCV_SBI_EXT_UNAVAILABLE)
320323
return NULL;
321324
return sbi_ext[i].ext_ptr;
322325
}

0 commit comments

Comments
 (0)