[{"id":1765139,"web_url":"http://patchwork.ozlabs.org/comment/1765139/","msgid":"<58b78bbb-429b-4072-ec9c-1409926b9d4a@canonical.com>","list_archive_url":null,"date":"2017-09-08T07:50:06","subject":"ACK: [PATCH][V2] lib: fwts_acpi_tables: add a new function to check\n\tReserved field","submitter":{"id":2900,"url":"http://patchwork.ozlabs.org/api/people/2900/","name":"Colin Ian King","email":"colin.king@canonical.com"},"content":"On 08/09/17 01:44, Alex Hung wrote:\n> Signed-off-by: Alex Hung \n> ---\n> src/lib/include/fwts_acpi_tables.h | 2 ++\n> src/lib/src/fwts_acpi_tables.c | 43 ++++++++++++++++++++++++++++++++++++++\n> 2 files changed, 45 insertions(+)\n> \n> diff --git a/src/lib/include/fwts_acpi_tables.h b/src/lib/include/fwts_acpi_tables.h\n> index 2527984..d478d99 100644\n> --- a/src/lib/include/fwts_acpi_tables.h\n> +++ b/src/lib/include/fwts_acpi_tables.h\n> @@ -53,6 +53,8 @@ bool fwts_acpi_obj_find(fwts_framework *fw, const char *obj_name);\n> \n> fwts_bool fwts_acpi_is_reduced_hardware(const fwts_acpi_table_fadt *fadt);\n> \n> +void fwts_acpi_reserved_zero_check(fwts_framework *fw, const char *table, const char *field, uint64_t value, uint8_t size, bool *passed);\n> +\n> #endif\n> \n> #endif\n> diff --git a/src/lib/src/fwts_acpi_tables.c b/src/lib/src/fwts_acpi_tables.c\n> index 3c3a63a..4ac70f8 100644\n> --- a/src/lib/src/fwts_acpi_tables.c\n> +++ b/src/lib/src/fwts_acpi_tables.c\n> @@ -1383,4 +1383,47 @@ bool fwts_acpi_obj_find(fwts_framework *fw, const char *obj_name)\n> \treturn found;\n> }\n> \n> +/*\n> + * fwts_acpi_reserved_zero_check()\n> + * verify whether the reserved field is zero\n> + */\n> +void fwts_acpi_reserved_zero_check(\n> +\tfwts_framework *fw,\n> +\tconst char *table,\n> +\tconst char *field,\n> +\tuint64_t value,\n> +\tuint8_t size,\n> +\tbool *passed)\n> +{\n> +\tif (value != 0) {\n> +\t\tchar label[20];\n> +\t\tstrncpy(label, table, 4);\t/* ACPI table name is 4 char long */\n> +\t\tstrncpy(label + 4, \"ReservedNonZero\", sizeof(label) - 4);\n> +\n> +\t\tswitch (size) {\n> +\t\tcase sizeof(uint8_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%2.2\" PRIx8 \" instead\", table, field, (uint8_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint16_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%4.4\" PRIx16 \" instead\", table, field, (uint16_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint32_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%8.8\" PRIx32 \" instead\", table, field, (uint32_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint64_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%16.16\" PRIx64 \" instead\", table, field, value);\n> +\t\t\tbreak;\n> +\t\t}\n> +\t\t*passed = false;\n> +\t}\n> +}\n> +\n> #endif\n> \n\nThanks Alex.\n\nAcked-by: Colin Ian King ","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=fwts-devel-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xpTxx5Xy1z9s71;\n\tFri, 8 Sep 2017 17:50:13 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1dqE3F-00079I-CP; Fri, 08 Sep 2017 07:50:09 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1dqE3D-00078T-Be\n\tfor fwts-devel@lists.ubuntu.com; Fri, 08 Sep 2017 07:50:07 +0000","from 1.general.cking.uk.vpn ([10.172.193.212])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1dqE3D-0005YB-2l; Fri, 08 Sep 2017 07:50:07 +0000"],"Subject":"ACK: [PATCH][V2] lib: fwts_acpi_tables: add a new function to check\n\tReserved field","To":"fwts-devel@lists.ubuntu.com","References":"<1504831492-16508-1-git-send-email-alex.hung@canonical.com>","From":"Colin Ian King ","Message-ID":"<58b78bbb-429b-4072-ec9c-1409926b9d4a@canonical.com>","Date":"Fri, 8 Sep 2017 08:50:06 +0100","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101\n\tThunderbird/45.8.0","MIME-Version":"1.0","In-Reply-To":"<1504831492-16508-1-git-send-email-alex.hung@canonical.com>","X-BeenThere":"fwts-devel@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Firmware Test Suite Development ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Errors-To":"fwts-devel-bounces@lists.ubuntu.com","Sender":"\"fwts-devel\" "}},{"id":1772530,"web_url":"http://patchwork.ozlabs.org/comment/1772530/","msgid":"<2970e799-0f26-74ee-5710-1e796bae4d0b@canonical.com>","list_archive_url":null,"date":"2017-09-21T08:24:37","subject":"ACK: [PATCH][V2] lib: fwts_acpi_tables: add a new function to check\n\tReserved field","submitter":{"id":14061,"url":"http://patchwork.ozlabs.org/api/people/14061/","name":"Ivan Hu","email":"ivan.hu@canonical.com"},"content":"On 09/08/2017 08:44 AM, Alex Hung wrote:\n> Signed-off-by: Alex Hung \n> ---\n> src/lib/include/fwts_acpi_tables.h | 2 ++\n> src/lib/src/fwts_acpi_tables.c | 43 ++++++++++++++++++++++++++++++++++++++\n> 2 files changed, 45 insertions(+)\n> \n> diff --git a/src/lib/include/fwts_acpi_tables.h b/src/lib/include/fwts_acpi_tables.h\n> index 2527984..d478d99 100644\n> --- a/src/lib/include/fwts_acpi_tables.h\n> +++ b/src/lib/include/fwts_acpi_tables.h\n> @@ -53,6 +53,8 @@ bool fwts_acpi_obj_find(fwts_framework *fw, const char *obj_name);\n> \n> fwts_bool fwts_acpi_is_reduced_hardware(const fwts_acpi_table_fadt *fadt);\n> \n> +void fwts_acpi_reserved_zero_check(fwts_framework *fw, const char *table, const char *field, uint64_t value, uint8_t size, bool *passed);\n> +\n> #endif\n> \n> #endif\n> diff --git a/src/lib/src/fwts_acpi_tables.c b/src/lib/src/fwts_acpi_tables.c\n> index 3c3a63a..4ac70f8 100644\n> --- a/src/lib/src/fwts_acpi_tables.c\n> +++ b/src/lib/src/fwts_acpi_tables.c\n> @@ -1383,4 +1383,47 @@ bool fwts_acpi_obj_find(fwts_framework *fw, const char *obj_name)\n> \treturn found;\n> }\n> \n> +/*\n> + * fwts_acpi_reserved_zero_check()\n> + * verify whether the reserved field is zero\n> + */\n> +void fwts_acpi_reserved_zero_check(\n> +\tfwts_framework *fw,\n> +\tconst char *table,\n> +\tconst char *field,\n> +\tuint64_t value,\n> +\tuint8_t size,\n> +\tbool *passed)\n> +{\n> +\tif (value != 0) {\n> +\t\tchar label[20];\n> +\t\tstrncpy(label, table, 4);\t/* ACPI table name is 4 char long */\n> +\t\tstrncpy(label + 4, \"ReservedNonZero\", sizeof(label) - 4);\n> +\n> +\t\tswitch (size) {\n> +\t\tcase sizeof(uint8_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%2.2\" PRIx8 \" instead\", table, field, (uint8_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint16_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%4.4\" PRIx16 \" instead\", table, field, (uint16_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint32_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%8.8\" PRIx32 \" instead\", table, field, (uint32_t)value);\n> +\t\t\tbreak;\n> +\t\tcase sizeof(uint64_t):\n> +\t\t\tfwts_failed(fw, LOG_LEVEL_MEDIUM, label,\n> +\t\t\t\t\"%4.4s %s field must be zero, got \"\n> +\t\t\t\t\"0x%16.16\" PRIx64 \" instead\", table, field, value);\n> +\t\t\tbreak;\n> +\t\t}\n> +\t\t*passed = false;\n> +\t}\n> +}\n> +\n> #endif\n> \n\nAcked-by: Ivan Hu ","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.ubuntu.com\n\t(client-ip=91.189.94.19; helo=huckleberry.canonical.com;\n\tenvelope-from=fwts-devel-bounces@lists.ubuntu.com;\n\treceiver=)","Received":["from huckleberry.canonical.com (huckleberry.canonical.com\n\t[91.189.94.19])\n\tby ozlabs.org (Postfix) with ESMTP id 3xyV5m06kKz9t3C;\n\tThu, 21 Sep 2017 18:24:43 +1000 (AEST)","from localhost ([127.0.0.1] helo=huckleberry.canonical.com)\n\tby huckleberry.canonical.com with esmtp (Exim 4.86_2)\n\t(envelope-from )\n\tid 1duwmo-0004EQ-D3; Thu, 21 Sep 2017 08:24:42 +0000","from youngberry.canonical.com ([91.189.89.112])\n\tby huckleberry.canonical.com with esmtps\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:128)\n\t(Exim 4.86_2) (envelope-from )\n\tid 1duwmn-0004E8-42\n\tfor fwts-devel@lists.ubuntu.com; Thu, 21 Sep 2017 08:24:41 +0000","from [175.181.226.10] (helo=[192.168.31.95])\n\tby youngberry.canonical.com with esmtpsa\n\t(TLS1.0:DHE_RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.76) (envelope-from )\n\tid 1duwmm-0008FK-Jj\n\tfor fwts-devel@lists.ubuntu.com; Thu, 21 Sep 2017 08:24:40 +0000"],"Subject":"ACK: [PATCH][V2] lib: fwts_acpi_tables: add a new function to check\n\tReserved field","To":"fwts-devel@lists.ubuntu.com","References":"<1504831492-16508-1-git-send-email-alex.hung@canonical.com>","From":"ivanhu ","Message-ID":"<2970e799-0f26-74ee-5710-1e796bae4d0b@canonical.com>","Date":"Thu, 21 Sep 2017 16:24:37 +0800","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<1504831492-16508-1-git-send-email-alex.hung@canonical.com>","Content-Language":"en-GB","X-BeenThere":"fwts-devel@lists.ubuntu.com","X-Mailman-Version":"2.1.20","Precedence":"list","List-Id":"Firmware Test Suite Development ","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Content-Transfer-Encoding":"base64","Content-Type":"text/plain; charset=\"utf-8\"; Format=\"flowed\"","Errors-To":"fwts-devel-bounces@lists.ubuntu.com","Sender":"\"fwts-devel\" "}}]