On Thu, 10 Nov 2022 19:02:45 +0000, Will Deacon <will@xxxxxxxxxx> wrote: > > From: Fuad Tabba <tabba@xxxxxxxxxx> > > Introduce a global table (and lock) to track pKVM instances at EL2, and > provide hypercalls that can be used by the untrusted host to create and > destroy pKVM VMs and their vCPUs. pKVM VM/vCPU state is directly > accessible only by the trusted hypervisor (EL2). > > Each pKVM VM is directly associated with an untrusted host KVM instance, > and is referenced by the host using an opaque handle. Future patches > will provide hypercalls to allow the host to initialize/set/get pKVM > VM/vCPU state using the opaque handle. > > Tested-by: Vincent Donnefort <vdonnefort@xxxxxxxxxx> > Signed-off-by: Fuad Tabba <tabba@xxxxxxxxxx> > Co-developed-by: Will Deacon <will@xxxxxxxxxx> > Signed-off-by: Will Deacon <will@xxxxxxxxxx> [...] > +static void unmap_donated_memory_noclear(void *va, size_t size) > +{ > + if (!va) > + return; > + > + __unmap_donated_memory(va, size); > +} This triggers a warning that persist until patch #19 (unused static function) and breaks my build (I've enabled CONFIG_WERROR). I'm going to stick a __maybe_unused there, and drop it once it is used... M. -- Without deviation from the norm, progress is not possible.