{"id":2225386,"url":"http://patchwork.ozlabs.org/api/1.2/patches/2225386/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/patch/20260420220804.71564-4-heinrich.schuchardt@canonical.com/","project":{"id":18,"url":"http://patchwork.ozlabs.org/api/1.2/projects/18/?format=json","name":"U-Boot","link_name":"uboot","list_id":"u-boot.lists.denx.de","list_email":"u-boot@lists.denx.de","web_url":null,"scm_url":null,"webscm_url":null,"list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20260420220804.71564-4-heinrich.schuchardt@canonical.com>","list_archive_url":null,"date":"2026-04-20T22:08:03","name":"[3/4] test: consider index parameter in acpi_find_table test","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"95446fe2d7a2f90682db27f22ff737beaa3dd339","submitter":{"id":82181,"url":"http://patchwork.ozlabs.org/api/1.2/people/82181/?format=json","name":"Heinrich Schuchardt","email":"heinrich.schuchardt@canonical.com"},"delegate":{"id":3184,"url":"http://patchwork.ozlabs.org/api/1.2/users/3184/?format=json","username":"sjg","first_name":"Simon","last_name":"Glass","email":"sjg@chromium.org"},"mbox":"http://patchwork.ozlabs.org/project/uboot/patch/20260420220804.71564-4-heinrich.schuchardt@canonical.com/mbox/","series":[{"id":500693,"url":"http://patchwork.ozlabs.org/api/1.2/series/500693/?format=json","web_url":"http://patchwork.ozlabs.org/project/uboot/list/?series=500693","date":"2026-04-20T22:08:00","name":"acpi: handle duplicate tables in acpi dump","version":1,"mbox":"http://patchwork.ozlabs.org/series/500693/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2225386/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2225386/checks/","tags":{},"related":[],"headers":{"Return-Path":"<u-boot-bounces@lists.denx.de>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (4096-bit key;\n unprotected) header.d=canonical.com header.i=@canonical.com\n header.a=rsa-sha256 header.s=20251003 header.b=K9Z67P2F;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=lists.denx.de\n (client-ip=2a01:238:438b:c500:173d:9f52:ddab:ee01; helo=phobos.denx.de;\n envelope-from=u-boot-bounces@lists.denx.de; receiver=patchwork.ozlabs.org)","phobos.denx.de;\n dmarc=pass (p=reject dis=none) header.from=canonical.com","phobos.denx.de;\n spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de","phobos.denx.de;\n\tdkim=pass (4096-bit key;\n unprotected) header.d=canonical.com header.i=@canonical.com\n header.b=\"K9Z67P2F\";\n\tdkim-atps=neutral","phobos.denx.de; dmarc=pass (p=reject dis=none)\n header.from=canonical.com","phobos.denx.de; spf=pass\n smtp.mailfrom=heinrich.schuchardt@canonical.com"],"Received":["from phobos.denx.de (phobos.denx.de\n [IPv6:2a01:238:438b:c500:173d:9f52:ddab:ee01])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4g004N5VBsz1yCv\n\tfor <incoming@patchwork.ozlabs.org>; Tue, 21 Apr 2026 08:08:36 +1000 (AEST)","from h2850616.stratoserver.net (localhost [IPv6:::1])\n\tby phobos.denx.de (Postfix) with ESMTP id 21602844F4;\n\tTue, 21 Apr 2026 00:08:21 +0200 (CEST)","by phobos.denx.de (Postfix, from userid 109)\n id EE135844EC; Tue, 21 Apr 2026 00:08:12 +0200 (CEST)","from smtp-relay-canonical-0.canonical.com\n (smtp-relay-canonical-0.canonical.com [185.125.188.120])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits))\n (No client certificate requested)\n by phobos.denx.de (Postfix) with ESMTPS id C4EE1844F4\n for <u-boot@lists.denx.de>; Tue, 21 Apr 2026 00:08:07 +0200 (CEST)","from LT03.fritz.box (ip-176-199-115-125.um44.pools.vodafone-ip.de\n [176.199.115.125])\n (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest\n SHA256)\n (No client certificate requested)\n by smtp-relay-canonical-0.canonical.com (Postfix) with ESMTPSA id 392A440D86;\n Mon, 20 Apr 2026 22:08:07 +0000 (UTC)"],"X-Spam-Checker-Version":"SpamAssassin 3.4.2 (2018-09-13) on phobos.denx.de","X-Spam-Level":"","X-Spam-Status":"No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,\n DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,\n RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE,SPF_PASS autolearn=ham\n autolearn_force=no version=3.4.2","DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com;\n s=20251003; t=1776722887;\n bh=rxIqISaqCtvCAlVi1tsmnO7fSoTkG2r1MefQ+mB84B0=;\n h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References:\n MIME-Version;\n b=K9Z67P2FBgE93cro50d3/OW5ff1XIAQ0CxAUbgGbes7iJChHWRXkYGEfhNJt2B31D\n ejCgsmWcuhxSFvZ6QPzpQthL3H/NksMJ0KL2k1H4Jqx5XlyWdL1rQbegMXZr4in50p\n 3ZMk0EqmQtE5RQUVOFXYkA2U6PWyYpNxBZeYw/KqTm/nE4LzRMc3fWIV3l8yWg4yj7\n 08WYvJ2URkbwRBbiD4ETr7cU1uqPWZ3dv26+iViLItIf5DxUiTMYLM9c57/uWl8yjD\n Q+cVFxq6Sc/GxKsTjkHL2GCGFBK2MBeL9X6dYJyFUfEfsnd2w4IfRG27A38mYm8YSz\n kDbQo1lQo0orcK2S/YRFXhRIlPCZVtKn38LTQNNpF1dBlxo6S7UVrBzNgz/m4IqKa4\n bqjcD5teEvZ6lU7HEj2/bxanzdhSFYVXyX9saKA3vjOQd3H5vErF8X9NkDNoVjkVAk\n 63wMaEJPJTD4iathVgA5v1wdhbsvmfbU/wV69RkBJDKNEOab1TlLCUn+acbiRdFU/e\n OnljUt6k4gptsSIvcMyJtwLNhW2KxrhG9ZpX2i4rdIHYTG3lX+G7WkfEhV8MInh6A8\n bnolZ5iyDx1XufEu5AoBcLwRk4gf1s7VwFSOdbbQTUNaVgsbhav+7W2XfLakHi8i1W\n 6PaZ1svg51zjkGZ51ZPTIkN4=","From":"Heinrich Schuchardt <heinrich.schuchardt@canonical.com>","To":"Simon Glass <sjg@chromium.org>","Cc":"Tom Rini <trini@konsulko.com>, Bin Meng <bmeng.cn@gmail.com>,\n u-boot@lists.denx.de,\n Heinrich Schuchardt <heinrich.schuchardt@canonical.com>","Subject":"[PATCH 3/4] test: consider index parameter in acpi_find_table test","Date":"Tue, 21 Apr 2026 00:08:03 +0200","Message-ID":"<20260420220804.71564-4-heinrich.schuchardt@canonical.com>","X-Mailer":"git-send-email 2.53.0","In-Reply-To":"<20260420220804.71564-1-heinrich.schuchardt@canonical.com>","References":"<20260420220804.71564-1-heinrich.schuchardt@canonical.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","X-BeenThere":"u-boot@lists.denx.de","X-Mailman-Version":"2.1.39","Precedence":"list","List-Id":"U-Boot discussion <u-boot.lists.denx.de>","List-Unsubscribe":"<https://lists.denx.de/options/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=unsubscribe>","List-Archive":"<https://lists.denx.de/pipermail/u-boot/>","List-Post":"<mailto:u-boot@lists.denx.de>","List-Help":"<mailto:u-boot-request@lists.denx.de?subject=help>","List-Subscribe":"<https://lists.denx.de/listinfo/u-boot>,\n <mailto:u-boot-request@lists.denx.de?subject=subscribe>","Errors-To":"u-boot-bounces@lists.denx.de","Sender":"\"U-Boot\" <u-boot-bounces@lists.denx.de>","X-Virus-Scanned":"clamav-milter 0.103.8 at phobos.denx.de","X-Virus-Status":"Clean"},"content":"* Add a duplicate table.\n* Check that both instances can be found.\n* Verify index bound checks.\n\nSigned-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>\n---\n test/dm/acpi.c | 16 +++++++++++++++-\n 1 file changed, 15 insertions(+), 1 deletion(-)","diff":"diff --git a/test/dm/acpi.c b/test/dm/acpi.c\nindex 8bec09f4e44..ba533763d96 100644\n--- a/test/dm/acpi.c\n+++ b/test/dm/acpi.c\n@@ -783,7 +783,7 @@ static int dm_test_acpi_find_table(struct unit_test_state *uts)\n \tstruct acpi_ctx ctx;\n \tulong acpi_start, addr;\n \tvoid *buf;\n-\tstruct acpi_table_header *table, *table1, *table2, *table3;\n+\tstruct acpi_table_header *table, *table1, *table2, *table3, *table4;\n \tstruct acpi_rsdp *rsdp;\n \tulong rsdt;\n \tulong xsdt;\n@@ -799,6 +799,7 @@ static int dm_test_acpi_find_table(struct unit_test_state *uts)\n \ttable3 = dm_test_write_test_table(&ctx, 3);\n \ttable1 = dm_test_write_test_table(&ctx, 1);\n \ttable2 = dm_test_write_test_table(&ctx, 2);\n+\ttable4 = dm_test_write_test_table(&ctx, 1);\n \n \t/* Retrieve RSDP, RSDT, XSDT */\n \trsdp = map_sysmem(gd_acpi_start(), 0);\n@@ -812,6 +813,11 @@ static int dm_test_acpi_find_table(struct unit_test_state *uts)\n \ttable = acpi_find_table(\"TST1\", 0);\n \tut_asserteq_ptr(table1, table);\n \tut_asserteq_strn(\"TST1\", table->signature);\n+\ttable = acpi_find_table(\"TST1\", 1);\n+\tut_asserteq_ptr(table4, table);\n+\tut_asserteq_strn(\"TST1\", table->signature);\n+\ttable = acpi_find_table(\"TST1\", 2);\n+\tut_assertnull(table);\n \ttable = acpi_find_table(\"TST2\", 0);\n \tut_asserteq_ptr(table2, table);\n \tut_asserteq_strn(\"TST2\", table->signature);\n@@ -823,6 +829,10 @@ static int dm_test_acpi_find_table(struct unit_test_state *uts)\n \trsdp->rsdt_address = 0;\n \ttable = acpi_find_table(\"TST1\", 0);\n \tut_asserteq_ptr(table1, table);\n+\ttable = acpi_find_table(\"TST1\", 1);\n+\tut_asserteq_ptr(table4, table);\n+\ttable = acpi_find_table(\"TST1\", 2);\n+\tut_assertnull(table);\n \ttable = acpi_find_table(\"TST2\", 0);\n \tut_asserteq_ptr(table2, table);\n \ttable = acpi_find_table(\"TST3\", 0);\n@@ -833,6 +843,10 @@ static int dm_test_acpi_find_table(struct unit_test_state *uts)\n \trsdp->xsdt_address = 0;\n \ttable = acpi_find_table(\"TST1\", 0);\n \tut_asserteq_ptr(table1, table);\n+\ttable = acpi_find_table(\"TST1\", 1);\n+\tut_asserteq_ptr(table4, table);\n+\ttable = acpi_find_table(\"TST1\", 2);\n+\tut_assertnull(table);\n \ttable = acpi_find_table(\"TST2\", 0);\n \tut_asserteq_ptr(table2, table);\n \ttable = acpi_find_table(\"TST3\", 0);\n","prefixes":["3/4"]}