+Edgar

On Wed, 5 Mar 2025, Philippe Mathieu-Daudé wrote:
> On 16/6/23 01:52, Stefano Stabellini wrote:
> > From: Vikram Garhwal <vikram.garh...@amd.com>
> > 
> > Add a new machine xenpvh which creates a IOREQ server to register/connect
> > with
> > Xen Hypervisor.
> > 
> > Optional: When CONFIG_TPM is enabled, it also creates a tpm-tis-device, adds
> > a
> > TPM emulator and connects to swtpm running on host machine via chardev
> > socket
> > and support TPM functionalities for a guest domain.
> > 
> > Extra command line for aarch64 xenpvh QEMU to connect to swtpm:
> >      -chardev socket,id=chrtpm,path=/tmp/myvtpm2/swtpm-sock \
> >      -tpmdev emulator,id=tpm0,chardev=chrtpm \
> >      -machine tpm-base-addr=0x0c000000 \
> > 
> > swtpm implements a TPM software emulator(TPM 1.2 & TPM 2) built on libtpms
> > and
> > provides access to TPM functionality over socket, chardev and CUSE
> > interface.
> > Github repo: https://github.com/stefanberger/swtpm
> > Example for starting swtpm on host machine:
> >      mkdir /tmp/vtpm2
> >      swtpm socket --tpmstate dir=/tmp/vtpm2 \
> >      --ctrl type=unixio,path=/tmp/vtpm2/swtpm-sock &
> > 
> > Signed-off-by: Vikram Garhwal <vikram.garh...@amd.com>
> > Signed-off-by: Stefano Stabellini <stefano.stabell...@amd.com>
> > Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
> > ---
> >   docs/system/arm/xenpvh.rst    |  34 +++++++
> >   docs/system/target-arm.rst    |   1 +
> >   hw/arm/meson.build            |   2 +
> >   hw/arm/xen_arm.c              | 181 ++++++++++++++++++++++++++++++++++
> >   include/hw/arm/xen_arch_hvm.h |   9 ++
> >   include/hw/xen/arch_hvm.h     |   2 +
> >   6 files changed, 229 insertions(+)
> >   create mode 100644 docs/system/arm/xenpvh.rst
> >   create mode 100644 hw/arm/xen_arm.c
> >   create mode 100644 include/hw/arm/xen_arch_hvm.h
> 
> 
> > diff --git a/include/hw/xen/arch_hvm.h b/include/hw/xen/arch_hvm.h
> > index 26674648d8..c7c515220d 100644
> > --- a/include/hw/xen/arch_hvm.h
> > +++ b/include/hw/xen/arch_hvm.h
> > @@ -1,3 +1,5 @@
> >   #if defined(TARGET_I386) || defined(TARGET_X86_64)
> >   #include "hw/i386/xen_arch_hvm.h"
> > +#elif defined(TARGET_ARM) || defined(TARGET_ARM_64)
> 
> I suppose this is a typo and you meant TARGET_AARCH64 :)
> Luckily TARGET_AARCH64 defines TARGET_ARM.

Hi Philippe, it looks like this is an older patch.

Edgar, is this patch still valid?

Reply via email to