From: Stefan Berger <stefanb@linux.vnet.ibm.com>
To: linuxppc-dev@lists.ozlabs.org, nikunj@linux.vnet.ibm.com,
aik@au1.ibm.com, pmac@au1.ibm.com
Cc: gcwilson@us.ibm.com, dimitris@us.ibm.com, latten@us.ibm.com,
lo1@us.ibm.com, stefanb@us.ibm.com,
Stefan Berger <stefanb@linux.vnet.ibm.com>
Subject: [PATCH 16/16] Add TPM firmware API call get-failure-reason
Date: Fri, 7 Aug 2015 21:55:05 -0400 [thread overview]
Message-ID: <1438998905-4085665-17-git-send-email-stefanb@linux.vnet.ibm.com> (raw)
In-Reply-To: <1438998905-4085665-1-git-send-email-stefanb@linux.vnet.ibm.com>
This patch adds support for the TPM firmware API call get-failure-reason.
Signed-off-by: Stefan Berger <stefanb@linux.vnet.ibm.com>
---
board-qemu/slof/vio-vtpm-cdriver.fs | 8 ++++++++
board-qemu/slof/vtpm-sml.fs | 7 +++++++
lib/libtpm/tcgbios.c | 16 ++++++++++++++++
lib/libtpm/tcgbios.h | 1 +
lib/libtpm/tpm.code | 10 ++++++++++
lib/libtpm/tpm.in | 1 +
slof/fs/tpm/tpm-static.fs | 12 ++++++++++++
7 files changed, 55 insertions(+)
diff --git a/board-qemu/slof/vio-vtpm-cdriver.fs b/board-qemu/slof/vio-vtpm-cdriver.fs
index 474c24c..18ee85a 100644
--- a/board-qemu/slof/vio-vtpm-cdriver.fs
+++ b/board-qemu/slof/vio-vtpm-cdriver.fs
@@ -80,6 +80,14 @@ false VALUE vtpm-debug?
;
\ firmware API call
+: get-failure-reason ( -- reason )
+ " get-failure-reason" call-forward IF
+ \ call-forward failed; return a value
+ 0 \ invalid
+ THEN
+;
+
+\ firmware API call
: hash-all ( data-ptr data-len hash-ptr -- )
" hash-all" call-forward IF
\ call-forward failed; clean up stack
diff --git a/board-qemu/slof/vtpm-sml.fs b/board-qemu/slof/vtpm-sml.fs
index 449f6fc..a37aa80 100644
--- a/board-qemu/slof/vtpm-sml.fs
+++ b/board-qemu/slof/vtpm-sml.fs
@@ -59,6 +59,13 @@ log-base LOG-SIZE tpm-set-log-parameters
vtpm-driver-get-state
;
+: get-failure-reason ( -- reason )
+ vtpm-debug? IF
+ ." Call to get-failure-reason" cr
+ THEN
+ vtpm-driver-get-failure-reason
+;
+
: hash-all ( data-ptr data-len hash-ptr -- )
vtpm-debug? IF
." Call to hash-all" cr
diff --git a/lib/libtpm/tcgbios.c b/lib/libtpm/tcgbios.c
index 02ad91a..7194141 100644
--- a/lib/libtpm/tcgbios.c
+++ b/lib/libtpm/tcgbios.c
@@ -1545,3 +1545,19 @@ uint32_t tpm_driver_get_state(void)
return td->getstate();
}
+
+/*
+ * tpm_driver_get_failure_reason: Function for interfacing with the firmware
+ * API
+ */
+uint32_t tpm_driver_get_failure_reason(void)
+{
+ struct tpm_driver *td = tpm_state.tpm_driver_to_use;
+
+ /* do not check for a working TPM here */
+
+ if (!td)
+ return PFW_DRV_ERROR_NO_FAILURE;
+
+ return td->geterror();
+}
diff --git a/lib/libtpm/tcgbios.h b/lib/libtpm/tcgbios.h
index 05cadb0..c7fbc29 100644
--- a/lib/libtpm/tcgbios.h
+++ b/lib/libtpm/tcgbios.h
@@ -39,6 +39,7 @@ uint32_t tpm_hash_all(const void *data, uint32_t datalen, void *hashptr);
uint32_t tpm_get_maximum_cmd_size(void);
uint32_t tpm_pass_through_to_tpm(unsigned char *buf, uint32_t cmdlen);
uint32_t tpm_driver_get_state(void);
+uint32_t tpm_driver_get_failure_reason(void);
/* flags returned by tpm_get_state */
#define TPM_STATE_ENABLED 1
diff --git a/lib/libtpm/tpm.code b/lib/libtpm/tpm.code
index 574c7eb..29e3b3d 100644
--- a/lib/libtpm/tpm.code
+++ b/lib/libtpm/tpm.code
@@ -163,6 +163,16 @@ PRIM(tpm_X2d_driver_X2d_get_X2d_state)
TOS.n = tpm_driver_get_state();
MIRP
+/****************************************************/
+/* Firmware API */
+/* SLOF: tpm-driver-get-error ( -- errcode) */
+/* LIBTPM: errcode = tpm_driver_get_error(void) */
+/****************************************************/
+PRIM(tpm_X2d_driver_X2d_get_X2d_failure_X2d_reason)
+ PUSH;
+ TOS.n = tpm_driver_get_failure_reason();
+MIRP
+
/*****************************************************************/
/* Firmware API */
/* SLOF: tpm-hash-all ( data-ptr data-len hash-ptr -- errcode) */
diff --git a/lib/libtpm/tpm.in b/lib/libtpm/tpm.in
index 3dd78f4..5f33d05 100644
--- a/lib/libtpm/tpm.in
+++ b/lib/libtpm/tpm.in
@@ -30,3 +30,4 @@ cod(tpm-hash-all)
cod(tpm-get-maximum-cmd-size)
cod(tpm-pass-through-to-tpm)
cod(tpm-driver-get-state)
+cod(tpm-driver-get-failure-reason)
diff --git a/slof/fs/tpm/tpm-static.fs b/slof/fs/tpm/tpm-static.fs
index 31ac815..cbd92cb 100644
--- a/slof/fs/tpm/tpm-static.fs
+++ b/slof/fs/tpm/tpm-static.fs
@@ -81,6 +81,18 @@ false VALUE vtpm-debug?
;
\ firmware API function
+: vtpm-driver-get-failure-reason ( -- reason )
+ vtpm-available? IF
+ tpm-driver-get-failure-reason
+ vtpm-debug? IF
+ ." VTPM: Return value from tpm-driver-get-failure-reason: " dup . cr
+ THEN
+ ELSE
+ -1 \ no failure
+ THEN
+;
+
+\ firmware API function
: vtpm-log-event ( event-ptr -- ok? )
vtpm-available? IF
tpm-log-event
--
1.9.3
prev parent reply other threads:[~2015-08-08 1:55 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-08-08 1:54 [PATCH 00/16] Add vTPM support to SLOF Stefan Berger
2015-08-08 1:54 ` [PATCH 01/16] Add a TPM driver implementation Stefan Berger
2015-08-08 1:54 ` [PATCH 02/16] Add TPM initialization support Stefan Berger
2015-08-08 1:54 ` [PATCH 03/16] Add sha1 implementation Stefan Berger
2015-08-08 1:54 ` [PATCH 04/16] Add initial support for logging Stefan Berger
2015-08-08 1:54 ` [PATCH 05/16] Extend internal firmware API Stefan Berger
2015-08-08 1:54 ` [PATCH 06/16] Return value of actual log in sml-get-handover-size Stefan Berger
2015-08-08 1:54 ` [PATCH 07/16] Perform some initial measurements Stefan Berger
2015-08-08 1:54 ` [PATCH 08/16] Add support for controlling the states of the TPM Stefan Berger
2015-08-08 1:54 ` [PATCH 09/16] Add support for a TPM menu to control the state " Stefan Berger
2015-08-08 1:54 ` [PATCH 10/16] Implement measurements of the master boot record Stefan Berger
2015-08-08 1:55 ` [PATCH 11/16] Measure the static core root of trust for measurements Stefan Berger
2015-08-08 1:55 ` [PATCH 12/16] Add TPM firmware API calls hash-all, log-event, hash-log-extend-event Stefan Berger
2015-08-08 1:55 ` [PATCH 13/16] Add TPM firmware API call get-maximum-cmd-size Stefan Berger
2015-08-08 1:55 ` [PATCH 14/16] Add TPM firmware API call pass-through-to-tpm Stefan Berger
2015-08-08 1:55 ` [PATCH 15/16] Add TPM firmware API call get-state Stefan Berger
2015-08-08 1:55 ` Stefan Berger [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1438998905-4085665-17-git-send-email-stefanb@linux.vnet.ibm.com \
--to=stefanb@linux.vnet.ibm.com \
--cc=aik@au1.ibm.com \
--cc=dimitris@us.ibm.com \
--cc=gcwilson@us.ibm.com \
--cc=latten@us.ibm.com \
--cc=linuxppc-dev@lists.ozlabs.org \
--cc=lo1@us.ibm.com \
--cc=nikunj@linux.vnet.ibm.com \
--cc=pmac@au1.ibm.com \
--cc=stefanb@us.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).