diff options
author | Jeremy Kerr <jk@ozlabs.org> | 2015-05-19 17:05:38 +0800 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2015-05-21 15:03:35 +1000 |
commit | 1e03bfbedc46ab7792f0fc4617df28e49af11cff (patch) | |
tree | 700a4d87af76b5016d400d940910b7906b47238e /hw | |
parent | f7b322e049bcf599511af475be340e86b3049001 (diff) | |
download | skiboot-1e03bfbedc46ab7792f0fc4617df28e49af11cff.zip skiboot-1e03bfbedc46ab7792f0fc4617df28e49af11cff.tar.gz skiboot-1e03bfbedc46ab7792f0fc4617df28e49af11cff.tar.bz2 |
prd: Don't expose full opal_prd_msg to kernel
The kernel is only accessing part of struct opal_prd_msg, and doesn't
need to know about the rest. This change moves the "internal" parts of
opal_prd_msg to a separate header.
In order for the kernel to pass full messages between userspace and
firmware, it needs the total size of the struct. We put this in the
header for the kernel to access.
Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'hw')
-rw-r--r-- | hw/prd.c | 24 |
1 files changed, 16 insertions, 8 deletions
@@ -89,7 +89,7 @@ static void prd_msg_consumed(void *data) uint8_t event = 0; lock(&events_lock); - switch (msg->type) { + switch (msg->hdr.type) { case OPAL_PRD_MSG_TYPE_ATTN: proc = msg->attn.proc; @@ -169,19 +169,20 @@ static void send_pending_events(void) prd_msg_inuse = true; prd_msg.token = 0; + prd_msg.hdr.size = sizeof(prd_msg); if (event & EVENT_ATTN) { - prd_msg.type = OPAL_PRD_MSG_TYPE_ATTN; + prd_msg.hdr.type = OPAL_PRD_MSG_TYPE_ATTN; populate_ipoll_msg(&prd_msg, proc); event = EVENT_ATTN; } else if (event & EVENT_OCC_ERROR) { - prd_msg.type = OPAL_PRD_MSG_TYPE_OCC_ERROR; + prd_msg.hdr.type = OPAL_PRD_MSG_TYPE_OCC_ERROR; prd_msg.occ_error.chip = proc; event = EVENT_OCC_ERROR; } else if (event & EVENT_OCC_RESET) { - prd_msg.type = OPAL_PRD_MSG_TYPE_OCC_RESET; + prd_msg.hdr.type = OPAL_PRD_MSG_TYPE_OCC_RESET; prd_msg.occ_reset.chip = proc; event = EVENT_OCC_RESET; @@ -325,13 +326,20 @@ static int64_t opal_prd_msg(struct opal_prd_msg *msg) { int rc; - switch (msg->type) { + /* fini is a little special: the kernel (which may not have the entire + * opal_prd_msg definition) can send a FINI message, so we don't check + * the full size */ + if (msg->hdr.size >= sizeof(struct opal_prd_msg_header) && + msg->hdr.type == OPAL_PRD_MSG_TYPE_FINI) + return prd_msg_handle_fini(); + + if (msg->hdr.size != sizeof(*msg)) + return OPAL_PARAMETER; + + switch (msg->hdr.type) { case OPAL_PRD_MSG_TYPE_INIT: rc = prd_msg_handle_init(msg); break; - case OPAL_PRD_MSG_TYPE_FINI: - rc = prd_msg_handle_fini(); - break; case OPAL_PRD_MSG_TYPE_ATTN_ACK: rc = prd_msg_handle_attn_ack(msg); break; |