ACK: [PATCH 2/2] srat: replace structure length checks by fwts_acpi_structure_length_check
ivanhu
ivan.hu at canonical.com
Wed Mar 6 02:34:06 UTC 2019
On 2/27/19 1:18 AM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
> src/acpi/srat/srat.c | 48 +++++++++++---------------------------------
> 1 file changed, 12 insertions(+), 36 deletions(-)
>
> diff --git a/src/acpi/srat/srat.c b/src/acpi/srat/srat.c
> index ced36d18..7adc9a78 100644
> --- a/src/acpi/srat/srat.c
> +++ b/src/acpi/srat/srat.c
> @@ -62,12 +62,8 @@ static void srat_check_local_apic_sapic_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_local_apic_sapic_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATLocalApicSapicAffinityLength",
> - "SRAT Local APIC/SPAIC Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_local_apic_sapic_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_local_apic_sapic_affinity))) {
> *passed = false;
> goto done;
> }
> @@ -120,12 +116,8 @@ static void srat_check_memory_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_memory_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATMemoryAffinityLength",
> - "SRAT Memory Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_memory_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_memory_affinity))) {
> *passed = false;
> goto done;
> }
> @@ -170,12 +162,8 @@ static void srat_check_local_x2apic_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_local_x2apic_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATLocalx2apicAffinityLength",
> - "SRAT Local x2APIC Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_local_x2apic_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_local_x2apic_affinity))) {
> *passed = false;
> goto done;
> }
> @@ -232,12 +220,8 @@ static void srat_check_gicc_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_gicc_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATGICCAffinityLength",
> - "SRAT GICC Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_gicc_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_gicc_affinity))) {
> *passed = false;
> goto done;
> }
> @@ -282,12 +266,8 @@ static void srat_check_its_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_its_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATITSAffinityLength",
> - "SRAT ITS Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_its_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_its_affinity))) {
> *passed = false;
> goto done;
> }
> @@ -329,12 +309,8 @@ static void srat_check_initiator_affinity(
> goto done;
> }
>
> - if (affinity->length != sizeof(fwts_acpi_table_initiator_affinity)) {
> - fwts_failed(fw, LOG_LEVEL_MEDIUM,
> - "SRATInitiatorAffinityLength",
> - "SRAT Initiator Affinity Length incorrect, got "
> - "%" PRIu8 ", expecting %zu",
> - affinity->length, sizeof(fwts_acpi_table_initiator_affinity));
> + if (!fwts_acpi_structure_length_check(fw, "SRAT", affinity->type,
> + affinity->length, sizeof(fwts_acpi_table_initiator_affinity))) {
> *passed = false;
> goto done;
> }
Acked-by: Ivan Hu <ivan.hu at canonical.com>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/fwts-devel/attachments/20190306/4f6ea24f/attachment.sig>
More information about the fwts-devel
mailing list