[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/5] tpm: convert tpm_util.c to use trace-events
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 3/5] tpm: convert tpm_util.c to use trace-events |
Date: |
Sun, 4 Mar 2018 21:08:59 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 |
On 03/04/2018 06:56 PM, Stefan Berger wrote:
> Signed-off-by: Stefan Berger <address@hidden>
Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/tpm/tpm_util.c | 29 ++++++++++++-----------------
> hw/tpm/trace-events | 7 +++++++
> 2 files changed, 19 insertions(+), 17 deletions(-)
>
> diff --git a/hw/tpm/tpm_util.c b/hw/tpm/tpm_util.c
> index 2de52a0..ee41757 100644
> --- a/hw/tpm/tpm_util.c
> +++ b/hw/tpm/tpm_util.c
> @@ -28,14 +28,7 @@
> #include "exec/memory.h"
> #include "sysemu/tpm_backend.h"
> #include "hw/qdev.h"
> -
> -#define DEBUG_TPM 0
> -
> -#define DPRINTF(fmt, ...) do { \
> - if (DEBUG_TPM) { \
> - fprintf(stderr, "tpm-util:"fmt"\n", ## __VA_ARGS__); \
> - } \
> -} while (0)
> +#include "trace.h"
>
> /* tpm backend property */
>
> @@ -279,10 +272,11 @@ int tpm_util_get_buffer_size(int tpm_fd, TPMVersion
> tpm_version,
>
> if (be32_to_cpu(tpm_resp.hdr.len) != sizeof(tpm_resp) ||
> be32_to_cpu(tpm_resp.len) != sizeof(uint32_t)) {
> - DPRINTF("tpm_resp->hdr.len = %u, expected = %zu\n",
> - be32_to_cpu(tpm_resp.hdr.len), sizeof(tpm_resp));
> - DPRINTF("tpm_resp->len = %u, expected = %zu\n",
> - be32_to_cpu(tpm_resp.len), sizeof(uint32_t));
> + trace_tpm_util_get_buffer_size_hdr_len(
> + be32_to_cpu(tpm_resp.hdr.len),
> + sizeof(tpm_resp));
> + trace_tpm_util_get_buffer_size_len(be32_to_cpu(tpm_resp.len),
> + sizeof(uint32_t));
> error_report("tpm_util: Got unexpected response to "
> "TPM_GetCapability; errcode: 0x%x",
> be32_to_cpu(tpm_resp.hdr.errcode));
> @@ -327,10 +321,11 @@ int tpm_util_get_buffer_size(int tpm_fd, TPMVersion
> tpm_version,
>
> if (be32_to_cpu(tpm2_resp.hdr.len) != sizeof(tpm2_resp) ||
> be32_to_cpu(tpm2_resp.count) != 2) {
> - DPRINTF("tpm2_resp->hdr.len = %u, expected = %zu\n",
> - be32_to_cpu(tpm2_resp.hdr.len), sizeof(tpm2_resp));
> - DPRINTF("tpm2_resp->len = %u, expected = %u\n",
> - be32_to_cpu(tpm2_resp.count), 2);
> + trace_tpm_util_get_buffer_size_hdr_len2(
> + be32_to_cpu(tpm2_resp.hdr.len),
> + sizeof(tpm2_resp));
> + trace_tpm_util_get_buffer_size_len2(
> + be32_to_cpu(tpm2_resp.count), 2);
> error_report("tpm_util: Got unexpected response to "
> "TPM2_GetCapability; errcode: 0x%x",
> be32_to_cpu(tpm2_resp.hdr.errcode));
> @@ -344,7 +339,7 @@ int tpm_util_get_buffer_size(int tpm_fd, TPMVersion
> tpm_version,
> return -EFAULT;
> }
>
> - DPRINTF("buffersize of device: %zu\n", *buffersize);
> + trace_tpm_util_get_buffer_size(*buffersize);
>
> return 0;
> }
> diff --git a/hw/tpm/trace-events b/hw/tpm/trace-events
> index 8557dd9..66b2b91 100644
> --- a/hw/tpm/trace-events
> +++ b/hw/tpm/trace-events
> @@ -7,3 +7,10 @@ tpm_crb_mmio_write(uint64_t addr, unsigned size, uint32_t
> val) "CRB write 0x" TA
> # hw/tpm/tpm_passthrough.c
> tpm_passthrough_handle_request(void *cmd) "processing command %p"
> tpm_passthrough_reset(void) "reset"
> +
> +# hw/tpm/tpm_util.c
> +tpm_util_get_buffer_size_hdr_len(uint32_t len, size_t expected)
> "tpm_resp->hdr.len = %u, expected = %zu"
> +tpm_util_get_buffer_size_len(uint32_t len, size_t expected) "tpm_resp->len =
> %u, expected = %zu"
> +tpm_util_get_buffer_size_hdr_len2(uint32_t len, size_t expected)
> "tpm2_resp->hdr.len = %u, expected = %zu"
> +tpm_util_get_buffer_size_len2(uint32_t len, size_t expected) "tpm2_resp->len
> = %u, expected = %zu"
> +tpm_util_get_buffer_size(size_t len) "buffersize of device: %zu"
>
- [Qemu-devel] [PATCH 0/5] Convert the TPM code to use tracing, Stefan Berger, 2018/03/04
- [Qemu-devel] [PATCH 2/5] tpm: convert tpm_passthrough.c to use trace-events, Stefan Berger, 2018/03/04
- [Qemu-devel] [PATCH 3/5] tpm: convert tpm_util.c to use trace-events, Stefan Berger, 2018/03/04
- Re: [Qemu-devel] [PATCH 3/5] tpm: convert tpm_util.c to use trace-events,
Philippe Mathieu-Daudé <=
- [Qemu-devel] [PATCH 4/5] tpm: convert tpm_emulator.c to use trace-events, Stefan Berger, 2018/03/04
- [Qemu-devel] [PATCH 5/5] tpm: convert tpm_tis.c to use trace-events, Stefan Berger, 2018/03/04
- Re: [Qemu-devel] [PATCH 0/5] Convert the TPM code to use tracing, Marc-André Lureau, 2018/03/05
- Re: [Qemu-devel] [PATCH 0/5] Convert the TPM code to use tracing, Stefan Hajnoczi, 2018/03/06