[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL v3 02/11] tpm-backend: Remove unneeded member variabl
From: |
Stefan Berger |
Subject: |
[Qemu-devel] [PULL v3 02/11] tpm-backend: Remove unneeded member variable from backend class |
Date: |
Fri, 13 Oct 2017 07:44:51 -0400 |
From: Amarnath Valluri <address@hidden>
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>
Signed-off-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 b0a9731..3708413 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 2d830d0..abedf3f 100644
--- a/tpm.c
+++ b/tpm.c
@@ -211,7 +211,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.5.5
- [Qemu-devel] [PULL v3 00/11] Merge tpm 2017/10/04, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 01/11] tpm: Use EMSGSIZE instead of EBADMSG to compile on OpenBSD, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 08/11] tpm-passthrough: move reusable code to utils, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 02/11] tpm-backend: Remove unneeded member variable from backend class,
Stefan Berger <=
- [Qemu-devel] [PULL v3 04/11] tpm-backend: Initialize and free data members in it's own methods, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 07/11] tpm-backend: Move realloc_buffer() implementation to tpm-tis model, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 05/11] tpm-backend: Made few interface methods optional, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 06/11] tpm-backend: Add new API to read backend TpmInfo, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 09/11] tpm: Added support for TPM emulator, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 03/11] tpm-backend: Move thread handling inside TPMBackend, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 11/11] specs: Describe the TPM support in QEMU, Stefan Berger, 2017/10/13
- [Qemu-devel] [PULL v3 10/11] tpm: Move tpm_cleanup() to right place, Stefan Berger, 2017/10/13