[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v5 1/8] tpm-backend: Remove unneeded member variable
From: |
Amarnath Valluri |
Subject: |
[Qemu-devel] [PATCH v5 1/8] tpm-backend: Remove unneeded member variable from backend class |
Date: |
Mon, 5 Jun 2017 15:45:04 +0300 |
TPMDriverOps inside TPMBackend is not required, as it is supposed to be a class
member. The only possible reason for keeping in TPMBackend was, to get the
backend type in tpm.c where dedicated backend api, tpm_backend_get_type() is
present.
Signed-off-by: Amarnath Valluri <address@hidden>
Reviewed-by: Marc-André Lureau <address@hidden>
Reviewed-by: Stefan Berger <address@hidden>
---
hw/tpm/tpm_passthrough.c | 4 ----
include/sysemu/tpm_backend.h | 1 -
tpm.c | 2 +-
3 files changed, 1 insertion(+), 6 deletions(-)
diff --git a/hw/tpm/tpm_passthrough.c b/hw/tpm/tpm_passthrough.c
index 9234eb3..a0baf5f 100644
--- a/hw/tpm/tpm_passthrough.c
+++ b/hw/tpm/tpm_passthrough.c
@@ -46,8 +46,6 @@
#define TPM_PASSTHROUGH(obj) \
OBJECT_CHECK(TPMPassthruState, (obj), TYPE_TPM_PASSTHROUGH)
-static const TPMDriverOps tpm_passthrough_driver;
-
/* data structures */
typedef struct TPMPassthruThreadParams {
TPMState *tpm_state;
@@ -462,8 +460,6 @@ static TPMBackend *tpm_passthrough_create(QemuOpts *opts,
const char *id)
/* let frontend set the fe_model to proper value */
tb->fe_model = -1;
- tb->ops = &tpm_passthrough_driver;
-
if (tpm_passthrough_handle_device_opts(opts, tb)) {
goto err_exit;
}
diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h
index b58f52d..e7f590d 100644
--- a/include/sysemu/tpm_backend.h
+++ b/include/sysemu/tpm_backend.h
@@ -50,7 +50,6 @@ struct TPMBackend {
enum TpmModel fe_model;
char *path;
char *cancel_path;
- const TPMDriverOps *ops;
QLIST_ENTRY(TPMBackend) list;
};
diff --git a/tpm.c b/tpm.c
index 9a7c711..0ee021a 100644
--- a/tpm.c
+++ b/tpm.c
@@ -258,7 +258,7 @@ static TPMInfo *qmp_query_tpm_inst(TPMBackend *drv)
res->model = drv->fe_model;
res->options = g_new0(TpmTypeOptions, 1);
- switch (drv->ops->type) {
+ switch (tpm_backend_get_type(drv)) {
case TPM_TYPE_PASSTHROUGH:
res->options->type = TPM_TYPE_OPTIONS_KIND_PASSTHROUGH;
tpo = g_new0(TPMPassthroughOptions, 1);
--
2.7.4
- [Qemu-devel] [PATCH v4 0/8] Provide support for the software TPM emulator, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 1/8] tpm-backend: Remove unneeded member variable from backend class,
Amarnath Valluri <=
- [Qemu-devel] [PATCH v5 3/8] tpm-backend: Initialize and free data members in it's own methods, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 2/8] tpm-backend: Move thread handling inside TPMBackend, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 4/8] tpm-backend: Made few interface methods optional, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 5/8] tmp backend: Add new api to read backend TpmInfo, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 6/8] tpm-backend: Move realloc_buffer() implementation to tpm-tis model, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 7/8] tpm-passthrough: move reusable code to utils, Amarnath Valluri, 2017/06/05
- [Qemu-devel] [PATCH v5 8/8] tpm: Added support for TPM emulator, Amarnath Valluri, 2017/06/05