From patchwork Fri May 17 15:02:23 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 797472 Delivered-To: patch@linaro.org Received: by 2002:adf:e68e:0:b0:351:d90a:5487 with SMTP id r14csp814940wrm; Fri, 17 May 2024 08:03:43 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCXFs/BJVYhuykJGH19bhln0w60CmRr4PqVi/sL45nfblVRlURX/ubfaIqV3ddufsqKJ5Bkc+ZLMBIvf8iTbaKaC X-Google-Smtp-Source: AGHT+IEsVRjprnWtKCuLL3nzlLRSkuElv88c6F4KKxDLs1hCwN58Djfparyg4ye5Zkvg+g1ftHQH X-Received: by 2002:a5b:d4b:0:b0:df4:4345:260b with SMTP id 3f1490d57ef6-df4434527d3mr7086215276.36.1715958223201; Fri, 17 May 2024 08:03:43 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715958223; cv=none; d=google.com; s=arc-20160816; b=bLRc7GB9Ywn83Kao0kGcaXy6EPizG7zkN64wrcsFujblEvVZBKbPllj1x+d8fSrlHw Lw3twon2dg6TXW1RGpZX551aC+MvJQfH4glO5UPTThgJ9ZO6+Ai3wz+GNl6s+fy9sfUz EzFO3srWr3pNqonROfvw+qEHxhce1C00qFJjrt7EQ/DMH2mG+DlD4qaJP/MGyKSg8JGt YJv53x1/kWIysD7O/DU5rYXWTufXeFp0MkS4TfpefEgwhiZz9F2d6PzcUcezmESp8uOa zOT+rWxrclMUzBJFkFuGcTE7uIHLzByFgArOqUNcay//6ivcFp2Nh6OCSOYo2eqCHGgX vF/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=+EqzMl8MHvvQIdbxxo0G9Us5dJc+dAGwQg0EfUlU33w=; fh=vPGgBKjG5M2Hhj91wBlzR/1e05e648xqCqV96evKOss=; b=MYIvI3y9XosNuJ3iZ12qiyfR7E1d6mX3GkMvYpDimU/k67yqQHYlXX3PzNmcKPHYhn Dk1ql/eQ1h2tIKsHE8MviKxCFIFB0wjScuAbcpoYpp59pxMwBBmiQGyBUulLf4V9d9j6 +1/40/0r+wYxx02fGv+e4338ONiJv26+d7d2ZGRhoFXlppWSaSX9rbtbZtxiGP0bIX5v OeceafGgIOzuyb04H8l0EXbGbQaGbc/QnEVhJ85EJR+tzkfu5VmzZvHPOhmWYB1A7YFc Aw6/N78CG7pcOjiFO+AxDQ+O9Mo7rkboy0imxRcd/spq46MUKXwIEBnCrJzUVudv9EW8 gvEQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=aR6yVKyi; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 3f1490d57ef6-debd3869f07si8621029276.436.2024.05.17.08.03.42 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 17 May 2024 08:03:43 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=aR6yVKyi; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7z6l-0004wx-TO; Fri, 17 May 2024 11:02:55 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7z6V-0004uO-Sn for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:48 -0400 Received: from mail-ed1-x531.google.com ([2a00:1450:4864:20::531]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1s7z6S-0005Nk-PJ for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:39 -0400 Received: by mail-ed1-x531.google.com with SMTP id 4fb4d7f45d1cf-56e6affdd21so1969720a12.3 for ; Fri, 17 May 2024 08:02:36 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715958154; x=1716562954; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=+EqzMl8MHvvQIdbxxo0G9Us5dJc+dAGwQg0EfUlU33w=; b=aR6yVKyi1k01OxaYReXTHpXIn9A9R9e5pjG4y9MK41YmPKP4iLokXbH54nbYlu/GOj ff1xWzqsAapOsEm4nG8+9St0vqsde6ym96idQXL1iG3qHTz6RlFw5frnPNb99/PqTh+Y VvoCJih1c782juIuOOxT67X+XpUz08Q73OWGYnvmjXCL/xCuN6FlImulMbh21lRfTxTr zOwRRQul5/DwQjmF7MLEgUEqi3QxOm10ERaDyIsOjG+aqyh87aMFHxyGWUdX/MN3Dgr2 vMc/YOoKCiXi2zq0lZM5MDN4fPsdSOy2cPNpfn32qXbJHQ17UBz0EXg03yCRGkDJ+kSV IJyg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715958154; x=1716562954; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=+EqzMl8MHvvQIdbxxo0G9Us5dJc+dAGwQg0EfUlU33w=; b=ki//4EKyVEltU5+NjkgUlgaX+Fm/GUMOIqcjCufHzGUktkLx25fkMCzXx2kekUFb08 C8yP29ayTHGUwLChKZAEvPqmtifnUaOrL1fsbyfrC/I8mF+rru8JkWehJhGN+TVsiou5 rJHFZwtD+8bcqhfz/vky2ivO4EYTy4ZwUS43Zn2V8GY3sytm+99MRi9vQjzyGvAGFgRB aQoiI9dj8X1lX7p0n//mbloF5GzwVrWUNL+cizc3Z35BHFu4PqfIb+XroqXg3JjxsCYK BUvG/oxrKBMQC3jpCH8SjQjmnyejeV15iJNxRNpid2CAe6f/1XmMVrFFp+2rfFdp93va 5JJA== X-Gm-Message-State: AOJu0YyhIIpXyygeLyB6yMe9GlGFsHySVSmsw1/ZJk1MKL5ykQuq1DDo aagGT3Yba51aCLTjJ+5kAr+EuYChtnHBzxQYAnTM9/oSS7lfbT0mPPNF9c/4gnhzYwpY1R+fpgr 5 X-Received: by 2002:a50:8757:0:b0:572:47be:831d with SMTP id 4fb4d7f45d1cf-5734d5ce595mr20099116a12.20.1715958154717; Fri, 17 May 2024 08:02:34 -0700 (PDT) Received: from localhost.localdomain ([149.14.240.163]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5733bea66f9sm11957369a12.15.2024.05.17.08.02.33 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 17 May 2024 08:02:34 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: =?utf-8?q?C=C3=A9dric_Le_Goater?= , Dongwon Kim , =?utf-8?q?Marc-Andr=C3=A9_Lureau?= , =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= Subject: [PULL 1/5] ui/console: Only declare variable fence_fd when CONFIG_GBM is defined Date: Fri, 17 May 2024 17:02:23 +0200 Message-ID: <20240517150227.32205-2-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20240517150227.32205-1-philmd@linaro.org> References: <20240517150227.32205-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::531; envelope-from=philmd@linaro.org; helo=mail-ed1-x531.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, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, 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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Cédric Le Goater This to avoid a build breakage : ../ui/gtk-egl.c: In function ‘gd_egl_draw’: ../ui/gtk-egl.c:73:9: error: unused variable ‘fence_fd’ [-Werror=unused-variable] 73 | int fence_fd; | ^~~~~~~~ Fixes: fa6426805b12 ("ui/console: Use qemu_dmabuf_set_..() helpers instead") Cc: Dongwon Kim Cc: Marc-André Lureau Signed-off-by: Cédric Le Goater Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20240515100520.574383-1-clg@redhat.com> Signed-off-by: Philippe Mathieu-Daudé Acked-by: Dongwon Kim --- ui/gtk-egl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ui/gtk-egl.c b/ui/gtk-egl.c index 0473f689c9..9831c10e1b 100644 --- a/ui/gtk-egl.c +++ b/ui/gtk-egl.c @@ -68,9 +68,9 @@ void gd_egl_draw(VirtualConsole *vc) GdkWindow *window; #ifdef CONFIG_GBM QemuDmaBuf *dmabuf = vc->gfx.guest_fb.dmabuf; + int fence_fd; #endif int ww, wh, ws; - int fence_fd; if (!vc->gfx.gls) { return; From patchwork Fri May 17 15:02:24 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 797476 Delivered-To: patch@linaro.org Received: by 2002:adf:e68e:0:b0:351:d90a:5487 with SMTP id r14csp815450wrm; Fri, 17 May 2024 08:04:27 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCVwsQfEt4cH+Es0+A/RfjFxlCrOy6tGRZlZVXhw/u5rlDQ7F+VdvBgVu4bgKttYbWGbKQYCAYFoAm0opUYSN7Lr X-Google-Smtp-Source: AGHT+IEYqJ0nLRCQMDmJw79lHmyitJQP0Yo/Fj3412LYlhZs5fIKan67E82br1y/ZeThk6/LhP+y X-Received: by 2002:a05:690c:3387:b0:627:7c28:32f3 with SMTP id 00721157ae682-6277c2836d9mr55474257b3.2.1715958267602; Fri, 17 May 2024 08:04:27 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715958267; cv=none; d=google.com; s=arc-20160816; b=FlyTYI1tgKMfhc453SkiV4YgYX/I6htM5wtwHYc8+8Xr2Zpw/xkQf9AN9vvek3jYMi iDGhi0afrsqoss8rsPwvyqa8AFjdvXOapJvF9c/yD2kWyEd/Lx/YP1usvANXDcITEiak gbYNKmpupHREBs8SxeoF79Aj2mLIl7FDYbTyzh1w0gWhnQ0J3Om+AOI6UvDuLP50TalV //kXQamRMJqWTNw6hllzA5HPcZzxl12JPtRztAj959dzsqzM5JJcg8tLWXu6yYhQQVa3 iNdjD3MV6sHXyiuAgPoJb5HPxjSc6PlMjEjnNDO4K8CiCpNM1xdHbnDXDW76wwyPOOsg +kvg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=prOBORLrKnIubByGiemAya5YeM7dN+tEWn5fMLTmbm4=; fh=KRI6VCZHMeg0ZgzbPSvWJNPxvSjTRUiYCNLt90+GnHs=; b=LMZe7vHausW9pMZBIrbZ65vKKJDjCXEohcV72BFjeFFQY7R2gRrWsCAZmzZ10nx1St 4zRkX5w+7tItu3MZhgnZYmdYOs7zVfrawqnDihwfHR9ne+qIHCqB0tv7AhzbCNLv7sEB 5BN7yKKeyvXTh4BAqqScxB8A98TU1CiplXXp0ejw01rkbOT94M5Mwg/aKjWislqhUl80 8jdztHEJZJvmWGkdJ+/GXw79xl0lD3sWxaY+ZOhnMd91UI0O/0cVMGBP3Rht2BRMbNDm JlZOB5nlRfEYOu7mu7jjhApu8sjDfU+Z0uZrVu4fwfQ134hSTGGh6sfObGWTV4vAnf4i 1NZA==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=BTAOo7MV; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 00721157ae682-6209e312c58si86617387b3.152.2024.05.17.08.04.27 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 17 May 2024 08:04:27 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=BTAOo7MV; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7z6q-00051u-E6; Fri, 17 May 2024 11:03:00 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7z6o-0004zo-H3 for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:58 -0400 Received: from mail-ed1-x531.google.com ([2a00:1450:4864:20::531]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1s7z6b-0005OP-FC for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:58 -0400 Received: by mail-ed1-x531.google.com with SMTP id 4fb4d7f45d1cf-56e1f3462caso5177698a12.3 for ; Fri, 17 May 2024 08:02:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715958160; x=1716562960; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=prOBORLrKnIubByGiemAya5YeM7dN+tEWn5fMLTmbm4=; b=BTAOo7MVw8FmgeeffQ5hpmckUijnZeFsyXxMTUxz6jIZ8+zys4Su+GL5AfLV7hWKRD MUrmybui38oMHeug1kb5PawjUPoNOghaZuvCxK1zG1wissDNrwydBIONp8VJg/AxElfG 3NWtypXutXR5oFzwGiXfntrBdTE/9pAff6/BhtLNz+zbD/sRRVmXh6UIClCDDFV4rfP8 Diup4kiMbGIw0Gvxv7hUeVWBGOUlOW8psl4JHWLLenB0IjhLzmj+HAVCfcDbNnP8q/aD YhJ2MeGlEY/lCe6jnQiN/auZyOghXEh3WEjo5AHilkT0Zzx1Lea6xH9T87QKVOvcRK/N CPvA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715958160; x=1716562960; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=prOBORLrKnIubByGiemAya5YeM7dN+tEWn5fMLTmbm4=; b=n9d0uCzGDqJsvsglGSpWfVa0mVfgkvbhvcQplGyZ/XNCBEqtRezR/1u8LXETyKq1/C ssY+jGxzFYNaR4ABmZGab+6CRln/nt+QkZ2mWtDQxIEinW5sUftOFojwKejMaHw4eKEI TJBsZWEuI2K87HGHZXmcWTRhXxRl4rbuYti2UpXZCVsU02bQ+HPqrIU9vH7k1iZgbUJx bfZgyk7zbHchbUU2A6SJaajsKqXNixM7wkCZzb7h10eYVVRFj7w+1h44zWV2mhkn13n4 TL/n3PfU6VCm37KJetz25/TdvpbRbuQHK4CVY8OTAxrZB9yC45Bb+p6mAlrLcHVygyrF ynqA== X-Gm-Message-State: AOJu0YyZKSOSCH2YMjMcA2Gm3hG4h4voRuuSLJrwjrCxnRjiYas2HDqv lWOCqQi0ULclXjHb62lNcFRbTb1ak1S85fQiTWv1i0CPvBHAmimrJqf0kMzf1qBf7WxMR4e6uLu t X-Received: by 2002:a50:d745:0:b0:572:4702:2227 with SMTP id 4fb4d7f45d1cf-5734d6de22dmr17150495a12.35.1715958160001; Fri, 17 May 2024 08:02:40 -0700 (PDT) Received: from localhost.localdomain ([149.14.240.163]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5733c322c61sm12370190a12.86.2024.05.17.08.02.39 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 17 May 2024 08:02:39 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: Gerd Hoffmann , qemu-stable@nongnu.org, =?utf-8?q?Phi?= =?utf-8?q?lippe_Mathieu-Daud=C3=A9?= Subject: [PULL 2/5] hw/pflash: fix block write start Date: Fri, 17 May 2024 17:02:24 +0200 Message-ID: <20240517150227.32205-3-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20240517150227.32205-1-philmd@linaro.org> References: <20240517150227.32205-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::531; envelope-from=philmd@linaro.org; helo=mail-ed1-x531.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, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, T_SPF_TEMPERROR=0.01 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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Gerd Hoffmann Move the pflash_blk_write_start() call. We need the offset of the first data write, not the offset for the setup (number-of-bytes) write. Without this fix u-boot can do block writes to the first flash block only. While being at it drop a leftover FIXME. Cc: qemu-stable@nongnu.org Resolves: https://gitlab.com/qemu-project/qemu/-/issues/2343 Fixes: 284a7ee2e290 ("hw/pflash: implement update buffer for block writes") Signed-off-by: Gerd Hoffmann Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20240516121237.534875-1-kraxel@redhat.com> Signed-off-by: Philippe Mathieu-Daudé --- hw/block/pflash_cfi01.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/hw/block/pflash_cfi01.c b/hw/block/pflash_cfi01.c index 1bda8424b9..c8f1cf5a87 100644 --- a/hw/block/pflash_cfi01.c +++ b/hw/block/pflash_cfi01.c @@ -518,10 +518,6 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr offset, break; case 0xe8: /* Write to buffer */ trace_pflash_write(pfl->name, "write to buffer"); - /* FIXME should save @offset, @width for case 1+ */ - qemu_log_mask(LOG_UNIMP, - "%s: Write to buffer emulation is flawed\n", - __func__); pfl->status |= 0x80; /* Ready! */ break; case 0xf0: /* Probe for AMD flash */ @@ -574,7 +570,6 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr offset, } pfl->counter = value; pfl->wcycle++; - pflash_blk_write_start(pfl, offset); break; case 0x60: if (cmd == 0xd0) { @@ -605,6 +600,9 @@ static void pflash_write(PFlashCFI01 *pfl, hwaddr offset, switch (pfl->cmd) { case 0xe8: /* Block write */ /* FIXME check @offset, @width */ + if (pfl->blk_offset == -1 && pfl->counter) { + pflash_blk_write_start(pfl, offset); + } if (!pfl->ro && (pfl->blk_offset != -1)) { pflash_data_write(pfl, offset, value, width, be); } else { From patchwork Fri May 17 15:02:25 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 797474 Delivered-To: patch@linaro.org Received: by 2002:adf:e68e:0:b0:351:d90a:5487 with SMTP id r14csp815167wrm; Fri, 17 May 2024 08:04:00 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCWA2A2YikTB/Pk5x7D+WzwSeCx1w1HkWYSHVtvte7Ljl9OiLMKkHTGDelYxdEPq2a49Ej02W7yiMc8NS0cY8w26 X-Google-Smtp-Source: AGHT+IH3cpHJHA8tUCx6JMhYVMPczERxvgICIwg1th4Xz9Rxuq2AVAC7nKRYPcNX9xsTErFP6zsm X-Received: by 2002:a05:6359:5a8c:b0:183:e72e:ce04 with SMTP id e5c5f4694b2df-193bb656ac6mr2341815355d.16.1715958239841; Fri, 17 May 2024 08:03:59 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715958239; cv=none; d=google.com; s=arc-20160816; b=zhqrgz/ZhJ7KpIbzq2r6xvZs3Y253XfO16PoUeKNLGzNrZFyTdXkovdl3Xnfwr6AxR VrUCHAXH+OprHAqheWtzWBJUQihFjnHFYcHMwPcdDMedIgX7V0L0MmVDRkEhWklXmmzs q+RgajXV3aGcwrVektVNPePEUWcwvZo11DXmHSTcGpUQBlCZc/fTGMFz+dz9YJUBuMue v+s4JDNidzLWFpEf/PWsEzN07n8DbQouzPuKnDIb0DdJxyQFL6EAHpDuV7nWo7ZkyBz4 mvhvvIV1Yo7hyMVPzEFAm8NPq7fB4uBIJHG6DSh1nDY/nJ1Br5ejnsWC0PRwWg6iSoyd ud1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=uJ3hD+YdmDQmGtwZeHY/6goxziw8ZaiYrGA/9n+kedo=; fh=TUD83fNAFRMrrDAgFWLt2leAEmWyoMlFWRqulCVtVQ8=; b=GE+G67+/b/LKoC9RyCigoVa6OD4XKda+YvSIlzt6auWHyzhw/AQD5UsA18qLPrR79D Ziy/2+tafo3HdeIayZpOqX2TfH1DOkK2z7UEKD54jCTXtsnLMguJValdCgiyE8knaWw7 JoK/XnElB1rgfrVUIJU3QDgeJTxKJm6YnYiz2pmktBD5v1CwFoklVEEa12ItrV1W3JdY MdaETu5QgySx9ja6svXReeNYBkDkR3d7JIVtksAGcSoyoikMO3u0u/Syt2ck6jPyE3ca 0oJW4zy+Iqsu7+6Yr53SaWryFn8cvMecmSb9eOggwPISTuMOnSUuQkGAal7n8S6EZkvz YmyQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kTIWomTF; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 00721157ae682-6209e7c1dddsi82236967b3.562.2024.05.17.08.03.59 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 17 May 2024 08:03:59 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=kTIWomTF; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7z6n-0004zX-Vy; Fri, 17 May 2024 11:02:58 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7z6i-0004v0-9E for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:52 -0400 Received: from mail-ed1-x532.google.com ([2a00:1450:4864:20::532]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1s7z6e-0005PC-Fx for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:52 -0400 Received: by mail-ed1-x532.google.com with SMTP id 4fb4d7f45d1cf-571be483ccaso5021859a12.2 for ; Fri, 17 May 2024 08:02:46 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715958165; x=1716562965; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=uJ3hD+YdmDQmGtwZeHY/6goxziw8ZaiYrGA/9n+kedo=; b=kTIWomTFSVDAeqZv5BrLtF6wQFXQ1CVagceMxk1Uy1Dlqj3JAD7Em+RVlH1STxmQF9 WKGjTUBcTRK1xzdjJUu463kV7ANZ7mGa0VBs65DT18MtrvjBN9ZKg3Sp9YyDApniFfpt f/1dgQAK6w5D+39KHMxYv6p3o1UbuEj2kiSny6BfLNFa2qHT7daAhvFERm4ax054R8yo Q5gkbelNBvlb9s54JmYLfyodthRGm59jaRMrnXa7kEir4a9ixw6B3gd6XZoyWAUBb5T1 mLy/POKtSN6uHJG1NTj9uPMLVQ+zV8D0X56LUSiFQF2DWGyjjyKvrldy6kYijpseRqaE CMbg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715958165; x=1716562965; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=uJ3hD+YdmDQmGtwZeHY/6goxziw8ZaiYrGA/9n+kedo=; b=l2j6xNcs+MpYBbeWRt2iOKKEGeI42p/3vSvfjLxZrOvNL4XP19LZmmFg5AC4x6oU9c rKVhI8xTqUSPX7TENcUv/YgZWiJv2xVhR14oJnk3NuVrBNmqisdtpB2mUNoTfFRVIbWA DWyN5ZvCK3c6VjmodDI+f9T2Jd9LMu+0gmGNhJPu9D7BqbYA7VgLGGreyCyyHD4b5vlN L45RSS49b36wNNKyBrC6ilOqiwzY9iqmVpyP2dAXBTuQRlOhqlnzab/rpvhhTfjpM8RJ Mgh/5EEUh5ru6Q/wBRjvEwkq5ib6o8GNH5qlfLKcCK8fJidGsdx2BvO/BHWrsDWE166n Z2xA== X-Gm-Message-State: AOJu0Yx59CP4YQFaBO+43h0OJs/kcKk8w34kYjeg/lg9bsnl5Zm0Lq8i muvPh6Hf8Ng0tTOI/rdFUn91+qhA4ayMKQQQUgKCRUHD4HiUp0tP0CspFyaWcWO2j25D6J5hIFj a X-Received: by 2002:a50:aa94:0:b0:572:a731:dd14 with SMTP id 4fb4d7f45d1cf-5734d6edacemr16342354a12.28.1715958165443; Fri, 17 May 2024 08:02:45 -0700 (PDT) Received: from localhost.localdomain ([149.14.240.163]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-5733bebb6casm11959706a12.29.2024.05.17.08.02.44 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 17 May 2024 08:02:44 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: =?utf-8?q?Daniel_P=2E_Berrang=C3=A9?= , Zhao Liu , =?utf-8?q?J=C3=A1n_Tomko?= , =?utf-8?q?Philippe_Mathieu-?= =?utf-8?q?Daud=C3=A9?= Subject: [PULL 3/5] hw/core: allow parameter=1 for SMP topology on any machine Date: Fri, 17 May 2024 17:02:25 +0200 Message-ID: <20240517150227.32205-4-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20240517150227.32205-1-philmd@linaro.org> References: <20240517150227.32205-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::532; envelope-from=philmd@linaro.org; helo=mail-ed1-x532.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, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, 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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Daniel P. Berrangé This effectively reverts commit 54c4ea8f3ae614054079395842128a856a73dbf9 Author: Zhao Liu Date: Sat Mar 9 00:01:37 2024 +0800 hw/core/machine-smp: Deprecate unsupported "parameter=1" SMP configurations but is not done as a 'git revert' since the part of the changes to the file hw/core/machine-smp.c which add 'has_XXX' checks remain desirable. Furthermore, we have to tweak the subsequently added unit test to account for differing warning message. The rationale for the original deprecation was: "Currently, it was allowed for users to specify the unsupported topology parameter as "1". For example, x86 PC machine doesn't support drawer/book/cluster topology levels, but user could specify "-smp drawers=1,books=1,clusters=1". This is meaningless and confusing, so that the support for this kind of configurations is marked deprecated since 9.0." There are varying POVs on the topic of 'unsupported' topology levels. It is common to say that on a system without hyperthreading, that there is always 1 thread. Likewise when new CPUs introduced a concept of multiple "dies', it was reasonable to say that all historical CPUs before that implicitly had 1 'die'. Likewise for the more recently introduced 'modules' and 'clusters' parameter'. From this POV, it is valid to set 'parameter=1' on the -smp command line for any machine, only a value > 1 is strictly an error condition. It doesn't cause any functional difficulty for QEMU, because internally the QEMU code is itself assuming that all "unsupported" parameters implicitly have a value of '1'. At the libvirt level, we've allowed applications to set 'parameter=1' when configuring a guest, and pass that through to QEMU. Deprecating this creates extra difficulty for because there's no info exposed from QEMU about which machine types "support" which parameters. Thus, libvirt can't know whether it is valid to pass 'parameter=1' for a given machine type, or whether it will trigger deprecation messages. Since there's no apparent functional benefit to deleting this deprecated behaviour from QEMU, and it creates problems for consumers of QEMU, remove this deprecation. Signed-off-by: Daniel P. Berrangé Reviewed-by: Zhao Liu Reviewed-by: Ján Tomko Message-ID: <20240513123358.612355-2-berrange@redhat.com> Signed-off-by: Philippe Mathieu-Daudé --- hw/core/machine-smp.c | 84 ++++++++++++------------------------- tests/unit/test-smp-parse.c | 8 ++-- 2 files changed, 31 insertions(+), 61 deletions(-) diff --git a/hw/core/machine-smp.c b/hw/core/machine-smp.c index 2b93fa99c9..5d8d7edcbd 100644 --- a/hw/core/machine-smp.c +++ b/hw/core/machine-smp.c @@ -118,76 +118,46 @@ void machine_parse_smp_config(MachineState *ms, } /* - * If not supported by the machine, a topology parameter must be - * omitted. + * If not supported by the machine, a topology parameter must + * not be set to a value greater than 1. */ - if (!mc->smp_props.modules_supported && config->has_modules) { - if (config->modules > 1) { - error_setg(errp, "modules not supported by this " - "machine's CPU topology"); - return; - } else { - /* Here modules only equals 1 since we've checked zero case. */ - warn_report("Deprecated CPU topology (considered invalid): " - "Unsupported modules parameter mustn't be " - "specified as 1"); - } + if (!mc->smp_props.modules_supported && + config->has_modules && config->modules > 1) { + error_setg(errp, + "modules > 1 not supported by this machine's CPU topology"); + return; } modules = modules > 0 ? modules : 1; - if (!mc->smp_props.clusters_supported && config->has_clusters) { - if (config->clusters > 1) { - error_setg(errp, "clusters not supported by this " - "machine's CPU topology"); - return; - } else { - /* Here clusters only equals 1 since we've checked zero case. */ - warn_report("Deprecated CPU topology (considered invalid): " - "Unsupported clusters parameter mustn't be " - "specified as 1"); - } + if (!mc->smp_props.clusters_supported && + config->has_clusters && config->clusters > 1) { + error_setg(errp, + "clusters > 1 not supported by this machine's CPU topology"); + return; } clusters = clusters > 0 ? clusters : 1; - if (!mc->smp_props.dies_supported && config->has_dies) { - if (config->dies > 1) { - error_setg(errp, "dies not supported by this " - "machine's CPU topology"); - return; - } else { - /* Here dies only equals 1 since we've checked zero case. */ - warn_report("Deprecated CPU topology (considered invalid): " - "Unsupported dies parameter mustn't be " - "specified as 1"); - } + if (!mc->smp_props.dies_supported && + config->has_dies && config->dies > 1) { + error_setg(errp, + "dies > 1 not supported by this machine's CPU topology"); + return; } dies = dies > 0 ? dies : 1; - if (!mc->smp_props.books_supported && config->has_books) { - if (config->books > 1) { - error_setg(errp, "books not supported by this " - "machine's CPU topology"); - return; - } else { - /* Here books only equals 1 since we've checked zero case. */ - warn_report("Deprecated CPU topology (considered invalid): " - "Unsupported books parameter mustn't be " - "specified as 1"); - } + if (!mc->smp_props.books_supported && + config->has_books && config->books > 1) { + error_setg(errp, + "books > 1 not supported by this machine's CPU topology"); + return; } books = books > 0 ? books : 1; - if (!mc->smp_props.drawers_supported && config->has_drawers) { - if (config->drawers > 1) { - error_setg(errp, "drawers not supported by this " - "machine's CPU topology"); - return; - } else { - /* Here drawers only equals 1 since we've checked zero case. */ - warn_report("Deprecated CPU topology (considered invalid): " - "Unsupported drawers parameter mustn't be " - "specified as 1"); - } + if (!mc->smp_props.drawers_supported && + config->has_drawers && config->drawers > 1) { + error_setg(errp, + "drawers > 1 not supported by this machine's CPU topology"); + return; } drawers = drawers > 0 ? drawers : 1; diff --git a/tests/unit/test-smp-parse.c b/tests/unit/test-smp-parse.c index 8994337e12..56165e6644 100644 --- a/tests/unit/test-smp-parse.c +++ b/tests/unit/test-smp-parse.c @@ -337,21 +337,21 @@ static const struct SMPTestData data_generic_invalid[] = { { /* config: -smp 2,dies=2 */ .config = SMP_CONFIG_WITH_DIES(T, 2, F, 0, T, 2, F, 0, F, 0, F, 0), - .expect_error = "dies not supported by this machine's CPU topology", + .expect_error = "dies > 1 not supported by this machine's CPU topology", }, { /* config: -smp 2,clusters=2 */ .config = SMP_CONFIG_WITH_CLUSTERS(T, 2, F, 0, T, 2, F, 0, F, 0, F, 0), - .expect_error = "clusters not supported by this machine's CPU topology", + .expect_error = "clusters > 1 not supported by this machine's CPU topology", }, { /* config: -smp 2,books=2 */ .config = SMP_CONFIG_WITH_BOOKS_DRAWERS(T, 2, F, 0, T, 2, F, 0, F, 0, F, 0, F, 0), - .expect_error = "books not supported by this machine's CPU topology", + .expect_error = "books > 1 not supported by this machine's CPU topology", }, { /* config: -smp 2,drawers=2 */ .config = SMP_CONFIG_WITH_BOOKS_DRAWERS(T, 2, T, 2, F, 0, F, 0, F, 0, F, 0, F, 0), - .expect_error = "drawers not supported by this machine's CPU topology", + .expect_error = "drawers > 1 not supported by this machine's CPU topology", }, { /* config: -smp 8,sockets=2,cores=4,threads=2,maxcpus=8 */ .config = SMP_CONFIG_GENERIC(T, 8, T, 2, T, 4, T, 2, T, 8), From patchwork Fri May 17 15:02:26 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 797477 Delivered-To: patch@linaro.org Received: by 2002:adf:e68e:0:b0:351:d90a:5487 with SMTP id r14csp815486wrm; Fri, 17 May 2024 08:04:31 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCVwtniSTa+Mkbd3QgOXbS1iO4IEkBMdmbDDBiCWMJXjPfEUO4KtzwcH/dTc3yatb5pM+Vu0a5o9j1z2A9M2yFxZ X-Google-Smtp-Source: AGHT+IFDm3o1bxPLoJRcBoRdHzI8umtln1lBFWafrwK5oJbUavgfDuDjrcZ7dg8yiKKIy9aMXFAP X-Received: by 2002:a05:690c:6c83:b0:620:2ec0:9797 with SMTP id 00721157ae682-622affc4341mr254621927b3.23.1715958271003; Fri, 17 May 2024 08:04:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715958270; cv=none; d=google.com; s=arc-20160816; b=fkRIhyPnAVFiG3onHRCJRk5yKcYkQTf78rDtFGckwj6Ql7+wcm+yvQrDXz5Mfr1qOs PeTVkt1BUTM0cL/Kb31uIavCAQvwAhNpndVkXSMYdKdICeMatrJMlhIjGilQP3HiiOKn iBpOjsCOkFyLSQS5OyOkAzwLeIM9onG7NQdv5RuwRNd6Olkui3fBba2o0VONfH0fYrIk eFgxSAqHyYO7uBVZY9663bsceYXyTFkLzzuRhHUwFetJgCbiGFf+q8NEse+LLwtj/fGX sXFA8T06XK1LhGaXKezmg2647fEeY45PdtuS+CmQJXiGy4DNtQMKXmdi45/pVmWoP8ew mGNw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=RFJQ8eXrLcyhgXUkRkABC/U14von2QfsgkjKz+WC79c=; fh=TUD83fNAFRMrrDAgFWLt2leAEmWyoMlFWRqulCVtVQ8=; b=DBggo0HDpQohvtwkC9LFdDBrDzcSd2KkEW74JiVPpYxawrHmz0ZjTHfSAb3XNi8xAY yVwEB0pUEfN4YGKESYejKCWfmrx+Qt+g2gTq+xHErO6PWt+zBNgZJfWKqAnXDzjdo0jo 48SdVYXI0JoWbEJ05ehQ+PyoduyTqc64i3lRT25UWKzXJdUNoTPCeyfIALlEvRWh2s2Z nw/mZsr/L6F68uHDTszKGZJD/MX62Qya7k16ZmxLPq5WUfLNlr0kfj8emcSAyvKuzUzF MAaSpQzKZg1R8QCLksfKkbiUMxdl4OHF/JPT4o3L7+pmqMqc2sFbligJCe862FEJ8Zb7 Pyrg==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=gc2u0LsB; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 00721157ae682-6209e70dc7dsi82090537b3.280.2024.05.17.08.04.30 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 17 May 2024 08:04:30 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=gc2u0LsB; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7z6y-0005AY-Ap; Fri, 17 May 2024 11:03:08 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7z6u-00057i-CZ for qemu-devel@nongnu.org; Fri, 17 May 2024 11:03:04 -0400 Received: from mail-ej1-x632.google.com ([2a00:1450:4864:20::632]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1s7z6i-0005Pp-AS for qemu-devel@nongnu.org; Fri, 17 May 2024 11:03:03 -0400 Received: by mail-ej1-x632.google.com with SMTP id a640c23a62f3a-a5a8cd78701so491995166b.2 for ; Fri, 17 May 2024 08:02:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715958170; x=1716562970; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=RFJQ8eXrLcyhgXUkRkABC/U14von2QfsgkjKz+WC79c=; b=gc2u0LsB9ul/DHWRO66ULOf7EZmlomg+FoaGtZ/jClqsnh28N8tGgG6XogugqCB1Y0 iIK9SevZDGiWz3fdfW7WodfHl5pMa1LofoxFddt7Gh9oFX5+8Xduhs9vPIJUVGmvXrDp oNc09P8Vn21Y1m6AuFzGX9YT9rdlkLxIqM4RMMIInNNAC3VPEZE7GZWtnuyCOqWTUxwZ OghH9utl2D5y1DH/NQTBZZ7si6Ijk0VdX4ydOxfx5O9nv+JmWoNuONvCGr2QJ7nfEWCR h798O2S7ZljGkEGNkRRu1pW7oz5dyxHbJOPm4s6IRCCbGxGBtu3EU6Rzwat95OsXABWg KEkQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715958170; x=1716562970; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=RFJQ8eXrLcyhgXUkRkABC/U14von2QfsgkjKz+WC79c=; b=HEeJ3Pcp6fknE7SJoRKrUhI0ahY9SgXWTbaOGqixvZIvrplntuKFBQ55KGHnO15UEV j2Rj3EMqWyfdzgr20/Nmzr7S6Lsl3dGqg2vQFgviYKe44nyh/hkM3DTIDb8gGG+hUdGe T0q2d9S8BSsmWRBYMOO8znLThzeexakppXCRkOmdry+LjVdQsHu3KeSD4EjSgCAJPWXM 1nuyu9tLpA00TcmiP2QuaY/WjKWCo5vhCwcbd/KI3f9KrTAwWcaDheKttRZDsufEimIu tZxJJCu8mI9Q9I+pqZpKePeB+M9xWJq0sFcTxMM0/d0WqjI6nF4z7oxU1/TslCPVOuvS WTzA== X-Gm-Message-State: AOJu0YwdJjeRZTDQzlJCQLJH+zq2kjKzSoFbZo852SjsapmtuWkdyzg2 Lnk7s1pTqcyKAZVriLOjlBFdS6TdyZVFvDw+rmSqy8Je5fa/8sawTvhFFSs1HqQPcniM6z3Pezw 4 X-Received: by 2002:a17:906:3a8d:b0:a59:be8a:bd6f with SMTP id a640c23a62f3a-a5a2d65f272mr1374782466b.61.1715958170359; Fri, 17 May 2024 08:02:50 -0700 (PDT) Received: from localhost.localdomain ([149.14.240.163]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a5a17b17886sm1130572866b.210.2024.05.17.08.02.49 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 17 May 2024 08:02:49 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: =?utf-8?q?Daniel_P=2E_Berrang=C3=A9?= , Zhao Liu , =?utf-8?q?J=C3=A1n_Tomko?= , =?utf-8?q?Philippe_Mathieu-?= =?utf-8?q?Daud=C3=A9?= Subject: [PULL 4/5] tests: add testing of parameter=1 for SMP topology Date: Fri, 17 May 2024 17:02:26 +0200 Message-ID: <20240517150227.32205-5-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20240517150227.32205-1-philmd@linaro.org> References: <20240517150227.32205-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::632; envelope-from=philmd@linaro.org; helo=mail-ej1-x632.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, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, SPF_HELO_NONE=0.001, 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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Daniel P. Berrangé Validate that it is possible to pass 'parameter=1' for any SMP topology parameter, since unsupported parameters are implicitly considered to always have a value of 1. Signed-off-by: Daniel P. Berrangé Reviewed-by: Zhao Liu Reviewed-by: Ján Tomko Message-ID: <20240513123358.612355-3-berrange@redhat.com> Signed-off-by: Philippe Mathieu-Daudé --- tests/unit/test-smp-parse.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/tests/unit/test-smp-parse.c b/tests/unit/test-smp-parse.c index 56165e6644..9fdba24fce 100644 --- a/tests/unit/test-smp-parse.c +++ b/tests/unit/test-smp-parse.c @@ -330,6 +330,14 @@ static const struct SMPTestData data_generic_valid[] = { .config = SMP_CONFIG_GENERIC(T, 8, T, 2, T, 4, T, 2, T, 16), .expect_prefer_sockets = CPU_TOPOLOGY_GENERIC(8, 2, 4, 2, 16), .expect_prefer_cores = CPU_TOPOLOGY_GENERIC(8, 2, 4, 2, 16), + }, { + /* + * Unsupported parameters are always allowed to be set to '1' + * config: -smp 8,books=1,drawers=1,sockets=2,modules=1,dies=1,cores=2,threads=2,maxcpus=8 + * expect: cpus=8,sockets=2,cores=2,threads=2,maxcpus=8 */ + .config = SMP_CONFIG_WITH_FULL_TOPO(8, 1, 1, 2, 1, 1, 2, 2, 8), + .expect_prefer_sockets = CPU_TOPOLOGY_GENERIC(8, 2, 2, 2, 8), + .expect_prefer_cores = CPU_TOPOLOGY_GENERIC(8, 2, 2, 2, 8), }, }; From patchwork Fri May 17 15:02:27 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= X-Patchwork-Id: 797475 Delivered-To: patch@linaro.org Received: by 2002:adf:e68e:0:b0:351:d90a:5487 with SMTP id r14csp815373wrm; Fri, 17 May 2024 08:04:19 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCXIcZthClKgdq/FXD9hpsJrbeBPrx73IuzVBpWhgn/qdYSiOZvKXWJpCl1pN5fHgSbjkOPJAlh0qeDX3r1/jmNQ X-Google-Smtp-Source: AGHT+IGaJ2s9/D30PYUuMRdx/1DoN9RGNSOze4lUOKPRasAOmJbXB8Y5mz8eMlPlA8H3QafABIW6 X-Received: by 2002:a25:d04d:0:b0:de4:5c35:62d with SMTP id 3f1490d57ef6-dee4f314afamr21700575276.52.1715958259512; Fri, 17 May 2024 08:04:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715958259; cv=none; d=google.com; s=arc-20160816; b=SK1tqzc++Id6TJD0ew42oPWwaQ5I00MKNNFUbnax4NaOTtOZea/grzcuPGTaRK1W8D B3QIKojfZ6ncXdVyrRSGmqyTKUqEFpRSIBXntwZ4j4ORzrSgsQ1s3JWTjcCSZFJarMJ/ zAAn3aobaET22JZP1xRO8UY16XCCY8CHhZLbGkVSPBotP7xgG9LAx7iqE6BM4kmf8KH4 W/+4gPwW+EJq5szu515KAuBlmwmZiVSjisEvhKvZacsVmkbL9KA9aDD87xpjTXWgqZjH 57ENeCkBwq4uP52ZTeE+1XcxzxJoM9pqNWa988TvdCTIchK0qYJ7Vj94cm2VZq/XPAEI oIhg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:list-subscribe:list-help:list-post:list-archive :list-unsubscribe:list-id:precedence:content-transfer-encoding :mime-version:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=CZ4R0GpwuiTGFZYxz5B+fpMVNyC6MJRh/IKnd5gDIFo=; fh=ePyGoEiUVsGq5muf6+HqrCJFUwgxhWNYwaYwKs9b+Tk=; b=lSZ6K/ke+IRFU6glK9n3DzMB7gvXV431a/bPo9dyjHzDBMALdq3epJg4KMchXIIbGf xzcl6//cdSO3dR2pEIFkFCPjuSlP3H6IqkPLp3nbo3qk/SBg/3VtcvAbiCale2ATFV/9 f4uNx+B8FuPcq9buQaTWD/1dpO4QvoZHsVI5sQ3cVAtq73WchFHjKgrst+O5vb1oXtK2 44RP6xi2YX6tedaeCh+hfzwXtC1dbaYSRl8V1g1blYlnrY5ESIJpjmcz6xZDThxTA9Bk ROtuJmCVAfZgeeovAOZzW+N4kgTxG/ZSOunxtBYeHxnpOHUdaS9EvywmhW9z0ifDSmYp MjhQ==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=fEqAGDOJ; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id 3f1490d57ef6-decd2f716c1si8491999276.673.2024.05.17.08.04.19 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 17 May 2024 08:04:19 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=fEqAGDOJ; spf=pass (google.com: domain of qemu-devel-bounces+patch=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+patch=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7z6q-00051a-9F; Fri, 17 May 2024 11:03:00 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7z6o-0004zq-RV for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:58 -0400 Received: from mail-ej1-x62b.google.com ([2a00:1450:4864:20::62b]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1s7z6n-0005QV-8n for qemu-devel@nongnu.org; Fri, 17 May 2024 11:02:58 -0400 Received: by mail-ej1-x62b.google.com with SMTP id a640c23a62f3a-a59e4136010so514879566b.3 for ; Fri, 17 May 2024 08:02:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1715958175; x=1716562975; darn=nongnu.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=CZ4R0GpwuiTGFZYxz5B+fpMVNyC6MJRh/IKnd5gDIFo=; b=fEqAGDOJK1oItUQ4vhKuDvBvDFXISaGMwkUtHoUqXBih+NB0f56QPJqjcJbnXfXqf8 iENDaUdQX5HmbT0dJVc+tDvELlCWyQWfdl6YM/TlC6Vcuf8FmrnhPR6sfaZNl7tuQMWR PZnu6jDnnWnd34Qzdt0Efrljz3eqCqNmCXqLKBxJsAtJoN53+GSx4FMAiXryxYtQEOEl ZymReNmfbuKfpQQuFWeL2d00zm9lCaJ6qOph0AkbRyojd3x+wKAaoxsV/ljFU09HSp8v xlkeVP/FrqXVO3mgQcelste2cYC2q4+/PjvfQznHOUoUz8+Bvt4GhYawXQ+BBBSGMFGR F+Rg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1715958175; x=1716562975; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=CZ4R0GpwuiTGFZYxz5B+fpMVNyC6MJRh/IKnd5gDIFo=; b=vNcIFvSlwneF2mU9YBwRhdRmaipNivuPL2W9o6vyqqw+TwOvvsrh8/t7aUnAky2Kxw wOU6AxNcbLShQgoqDL8zJRbCDVgvWewL6tEq049C8NX9u7XR7NkbJQ2KKJpuftGk9L3o QB2afaGmQfB9MBGw7g5nUsrKboJP1VNB0M7mek7wtIASGuzWNbwxM6DDIUVyCymh04jh ABz03fUp+pJQjN9k066jNZ81jEYaTWF/9tFZhbdPzUy2AhFu+Risp8A8iWoV2cygmi6K VOonY9IyRDVe+lxMzaGsqyNdcx9xKpob84msSYPOI85Vf65vvHTxZNva+DJcNoxb2Vj6 1JsA== X-Gm-Message-State: AOJu0YzED+PuuuscOy3UAGUEP2B3GSmez5cMLzbuHxRZXuJqEDPXcDwt fZYS9DTc0eEtZvx3TkJuwJbx8Sm7INv2IK4H4KsAub/MKjMZbIpv5u6kZ3GhkJAzOLEvNrxKnyf z X-Received: by 2002:a17:906:15cd:b0:a59:c9ce:3386 with SMTP id a640c23a62f3a-a5a2d5cb0c1mr1344291466b.33.1715958175249; Fri, 17 May 2024 08:02:55 -0700 (PDT) Received: from localhost.localdomain ([149.14.240.163]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-a5a17b17555sm1120578666b.189.2024.05.17.08.02.54 (version=TLS1_3 cipher=TLS_CHACHA20_POLY1305_SHA256 bits=256/256); Fri, 17 May 2024 08:02:54 -0700 (PDT) From: =?utf-8?q?Philippe_Mathieu-Daud=C3=A9?= To: qemu-devel@nongnu.org Cc: Gustavo Romero , =?utf-8?q?Philippe_Mathieu-D?= =?utf-8?q?aud=C3=A9?= Subject: [PULL 5/5] tests: Gently exit from GDB when tests complete Date: Fri, 17 May 2024 17:02:27 +0200 Message-ID: <20240517150227.32205-6-philmd@linaro.org> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20240517150227.32205-1-philmd@linaro.org> References: <20240517150227.32205-1-philmd@linaro.org> MIME-Version: 1.0 Received-SPF: pass client-ip=2a00:1450:4864:20::62b; envelope-from=philmd@linaro.org; helo=mail-ej1-x62b.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, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, 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: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+patch=linaro.org@nongnu.org Sender: qemu-devel-bounces+patch=linaro.org@nongnu.org From: Gustavo Romero GDB commit a207f6b3a38 ('Rewrite "python" command exception handling') changed how exit() called from Python scripts loaded by GDB behave, turning it into an exception instead of a generic error code that is returned. This change caused several QEMU tests to crash with the following exception: Python Exception : 0 Error occurred in Python: 0 This happens because in tests/guest-debug/test_gdbstub.py exit is called after the tests have completed. This commit fixes it by politely asking GDB to exit via gdb.execute, passing the proper fail_count to be reported to 'make', instead of abruptly calling exit() from the Python script. Signed-off-by: Gustavo Romero Reviewed-by: Philippe Mathieu-Daudé Message-ID: <20240515173132.2462201-4-gustavo.romero@linaro.org> Signed-off-by: Philippe Mathieu-Daudé --- tests/guest-debug/test_gdbstub.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/guest-debug/test_gdbstub.py b/tests/guest-debug/test_gdbstub.py index 7f71d34da1..46fbf98f0c 100644 --- a/tests/guest-debug/test_gdbstub.py +++ b/tests/guest-debug/test_gdbstub.py @@ -57,4 +57,4 @@ def main(test, expected_arch=None): pass print("All tests complete: {} failures".format(fail_count)) - exit(fail_count) + gdb.execute(f"exit {fail_count}")