diff mbox series

[V6,12/21] core/pldm: Send a system firmware Graceful Restart request

Message ID 20220913102705.65506-13-clombard@linux.vnet.ibm.com
State Superseded
Headers show
Series Implement MCTP and PLDM features | expand

Commit Message

Christophe Lombard Sept. 13, 2022, 10:26 a.m. UTC 
Set the state information of the PLDM effecter identified by:
the entity type (PLDM_ENTITY_SYS_FIRMWARE) and the state set
PLDM_STATE_SET_SW_TERMINATION_STATUS with the effecter state:
PLDM_SW_TERM_GRACEFUL_RESTART_REQUESTED to request a platform restart.

Reviewed-by: Abhishek Singh Tomar <abhishek at linux.ibm.com>
Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
---
 core/pldm/pldm-platform-requests.c | 75 ++++++++++++++++++++++++++++++
 include/pldm.h                     |  5 ++
 2 files changed, 80 insertions(+)

Comments

Abhishek Singh Tomar Sept. 29, 2022, 4:57 p.m. UTC | #1 
On Tue, Sep 13, 2022 at 12:26:56PM +0200, Christophe Lombard wrote:
> Set the state information of the PLDM effecter identified by:
> the entity type (PLDM_ENTITY_SYS_FIRMWARE) and the state set
> PLDM_STATE_SET_SW_TERMINATION_STATUS with the effecter state:
> PLDM_SW_TERM_GRACEFUL_RESTART_REQUESTED to request a platform restart.
> 
> Reviewed-by: Abhishek Singh Tomar <abhishek at linux.ibm.com>
Reviewed-by: Abhishek Singh Tomar <abhishek@linux.ibm.com>
> Signed-off-by: Christophe Lombard <clombard@linux.vnet.ibm.com>
> ---
>  core/pldm/pldm-platform-requests.c | 75 ++++++++++++++++++++++++++++++
>  include/pldm.h                     |  5 ++
>  2 files changed, 80 insertions(+)
> 
> diff --git a/core/pldm/pldm-platform-requests.c b/core/pldm/pldm-platform-requests.c
> index 93b98438..99d58158 100644
> --- a/core/pldm/pldm-platform-requests.c
> +++ b/core/pldm/pldm-platform-requests.c
> @@ -39,6 +39,49 @@ static void pdr_init_complete(bool success)
>  	pdr_ready = true;
>  }
> 
> +/*
> + * Search the matching record and return the effecter id.
> + * PDR type = PLDM_STATE_EFFECTER_PDR
> + */
> +static int find_effecter_id_by_state_set_Id(uint16_t entity_type,
> +					    uint16_t state_set_id,
> +					    uint16_t *effecter_id)
> +{
> +	struct state_effecter_possible_states *possible_states;
> +	struct pldm_state_effecter_pdr *state_effecter_pdr;
> +	const pldm_pdr_record *record = NULL;
> +	uint8_t *outData = NULL;
> +	uint32_t size;
> +
> +	do {
> +		/* Find (first) PDR record by PLDM_STATE_EFFECTER_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_EFFECTER_PDR,
> +				record, /* PDR record handle */
> +				&outData, &size);
> +
> +		if (record) {
> +			state_effecter_pdr = (struct pldm_state_effecter_pdr *) outData;
> +
> +			*effecter_id = le16_to_cpu(state_effecter_pdr->effecter_id);
> +
> +			possible_states = (struct state_effecter_possible_states *)
> +				state_effecter_pdr->possible_states;
> +
> +			if ((le16_to_cpu(state_effecter_pdr->entity_type) == entity_type) &&
> +			    (le16_to_cpu(possible_states->state_set_id) == state_set_id))
> +				return OPAL_SUCCESS;
> +		}
> +
> +	} while (record);
> +
> +	return OPAL_PARAMETER;
> +}
> +
>  struct set_effecter_state_response {
>  	uint8_t completion_code;
>  };
> @@ -115,6 +158,38 @@ static int set_state_effecter_states_req(uint16_t effecter_id,
>  	return OPAL_SUCCESS;
>  }
> 
> +/*
> + * entity_type:  System Firmware
> + * state_set:    Software Termination Status(129)
> + * states:       Graceful Restart Requested(6)
> + */
> +int pldm_platform_restart(void)
> +{
> +	set_effecter_state_field field;
> +	uint16_t effecter_id;
> +	int rc;
> +
> +	if (!pdr_ready)
> +		return OPAL_HARDWARE;
> +
> +	rc = find_effecter_id_by_state_set_Id(
> +				PLDM_ENTITY_SYS_FIRMWARE,
> +				PLDM_STATE_SET_SW_TERMINATION_STATUS,
> +				&effecter_id);
> +	if (rc) {
> +		prlog(PR_ERR, "%s - effecter id not found\n", __func__);
> +		return rc;
> +	}
> +
> +	field.set_request = PLDM_REQUEST_SET;
> +	field.effecter_state = PLDM_SW_TERM_GRACEFUL_RESTART_REQUESTED;
> +
> +	prlog(PR_INFO, "sending system firmware Graceful Restart request (effecter_id: %d)\n",
> +			effecter_id);
> +
> +	return set_state_effecter_states_req(effecter_id, &field, true);
> +}
> +
>  struct get_pdr_response {
>  	uint8_t completion_code;
>  	uint32_t next_record_hndl;
> diff --git a/include/pldm.h b/include/pldm.h
> index 3aae560c..2429b5e5 100644
> --- a/include/pldm.h
> +++ b/include/pldm.h
> @@ -15,4 +15,9 @@ int pldm_mctp_init(uint8_t mode);
>   */
>  void pldm_mctp_exit(void);
> 
> +/**
> + * Send a system firmware Graceful Restart request
> + */
> +int pldm_platform_restart(void);
> +
>  #endif /* __PLDM_H__ */
> -- 
> 2.37.3
> 
> _______________________________________________
> Skiboot mailing list
> Skiboot@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/skiboot
diff mbox series

Patch

diff --git a/core/pldm/pldm-platform-requests.c b/core/pldm/pldm-platform-requests.c
index 93b98438..99d58158 100644
--- a/core/pldm/pldm-platform-requests.c
+++ b/core/pldm/pldm-platform-requests.c
@@ -39,6 +39,49 @@  static void pdr_init_complete(bool success)
 	pdr_ready = true;
 }
 
