[Skiboot] [PATCH V5 12/15] core/pldm: Update boot progress state
Christophe Lombard
clombard at linux.ibm.com
Fri Jul 21 19:07:18 AEST 2023
PLDM Event Messages are PLDM monitoring and control messages that are used
by a PLDM terminus to synchronously or asynchronously report PLDM events
to a central party called the PLDM Event Receiver.
This patch allows to send a:
- generic sensor events (events related to PLDM numeric and state sensors).
- boot progress sensor event.
Signed-off-by: Christophe Lombard <clombard at linux.ibm.com>
---
core/pldm/pldm-platform-requests.c | 267 +++++++++++++++++++++++++++++
core/pldm/pldm-responder.c | 1 +
include/pldm.h | 12 ++
3 files changed, 280 insertions(+)
diff --git a/core/pldm/pldm-platform-requests.c b/core/pldm/pldm-platform-requests.c
index 8bd373cc..cb0ff944 100644
--- a/core/pldm/pldm-platform-requests.c
+++ b/core/pldm/pldm-platform-requests.c
@@ -72,6 +72,51 @@ int pldm_platform_pdr_find_record(uint32_t record_handle,
return OPAL_SUCCESS;
}
+/*
+ * Search the matching record and return the sensor id.
+ * PDR type = PLDM_STATE_SENSOR_PDR
+ */
+static int find_sensor_id_by_state_set_Id(uint16_t entity_type,
+ uint16_t state_set_id,
+ uint16_t *sensor_id,
+ uint16_t terminus_handle)
+{
+ struct state_sensor_possible_states *possible_states;
+ struct pldm_state_sensor_pdr *state_sensor_pdr;
+ const pldm_pdr_record *record = NULL;
+ uint8_t *outData = NULL;
+ uint32_t size;
+
+ do {
+ /* Find (first) PDR record by PLDM_STATE_SENSOR_PDR type
+ * if record not NULL, then search will begin from this
+ * record's next record
+ */
+ record = pldm_pdr_find_record_by_type(
+ pdrs_repo, /* PDR repo handle */
+ PLDM_STATE_SENSOR_PDR,
+ record, /* PDR record handle */
+ &outData, &size);
+
+ if (record) {
+ state_sensor_pdr = (struct pldm_state_sensor_pdr *) outData;
+
+ *sensor_id = le16_to_cpu(state_sensor_pdr->sensor_id);
+
+ possible_states = (struct state_sensor_possible_states *)
+ state_sensor_pdr->possible_states;
+
+ if ((le16_to_cpu(state_sensor_pdr->entity_type) == entity_type) &&
+ (le16_to_cpu(state_sensor_pdr->terminus_handle) == terminus_handle) &&
+ (le16_to_cpu(possible_states->state_set_id) == state_set_id))
+ return OPAL_SUCCESS;
+ }
+
+ } while (record);
+
+ return OPAL_PARAMETER;
+}
+
/*
* Search the matching record and return the effecter id.
* PDR type = PLDM_STATE_EFFECTER_PDR
@@ -263,6 +308,228 @@ int pldm_platform_restart(void)
return set_state_effecter_states_req(effecter_id, &field, true);
}
+static int send_sensor_state_changed_event(uint16_t state_set_id,
+ uint16_t sensor_id,
+ uint8_t sensor_offset,
+ uint8_t sensor_state,
+ bool no_timeout)
+{
+ size_t event_data_size = 0, actual_event_data_size;
+ size_t response_len, payload_len, data_size;
+ uint8_t *event_data = NULL;
+ struct pldm_tx_data *tx;
+ void *response_msg;
+ int rc, i;
+
+ struct pldm_platform_event_message_req event_message_req = {
+ .format_version = PLDM_PLATFORM_EVENT_MESSAGE_FORMAT_VERSION,
+ .tid = HOST_TID,
+ .event_class = PLDM_SENSOR_EVENT,
+ };
+
+ struct pldm_platform_event_message_resp response;
+
+ prlog(PR_DEBUG, "%s - state_set_id: %d, sensor_id: %d, sensor_state: %d\n",
+ __func__, state_set_id, sensor_id, sensor_state);
+
+ /*
+ * The first time around this loop, event_data is nullptr which
+ * instructs the encoder to not actually do the encoding, but
+ * rather fill out actual_change_records_size with the correct
+ * size, stop and return PLDM_SUCCESS. Then we allocate the
+ * proper amount of memory and call the encoder again, which
+ * will cause it to actually encode the message.
+ */
+ for (i = 0; i < 2; i++) {
+ rc = encode_sensor_event_data(
+ (struct pldm_sensor_event_data *)event_data,
+ event_data_size,
+ sensor_id,
+ PLDM_STATE_SENSOR_STATE,
+ sensor_offset,
+ sensor_state,
+ sensor_state,
+ &actual_event_data_size);
+ if (rc) {
+ prlog(PR_ERR, "encode PldmSensorChgEventData Error, rc: %d\n", rc);
+ return OPAL_PARAMETER;
+ }
+
+ if (event_data == NULL) {
+ event_data_size = actual_event_data_size;
+ event_data = zalloc(event_data_size);
+ if (!event_data) {
+ prlog(PR_ERR, "failed to allocate event data (size: 0x%lx)\n", event_data_size);
+ return OPAL_NO_MEM;
+ }
+ }
+ }
+
+ /* Send the event request */
+ payload_len = PLDM_PLATFORM_EVENT_MESSAGE_MIN_REQ_BYTES + event_data_size;
+
+ data_size = sizeof(struct pldm_msg_hdr) +
+ sizeof(struct pldm_platform_event_message_req) +
+ event_data_size;
+ tx = zalloc(sizeof(struct pldm_tx_data) + data_size);
+ if (!tx)
+ return OPAL_NO_MEM;
+ tx->data_size = data_size - 1;
+
+ /* Encode the platform event message request */
+ rc = encode_platform_event_message_req(
+ DEFAULT_INSTANCE_ID,
+ event_message_req.format_version,
+ event_message_req.tid,
+ event_message_req.event_class,
+ (const uint8_t *)event_data,
+ event_data_size,
+ (struct pldm_msg *)tx->data,
+ payload_len);
+ if (rc != PLDM_SUCCESS) {
+ prlog(PR_ERR, "Encode PlatformEventMessage Error, rc: %d\n", rc);
+ free(event_data);
+ free(tx);
+ return OPAL_PARAMETER;
+ }
+ free(event_data);
+
+ /* Send and get the response message bytes.
+ * It may happen that for some commands, the responder does not
+ * have time to respond.
+ */
+ if (no_timeout) {
+ rc = pldm_mctp_message_tx(tx);
+ if (rc)
+ prlog(PR_ERR, "Failed to send PlatformEventMessage request, rc = %d\n", rc);
+ free(tx);
+ return rc;
+ }
+
+ /* Send and get the response message bytes */
+ rc = pldm_requester_queue_and_wait(tx,
+ &response_msg, &response_len);
+ if (rc) {
+ prlog(PR_ERR, "Communication Error, req: PlatformEventMessage, rc: %d\n", rc);
+ free(tx);
+ return rc;
+ }
+ free(tx);
+
+ /* Decode the message */
+ payload_len = response_len - sizeof(struct pldm_msg_hdr);
+ rc = decode_platform_event_message_resp(
+ response_msg,
+ payload_len,
+ &response.completion_code,
+ &response.platform_event_status);
+ if (rc != PLDM_SUCCESS || response.completion_code != PLDM_SUCCESS) {
+ prlog(PR_ERR, "Decode PlatformEventMessage Error, rc: %d, cc: %d, pes: %d\n",
+ rc, response.completion_code,
+ response.platform_event_status);
+ free(response_msg);
+ return OPAL_PARAMETER;
+ }
+
+ free(response_msg);
+
+ return OPAL_SUCCESS;
+}
+
+#define BOOT_STATE_SENSOR_INDEX 0
+
+int pldm_platform_send_progress_state_change(
+ enum pldm_state_set_boot_progress_state_values state)
+{
+ struct state_sensor_possible_states *possible_states;
+ struct pldm_state_sensor_pdr *sensor_pdr = NULL;
+ const pldm_pdr_record *record = NULL;
+ uint16_t terminus_handle;
+ uint8_t *outData = NULL;
+ uint16_t sensor_id = 0;
+ uint32_t size;
+
+ if (!pdr_ready)
+ return OPAL_HARDWARE;
+
+ prlog(PR_INFO, "Setting boot progress, state: %d\n", state);
+
+ do {
+ /* Find (first) PDR record by PLDM_STATE_SENSOR_PDR type
+ * if record not NULL, then search will begin from this
+ * record's next record
+ */
+ record = pldm_pdr_find_record_by_type(
+ pdrs_repo, /* PDR repo handle */
+ PLDM_STATE_SENSOR_PDR,
+ record, /* PDR record handle */
+ &outData, &size);
+
+ if (record) {
+ sensor_pdr = (struct pldm_state_sensor_pdr *) outData;
+ terminus_handle = le16_to_cpu(sensor_pdr->terminus_handle);
+
+ if ((le16_to_cpu(sensor_pdr->entity_type) == PLDM_ENTITY_SYS_BOARD) &&
+ (terminus_handle == HOST_TID)) {
+ possible_states = (struct state_sensor_possible_states *)
+ sensor_pdr->possible_states;
+
+ if (le16_to_cpu(possible_states->state_set_id) ==
+ PLDM_STATE_SET_BOOT_PROGRESS){
+ sensor_id = le16_to_cpu(sensor_pdr->sensor_id);
+ break;
+ }
+ }
+ }
+
+ } while (record);
+
+ if (sensor_id == 0)
+ return OPAL_PARAMETER;
+
+ return send_sensor_state_changed_event(
+ PLDM_STATE_SET_BOOT_PROGRESS,
+ sensor_id,
+ BOOT_STATE_SENSOR_INDEX,
+ state,
+ false);
+}
+
+#define SW_TERM_GRACEFUL_SHUTDOWN_INDEX 0
+
+/*
+ * entity_type: System Firmware
+ * state_set: Software Termination Status(129)
+ * states: Graceful Shutdown Requested(7)
+ */
+int pldm_platform_initiate_shutdown(void)
+{
+ uint16_t sensor_id;
+ int rc;
+
+ if (!pdr_ready)
+ return OPAL_HARDWARE;
+
+ rc = find_sensor_id_by_state_set_Id(
+ PLDM_ENTITY_SYSTEM_CHASSIS,
+ PLDM_STATE_SET_SW_TERMINATION_STATUS,
+ &sensor_id, HOST_TID);
+ if (rc) {
+ prlog(PR_ERR, "%s - sensor id not found\n", __func__);
+ return rc;
+ }
+
+ prlog(PR_INFO, "sending system firmware Graceful Shutdown request (sensor_id: %d)\n",
+ sensor_id);
+
+ return send_sensor_state_changed_event(
+ PLDM_STATE_SET_SW_TERMINATION_STATUS,
+ sensor_id,
+ SW_TERM_GRACEFUL_SHUTDOWN_INDEX,
+ PLDM_SW_TERM_GRACEFUL_SHUTDOWN,
+ true);
+}
+
static int add_state_sensor_pdr(pldm_pdr *repo,
uint32_t *record_handle,
uint16_t entity_type,
diff --git a/core/pldm/pldm-responder.c b/core/pldm/pldm-responder.c
index d42d9f52..7dbde7db 100644
--- a/core/pldm/pldm-responder.c
+++ b/core/pldm/pldm-responder.c
@@ -792,6 +792,7 @@ static int platform_set_state_effecter_states_handler(const struct pldm_rx_data
cpu_to_be64(SOFT_REBOOT));
}
+ pldm_platform_initiate_shutdown();
break;
case PLDM_STATE_SET_BOOT_RESTART_CAUSE_WARM_RESET:
diff --git a/include/pldm.h b/include/pldm.h
index 80ee85c0..e33de7dc 100644
--- a/include/pldm.h
+++ b/include/pldm.h
@@ -6,6 +6,7 @@
#define __PLDM_H__
#include <skiboot.h>
+#include <pldm/include/libpldm/state_set.h>
/**
* Handle PLDM messages received from MCTP
@@ -33,6 +34,11 @@ int pldm_platform_power_off(void);
*/
int pldm_platform_restart(void);
+/**
+ * Send a system firmware Graceful Shutdown request
+ */
+int pldm_platform_initiate_shutdown(void);
+
/**
* Update the firmware version device-tree field
*/
@@ -53,4 +59,10 @@ bool pldm_lid_files_exit(struct blocklevel_device *bl);
*/
int pldm_watchdog_init(void);
+/**
+ * Update boot progress state
+ */
+int pldm_platform_send_progress_state_change(
+ enum pldm_state_set_boot_progress_state_values state);
+
#endif /* __PLDM_H__ */
--
2.41.0
More information about the Skiboot
mailing list