Re: [PULL 07/11] sysemu: tpm: Add a stub function for TPM_IS_CRB

2022-05-18 Thread Stefan Berger




On 5/18/22 21:17, Alex Williamson wrote:

On Wed, 18 May 2022 20:46:02 -0400
Stefan Berger  wrote:


On 5/6/22 16:48, Alex Williamson wrote:

From: Eric Auger 

In a subsequent patch, VFIO will need to recognize if
a memory region owner is a TPM CRB device. Hence VFIO
needs to use TPM_IS_CRB() even if CONFIG_TPM is unset. So
let's add a stub function.

Signed-off-by: Eric Auger 
Suggested-by: Cornelia Huck 
Reviewed-by: Stefan Berger 


Oops, a typo here: linnux -> linux


Yup, sorry, didn't notice until it was too late, but that was as
provided by you:


I know...



https://lore.kernel.org/all/96940f79-a6e0-d14f-5d74-abe280846...@linux.ibm.com/

Thanks,

Alex





Re: [PULL 07/11] sysemu: tpm: Add a stub function for TPM_IS_CRB

2022-05-18 Thread Alex Williamson
On Wed, 18 May 2022 20:46:02 -0400
Stefan Berger  wrote:

> On 5/6/22 16:48, Alex Williamson wrote:
> > From: Eric Auger 
> > 
> > In a subsequent patch, VFIO will need to recognize if
> > a memory region owner is a TPM CRB device. Hence VFIO
> > needs to use TPM_IS_CRB() even if CONFIG_TPM is unset. So
> > let's add a stub function.
> > 
> > Signed-off-by: Eric Auger 
> > Suggested-by: Cornelia Huck 
> > Reviewed-by: Stefan Berger   
> 
> Oops, a typo here: linnux -> linux

Yup, sorry, didn't notice until it was too late, but that was as
provided by you:

https://lore.kernel.org/all/96940f79-a6e0-d14f-5d74-abe280846...@linux.ibm.com/

Thanks,

Alex




Re: [PULL 07/11] sysemu: tpm: Add a stub function for TPM_IS_CRB

2022-05-18 Thread Stefan Berger




On 5/6/22 16:48, Alex Williamson wrote:

From: Eric Auger 

In a subsequent patch, VFIO will need to recognize if
a memory region owner is a TPM CRB device. Hence VFIO
needs to use TPM_IS_CRB() even if CONFIG_TPM is unset. So
let's add a stub function.

Signed-off-by: Eric Auger 
Suggested-by: Cornelia Huck 
Reviewed-by: Stefan Berger 


Oops, a typo here: linnux -> linux



[PULL 07/11] sysemu: tpm: Add a stub function for TPM_IS_CRB

2022-05-06 Thread Alex Williamson
From: Eric Auger 

In a subsequent patch, VFIO will need to recognize if
a memory region owner is a TPM CRB device. Hence VFIO
needs to use TPM_IS_CRB() even if CONFIG_TPM is unset. So
let's add a stub function.

Signed-off-by: Eric Auger 
Suggested-by: Cornelia Huck 
Reviewed-by: Stefan Berger 
Link: https://lore.kernel.org/r/20220506132510.1847942-2-eric.au...@redhat.com
Signed-off-by: Alex Williamson 
---
 include/sysemu/tpm.h |6 ++
 1 file changed, 6 insertions(+)

diff --git a/include/sysemu/tpm.h b/include/sysemu/tpm.h
index 68b2206463c5..fb40e30ff60e 100644
--- a/include/sysemu/tpm.h
+++ b/include/sysemu/tpm.h
@@ -80,6 +80,12 @@ static inline TPMVersion tpm_get_version(TPMIf *ti)
 #define tpm_init()  (0)
 #define tpm_cleanup()
 
+/* needed for an alignment check in non-tpm code */
+static inline Object *TPM_IS_CRB(Object *obj)
+{
+ return NULL;
+}
+
 #endif /* CONFIG_TPM */
 
 #endif /* QEMU_TPM_H */