+/*
+ * Search the matching record and return the effecter id.
+ * PDR type = PLDM_STATE_EFFECTER_PDR
+ */
+static int find_effecter_id_by_state_set_Id(uint16_t entity_type,
+					    uint16_t state_set_id,
+					    uint16_t *effecter_id)
+{
+	struct state_effecter_possible_states *possible_states;
+	struct pldm_state_effecter_pdr *state_effecter_pdr;
+	const pldm_pdr_record *record = NULL;
+	uint8_t *outData = NULL;
+	uint32_t size;
+
+	do {
+		/* Find (first) PDR record by PLDM_STATE_EFFECTER_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_EFFECTER_PDR,
+				record, /* PDR record handle */
+				&outData, &size);
+
+		if (record) {
+			state_effecter_pdr = (struct pldm_state_effecter_pdr *) outData;
+
+			*effecter_id = le16_to_cpu(state_effecter_pdr->effecter_id);
+
+			possible_states = (struct state_effecter_possible_states *)
+				state_effecter_pdr->possible_states;
+
+			if ((le16_to_cpu(state_effecter_pdr->entity_type) == entity_type) &&
+			    (le16_to_cpu(possible_states->state_set_id) == state_set_id))
+				return OPAL_SUCCESS;
+		}
+
+	} while (record);
+
+	return OPAL_PARAMETER;
+}
+
 struct set_effecter_state_response {
 	uint8_t completion_code;
 };
@@ -115,6 +158,38 @@  static int set_state_effecter_states_req(uint16_t effecter_id,
 	return OPAL_SUCCESS;
 }
 
+/*
+ * entity_type:  System Firmware
+ * state_set:    Software Termination Status(129)
+ * states:       Graceful Restart Requested(6)
+ */
+int pldm_platform_restart(void)
+{
+	set_effecter_state_field field;
+	uint16_t effecter_id;
+	int rc;
+
+	if (!pdr_ready)
+		return OPAL_HARDWARE;
+
+	rc = find_effecter_id_by_state_set_Id(
+				PLDM_ENTITY_SYS_FIRMWARE,
+				PLDM_STATE_SET_SW_TERMINATION_STATUS,
+				&effecter_id);
+	if (rc) {
+		prlog(PR_ERR, "%s - effecter id not found\n", __func__);
+		return rc;
+	}
+
+	field.set_request = PLDM_REQUEST_SET;
+	field.effecter_state = PLDM_SW_TERM_GRACEFUL_RESTART_REQUESTED;
+
+	prlog(PR_INFO, "sending system firmware Graceful Restart request (effecter_id: %d)\n",
+			effecter_id);
+
+	return set_state_effecter_states_req(effecter_id, &field, true);
+}
+
 struct get_pdr_response {
 	uint8_t completion_code;
 	uint32_t next_record_hndl;
diff --git a/include/pldm.h b/include/pldm.h
index 3aae560c..2429b5e5 100644
--- a/include/pldm.h
+++ b/include/pldm.h
@@ -15,4 +15,9 @@  int pldm_mctp_init(uint8_t mode);
  */
 void pldm_mctp_exit(void);
 
+/**
+ * Send a system firmware Graceful Restart request
+ */
+int pldm_platform_restart(void);
+
 #endif /* __PLDM_H__ */