Message ID | 20171009225623.29232-28-marcandre.lureau@redhat.com |
---|---|
State | New |
Headers | show |
Series | TPM: code cleanup & CRB device | expand |
On 10/09/2017 06:56 PM, Marc-André Lureau wrote: > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> > --- > include/sysemu/tpm_backend.h | 12 ------------ > backends/tpm.c | 42 ------------------------------------------ > tpm.c | 6 ------ > 3 files changed, 60 deletions(-) > > diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h > index b12ae5b625..a893e586ae 100644 > --- a/include/sysemu/tpm_backend.h > +++ b/include/sysemu/tpm_backend.h > @@ -81,8 +81,6 @@ struct TPMBackendClass { > > TpmTypeOptions *(*get_tpm_options)(TPMBackend *t); > > - void (*opened)(TPMBackend *s, Error **errp); > - > void (*handle_request)(TPMBackend *s, TPMBackendCmd *cmd); > }; > > @@ -172,16 +170,6 @@ bool tpm_backend_get_tpm_established_flag(TPMBackend *s); > */ > int tpm_backend_reset_tpm_established_flag(TPMBackend *s, uint8_t locty); > > -/** > - * tpm_backend_open: > - * @s: the backend to open > - * @errp: a pointer to return the #Error object if an error occurs. > - * > - * This function will open the backend if it is not already open. Calling this > - * function on an already opened backend will not result in an error. > - */ > -void tpm_backend_open(TPMBackend *s, Error **errp); > - > /** > * tpm_backend_get_tpm_version: > * @s: the backend to call into > diff --git a/backends/tpm.c b/backends/tpm.c > index 0c48d18775..7e636fbc7a 100644 > --- a/backends/tpm.c > +++ b/backends/tpm.c > @@ -159,52 +159,10 @@ TPMInfo *tpm_backend_query_tpm(TPMBackend *s) > return info; > } > > -static bool tpm_backend_prop_get_opened(Object *obj, Error **errp) > -{ > - TPMBackend *s = TPM_BACKEND(obj); > - > - return s->opened; > -} > - > -void tpm_backend_open(TPMBackend *s, Error **errp) > -{ > - object_property_set_bool(OBJECT(s), true, "opened", errp); > -} > - > -static void tpm_backend_prop_set_opened(Object *obj, bool value, Error **errp) > -{ > - TPMBackend *s = TPM_BACKEND(obj); > - TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s); > - Error *local_err = NULL; > - > - if (value == s->opened) { > - return; > - } > - > - if (!value && s->opened) { > - error_setg(errp, QERR_PERMISSION_DENIED); > - return; > - } > - > - if (k->opened) { > - k->opened(s, &local_err); > - if (local_err) { > - error_propagate(errp, local_err); > - return; > - } > - } > - > - s->opened = true; > -} > - > static void tpm_backend_instance_init(Object *obj) > { > TPMBackend *s = TPM_BACKEND(obj); > > - object_property_add_bool(obj, "opened", > - tpm_backend_prop_get_opened, > - tpm_backend_prop_set_opened, > - NULL); > s->bh = qemu_bh_new(tpm_backend_request_completed_bh, s); > } > > diff --git a/tpm.c b/tpm.c > index ce1543fcb4..a46ee5f144 100644 > --- a/tpm.c > +++ b/tpm.c > @@ -134,12 +134,6 @@ static int tpm_init_tpmdev(void *dummy, QemuOpts *opts, Error **errp) > return 1; > } > > - tpm_backend_open(drv, &local_err); > - if (local_err) { > - error_report_err(local_err); > - return 1; > - } > - > QLIST_INSERT_HEAD(&tpm_backends, drv, list); > > return 0; Since nothing is setting the "opened' anymore, would anyone notice because this has changed or is something else doing that now? Otherwise I don't mind removing it... Stefan
Hi ----- Original Message ----- > On 10/09/2017 06:56 PM, Marc-André Lureau wrote: > > Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> > > --- > > include/sysemu/tpm_backend.h | 12 ------------ > > backends/tpm.c | 42 > > ------------------------------------------ > > tpm.c | 6 ------ > > 3 files changed, 60 deletions(-) > > > > diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h > > index b12ae5b625..a893e586ae 100644 > > --- a/include/sysemu/tpm_backend.h > > +++ b/include/sysemu/tpm_backend.h > > @@ -81,8 +81,6 @@ struct TPMBackendClass { > > > > TpmTypeOptions *(*get_tpm_options)(TPMBackend *t); > > > > - void (*opened)(TPMBackend *s, Error **errp); > > - > > void (*handle_request)(TPMBackend *s, TPMBackendCmd *cmd); > > }; > > > > @@ -172,16 +170,6 @@ bool tpm_backend_get_tpm_established_flag(TPMBackend > > *s); > > */ > > int tpm_backend_reset_tpm_established_flag(TPMBackend *s, uint8_t locty); > > > > -/** > > - * tpm_backend_open: > > - * @s: the backend to open > > - * @errp: a pointer to return the #Error object if an error occurs. > > - * > > - * This function will open the backend if it is not already open. Calling > > this > > - * function on an already opened backend will not result in an error. > > - */ > > -void tpm_backend_open(TPMBackend *s, Error **errp); > > - > > /** > > * tpm_backend_get_tpm_version: > > * @s: the backend to call into > > diff --git a/backends/tpm.c b/backends/tpm.c > > index 0c48d18775..7e636fbc7a 100644 > > --- a/backends/tpm.c > > +++ b/backends/tpm.c > > @@ -159,52 +159,10 @@ TPMInfo *tpm_backend_query_tpm(TPMBackend *s) > > return info; > > } > > > > -static bool tpm_backend_prop_get_opened(Object *obj, Error **errp) > > -{ > > - TPMBackend *s = TPM_BACKEND(obj); > > - > > - return s->opened; > > -} > > - > > -void tpm_backend_open(TPMBackend *s, Error **errp) > > -{ > > - object_property_set_bool(OBJECT(s), true, "opened", errp); > > -} > > - > > -static void tpm_backend_prop_set_opened(Object *obj, bool value, Error > > **errp) > > -{ > > - TPMBackend *s = TPM_BACKEND(obj); > > - TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s); > > - Error *local_err = NULL; > > - > > - if (value == s->opened) { > > - return; > > - } > > - > > - if (!value && s->opened) { > > - error_setg(errp, QERR_PERMISSION_DENIED); > > - return; > > - } > > - > > - if (k->opened) { > > - k->opened(s, &local_err); > > - if (local_err) { > > - error_propagate(errp, local_err); > > - return; > > - } > > - } > > - > > - s->opened = true; > > -} > > - > > static void tpm_backend_instance_init(Object *obj) > > { > > TPMBackend *s = TPM_BACKEND(obj); > > > > - object_property_add_bool(obj, "opened", > > - tpm_backend_prop_get_opened, > > - tpm_backend_prop_set_opened, > > - NULL); > > s->bh = qemu_bh_new(tpm_backend_request_completed_bh, s); > > } > > > > diff --git a/tpm.c b/tpm.c > > index ce1543fcb4..a46ee5f144 100644 > > --- a/tpm.c > > +++ b/tpm.c > > @@ -134,12 +134,6 @@ static int tpm_init_tpmdev(void *dummy, QemuOpts > > *opts, Error **errp) > > return 1; > > } > > > > - tpm_backend_open(drv, &local_err); > > - if (local_err) { > > - error_report_err(local_err); > > - return 1; > > - } > > - > > QLIST_INSERT_HEAD(&tpm_backends, drv, list); > > > > return 0; > > Since nothing is setting the "opened' anymore, would anyone notice > because this has changed or is something else doing that now? Otherwise > I don't mind removing it... Was it ever used? I see the property was added in commit 8f0605cc9caacbcc647a6df9ae541ed2da4b9bb0. Code adapted from rng backend. But rng backend set "opened" on UserCreatableClass.complete() (TPM backend doesn't, but probably could - I have an experimental patch for that) I think we can remove it that code safely.
On 10/10/2017 06:27 PM, Marc-André Lureau wrote: > Hi > > ----- Original Message ----- >> On 10/09/2017 06:56 PM, Marc-André Lureau wrote: >>> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> >>> --- >>> include/sysemu/tpm_backend.h | 12 ------------ >>> backends/tpm.c | 42 >>> ------------------------------------------ >>> tpm.c | 6 ------ >>> 3 files changed, 60 deletions(-) >>> >>> diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h >>> index b12ae5b625..a893e586ae 100644 >>> --- a/include/sysemu/tpm_backend.h >>> +++ b/include/sysemu/tpm_backend.h >>> @@ -81,8 +81,6 @@ struct TPMBackendClass { >>> >>> TpmTypeOptions *(*get_tpm_options)(TPMBackend *t); >>> >>> - void (*opened)(TPMBackend *s, Error **errp); >>> - >>> void (*handle_request)(TPMBackend *s, TPMBackendCmd *cmd); >>> }; >>> >>> @@ -172,16 +170,6 @@ bool tpm_backend_get_tpm_established_flag(TPMBackend >>> *s); >>> */ >>> int tpm_backend_reset_tpm_established_flag(TPMBackend *s, uint8_t locty); >>> >>> -/** >>> - * tpm_backend_open: >>> - * @s: the backend to open >>> - * @errp: a pointer to return the #Error object if an error occurs. >>> - * >>> - * This function will open the backend if it is not already open. Calling >>> this >>> - * function on an already opened backend will not result in an error. >>> - */ >>> -void tpm_backend_open(TPMBackend *s, Error **errp); >>> - >>> /** >>> * tpm_backend_get_tpm_version: >>> * @s: the backend to call into >>> diff --git a/backends/tpm.c b/backends/tpm.c >>> index 0c48d18775..7e636fbc7a 100644 >>> --- a/backends/tpm.c >>> +++ b/backends/tpm.c >>> @@ -159,52 +159,10 @@ TPMInfo *tpm_backend_query_tpm(TPMBackend *s) >>> return info; >>> } >>> >>> -static bool tpm_backend_prop_get_opened(Object *obj, Error **errp) >>> -{ >>> - TPMBackend *s = TPM_BACKEND(obj); >>> - >>> - return s->opened; >>> -} >>> - >>> -void tpm_backend_open(TPMBackend *s, Error **errp) >>> -{ >>> - object_property_set_bool(OBJECT(s), true, "opened", errp); >>> -} >>> - >>> -static void tpm_backend_prop_set_opened(Object *obj, bool value, Error >>> **errp) >>> -{ >>> - TPMBackend *s = TPM_BACKEND(obj); >>> - TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s); >>> - Error *local_err = NULL; >>> - >>> - if (value == s->opened) { >>> - return; >>> - } >>> - >>> - if (!value && s->opened) { >>> - error_setg(errp, QERR_PERMISSION_DENIED); >>> - return; >>> - } >>> - >>> - if (k->opened) { >>> - k->opened(s, &local_err); >>> - if (local_err) { >>> - error_propagate(errp, local_err); >>> - return; >>> - } >>> - } >>> - >>> - s->opened = true; >>> -} >>> - >>> static void tpm_backend_instance_init(Object *obj) >>> { >>> TPMBackend *s = TPM_BACKEND(obj); >>> >>> - object_property_add_bool(obj, "opened", >>> - tpm_backend_prop_get_opened, >>> - tpm_backend_prop_set_opened, >>> - NULL); >>> s->bh = qemu_bh_new(tpm_backend_request_completed_bh, s); >>> } >>> >>> diff --git a/tpm.c b/tpm.c >>> index ce1543fcb4..a46ee5f144 100644 >>> --- a/tpm.c >>> +++ b/tpm.c >>> @@ -134,12 +134,6 @@ static int tpm_init_tpmdev(void *dummy, QemuOpts >>> *opts, Error **errp) >>> return 1; >>> } >>> >>> - tpm_backend_open(drv, &local_err); >>> - if (local_err) { >>> - error_report_err(local_err); >>> - return 1; >>> - } >>> - >>> QLIST_INSERT_HEAD(&tpm_backends, drv, list); >>> >>> return 0; >> Since nothing is setting the "opened' anymore, would anyone notice >> because this has changed or is something else doing that now? Otherwise >> I don't mind removing it... > Was it ever used? I see the property was added in commit 8f0605cc9caacbcc647a6df9ae541ed2da4b9bb0. > > Code adapted from rng backend. But rng backend set "opened" on UserCreatableClass.complete() (TPM backend doesn't, but probably could - I have an experimental patch for that) > > I think we can remove it that code safely. > Reviewed-by: Stefan Berger <stefanb@linux.vnet.ibm.com>
diff --git a/include/sysemu/tpm_backend.h b/include/sysemu/tpm_backend.h index b12ae5b625..a893e586ae 100644 --- a/include/sysemu/tpm_backend.h +++ b/include/sysemu/tpm_backend.h @@ -81,8 +81,6 @@ struct TPMBackendClass { TpmTypeOptions *(*get_tpm_options)(TPMBackend *t); - void (*opened)(TPMBackend *s, Error **errp); - void (*handle_request)(TPMBackend *s, TPMBackendCmd *cmd); }; @@ -172,16 +170,6 @@ bool tpm_backend_get_tpm_established_flag(TPMBackend *s); */ int tpm_backend_reset_tpm_established_flag(TPMBackend *s, uint8_t locty); -/** - * tpm_backend_open: - * @s: the backend to open - * @errp: a pointer to return the #Error object if an error occurs. - * - * This function will open the backend if it is not already open. Calling this - * function on an already opened backend will not result in an error. - */ -void tpm_backend_open(TPMBackend *s, Error **errp); - /** * tpm_backend_get_tpm_version: * @s: the backend to call into diff --git a/backends/tpm.c b/backends/tpm.c index 0c48d18775..7e636fbc7a 100644 --- a/backends/tpm.c +++ b/backends/tpm.c @@ -159,52 +159,10 @@ TPMInfo *tpm_backend_query_tpm(TPMBackend *s) return info; } -static bool tpm_backend_prop_get_opened(Object *obj, Error **errp) -{ - TPMBackend *s = TPM_BACKEND(obj); - - return s->opened; -} - -void tpm_backend_open(TPMBackend *s, Error **errp) -{ - object_property_set_bool(OBJECT(s), true, "opened", errp); -} - -static void tpm_backend_prop_set_opened(Object *obj, bool value, Error **errp) -{ - TPMBackend *s = TPM_BACKEND(obj); - TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s); - Error *local_err = NULL; - - if (value == s->opened) { - return; - } - - if (!value && s->opened) { - error_setg(errp, QERR_PERMISSION_DENIED); - return; - } - - if (k->opened) { - k->opened(s, &local_err); - if (local_err) { - error_propagate(errp, local_err); - return; - } - } - - s->opened = true; -} - static void tpm_backend_instance_init(Object *obj) { TPMBackend *s = TPM_BACKEND(obj); - object_property_add_bool(obj, "opened", - tpm_backend_prop_get_opened, - tpm_backend_prop_set_opened, - NULL); s->bh = qemu_bh_new(tpm_backend_request_completed_bh, s); } diff --git a/tpm.c b/tpm.c index ce1543fcb4..a46ee5f144 100644 --- a/tpm.c +++ b/tpm.c @@ -134,12 +134,6 @@ static int tpm_init_tpmdev(void *dummy, QemuOpts *opts, Error **errp) return 1; } - tpm_backend_open(drv, &local_err); - if (local_err) { - error_report_err(local_err); - return 1; - } - QLIST_INSERT_HEAD(&tpm_backends, drv, list); return 0;
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com> --- include/sysemu/tpm_backend.h | 12 ------------ backends/tpm.c | 42 ------------------------------------------ tpm.c | 6 ------ 3 files changed, 60 deletions(-)