src/msgbuf/platform.h

#ifndef PLDM_MSGBUF_PLATFORM_H
#define PLDM_MSGBUF_PLATFORM_H

#include "../msgbuf.h"
#include <libpldm/base.h>
#include <libpldm/platform.h>

static inline int
pldm_msgbuf_extract_value_pdr_hdr(struct pldm_msgbuf *ctx,
				  struct pldm_value_pdr_hdr *hdr)
{
	pldm_msgbuf_extract(ctx, &hdr->record_handle);
	pldm_msgbuf_extract(ctx, &hdr->version);
	pldm_msgbuf_extract(ctx, &hdr->type);
	pldm_msgbuf_extract(ctx, &hdr->record_change_num);
	pldm_msgbuf_extract(ctx, &hdr->length);

	return pldm_msgbuf_validate(ctx);
}

/*
 * We use __attribute__((always_inline)) below so the compiler has visibility of
 * the switch() at the call site. It is often the case that the size of multiple
 * fields depends on the tag. Inlining thus gives the compiler visibility to
 * hoist one tag-based code-path condition to cover all invocations.
 */

__attribute__((always_inline)) static inline int
pldm_msgbuf_extract_sensor_data(struct pldm_msgbuf *ctx,
				enum pldm_sensor_readings_data_type tag,
				union_sensor_data_size *dst)
{
	switch (tag) {
	case PLDM_SENSOR_DATA_SIZE_UINT8:
		return pldm_msgbuf_extract(ctx, &dst->value_u8);
	case PLDM_SENSOR_DATA_SIZE_SINT8:
		return pldm_msgbuf_extract(ctx, &dst->value_s8);
	case PLDM_SENSOR_DATA_SIZE_UINT16:
		return pldm_msgbuf_extract(ctx, &dst->value_u16);
	case PLDM_SENSOR_DATA_SIZE_SINT16:
		return pldm_msgbuf_extract(ctx, &dst->value_s16);
	case PLDM_SENSOR_DATA_SIZE_UINT32:
		return pldm_msgbuf_extract(ctx, &dst->value_u32);
	case PLDM_SENSOR_DATA_SIZE_SINT32:
		return pldm_msgbuf_extract(ctx, &dst->value_s32);
	}

	return -PLDM_ERROR_INVALID_DATA;
}

/*
 * This API is bad, but it's because the caller's APIs are also bad. They should
 * have used the approach used by callers of pldm_msgbuf_extract_sensor_data()
 * above
 */
__attribute__((always_inline)) static inline int
pldm_msgbuf_extract_sensor_value(struct pldm_msgbuf *ctx,
				 enum pldm_effecter_data_size tag, uint8_t *val)
{
	switch (tag) {
	case PLDM_SENSOR_DATA_SIZE_UINT8:
		return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val);
	case PLDM_SENSOR_DATA_SIZE_SINT8:
		return pldm_msgbuf_extract_int8(ctx, (int8_t *)val);
	case PLDM_SENSOR_DATA_SIZE_UINT16:
		return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val);
	case PLDM_SENSOR_DATA_SIZE_SINT16:
		return pldm_msgbuf_extract_int16(ctx, (int16_t *)val);
	case PLDM_SENSOR_DATA_SIZE_UINT32:
		return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val);
	case PLDM_SENSOR_DATA_SIZE_SINT32:
		return pldm_msgbuf_extract_int32(ctx, (int32_t *)val);
	}

	return -PLDM_ERROR_INVALID_DATA;
}

__attribute__((always_inline)) static inline int
pldm_msgbuf_extract_range_field_format(struct pldm_msgbuf *ctx,
				       enum pldm_range_field_format tag,
				       union_range_field_format *dst)
{
	switch (tag) {
	case PLDM_RANGE_FIELD_FORMAT_UINT8:
		return pldm_msgbuf_extract(ctx, &dst->value_u8);
	case PLDM_RANGE_FIELD_FORMAT_SINT8:
		return pldm_msgbuf_extract(ctx, &dst->value_s8);
	case PLDM_RANGE_FIELD_FORMAT_UINT16:
		return pldm_msgbuf_extract(ctx, &dst->value_u16);
	case PLDM_RANGE_FIELD_FORMAT_SINT16:
		return pldm_msgbuf_extract(ctx, &dst->value_s16);
	case PLDM_RANGE_FIELD_FORMAT_UINT32:
		return pldm_msgbuf_extract(ctx, &dst->value_u32);
	case PLDM_RANGE_FIELD_FORMAT_SINT32:
		return pldm_msgbuf_extract(ctx, &dst->value_s32);
	case PLDM_RANGE_FIELD_FORMAT_REAL32:
		return pldm_msgbuf_extract(ctx, &dst->value_f32);
	}

	return -PLDM_ERROR_INVALID_DATA;
}

/* This API is bad, but it's because the caller's APIs are also bad */
__attribute__((always_inline)) static inline int
pldm_msgbuf_extract_effecter_value(struct pldm_msgbuf *ctx,
				   enum pldm_effecter_data_size tag,
				   uint8_t *val)
{
	switch (tag) {
	case PLDM_EFFECTER_DATA_SIZE_UINT8:
		return pldm_msgbuf_extract_uint8(ctx, (uint8_t *)val);
	case PLDM_EFFECTER_DATA_SIZE_SINT8:
		return pldm_msgbuf_extract_int8(ctx, (int8_t *)val);
	case PLDM_EFFECTER_DATA_SIZE_UINT16:
		return pldm_msgbuf_extract_uint16(ctx, (uint16_t *)val);
	case PLDM_EFFECTER_DATA_SIZE_SINT16:
		return pldm_msgbuf_extract_int16(ctx, (int16_t *)val);
	case PLDM_EFFECTER_DATA_SIZE_UINT32:
		return pldm_msgbuf_extract_uint32(ctx, (uint32_t *)val);
	case PLDM_EFFECTER_DATA_SIZE_SINT32:
		return pldm_msgbuf_extract_int32(ctx, (int32_t *)val);
	}

	return -PLDM_ERROR_INVALID_DATA;
}

#endif