On 11/10/2023 10:31 pm, Oleksii wrote: > On Tue, 2023-10-10 at 16:52 +0800, Andrew Cooper wrote: >> On 10/10/2023 3:58 pm, Oleksii wrote: >>>> Oleksii: I've not touched RISCV yet, because I don't know how the >>>> platforms >>>> typically look. I'm happy to default it active in RISCV too if >>>> that's the >>>> right thing to do. >>> We are still waiting for a platform with hypervisor extension >>> support >>> but I am using PDX in my Xen's repo. >> Yes, but are you using that because Xen wouldn't build without it, or >> because it's necessary for RISC-V platforms? > You are right I am using it because before this patch Xen can't be > built without PDX. > This is not necessary for RISC-V. At least platform I know they don't > use sparse RAM banks.
Ok thanks. I'll leave the code as is, but tweak the commit message to include RISC-V alongside x86. And as already noted, we can always revisit the decision in the future if things change. ~Andrew
