Hi, > On Oct 12, 2023, at 01:25, Andrew Cooper <[email protected]> wrote: > > 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.
Looks like the discussion has been settled so: Release-acked-by: Henry Wang <[email protected]> Kind regards, Henry > > ~Andrew
