tpm: add TPMBackendCmd to hold the request state

This simplifies a bit locality handling, and argument passing, and
could pave the way to queuing requests (if that makes sense).

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Reviewed-by: Stefan Berger <stefanb@linux.vnet.ibm.com>
Signed-off-by: Stefan Berger <stefanb@linux.vnet.ibm.com>
This commit is contained in:
Marc-André Lureau 2017-10-10 00:55:55 +02:00 committed by Stefan Berger
parent d1fd6b563d
commit 0e43b7e61c
6 changed files with 50 additions and 44 deletions

View File

@ -25,7 +25,7 @@ static void tpm_backend_worker_thread(gpointer data, gpointer user_data)
TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s); TPMBackendClass *k = TPM_BACKEND_GET_CLASS(s);
assert(k->handle_request != NULL); assert(k->handle_request != NULL);
k->handle_request(s); k->handle_request(s, (TPMBackendCmd *)data);
} }
static void tpm_backend_thread_end(TPMBackend *s) static void tpm_backend_thread_end(TPMBackend *s)
@ -76,9 +76,9 @@ bool tpm_backend_had_startup_error(TPMBackend *s)
return s->had_startup_error; return s->had_startup_error;
} }
void tpm_backend_deliver_request(TPMBackend *s) void tpm_backend_deliver_request(TPMBackend *s, TPMBackendCmd *cmd)
{ {
g_thread_pool_push(s->thread_pool, NULL, NULL); g_thread_pool_push(s->thread_pool, cmd, NULL);
} }
void tpm_backend_reset(TPMBackend *s) void tpm_backend_reset(TPMBackend *s)

View File

@ -172,28 +172,29 @@ static int tpm_emulator_set_locality(TPMEmulator *tpm_emu, uint8_t locty_number)
return 0; return 0;
} }
static void tpm_emulator_handle_request(TPMBackend *tb) static void tpm_emulator_handle_request(TPMBackend *tb, TPMBackendCmd *cmd)
{ {
TPMEmulator *tpm_emu = TPM_EMULATOR(tb); TPMEmulator *tpm_emu = TPM_EMULATOR(tb);
TPMLocality *locty = NULL;
bool selftest_done = false;
Error *err = NULL; Error *err = NULL;
DPRINTF("processing TPM command"); DPRINTF("processing TPM command");
locty = tb->tpm_state->locty_data; if (tpm_emulator_set_locality(tpm_emu, tb->tpm_state->locty_number) < 0) {
if (tpm_emulator_set_locality(tpm_emu, goto error;
tb->tpm_state->locty_number) < 0 ||
tpm_emulator_unix_tx_bufs(tpm_emu, locty->w_buffer.buffer,
locty->w_offset, locty->r_buffer.buffer,
locty->r_buffer.size, &selftest_done,
&err) < 0) {
tpm_util_write_fatal_error_response(locty->r_buffer.buffer,
locty->r_buffer.size);
error_report_err(err);
} }
tb->recv_data_callback(tb->tpm_state, selftest_done); if (tpm_emulator_unix_tx_bufs(tpm_emu, cmd->in, cmd->in_len,
cmd->out, cmd->out_len,
&cmd->selftest_done, &err) < 0) {
goto error;
}
tb->recv_data_callback(tb->tpm_state);
return;
error:
tpm_util_write_fatal_error_response(cmd->out, cmd->out_len);
error_report_err(err);
} }
static int tpm_emulator_probe_caps(TPMEmulator *tpm_emu) static int tpm_emulator_probe_caps(TPMEmulator *tpm_emu)

View File

@ -26,6 +26,7 @@ struct TPMState {
uint8_t locty_number; uint8_t locty_number;
TPMLocality *locty_data; TPMLocality *locty_data;
TPMBackendCmd cmd;
char *backend; char *backend;
TPMBackend *be_driver; TPMBackend *be_driver;

View File

@ -137,30 +137,16 @@ err_exit:
return ret; return ret;
} }
static int tpm_passthrough_unix_transfer(TPMPassthruState *tpm_pt, static void tpm_passthrough_handle_request(TPMBackend *tb, TPMBackendCmd *cmd)
const TPMLocality *locty_data,
bool *selftest_done)
{
return tpm_passthrough_unix_tx_bufs(tpm_pt,
locty_data->w_buffer.buffer,
locty_data->w_offset,
locty_data->r_buffer.buffer,
locty_data->r_buffer.size,
selftest_done);
}
static void tpm_passthrough_handle_request(TPMBackend *tb)
{ {
TPMPassthruState *tpm_pt = TPM_PASSTHROUGH(tb); TPMPassthruState *tpm_pt = TPM_PASSTHROUGH(tb);
bool selftest_done = false;
DPRINTF("tpm_passthrough: processing command\n"); DPRINTF("tpm_passthrough: processing command %p\n", cmd);
tpm_passthrough_unix_transfer(tpm_pt, tpm_passthrough_unix_tx_bufs(tpm_pt, cmd->in, cmd->in_len,
tb->tpm_state->locty_data, cmd->out, cmd->out_len, &cmd->selftest_done);
&selftest_done);
tb->recv_data_callback(tb->tpm_state, selftest_done); tb->recv_data_callback(tb->tpm_state);
} }
static void tpm_passthrough_reset(TPMBackend *tb) static void tpm_passthrough_reset(TPMBackend *tb)

