Re: [PATCH for-4.18 v5] xen/pdx: Make CONFIG_PDX_COMPRESSION a common Kconfig option

[Oleksii <oleksii.kurochko@xxxxxxxxx>]

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.

> 
> This patch fixes the problem where PDX had the illusion of being
> necessary, but was actually mandatory.
> 
> ~Andrew

~ Oleksii