Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2217854/?format=api
{ "id": 2217854, "url": "http://patchwork.ozlabs.org/api/patches/2217854/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260330185236.1905636-6-peter.maydell@linaro.org/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260330185236.1905636-6-peter.maydell@linaro.org>", "list_archive_url": null, "date": "2026-03-30T18:52:36", "name": "[PULL,5/5] hw/arm/xilinx_zynq: Use strcasecmp to parse boot-mode option values", "commit_ref": null, "pull_url": null, "state": "not-applicable", "archived": false, "hash": "64794d0bc8d21ea9bcce5cf2a3e7e05e6ba66408", "submitter": { "id": 5111, "url": "http://patchwork.ozlabs.org/api/people/5111/?format=api", "name": "Peter Maydell", "email": "peter.maydell@linaro.org" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/patch/20260330185236.1905636-6-peter.maydell@linaro.org/mbox/", "series": [ { "id": 498079, "url": "http://patchwork.ozlabs.org/api/series/498079/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=498079", "date": "2026-03-30T18:52:33", "name": "[PULL,1/5] target/arm: Don't skip access flag fault for AccessType_AT", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/498079/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2217854/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2217854/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@legolas.ozlabs.org", "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=linaro.org header.i=@linaro.org header.a=rsa-sha256\n header.s=google header.b=jwaFwx6S;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)" ], "Received": [ "from lists.gnu.org (lists.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fl0lX1nQHz1xtJ\n\tfor <incoming@patchwork.ozlabs.org>; Tue, 31 Mar 2026 05:54:00 +1100 (AEDT)", "from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1w7Hjs-0007n7-Fa; Mon, 30 Mar 2026 14:53:29 -0400", "from eggs.gnu.org ([2001:470:142:3::10])\n by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <peter.maydell@linaro.org>)\n id 1w7HjE-0007i4-Sh\n for qemu-devel@nongnu.org; Mon, 30 Mar 2026 14:52:49 -0400", "from mail-wr1-x429.google.com ([2a00:1450:4864:20::429])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)\n (Exim 4.90_1) (envelope-from <peter.maydell@linaro.org>)\n id 1w7HjD-00039j-5G\n for qemu-devel@nongnu.org; Mon, 30 Mar 2026 14:52:48 -0400", "by mail-wr1-x429.google.com with SMTP id\n ffacd0b85a97d-43b87970468so4387716f8f.3\n for <qemu-devel@nongnu.org>; Mon, 30 Mar 2026 11:52:45 -0700 (PDT)", "from lanath.. (wildly.archaic.org.uk. [81.2.115.145])\n by smtp.gmail.com with ESMTPSA id\n ffacd0b85a97d-43cf21f1749sm21229400f8f.14.2026.03.30.11.52.43\n for <qemu-devel@nongnu.org>\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Mon, 30 Mar 2026 11:52:43 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=linaro.org; s=google; t=1774896764; x=1775501564; darn=nongnu.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:to:from:from:to:cc:subject:date:message-id\n :reply-to; bh=Xv4fXudi8WmA4+DY+5K9JiStPbSVSJvSdVrxjTUqZ30=;\n b=jwaFwx6SGUF18VvmcJ6DxZs4fqSG7KLo/eaZF3JDFuqyaQU1mUCDsX2Eh7RwHtBTA8\n xZ3ENvZ1gA7CpoqVKG47+ciOCFxvLxwzw29nN/HljPvup7/r4bgQPZ0A2VDSaC0jHQtw\n ck1lCvj93TF/8/WHDHBKVJRzDuolQtQLYBrfsoEWselN1EPT7KpWvRYLwyy2CC4CzoEi\n Zz7GGxprdW3j6PVx5nTBBs3VGGdp3t+9Y6Alt/jXpbjvShWkLWDKcLJyKMIpqIbMert6\n SRiI2q4XFZn6vbvrBvXFzxS+TpZ6itC8fU9BUMUfpDQEXRO1Uc4bBOi0+WCNUzWC02Mh\n PYxA==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1774896764; x=1775501564;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:to:from:x-gm-gg:x-gm-message-state:from:to\n :cc:subject:date:message-id:reply-to;\n bh=Xv4fXudi8WmA4+DY+5K9JiStPbSVSJvSdVrxjTUqZ30=;\n b=jderCelg1vmoCZ4fyD4zDQFNFNaFS3QyPWiYd9Mh7LMA1YBewC7A0ZoAid7iOz1JJz\n QzKppND8RsBpi1KhUZmog8c3P8PfkH9J6tHDFOpXyVtoTYlTcZy10M5XuLD7PEHEOQRZ\n dLGbNecSuL51YnqebleljAX7e2Z4+Rk5rYBdFgsdhnZ6PFRWDeoVbTjoPXVa5LONi3me\n KTe6WLgbUdJsPAbsD8ohdgFbYvBtn+asUQE1E1j1I6GAIhG+bBh3XkhgckC/VFSbfCx/\n j4+9ZPq+eIWWNqmgT81ZD4bylw+K3CDVXIk/npE3MJO1TSayIPZFx65klTftDlI45FSV\n kApw==", "X-Gm-Message-State": "AOJu0YzOV+aR9jT8/pXZBB0zaWQkQhg9WVT0uI0xn2ORwiFHrF43hTwH\n DlzSxAATQNd6SM3urWnn/72ILeEdEySPDiLVQwTZ4hpZiZFBiUu7ZKDioGvUIyMWMgQxp+9KHXH\n j99WuXqg=", "X-Gm-Gg": "ATEYQzw6YkYSEAd+0NjFi+A/plskVZpJJVqGlT0FgFih+u0noalba7NPlOJarueg20d\n tSC8OY3Az5w99P/Adj2KtrMkDw4afa1lEn11gGZ0e8qlMldX9avoni3l1ov4h8v6WHUIsY11MDZ\n dRG2bfBm9StOevoMXfNpXclAJMD/vQOMdcGTWkqdQmjYMrLwYKwAJv7F9JqlYpvh29oAkVR6CKP\n 8KKFZchdFf0feUipOKJClMheflZosPexR+oGU8rgwT9C441z6vGjsNCTpi4E2KUEhXvf3nZlnLU\n pbHw6BXRk3YkkGYeptZ9trOBajzFuKH3ux+df0TDfyLGYPkWs9D8Ohgawctfp/4EO/aP0z90OJ/\n ta3HtD2SsDHBmTXJdfCBRsjSkKaK6+/f3gIv5JUbOK7inTrS+BUfM/lpw05A3hajXWeedCrS/yC\n xDCLBHpTC1KE8DstumW9HAp/BJ4ylWvp4PHGelqCLzNpZAwF2MFehFmrPjhBJxfEoTIjlXXS84n\n pBuEDsOCCrIyOkqkw/lfeiMnR7tz1k=", "X-Received": "by 2002:a05:6000:40db:b0:43c:fd0a:5c6c with SMTP id\n ffacd0b85a97d-43cfd0a5fc9mr9798822f8f.0.1774896764197;\n Mon, 30 Mar 2026 11:52:44 -0700 (PDT)", "From": "Peter Maydell <peter.maydell@linaro.org>", "To": "qemu-devel@nongnu.org", "Subject": "[PULL 5/5] hw/arm/xilinx_zynq: Use strcasecmp to parse boot-mode\n option values", "Date": "Mon, 30 Mar 2026 19:52:36 +0100", "Message-ID": "<20260330185236.1905636-6-peter.maydell@linaro.org>", "X-Mailer": "git-send-email 2.43.0", "In-Reply-To": "<20260330185236.1905636-1-peter.maydell@linaro.org>", "References": "<20260330185236.1905636-1-peter.maydell@linaro.org>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Received-SPF": "pass client-ip=2a00:1450:4864:20::429;\n envelope-from=peter.maydell@linaro.org; helo=mail-wr1-x429.google.com", "X-Spam_score_int": "-20", "X-Spam_score": "-2.1", "X-Spam_bar": "--", "X-Spam_report": "(-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,\n DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1,\n RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,\n SPF_PASS=-0.001 autolearn=ham autolearn_force=no", "X-Spam_action": "no action", "X-BeenThere": "qemu-devel@nongnu.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "qemu development <qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>", "List-Archive": "<https://lists.nongnu.org/archive/html/qemu-devel>", "List-Post": "<mailto:qemu-devel@nongnu.org>", "List-Help": "<mailto:qemu-devel-request@nongnu.org?subject=help>", "List-Subscribe": "<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org" }, "content": "In zynq_set_boot_mode() where we parse the string the user has set\nthe boot-mode option to, we use strncasecmp(str, \"qspi\", 4) and so\non. This is wrong, because it means that we will ignore any trailing\njunk on the end of the option string, and handle\n -machine boot-mode=sdXYZZY\nthe same as\n -machine boot-mode=sd\n\nIn the documentation we say:\n Supported values are ``jtag``, ``sd``, ``qspi`` and ``nor``.\nand that's obviously what we meant to implement.\n\nThe correct tool for this job is a simple strcasecmp operation.\nSwitch to that.\n\nWe use the g_ascii_strcasecmp() rather than plain strcasecmp()\nbecause we're comparing ASCII strings here and don't want the\npotentially locale-specific behaviour that strcasecmp() implies (and\nwe're trying to standardize on the glib function for this kind of\nstring comparison).\n\nFixes: 7df3747c92d13 (\"hw/arm/xilinx_zynq: Add boot-mode property\")\nSigned-off-by: Peter Maydell <peter.maydell@linaro.org>\nReviewed-by: Alistair Francis <alistair.francis@wdc.com>\nReviewed-by: Kostiantyn Kostiuk <kkostiuk@redhat.com>\nReviewed-by: Paolo Bonzini <pbonzini@redhat.com>\nMessage-id: 20260327145012.907264-1-peter.maydell@linaro.org\n---\n hw/arm/xilinx_zynq.c | 8 ++++----\n 1 file changed, 4 insertions(+), 4 deletions(-)", "diff": "diff --git a/hw/arm/xilinx_zynq.c b/hw/arm/xilinx_zynq.c\nindex d43f36b718..9dcded9219 100644\n--- a/hw/arm/xilinx_zynq.c\n+++ b/hw/arm/xilinx_zynq.c\n@@ -186,13 +186,13 @@ static void zynq_set_boot_mode(Object *obj, const char *str,\n ZynqMachineState *m = ZYNQ_MACHINE(obj);\n uint8_t mode = 0;\n \n- if (!strncasecmp(str, \"qspi\", 4)) {\n+ if (!g_ascii_strcasecmp(str, \"qspi\")) {\n mode = 1;\n- } else if (!strncasecmp(str, \"sd\", 2)) {\n+ } else if (!g_ascii_strcasecmp(str, \"sd\")) {\n mode = 5;\n- } else if (!strncasecmp(str, \"nor\", 3)) {\n+ } else if (!g_ascii_strcasecmp(str, \"nor\")) {\n mode = 2;\n- } else if (!strncasecmp(str, \"jtag\", 4)) {\n+ } else if (!g_ascii_strcasecmp(str, \"jtag\")) {\n mode = 0;\n } else {\n error_setg(errp, \"%s boot mode not supported\", str);\n", "prefixes": [ "PULL", "5/5" ] }