Re: [PATCH v1 3/6] soc: mediatek: virt: geniezone: Introduce GenieZone hypervisor support

[Matthias Brugger]

On 13/04/2023 14:55, Krzysztof Kozlowski wrote:
> On 13/04/2023 11:07, Yi-De Wu wrote:
> > From: "Yingshiuan Pan" <yingshiuan.pan@xxxxxxxxxxxx>
> >
> > GenieZone is MediaTek proprietary hypervisor solution, and it is running
> > in EL2 stand alone as a type-I hypervisor. This patch exports a set of
> > ioctl interfaces for userspace VMM (e.g., crosvm) to operate guest VMs
> > lifecycle (creation, running, and destroy) on GenieZone.
> >
> > Signed-off-by: Yingshiuan Pan <yingshiuan.pan@xxxxxxxxxxxx>
> > Signed-off-by: Yi-De Wu <yi-de.wu@xxxxxxxxxxxx>
> > ---
> >   arch/arm64/include/uapi/asm/gzvm_arch.h       |  79 ++++
> >   drivers/soc/mediatek/Kconfig                  |   2 +
> >   drivers/soc/mediatek/Makefile                 |   1 +
> >   drivers/soc/mediatek/virt/geniezone/Kconfig   |  17 +
>
> Hypervisor drivers do not go to soc. Stop shoving there everything from
> your downstream. Find appropriate directory, e.g. maybe drivers/virt.

Acked, what is the reason you want to add this to drivers/soc instead of 
drivers/virt?

Regards,
Matthias

> See:
> https://lore.kernel.org/all/20230304010632.2127470-1-quic_eberman@xxxxxxxxxxx/
>
> You should follow that discussion as well and be sure that all concerns
> raised for Gunyah are solved also here.
>
> Best regards,
> Krzysztof