View File

@ -215,7 +215,15 @@ static void tpm_tis_tpm_send(TPMState *s, uint8_t locty)
*/ */
tis->loc[locty].state = TPM_TIS_STATE_EXECUTION; tis->loc[locty].state = TPM_TIS_STATE_EXECUTION;
tpm_backend_deliver_request(s->be_driver); s->cmd = (TPMBackendCmd) {
.locty = locty,
.in = s->locty_data->w_buffer.buffer,
.in_len = s->locty_data->w_offset,
.out = s->locty_data->r_buffer.buffer,
.out_len = s->locty_data->r_buffer.size
};
tpm_backend_deliver_request(s->be_driver, &s->cmd);
} }
/* raise an interrupt if allowed */ /* raise an interrupt if allowed */
@ -352,7 +360,7 @@ static void tpm_tis_receive_bh(void *opaque)
{ {
TPMState *s = opaque; TPMState *s = opaque;
TPMTISEmuState *tis = &s->s.tis; TPMTISEmuState *tis = &s->s.tis;
uint8_t locty = s->locty_number; uint8_t locty = s->cmd.locty;
tpm_tis_sts_set(&tis->loc[locty], tpm_tis_sts_set(&tis->loc[locty],
TPM_TIS_STS_VALID | TPM_TIS_STS_DATA_AVAILABLE); TPM_TIS_STS_VALID | TPM_TIS_STS_DATA_AVAILABLE);
@ -371,11 +379,11 @@ static void tpm_tis_receive_bh(void *opaque)
/* /*
* Callback from the TPM to indicate that the response was received. * Callback from the TPM to indicate that the response was received.
*/ */
static void tpm_tis_receive_cb(TPMState *s, static void tpm_tis_receive_cb(TPMState *s)
bool is_selftest_done)
{ {
TPMTISEmuState *tis = &s->s.tis; TPMTISEmuState *tis = &s->s.tis;
uint8_t locty = s->locty_number; bool is_selftest_done = s->cmd.selftest_done;
uint8_t locty = s->cmd.locty;
uint8_t l; uint8_t l;
if (is_selftest_done) { if (is_selftest_done) {

View File

@ -30,7 +30,16 @@
typedef struct TPMBackendClass TPMBackendClass; typedef struct TPMBackendClass TPMBackendClass;
typedef struct TPMBackend TPMBackend; typedef struct TPMBackend TPMBackend;
typedef void (TPMRecvDataCB)(TPMState *, bool selftest_done); typedef void (TPMRecvDataCB)(TPMState *);
typedef struct TPMBackendCmd {
uint8_t locty;
const uint8_t *in;
uint32_t in_len;
uint8_t *out;
uint32_t out_len;
bool selftest_done;
} TPMBackendCmd;
struct TPMBackend { struct TPMBackend {
Object parent; Object parent;
@ -76,7 +85,7 @@ struct TPMBackendClass {
void (*opened)(TPMBackend *s, Error **errp); void (*opened)(TPMBackend *s, Error **errp);
void (*handle_request)(TPMBackend *s); void (*handle_request)(TPMBackend *s, TPMBackendCmd *cmd);
}; };
/** /**
@ -121,11 +130,12 @@ bool tpm_backend_had_startup_error(TPMBackend *s);
/** /**
* tpm_backend_deliver_request: * tpm_backend_deliver_request:
* @s: the backend to send the request to * @s: the backend to send the request to
* @cmd: the command to deliver
* *
* Send a request to the backend. The backend will then send the request * Send a request to the backend. The backend will then send the request
* to the TPM implementation. * to the TPM implementation.
*/ */
void tpm_backend_deliver_request(TPMBackend *s); void tpm_backend_deliver_request(TPMBackend *s, TPMBackendCmd *cmd);
/** /**
* tpm_backend_reset: * tpm_backend_reset: