[Skiboot] [PATCH V2 11/21] core/pldm: Encode state effecter request

Abhishek SIngh Tomar abhishek at linux.ibm.com
Wed Mar 16 22:41:51 AEDT 2022


On Fri, Mar 04, 2022 at 02:11:44PM +0100, Christophe Lombard wrote:
> PLDM effecters provide a general mechanism for controlling or configuring a
> state or numeric setting of an entity.
> The entity state and numeric setting values are written into an effecter.
> 
> PLDM commands are specified for writing the state or numeric setting to an
> effecter. Effecters are identified by and accessed using an EffecterID that
> is unique for each effecter within a given terminus.
> 
> PLDM State Effecters provide a regular command structure for setting state
> information in order to change the state of an entity.
> 
> The SetStateEffecterStates command is used to set the state of one or more
> effecters within a PLDM State Effecter.
> 
> Signed-off-by: Christophe Lombard <clombard at linux.vnet.ibm.com>
Reviewed-by: Abhishek Singh Tomar <abhishek at linux.ibm.com>
> ---
>  core/pldm/pldm-platform-requests.c | 70 ++++++++++++++++++++++++++++++
>  1 file changed, 70 insertions(+)
> 
> diff --git a/core/pldm/pldm-platform-requests.c b/core/pldm/pldm-platform-requests.c
> index b416cc76..725d1a2e 100644
> --- a/core/pldm/pldm-platform-requests.c
> +++ b/core/pldm/pldm-platform-requests.c
> @@ -32,6 +32,76 @@ static void pdr_init_complete(bool success)
>  	pdr_ready = true;
>  }
>  
> +struct set_effecter_state_response {
> +	uint8_t completion_code;
> +};
> +
> +
> +
> +/*
> + * Create and send a PLDM request message for SetStateEffecterStates.
> + */
> +static int set_state_effecter_states_req(uint16_t effecter_id,
> +					 set_effecter_state_field *field,
> +					 bool no_timeout)
> +{
> +	char request_msg[PKT_SIZE(struct pldm_set_state_effecter_states_req)];
> +	size_t response_len, payload_len;
> +	void *response_msg;
> +	int rc;
> +
> +	struct pldm_set_state_effecter_states_req states_req = {
> +		.effecter_id = effecter_id,
> +		.comp_effecter_count = 1
> +	};
> +
> +	struct set_effecter_state_response response;
> +
> +	/* Encode the state effecter states request */
> +	rc = encode_set_state_effecter_states_req(
> +			DEFAULT_INSTANCE_ID,
> +			states_req.effecter_id,
> +			states_req.comp_effecter_count,
> +			field,
> +			(struct pldm_msg *)request_msg);
> +	if (rc != PLDM_SUCCESS) {
> +		prlog(PR_ERR, "Encode SetStateEffecter Error (rc: %d)\n",
> +			      rc);
> +		return OPAL_PARAMETER;
> +	}
> +
> +	/* Send and get the response message bytes */
> +	rc = pldm_do_request(BMC_EID, request_msg, sizeof(request_msg),
> +			     &response_msg, &response_len);
> +	if (rc) {
> +		/* it may happen that for some commands, the responder
> +		 * does not have time to respond
> +		 */
> +		if (no_timeout && (rc == OPAL_TIMEOUT)) {
> +			free(response_msg);
> +			return OPAL_SUCCESS;
> +		}
> +		prlog(PR_ERR, "PLDM: Communication Error (req: SetStateEffecter, rc: %d)\n", rc);
> +		return OPAL_PARAMETER;
> +	}
> +
> +	/* Decode the message */
> +	payload_len = response_len - sizeof(struct pldm_msg_hdr);
> +	rc = decode_set_state_effecter_states_resp(
> +				response_msg,
> +				payload_len,
> +				&response.completion_code);
> +	if (rc != PLDM_SUCCESS || response.completion_code != PLDM_SUCCESS) {
> +		prlog(PR_ERR, "Decode SetStateEffecter Error (rc: %d, cc: %d)\n",
> +			      rc, response.completion_code);
> +		return OPAL_PARAMETER;
> +	}
> +
> +	free(response_msg);
> +
> +	return OPAL_SUCCESS;
> +}
> +
>  struct get_pdr_response {
>  	uint8_t completion_code;
>  	uint32_t next_record_hndl;
> -- 
> 2.35.1
> 
> _______________________________________________
> Skiboot mailing list
> Skiboot at lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/skiboot


More information about the Skiboot mailing list