From patchwork Wed Nov 27 13:57:29 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887054 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 86E20D6ACD8 for ; Wed, 27 Nov 2024 13:58:15 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIY3-0003eq-F9; Wed, 27 Nov 2024 08:57:43 -0500 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 1tGIXx-0003dt-SW for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:39 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIXw-0007hA-9W for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:37 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715854; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=toqdy82Y55iUyJz9RE8D3pVN23M/G0bs/onY8PVnesI=; b=I3FmuQA1gKseA51Lv3AMA8FLrpnRTDvAFIj07BM8noP/luC320BOgjSS1E31/vRqZLTYzB Gc5A410SEg7NAiyFmnxyEc0eQqE2tfHXNBjegTWtRevGNjrk5GRchT87zd9YBsj2JkdLDs DOq2gCLdLtyvViZF2lehmOcqTyG4Tjs= Received: from mail-wm1-f72.google.com (mail-wm1-f72.google.com [209.85.128.72]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-131-K4sLnw8HMiKCj0OYeTF8hA-1; Wed, 27 Nov 2024 08:57:33 -0500 X-MC-Unique: K4sLnw8HMiKCj0OYeTF8hA-1 X-Mimecast-MFC-AGG-ID: K4sLnw8HMiKCj0OYeTF8hA Received: by mail-wm1-f72.google.com with SMTP id 5b1f17b1804b1-4349d895ef8so35419465e9.0 for ; Wed, 27 Nov 2024 05:57:33 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715852; x=1733320652; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=toqdy82Y55iUyJz9RE8D3pVN23M/G0bs/onY8PVnesI=; b=poydHEOQw06pbEUqeSnYM6tubQeXEgiiSvzL5Ue73jkeceTM9sTtNY5GtcRJgrcgQw 6JOz1jnGIsFSzH99auTu1oNZ3AdfWLGvMe75Z/CBzVUm6wtszdVm5T/0k9tQPFKg4eP+ t0InmeXMQhpQzOebT0ZuarYwT+P1le141t7W4YLlzzLFJ8AS1HpxFtTkBy8mas5nJEFn cJ9cR6i0roUIkJH8Ilaa1SrzJoxodJ1hvEDBeliLYRRCz359GIF7k+dNbeOpCFMUCSGt VP2jKylsZg6sZUxCQK1xywN/yLAOYQdLrEeL2mHosvsph8KQ4W/1HA8TEA7fPUIsTRyc l5hQ== X-Gm-Message-State: AOJu0Yw5EPJwEMjqkWItgeoMjK489Mq1zayUxHl0q3nLA4u3soR+WTxP xKkNNNIuj88wMkK5GlzCV5k4kWQ4L3kh9lYlUJiIAwoCGB6cc6pWQhk/doJyVAmJuBntLwMj/SJ HpC/ZcOzcjAaraCfBEX8ugoTrata6voIi0d9bM3qyAww6qFiZHJDaiLEafhkH/7HXAdSMrA55mt Vg73SWMmTKyo5Ho8Pbqtrw0f31AXysuEy8 X-Gm-Gg: ASbGncsRDZ17WHK+xMGeRChO642L7SCdmiJBCPeU10SlGJazmHSC6EJJoQxOwhrWfu8 nzWqmig6Z7OPqJC9kkn6U8GwWZiLnfbteVA2VSMXAmhevWilv25GYcs+OKFZs80yTvCdl/AnE7C JH5Sljr3yeRYiwB1vqRmUSvXynUR49BKqmcBl7yBrLlqv3sGHLQDxyBnokWW9M/8+Wvw0dY0haZ A1Vvzpgmf64OkAqD0HdV0eZoyWL18R0sKaVrUfbOu68 X-Received: by 2002:a05:600c:4f12:b0:431:542d:2599 with SMTP id 5b1f17b1804b1-434a9de8601mr27605775e9.22.1732715852326; Wed, 27 Nov 2024 05:57:32 -0800 (PST) X-Google-Smtp-Source: AGHT+IGNgjYU7nfRqAFwfmEvL9HfqW5GMuNiUVj84Jwzcu+4+Adu/CsUKRoQkpUS7aX3NNJJd3TN2A== X-Received: by 2002:a05:600c:4f12:b0:431:542d:2599 with SMTP id 5b1f17b1804b1-434a9de8601mr27605465e9.22.1732715851858; Wed, 27 Nov 2024 05:57:31 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-434aa7fa3d4sm21961015e9.44.2024.11.27.05.57.30 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:31 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:29 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , zuoboqun , Gao Shiyuan , Stefano Garzarella , Jason Wang Subject: [PULL 01/13] vhost_net: fix assertion triggered by batch of host notifiers processing Message-ID: <16f6804c460109e8b126cc710a0072f9f83bd230.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: zuoboqun When the backend of vhost_net restarts during the vm is running, vhost_net is stopped and started. The virtio_device_grab_ioeventfd() fucntion in vhost_net_enable_notifiers() will result in a call to virtio_bus_set_host_notifier()(assign=false). And now virtio_device_grab_ioeventfd() is batched in a single transaction with virtio_bus_set_host_notifier()(assign=true). This triggers the following assertion: kvm_mem_ioeventfd_del: error deleting ioeventfd: Bad file descriptor This patch moves virtio_device_grab_ioeventfd() out of the batch to fix this problem. To be noted that the for loop to release ioeventfd should start from i+1, not i, because the i-th ioeventfd has already been released in vhost_dev_disable_notifiers_nvqs(). Fixes: 6166799f6 ("vhost_net: configure all host notifiers in a single MR transaction") Signed-off-by: Zuo Boqun Reported-by: Gao Shiyuan Message-Id: <20241115080312.3184-1-zuoboqun@baidu.com> Acked-by: Stefano Garzarella Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/net/vhost_net.c | 35 ++++++++++++++++++++++++----------- 1 file changed, 24 insertions(+), 11 deletions(-) diff --git a/hw/net/vhost_net.c b/hw/net/vhost_net.c index 997aab0557..891f235a0a 100644 --- a/hw/net/vhost_net.c +++ b/hw/net/vhost_net.c @@ -229,9 +229,24 @@ static int vhost_net_enable_notifiers(VirtIODevice *dev, int nvhosts = data_queue_pairs + cvq; struct vhost_net *net; struct vhost_dev *hdev; - int r, i, j; + int r, i, j, k; NetClientState *peer; + /* + * We will pass the notifiers to the kernel, make sure that QEMU + * doesn't interfere. + */ + for (i = 0; i < nvhosts; i++) { + r = virtio_device_grab_ioeventfd(dev); + if (r < 0) { + error_report("vhost %d binding does not support host notifiers", i); + for (k = 0; k < i; k++) { + virtio_device_release_ioeventfd(dev); + } + return r; + } + } + /* * Batch all the host notifiers in a single transaction to avoid * quadratic time complexity in address_space_update_ioeventfds(). @@ -247,16 +262,6 @@ static int vhost_net_enable_notifiers(VirtIODevice *dev, net = get_vhost_net(peer); hdev = &net->dev; - /* - * We will pass the notifiers to the kernel, make sure that QEMU - * doesn't interfere. - */ - r = virtio_device_grab_ioeventfd(dev); - if (r < 0) { - error_report("binding does not support host notifiers"); - memory_region_transaction_commit(); - goto fail_nvhosts; - } for (j = 0; j < hdev->nvqs; j++) { r = virtio_bus_set_host_notifier(VIRTIO_BUS(qbus), @@ -277,6 +282,14 @@ static int vhost_net_enable_notifiers(VirtIODevice *dev, return 0; fail_nvhosts: vhost_net_disable_notifiers_nvhosts(dev, ncs, data_queue_pairs, i); + /* + * This for loop starts from i+1, not i, because the i-th ioeventfd + * has already been released in vhost_dev_disable_notifiers_nvqs(). + */ + for (k = i + 1; k < nvhosts; k++) { + virtio_device_release_ioeventfd(dev); + } + return r; } From patchwork Wed Nov 27 13:57:32 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887063 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5638ED6ACD9 for ; Wed, 27 Nov 2024 14:00:23 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIY5-0003fF-MN; Wed, 27 Nov 2024 08:57:45 -0500 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 1tGIY0-0003eO-5v for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:41 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIXy-0007hY-K9 for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:39 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715857; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=EHliCKGIV0A4mTHvD8Vp0gNJsTqr7mAyIUNPjljDA4U=; b=LLHahnzd7fH18scFRPH//YvQhfn/9Ayf93ng4EMgQeQjqk/zNL3T0r1cBy25WTlUi3rq6a BBASQGZLGgEkGRqb2Ieda/3b5T6n97HUF+rx8qxcnsXNAgqdGz40bHhdQTrGjnlgJsZg2D 0MJ2tjazbpJ9PfO3V3FIIDxqJ3yVk/c= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-595-UpfmxzluPEu5ipnY1TgijQ-1; Wed, 27 Nov 2024 08:57:36 -0500 X-MC-Unique: UpfmxzluPEu5ipnY1TgijQ-1 X-Mimecast-MFC-AGG-ID: UpfmxzluPEu5ipnY1TgijQ Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-38240d9ed31so4052271f8f.3 for ; Wed, 27 Nov 2024 05:57:36 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715855; x=1733320655; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=EHliCKGIV0A4mTHvD8Vp0gNJsTqr7mAyIUNPjljDA4U=; b=tItsSTCHJSfZaFNJ4hDClaUK1TIwNrrJpNlBntH59zAcKJLcsFthrNAs4iTn1gMQz9 n5y5ZmJVf4RPFsMXCXwsI2SAUzGtXf+CZ7kGVOYcXuZhwvVksad0o1b0cQRuFphWRHlR p3+hPvG31DktRUL53wn9mSUZJLS+npjA6KcM++HYK7vjyA9AE4Uaqu77dx9l/aHqXchI vLlis9ry92bkn+4XoitrG7Yx1iYYz1PdliGXnGCfsDjjf27bBQqqX7snCX1l5QuMGWiV QioCxVHoTKNX5f9a/WM5+0ggtoeK/8Usji3N4TL0nRAg3yC+3HoZJErRRj4ALtEH6H5V TOew== X-Gm-Message-State: AOJu0Yy0yVFpl1aeLvKgnxY2QBZQAFq+zcVwCm/Sa8ZvhIR2JgmuXPhE lc0X4VuuNiNQTkS/VOsESDwy7XiJFHfbLt5O5P0AsejqDytSKRU5dMRs5IK9ma/fAzOy3t7C2zm 56GdcmxVqd+oAoH0KynpAoe2xu/uyiBNmORiYfisIqQTETf6ZQSkH3TxDB6Hq1n8hlDr3duzVYF Mg28tqcEudhrV9vOeVFm8IH3V9mplPRtYR X-Gm-Gg: ASbGncuuz65U6BPqYaL9mRADCgbW7D3nBq/xW9TVczHtSZDux2RD8S+/Snuzq8wL4YK B/o8EyQgd1fx0t7f9Q2SSsp2J1jQTnCpAwxpOlr6Ldu8egjj9fbItj7qh6oIhHO1qp4lPXSxBlN FcKY2iqSrKOLW35VRwewV7T0aiKAOdL2lpZmZHGc76tshuf36rmgMrQzBGJSGLMpGuUx4TNKfxR nuXpoN72EBiUPso7/DYJYNFlLl8eEbedduEVnMpaHo5 X-Received: by 2002:a5d:5f52:0:b0:382:4e57:48e7 with SMTP id ffacd0b85a97d-385c6eb58eemr2268052f8f.10.1732715854978; Wed, 27 Nov 2024 05:57:34 -0800 (PST) X-Google-Smtp-Source: AGHT+IEqwMJrrUGS/D6SdJy43XgOsUdKIbwq69vAoCNe8GS+KbbEELU0Zba47aNtqI2v7RsrQrodrg== X-Received: by 2002:a5d:5f52:0:b0:382:4e57:48e7 with SMTP id ffacd0b85a97d-385c6eb58eemr2268031f8f.10.1732715854565; Wed, 27 Nov 2024 05:57:34 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fafe207sm16447042f8f.36.2024.11.27.05.57.33 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:33 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:32 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Salil Mehta , Igor Mammedov , Ani Sinha Subject: [PULL 02/13] qtest: allow ACPI DSDT Table changes Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Salil Mehta list changed files in tests/qtest/bios-tables-test-allowed-diff.h Message-ID: <20241106100047.18901c9d@imammedo.users.ipa.redhat.com> Signed-off-by: Salil Mehta Signed-off-by: Igor Mammedov Message-Id: <20241112170258.2996640-2-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test-allowed-diff.h | 41 +++++++++++++++++++++ 1 file changed, 41 insertions(+) diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index dfb8523c8b..512d40665d 100644 --- a/tests/qtest/bios-tables-test-allowed-diff.h +++ b/tests/qtest/bios-tables-test-allowed-diff.h @@ -1 +1,42 @@ /* List of comma-separated changed AML files to ignore */ +"tests/data/acpi/x86/pc/DSDT", +"tests/data/acpi/x86/pc/DSDT.acpierst", +"tests/data/acpi/x86/pc/DSDT.acpihmat", +"tests/data/acpi/x86/pc/DSDT.bridge", +"tests/data/acpi/x86/pc/DSDT.cphp", +"tests/data/acpi/x86/pc/DSDT.dimmpxm", +"tests/data/acpi/x86/pc/DSDT.hpbridge", +"tests/data/acpi/x86/pc/DSDT.hpbrroot", +"tests/data/acpi/x86/pc/DSDT.ipmikcs", +"tests/data/acpi/x86/pc/DSDT.memhp", +"tests/data/acpi/x86/pc/DSDT.nohpet", +"tests/data/acpi/x86/pc/DSDT.numamem", +"tests/data/acpi/x86/pc/DSDT.roothp", +"tests/data/acpi/x86/q35/DSDT", +"tests/data/acpi/x86/q35/DSDT.acpierst", +"tests/data/acpi/x86/q35/DSDT.acpihmat", +"tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator", +"tests/data/acpi/x86/q35/DSDT.applesmc", +"tests/data/acpi/x86/q35/DSDT.bridge", +"tests/data/acpi/x86/q35/DSDT.core-count", +"tests/data/acpi/x86/q35/DSDT.core-count2", +"tests/data/acpi/x86/q35/DSDT.cphp", +"tests/data/acpi/x86/q35/DSDT.cxl", +"tests/data/acpi/x86/q35/DSDT.dimmpxm", +"tests/data/acpi/x86/q35/DSDT.ipmibt", +"tests/data/acpi/x86/q35/DSDT.ipmismbus", +"tests/data/acpi/x86/q35/DSDT.ivrs", +"tests/data/acpi/x86/q35/DSDT.memhp", +"tests/data/acpi/x86/q35/DSDT.mmio64", +"tests/data/acpi/x86/q35/DSDT.multi-bridge", +"tests/data/acpi/x86/q35/DSDT.noacpihp", +"tests/data/acpi/x86/q35/DSDT.nohpet", +"tests/data/acpi/x86/q35/DSDT.numamem", +"tests/data/acpi/x86/q35/DSDT.pvpanic-isa", +"tests/data/acpi/x86/q35/DSDT.thread-count", +"tests/data/acpi/x86/q35/DSDT.thread-count2", +"tests/data/acpi/x86/q35/DSDT.tis.tpm12", +"tests/data/acpi/x86/q35/DSDT.tis.tpm2", +"tests/data/acpi/x86/q35/DSDT.type4-count", +"tests/data/acpi/x86/q35/DSDT.viot", +"tests/data/acpi/x86/q35/DSDT.xapic", From patchwork Wed Nov 27 13:57:34 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887065 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A9105D6ACD9 for ; Wed, 27 Nov 2024 14:00:29 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIY6-0003fR-SB; Wed, 27 Nov 2024 08:57:47 -0500 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 1tGIY3-0003em-DX for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:43 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIY1-0007iI-N6 for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:43 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715861; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=P3wEkJJcev47yGejLPspLgvfrlRtXLqP/wDtBuhCwFs=; b=SgZgwPorDo6dOKUaG6FgfuQU9b97b5a+jC3IWyJzyO7WWzdRACIZCRDicRHI7cNGryeObu y9T7XPhcqLl1QnbU7kp6TLnp9kQu+z2hZM8dZgOVCH7rYTQagy0HjtVPXrkbTsmWgoybA1 HtU+v3wha3TN+DGW63HGKkYrCOhT4U0= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-639-aJiqfwVIOvKZfVwO0TSo_w-1; Wed, 27 Nov 2024 08:57:39 -0500 X-MC-Unique: aJiqfwVIOvKZfVwO0TSo_w-1 X-Mimecast-MFC-AGG-ID: aJiqfwVIOvKZfVwO0TSo_w Received: by mail-wr1-f70.google.com with SMTP id ffacd0b85a97d-382450d158fso3295797f8f.1 for ; Wed, 27 Nov 2024 05:57:39 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715858; x=1733320658; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=P3wEkJJcev47yGejLPspLgvfrlRtXLqP/wDtBuhCwFs=; b=UWC1fbFJdx6YJVW5cNtWvz6WH0TVLq2YLa977NDPD56iuqlU9/0gisTquY0AYncK9V lcIxnwRNrGYBu2VPAl+M/QDvmWWCKBNfAgdDQN7mGI+7SY+yqW6qZgVbuum8oAiSNF1f jK14j2FI9bWAKkE3tYzwsT4n0HLq9KEnBIrLIawvopr0bp3MkHygy/bGeE6h1f9j20O9 BpGQimVdbPD6BFIniaTFgSrvsIVr3hq5c6iXNPBvdPX2Gteuqj0SKp9WStHZb5XhyeOX c3uDHB1M2yiny0A0j4nvPzCu6xzuDOMYuSwHLa7FRPlAIXp0CgS/EFfAx0yh4RZ9GGr/ K+zg== X-Gm-Message-State: AOJu0Yx3KrbZRSyoLP7aaR5L8nOUVkGJBTfz7+T3LkUOBNokY6i3qRA8 i4LVIUNXMDjR7VaNix7VFJMYSS/wJqBosLm7HvfOq2bBkSxM+iSpNQSOV33k8Cjj38+Tt+qTIac Qf6hDimnnt2Jow/AOAqUseceuCtqLPK7Fm0E/UfmN9cEqX5LdM8Qk/GnPXpk8ewnP2DytlBR+bP BsVWZm2bwjy0dsT7fl33RLRCykgoPmPGEM X-Gm-Gg: ASbGncv9OsYvuFkuJv1/5uLhHuNq6kEpX/ON8ziA3KVPh7ekX+jgO7uK0WQVyNVr/l1 kA+SU73hqfqO0VW2LNGCs+ZwiT552R9GOgu5cKNiMpMG76qFXeT07aeFJa0830bosMMYxKnGV7d XpckJAPzG+MmrthJ/QggHA4g1f4LJZ0DIPhjrT/22raxUbkS6xregnZ2hXl2HlfGjvr7KvvpNyG 5slaKGE9XJXEOy9az3mX4FrhnssCGHTjBULONSmKKUn X-Received: by 2002:a05:6000:2ae:b0:382:49f3:4f8d with SMTP id ffacd0b85a97d-385c6eb698cmr2800773f8f.15.1732715858131; Wed, 27 Nov 2024 05:57:38 -0800 (PST) X-Google-Smtp-Source: AGHT+IF8tAWAPAhc356fDV5VV3PpkJqaXk52M8jmEOqJ6EaO1ogjfR/59gUpnWqKc/JcER2BuPSoAg== X-Received: by 2002:a05:6000:2ae:b0:382:49f3:4f8d with SMTP id ffacd0b85a97d-385c6eb698cmr2800737f8f.15.1732715857676; Wed, 27 Nov 2024 05:57:37 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbeddf4sm16221472f8f.93.2024.11.27.05.57.35 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:37 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:34 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Igor Mammedov , Ani Sinha , Eduardo Habkost , Marcel Apfelbaum , Philippe =?utf-8?q?Mathieu-D?= =?utf-8?q?aud=C3=A9?= , Yanan Wang , Zhao Liu Subject: [PULL 03/13] Revert "hw/acpi: Update ACPI `_STA` method with QOM vCPU ACPI Hotplug states" Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Igor Mammedov This reverts commit bf1ecc8dad6061914730a2a2d57af6b37c3a4f8d which broke cpu hotplug in x86 after migration to older QEMU Fixes: bf1ecc8dad606 (w/acpi: Update ACPI `_STA` method with QOM vCPU ACPI Hotplug states) Signed-off-by: Igor Mammedov Message-Id: <20241112170258.2996640-3-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/core/cpu.h | 1 - hw/acpi/cpu.c | 38 ++++---------------------------------- 2 files changed, 4 insertions(+), 35 deletions(-) diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h index db8a6fbc6e..e7de77dc6d 100644 --- a/include/hw/core/cpu.h +++ b/include/hw/core/cpu.h @@ -159,7 +159,6 @@ struct CPUClass { void (*query_cpu_fast)(CPUState *cpu, CpuInfoFast *value); int64_t (*get_arch_id)(CPUState *cpu); bool (*cpu_persistent_status)(CPUState *cpu); - bool (*cpu_enabled_status)(CPUState *cpu); void (*set_pc)(CPUState *cpu, vaddr value); vaddr (*get_pc)(CPUState *cpu); int (*gdb_read_register)(CPUState *cpu, GByteArray *buf, int reg); diff --git a/hw/acpi/cpu.c b/hw/acpi/cpu.c index 23443f09a5..9b03b4292e 100644 --- a/hw/acpi/cpu.c +++ b/hw/acpi/cpu.c @@ -50,18 +50,6 @@ void acpi_cpu_ospm_status(CPUHotplugState *cpu_st, ACPIOSTInfoList ***list) } } -static bool check_cpu_enabled_status(DeviceState *dev) -{ - CPUClass *k = dev ? CPU_GET_CLASS(dev) : NULL; - CPUState *cpu = CPU(dev); - - if (cpu && (!k->cpu_enabled_status || k->cpu_enabled_status(cpu))) { - return true; - } - - return false; -} - static uint64_t cpu_hotplug_rd(void *opaque, hwaddr addr, unsigned size) { uint64_t val = 0; @@ -75,11 +63,10 @@ static uint64_t cpu_hotplug_rd(void *opaque, hwaddr addr, unsigned size) cdev = &cpu_st->devs[cpu_st->selector]; switch (addr) { case ACPI_CPU_FLAGS_OFFSET_RW: /* pack and return is_* fields */ - val |= check_cpu_enabled_status(DEVICE(cdev->cpu)) ? 1 : 0; + val |= cdev->cpu ? 1 : 0; val |= cdev->is_inserting ? 2 : 0; val |= cdev->is_removing ? 4 : 0; val |= cdev->fw_remove ? 16 : 0; - val |= cdev->cpu ? 32 : 0; trace_cpuhp_acpi_read_flags(cpu_st->selector, val); break; case ACPI_CPU_CMD_DATA_OFFSET_RW: @@ -362,7 +349,6 @@ const VMStateDescription vmstate_cpu_hotplug = { #define CPU_REMOVE_EVENT "CRMV" #define CPU_EJECT_EVENT "CEJ0" #define CPU_FW_EJECT_EVENT "CEJF" -#define CPU_PRESENT "CPRS" void build_cpus_aml(Aml *table, MachineState *machine, CPUHotplugFeatures opts, build_madt_cpu_fn build_madt_cpu, hwaddr base_addr, @@ -423,9 +409,7 @@ void build_cpus_aml(Aml *table, MachineState *machine, CPUHotplugFeatures opts, aml_append(field, aml_named_field(CPU_EJECT_EVENT, 1)); /* tell firmware to do device eject, write only */ aml_append(field, aml_named_field(CPU_FW_EJECT_EVENT, 1)); - /* 1 if present, read only */ - aml_append(field, aml_named_field(CPU_PRESENT, 1)); - aml_append(field, aml_reserved_field(2)); + aml_append(field, aml_reserved_field(3)); aml_append(field, aml_named_field(CPU_COMMAND, 8)); aml_append(cpu_ctrl_dev, field); @@ -455,7 +439,6 @@ void build_cpus_aml(Aml *table, MachineState *machine, CPUHotplugFeatures opts, Aml *ctrl_lock = aml_name("%s.%s", cphp_res_path, CPU_LOCK); Aml *cpu_selector = aml_name("%s.%s", cphp_res_path, CPU_SELECTOR); Aml *is_enabled = aml_name("%s.%s", cphp_res_path, CPU_ENABLED); - Aml *is_present = aml_name("%s.%s", cphp_res_path, CPU_PRESENT); Aml *cpu_cmd = aml_name("%s.%s", cphp_res_path, CPU_COMMAND); Aml *cpu_data = aml_name("%s.%s", cphp_res_path, CPU_DATA); Aml *ins_evt = aml_name("%s.%s", cphp_res_path, CPU_INSERT_EVENT); @@ -484,26 +467,13 @@ void build_cpus_aml(Aml *table, MachineState *machine, CPUHotplugFeatures opts, { Aml *idx = aml_arg(0); Aml *sta = aml_local(0); - Aml *ifctx2; - Aml *else_ctx; aml_append(method, aml_acquire(ctrl_lock, 0xFFFF)); aml_append(method, aml_store(idx, cpu_selector)); aml_append(method, aml_store(zero, sta)); - ifctx = aml_if(aml_equal(is_present, one)); + ifctx = aml_if(aml_equal(is_enabled, one)); { - ifctx2 = aml_if(aml_equal(is_enabled, one)); - { - /* cpu is present and enabled */ - aml_append(ifctx2, aml_store(aml_int(0xF), sta)); - } - aml_append(ifctx, ifctx2); - else_ctx = aml_else(); - { - /* cpu is present but disabled */ - aml_append(else_ctx, aml_store(aml_int(0xD), sta)); - } - aml_append(ifctx, else_ctx); + aml_append(ifctx, aml_store(aml_int(0xF), sta)); } aml_append(method, ifctx); aml_append(method, aml_release(ctrl_lock)); From patchwork Wed Nov 27 13:57:37 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887055 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 65C5AD6ACD2 for ; Wed, 27 Nov 2024 13:58:37 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIY8-0003fw-VL; Wed, 27 Nov 2024 08:57:48 -0500 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 1tGIY5-0003fE-IE for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:45 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIY4-0007iV-3o for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715863; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=wJrhS8Qk1ShMNFRxlemP9veo4SdCGzL5pa49imArvN0=; b=WVJ+u7rbltc+0OLz3iKlnblrk1vYbDj90j0yGXN7XEU+NW35b62NUVu2CagcTRzN3GVzsF suMmhbef0IeaoX63qFHbaJFRhLtJkU1gOshPfdQPlLHTzHTRD7/hH1F51lU1RsNKtwdsZs t27V5V53JvD03wg7F/W71XuX7PHMH4s= Received: from mail-wm1-f69.google.com (mail-wm1-f69.google.com [209.85.128.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-689-vZWrfruuPDC5hnc2ekfLjA-1; Wed, 27 Nov 2024 08:57:42 -0500 X-MC-Unique: vZWrfruuPDC5hnc2ekfLjA-1 X-Mimecast-MFC-AGG-ID: vZWrfruuPDC5hnc2ekfLjA Received: by mail-wm1-f69.google.com with SMTP id 5b1f17b1804b1-434a4ad4798so17092855e9.0 for ; Wed, 27 Nov 2024 05:57:42 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715861; x=1733320661; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=wJrhS8Qk1ShMNFRxlemP9veo4SdCGzL5pa49imArvN0=; b=sCzsXQXxy1y2FGglvTdDg+MyLK7zUmpELpDft/1ARRJTnt3D1cY+/IJO+zoP+S5Oio KDT4fICids+locIZytm8CQ36pktN8cIyxwhBM+qvThevKthrVjWYVvfbatEi8RRdrzAL ea6LeCgf+2oVbPpYxT0odhzE4w7sB9cTeEW0Mi7z1MUbhI6E+aX97qyK75GeF0qlpn/Q HiTVekYkUsi5s3z+jKKqZfQvTVAhts8skmr4N30+lFtLZkRf0cl/EViWmBMuCYW8SJpc 9HDUQwYJFySfPAjA2lNp8QGNRGmsa0TdkQTu18ia5PFWkh0z6gpAUzLOWSHWxk7D7XPy x0MQ== X-Gm-Message-State: AOJu0YwTCye3+gCSfa9hnPrC8Q4TmDm0dRZLUMYeLuRSmH37Rzl5Io0u UBP+zPixpHn70OriAOGB7/pj2IJHAQSFEJ3c0UgVye4jrFL+LWOSJ76kg7o+UO6RZVJmKi7uoXz HWu11H8BOKbSyHfeIhILpOEQrfIDOxcI/soxEppciZKZqHR2Y67KX2sCzdXjJ0lTdPCzRvqpu5g J/yNryIVBXIKAnHQCm6j4P9eGavnyeNGer X-Gm-Gg: ASbGncvVEZEcxTpZgF0gEWPeIh8ydirInbJTNWH7loArvTZWBwld94BsY44xkNOr10v FHx/wVGTa3m61w8JtCfsKPC+jzkV0nbu2hf6EDytO1n9JaZN+eHCX1NrVqwK+MyB0Lv1exdRXzi 7sQ2vw5zIoW1jfui4YlextwFWdpfdLBGAgfaqbNHqQA/aWcg5zc1qZh95RLuJzD6s9kVc5XoV5P pEXzE47Bi/3HPAOGs+glDXhZv044EPbhIR3ngjWyup3 X-Received: by 2002:a05:600c:56d7:b0:434:a468:4a57 with SMTP id 5b1f17b1804b1-434a9de8d3amr24432445e9.26.1732715860897; Wed, 27 Nov 2024 05:57:40 -0800 (PST) X-Google-Smtp-Source: AGHT+IELN1x1hmuo8yUsYGRVwB9aTk0Dg6sfAIxKvvrxbnP0P6rjOTRz9ZyWqqhd/yqJjLk/8dIptg== X-Received: by 2002:a05:600c:56d7:b0:434:a468:4a57 with SMTP id 5b1f17b1804b1-434a9de8d3amr24432225e9.26.1732715860541; Wed, 27 Nov 2024 05:57:40 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-434aa7804fbsm22358165e9.21.2024.11.27.05.57.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:39 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:37 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Igor Mammedov , Ani Sinha , Eduardo Habkost , Marcel Apfelbaum , Philippe =?utf-8?q?Mathieu-D?= =?utf-8?q?aud=C3=A9?= , Yanan Wang , Zhao Liu Subject: [PULL 04/13] Revert "hw/acpi: Make CPUs ACPI `presence` conditional during vCPU hot-unplug" Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Igor Mammedov This reverts commit 2d6cfbaf174b91dfa9a50065f7494634afb39c23. The patch is supposed to be part of ARM CPU hotplug series and has not value on its own without it. The series however is still in RFC stage and outside of scope 9.2 release. On top of that it introduces not needed callback that pokes directly into CPU state without any need for that. Instead properties and AML generator option should be used to configure static platform depended vCPU presence state. Drop the patch so that corrected version could be posted along with ARM CPU hotplug series and properly reviewed in relevant context. That also helps us to keep history cleaner with new patch being against original code vs a string of fixups on top of current mess. Signed-off-by: Igor Mammedov Message-Id: <20241112170258.2996640-4-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- include/hw/core/cpu.h | 1 - hw/acpi/cpu.c | 15 +-------------- 2 files changed, 1 insertion(+), 15 deletions(-) diff --git a/include/hw/core/cpu.h b/include/hw/core/cpu.h index e7de77dc6d..c3ca0babcb 100644 --- a/include/hw/core/cpu.h +++ b/include/hw/core/cpu.h @@ -158,7 +158,6 @@ struct CPUClass { void (*dump_state)(CPUState *cpu, FILE *, int flags); void (*query_cpu_fast)(CPUState *cpu, CpuInfoFast *value); int64_t (*get_arch_id)(CPUState *cpu); - bool (*cpu_persistent_status)(CPUState *cpu); void (*set_pc)(CPUState *cpu, vaddr value); vaddr (*get_pc)(CPUState *cpu); int (*gdb_read_register)(CPUState *cpu, GByteArray *buf, int reg); diff --git a/hw/acpi/cpu.c b/hw/acpi/cpu.c index 9b03b4292e..5cb60ca8bc 100644 --- a/hw/acpi/cpu.c +++ b/hw/acpi/cpu.c @@ -233,17 +233,6 @@ void cpu_hotplug_hw_init(MemoryRegion *as, Object *owner, memory_region_add_subregion(as, base_addr, &state->ctrl_reg); } -static bool should_remain_acpi_present(DeviceState *dev) -{ - CPUClass *k = CPU_GET_CLASS(dev); - /* - * A system may contain CPUs that are always present on one die, NUMA node, - * or socket, yet may be non-present on another simultaneously. Check from - * architecture specific code. - */ - return k->cpu_persistent_status && k->cpu_persistent_status(CPU(dev)); -} - static AcpiCpuStatus *get_cpu_status(CPUHotplugState *cpu_st, DeviceState *dev) { CPUClass *k = CPU_GET_CLASS(dev); @@ -300,9 +289,7 @@ void acpi_cpu_unplug_cb(CPUHotplugState *cpu_st, return; } - if (!should_remain_acpi_present(dev)) { - cdev->cpu = NULL; - } + cdev->cpu = NULL; } static const VMStateDescription vmstate_cpuhp_sts = { From patchwork Wed Nov 27 13:57:40 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887057 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2DD43D6ACD7 for ; Wed, 27 Nov 2024 13:59:44 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYG-0003hV-6F; Wed, 27 Nov 2024 08:57:56 -0500 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 1tGIYD-0003gr-KZ for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:54 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYA-0007j4-Ga for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:53 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715869; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=KhFH7QeYVwBF3yejmS3ivhVwIE14VqwZtb3PUinQoHc=; b=JTzSi4kLgbHkaqlLpnv2VADioJLFY2vd3lpHoqC3PM8FvmTkNyMbyRXrUGs4ZiC9xZSUQs afCqpH3OF0TufLQ9apFcL/0vjy/hr2LQJqjpImsuLq/UpUWdqUELe7G80Q5XxyE7rrmohj TseL19/A1YW+7SmCx6hFDq244eIILFI= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-124-VHXUb51CMEiDthTLfGRQEw-1; Wed, 27 Nov 2024 08:57:47 -0500 X-MC-Unique: VHXUb51CMEiDthTLfGRQEw-1 X-Mimecast-MFC-AGG-ID: VHXUb51CMEiDthTLfGRQEw Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-3825a721afaso4020392f8f.0 for ; Wed, 27 Nov 2024 05:57:47 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715865; x=1733320665; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=KhFH7QeYVwBF3yejmS3ivhVwIE14VqwZtb3PUinQoHc=; b=w9XDx0gFtCE1Q85m+k2yxBFaGOlSLsrfsxVIITPnQmGo5Ksns/7t6grcsBzrwa8rmK GxNUjuqn+oRgruXIGJ02116NK7mPhpo6SUNvfO8q7Hccx37pjt68l5Hqs+9KRZfu3JBD 7hoMvI10s6/2b07ffksheVaVhNBCbITD7vhpOgaT4l0CsqlXT1Lyv67zBWkneBHYXGso rHnyhN97N3OFyKzYKIOfdMQSAdvFDDCas+Ax1ohitakZq9acHqt1DCvyEUcMkSjmbKdA PJKmyeTcmFJGgc4D2kK6YuvnXPFNO4LoDHhq/VaKCpYO/2+t0CphFuBGa1f8d1JIwxRG ZFGg== X-Gm-Message-State: AOJu0YwfRqgHjelKHmGRkbYj1udcinJey+fx1PkJcukLWqud1vuFd/cw 1pxOHAT1no2qEfuR7RYHRxdzJ9ZEYTouuijA4WQa+gqExepOuZrR0TCzpIDL6RMO8AVdIF3YsKa 66rYxQjLKqnGeaQ8wBROd4FHcVFgFog9Wlfz3OWrmTTjlUnH6pdyazGg4MNTfSODEgHXPiYY7AT yeBKMuEETrysaen2EQb2mecIpvaOJk/q2A X-Gm-Gg: ASbGncv/GTr1EqAfkqHV0opsMfjeqTtjgtbuWLkDdMUtOhnEfh/jXaU8RA5ZnAE7WDZ sZHbpq2bqWPCbWu8u6lfbuqODgvvAnN66gbOrK4UggAwcAj49zcYsVONOtgVenwgGF22bKK9Dhw epi3K3E1Slp/77FcySydRpz6h+Cp0aVrqY4eJiEAoEoCrhYTb7vAx3c2IulRefQnctrgPVqrxCP 8+/ZVGJD9FJG0VrcSBgwXPyWj0INMQUJFZ0PplkUon6 X-Received: by 2002:a5d:648a:0:b0:382:485b:363e with SMTP id ffacd0b85a97d-385c6cca1bbmr2458681f8f.6.1732715863701; Wed, 27 Nov 2024 05:57:43 -0800 (PST) X-Google-Smtp-Source: AGHT+IEARjgAQrUgrHJXP6IWBnB+Hwb+N7pbpBFn5gNfTPuf9Nr+XlResGR9V8g0eFEzXMB+LMMsOA== X-Received: by 2002:a5d:648a:0:b0:382:485b:363e with SMTP id ffacd0b85a97d-385c6cca1bbmr2458643f8f.6.1732715863138; Wed, 27 Nov 2024 05:57:43 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbedad6sm16330995f8f.91.2024.11.27.05.57.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:42 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:40 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Igor Mammedov , Ani Sinha Subject: [PULL 05/13] tests/acpi: update expected blobs Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.133.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.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, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Igor Mammedov Expected AML return to the state before bf1ecc8dad606 (w/acpi: Update ACPI `_STA` method with QOM vCPU ACPI Hotplug states) droping not needed CPRS and _STA logic that broke cpu hotplug @@ -2887,7 +2887,6 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001) CRMV, 1, CEJ0, 1, CEJF, 1, - CPRS, 1, Offset (0x05), CCMD, 8 } @@ -2922,16 +2921,9 @@ DefinitionBlock ("", "DSDT", 1, "BOCHS ", "BXPC ", 0x00000001) Acquire (\_SB.PCI0.PRES.CPLK, 0xFFFF) \_SB.PCI0.PRES.CSEL = Arg0 Local0 = Zero - If ((\_SB.PCI0.PRES.CPRS == One)) + If ((\_SB.PCI0.PRES.CPEN == One)) { - If ((\_SB.PCI0.PRES.CPEN == One)) - { - Local0 = 0x0F - } - Else - { - Local0 = 0x0D - } + Local0 = 0x0F } Signed-off-by: Igor Mammedov Message-Id: <20241112170258.2996640-5-imammedo@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test-allowed-diff.h | 41 ------------------ tests/data/acpi/x86/pc/DSDT | Bin 8560 -> 8526 bytes tests/data/acpi/x86/pc/DSDT.acpierst | Bin 8471 -> 8437 bytes tests/data/acpi/x86/pc/DSDT.acpihmat | Bin 9885 -> 9851 bytes tests/data/acpi/x86/pc/DSDT.bridge | Bin 15431 -> 15397 bytes tests/data/acpi/x86/pc/DSDT.cphp | Bin 9024 -> 8990 bytes tests/data/acpi/x86/pc/DSDT.dimmpxm | Bin 10214 -> 10180 bytes tests/data/acpi/x86/pc/DSDT.hpbridge | Bin 8511 -> 8477 bytes tests/data/acpi/x86/pc/DSDT.hpbrroot | Bin 5067 -> 5033 bytes tests/data/acpi/x86/pc/DSDT.ipmikcs | Bin 8632 -> 8598 bytes tests/data/acpi/x86/pc/DSDT.memhp | Bin 9919 -> 9885 bytes tests/data/acpi/x86/pc/DSDT.nohpet | Bin 8418 -> 8384 bytes tests/data/acpi/x86/pc/DSDT.numamem | Bin 8566 -> 8532 bytes tests/data/acpi/x86/pc/DSDT.roothp | Bin 12353 -> 12319 bytes tests/data/acpi/x86/q35/DSDT | Bin 8389 -> 8355 bytes tests/data/acpi/x86/q35/DSDT.acpierst | Bin 8406 -> 8372 bytes tests/data/acpi/x86/q35/DSDT.acpihmat | Bin 9714 -> 9680 bytes .../acpi/x86/q35/DSDT.acpihmat-noinitiator | Bin 8668 -> 8634 bytes tests/data/acpi/x86/q35/DSDT.applesmc | Bin 8435 -> 8401 bytes tests/data/acpi/x86/q35/DSDT.bridge | Bin 12002 -> 11968 bytes tests/data/acpi/x86/q35/DSDT.core-count | Bin 12947 -> 12913 bytes tests/data/acpi/x86/q35/DSDT.core-count2 | Bin 33804 -> 33770 bytes tests/data/acpi/x86/q35/DSDT.cphp | Bin 8853 -> 8819 bytes tests/data/acpi/x86/q35/DSDT.cxl | Bin 13180 -> 13146 bytes tests/data/acpi/x86/q35/DSDT.dimmpxm | Bin 10043 -> 10009 bytes tests/data/acpi/x86/q35/DSDT.ipmibt | Bin 8464 -> 8430 bytes tests/data/acpi/x86/q35/DSDT.ipmismbus | Bin 8477 -> 8443 bytes tests/data/acpi/x86/q35/DSDT.ivrs | Bin 8406 -> 8372 bytes tests/data/acpi/x86/q35/DSDT.memhp | Bin 9748 -> 9714 bytes tests/data/acpi/x86/q35/DSDT.mmio64 | Bin 9519 -> 9485 bytes tests/data/acpi/x86/q35/DSDT.multi-bridge | Bin 13242 -> 13208 bytes tests/data/acpi/x86/q35/DSDT.noacpihp | Bin 8269 -> 8235 bytes tests/data/acpi/x86/q35/DSDT.nohpet | Bin 8247 -> 8213 bytes tests/data/acpi/x86/q35/DSDT.numamem | Bin 8395 -> 8361 bytes tests/data/acpi/x86/q35/DSDT.pvpanic-isa | Bin 8490 -> 8456 bytes tests/data/acpi/x86/q35/DSDT.thread-count | Bin 12947 -> 12913 bytes tests/data/acpi/x86/q35/DSDT.thread-count2 | Bin 33804 -> 33770 bytes tests/data/acpi/x86/q35/DSDT.tis.tpm12 | Bin 8995 -> 8961 bytes tests/data/acpi/x86/q35/DSDT.tis.tpm2 | Bin 9021 -> 8987 bytes tests/data/acpi/x86/q35/DSDT.type4-count | Bin 18623 -> 18589 bytes tests/data/acpi/x86/q35/DSDT.viot | Bin 14646 -> 14612 bytes tests/data/acpi/x86/q35/DSDT.xapic | Bin 35752 -> 35718 bytes 42 files changed, 41 deletions(-) diff --git a/tests/data/acpi/x86/pc/DSDT b/tests/data/acpi/x86/pc/DSDT index f68a32e6068788c3fdfaff139206b725ba2bf1d1..8b8235fe79e2fa08a6f840c8479edb75f5a047b9 100644 GIT binary patch delta 76 zcmez1bk2#(CD@o=P99EwyZB(I_~<5&$!B=t7&Rtq^C~bgFi#HR)nM{9 go?OFg#^%B1930{}*^W<-i6LR~e_rX$3VZ>=07R=4p8x;= delta 125 zcmX@-^udYCCD*Nn-PWAb`lQ#13)G5RbZ4FS%c1_42?!C<8eq%ehD{TK_l_!Abg7I5)y J{>vLE3;@@vBVGUi diff --git a/tests/data/acpi/x86/pc/DSDT.acpierst b/tests/data/acpi/x86/pc/DSDT.acpierst index 0fd79699ebbfb650a5bbe49faecbebf2fc13a86c..06829b9c6c6d726d955dc7c99bc9f42448e22aeb 100644 GIT binary patch delta 76 zcmV-S0JHy>LiIrkL{mgm^&kKM0kW|QxeN(UK3`KpUt5AmlgtcW0VtCz4Ilyl1Cvq> iC<0A0lY9*}21o`&Q&d5dJ`OGd0AQ2<4I8r{4)+pssulqN delta 125 zcmezBINgcMCDP99E2$M|5U_~<6z$!B=t7 fC<075lY9+90!RjvzYR5$3l18Skq{NLKn`6P2n`eu delta 108 zcmezEGuM~PCD*Nn-PWAb`lQ#13)G5RbZ4FS%c1_42?!C<8eq%ehD{TK_l_!Abg7I5)y J{>!VO2LR(fBWeHu diff --git a/tests/data/acpi/x86/pc/DSDT.cphp b/tests/data/acpi/x86/pc/DSDT.cphp index 1079ff81c1ab05f0d5991835b3a0b014687924f4..045a52e75b7fcd4e5f840a758c548231498b96e4 100644 GIT binary patch delta 69 zcmV-L0J{IcMxI6rL{mgm9wPt%0nM=rxeN(UTVGQ`Ut5AmlgtcW0VtCz4Ilyl1Cvq> bC<0ATlY9+w0!Rjv@C`MSJPsPOTn^+E!a5Wv delta 108 zcmbQ|cEF9xCD43t0=ecoQZs5D?w$ I&v#w~08Tv}v;Y7A diff --git a/tests/data/acpi/x86/pc/DSDT.dimmpxm b/tests/data/acpi/x86/pc/DSDT.dimmpxm index 34fe3fcad9dc245eb173ca5a9655d17f15bd42ef..205219b99d903555125c4b07fc047c42993eb338 100644 GIT binary patch delta 96 zcmaFnf5e~5CDz&#S=1z&tsPSA)sT zb@Bw>NG1=q$**}$rKBdu=(B*-1~_{f1O&MTI|sP>F&1#~Crp+X5Z#=^S0xDmhw>Y! delta 103 zcmX@&|IDAuCD4FZB(gTYECOYupwE@Um>;@xb*H$f5r Dad91& diff --git a/tests/data/acpi/x86/pc/DSDT.hpbridge b/tests/data/acpi/x86/pc/DSDT.hpbridge index 33c7529f5c8a6ec651f979b3cebadb18774a781f..8fa8b519ec65bd5099c45f4e1c85b11b47a23845 100644 GIT binary patch delta 76 zcmdn*G}npCCD*Nn-PWAb`lQ#13)G5RbZ4FS%c1_42?!C<8eq%ehD{TK_l_!Abg7I5)y J{>!T%3;?JoBF+E+ diff --git a/tests/data/acpi/x86/pc/DSDT.hpbrroot b/tests/data/acpi/x86/pc/DSDT.hpbrroot index 2661170c833e23ae1985d31512a81a29afc858bd..01719462a72fd6d40ce433dac601e4b94eae574c 100644 GIT binary patch delta 98 zcmX@DzEYjbCD3`4>KsmU?=EFdib&YlJVL9W5h0j_?G1zh}_FY)|j0{}jp B8;bw{ delta 131 zcmZ3fep;Q&CD iC<0A0lY9*}21o`&Q&d5dJ`OGd0AQ2<4I8r{4ww^F-xWOo delta 125 zcmbQ{yu+ExCD*Nn-PWAb`lQ#13)G5RbZ4FS%c1_42?!C<8eq%ehD{TK_l_!Abg7I5)y J{>wX47yyb?BK!aV diff --git a/tests/data/acpi/x86/pc/DSDT.memhp b/tests/data/acpi/x86/pc/DSDT.memhp index 6ede4361f4b7bf18c496d7ca117310675126bbbf..03ff464ba4e72082fce0921815cfc09ca20b561a 100644 GIT binary patch delta 76 zcmV-S0JHzUO`S~&L{mgmohASP0b8*OxeN(KKVMTqUt5AmlgtcW0VtCz4Ilyl1Cvq> iC;~$?lY9*}21o`&Q&d5dJ`OGd0AQ2<4I8r{4xbo4h!ooZ delta 125 zcmbR1yWf|~CD_Are@}oWAs@-8Uma>4FZB(gTYD{NMQ=Q`Y{%8@h2=~E#Tta J{Firu1OUv#BbopJ diff --git a/tests/data/acpi/x86/pc/DSDT.nohpet b/tests/data/acpi/x86/pc/DSDT.nohpet index 9d6040733f57f243056ee8f7601af84dd3cf2fa2..b081030f0ed171e52b13e28cfdc8770a04c2806e 100644 GIT binary patch delta 76 zcmV-S0JHz%LBK%@L{mgmz#sqs0W+}*D+~!wK3`KpUt5AmlRgYy0VtE53?Kpk1CzuI iC<0A0lkN;Q21o`&Q&d5dunjH(0AQ1G4I8tP4Nwz7&=p4j delta 125 zcmX@$_{fpVCDP99Eo$M|5U_~<6z$!B=t7S?YsTctF?l_&shRoY7=0Fyh5%<(CDu1FqxnWIc1ccuyZB(I_~<5&$x@PWj2e?GB^8($m?tlg)L`;8 go_tEujLn11IXJ{|a*vcA6GOseUn%L$c~aqQ02K=qHUIzs delta 108 zcmZ4Nc+`>0CDAX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQOb!80A}|c$^ZZW diff --git a/tests/data/acpi/x86/q35/DSDT.acpierst b/tests/data/acpi/x86/q35/DSDT.acpierst index f91cbe55fcfeea319babf7c9a0c6a6ccdc3320d1..46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b 100644 GIT binary patch delta 76 zcmccSxW$pnCD2b8v{`AX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQ7V`X09c|Ot^fc4 diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat b/tests/data/acpi/x86/q35/DSDT.acpihmat index 0949fb9d67c70dc882e50501ece421114ad8080b..61c5bd52a42242e85090934e8e45bf01642609d6 100644 GIT binary patch delta 96 zcmez5eZiZ{CD*F7d%m@zG5llcgl%7&RtWN-8ihFi&0}slnu9 zIr)^NBa;W)WL_y#DXGaZ`Ya%|0nVNV0YR?8&H=7|j0If$36mXVL^tAX8JH%| zmegSKw4Qua(vj1Z!#Oy_kuzF(vZ9ReL{mgmx*-4n0ll#b2N?-QP+wC*Ut5AmlNuRb0VtDs86W}x1CyW` iC;~)9lgb%c21o`&Q&d5duNp1_0AQ1F8XL2b8cPRMBNdqd delta 125 zcmdnxe8-u~CDAX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQL2Cq0GY8KasU7T diff --git a/tests/data/acpi/x86/q35/DSDT.bridge b/tests/data/acpi/x86/q35/DSDT.bridge index 3464f552974672bde25eb15f1c93c309c57ef5cb..d9938dba8fa5d405f7696c0dbdc24f3ae42ec934 100644 GIT binary patch delta 76 zcmaD9dmxs}CDQ`zV&nCD4FZB(gTYEC8%jyDEo3d=;!T*mQATug IqEx&H0I`T3t^fc4 diff --git a/tests/data/acpi/x86/q35/DSDT.core-count b/tests/data/acpi/x86/q35/DSDT.core-count index 08f5d5f54bcb61235b98fc85bb814046dd038c13..a24b04cbdbf09383b933a42a2a15182545543a87 100644 GIT binary patch delta 71 zcmV-N0J#5?W$|PRL{mgmaWVh^0k5$N2N?>DTLE8FLSI{gNRt~GUI8eRdKn-B00Wbu d87KmZP_xS!vjPH029tmsHIvL78nf&irxpI;78U>i delta 110 zcmeyEGC7sYCD>Gcqtu zo+GKj)E>0?nB-%qo diff --git a/tests/data/acpi/x86/q35/DSDT.core-count2 b/tests/data/acpi/x86/q35/DSDT.core-count2 index d29a7108f82110ce9f9b4e006501215d41c5420a..3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8 100644 GIT binary patch delta 100 zcmeC_V0zWg#N`s~67s5llNU;A zFg0gvJ}nu~!{os>Sx3!ON@{Y9J_|^3fU~DTK#*&&bAYQKV*wX`!sJ8^(ames;$s1w CTO4Nq delta 126 zcmaFW&eYSv#N`s~62jBMz`*FVk&9iDtD}@HKG-Qfy2*F4v}7Ek?&K;-1$O6vpkPJ@ zrpa?8HJBQ*HXoCW=izkaa1IV}>&YlJVL9W4I aos*;0q}dj-7I5(zZ%vD0Bsc$ delta 108 zcmezDGS!vKCDQ!(?&G5RbZ4FS%c1_42?!CAX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQR=4z0JAtBNdN!< diff --git a/tests/data/acpi/x86/q35/DSDT.dimmpxm b/tests/data/acpi/x86/q35/DSDT.dimmpxm index 1db0bf454a203006f866e6752d06422ae675cbd3..228374b55bd544116e359f659e546fc66cf8a895 100644 GIT binary patch delta 91 zcmdn(H`9;HCDNE4v?QlTKzy)Md~}n?WIf3^Mvck6k_t==%#*iBYB0IE tPQE7@$>hN{*;2|>QfhLHJ_|@~fU~DTK#*&&bAYQKV*wZc<|R_oc>rS!7_9&R delta 132 zcmbQ~x7&})CDAX8JH%o zm(*Z#cAtDxGLp@e!#Oy_aq>bbJxhj!1?H1u^jSdq0-QY!0)kwF!8#U5VG6tYF&1#~ QCoE(w;NsofBsHA}0HD?-HUIzs diff --git a/tests/data/acpi/x86/q35/DSDT.ipmibt b/tests/data/acpi/x86/q35/DSDT.ipmibt index 25f43ae8efb55364a739e6b5e3cb4e71e61862b0..45f911ada5645f158f3d6c0c430ec1d52cadc5d8 100644 GIT binary patch delta 76 zcmbQ>^v;pXCD`WA;Wac1ccuyZB(I_~<5&$x@PWj2e?GB^8($m?tlg)L`;8 go_tEujLn11IXJ{|a*vcA6GOseUn%L$c~Xnm06Z5J7XSbN delta 108 zcmaFoIKhd_CDAX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQL2{>06xVXs{jB1 diff --git a/tests/data/acpi/x86/q35/DSDT.ipmismbus b/tests/data/acpi/x86/q35/DSDT.ipmismbus index 32bcd25bda9e9d2775790385f8da6a11e9d5cb46..e5d6811bee1233d74236453c49060390d74d4416 100644 GIT binary patch delta 76 zcmV-S0JHy{Li<4qL{mgm`yc=S0o<_)2N?-ZK3`KpUt5AmlNuRb0VtDs86W}x1CyW` iC<0A0lgb%321o`&Q&d5dj2bQi0AQ0%8XL228m|Uoloh-H delta 108 zcmezEIM<2GCD02)FaPyhe` diff --git a/tests/data/acpi/x86/q35/DSDT.ivrs b/tests/data/acpi/x86/q35/DSDT.ivrs index f91cbe55fcfeea319babf7c9a0c6a6ccdc3320d1..46fd25400b7c00ee9149ddb64cb5d5bd73f6a82b 100644 GIT binary patch delta 76 zcmccSxW$pnCD2b8v{`AX8JH%| zmegSKGM#)>(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQ7V`X09c|Ot^fc4 diff --git a/tests/data/acpi/x86/q35/DSDT.memhp b/tests/data/acpi/x86/q35/DSDT.memhp index be90eb71d8dda8fe54c79ffffe103986ee06ae3a..5ce081187a578ba7145a9ba20d30be36c13b7663 100644 GIT binary patch delta 76 zcmbQ@^U0gbCD50iyCkQ(V|=hvd~}oVWGTrwM%~Gkk_zn30YSlx3`~<} zOKLDVm`y$^X~yKrG5N2gshIiX7=0Fyh5%66_M9ugbu{sJ)SkU6Rw?F+SKSKDxTqweHNNd4FZB(gTYEC8%jyDEo3d=;!T*mQATug IqLjHL0EhJ+B>(^b diff --git a/tests/data/acpi/x86/q35/DSDT.noacpihp b/tests/data/acpi/x86/q35/DSDT.noacpihp index 45cc2bcffa42d73db110afd5075556dcfe5d9936..8bc16887e1c963c61aaecf71712a09c0554f6d67 100644 GIT binary patch delta 75 zcmX@>u-bvkCDNGPy72LuH(GB8c{ zlh9!DGM!u@VaDXjF?oZ8shRoY7=0Fyh5%nCDoVSr39y=V|=hvd~}oVAX8JH&X zN@_59nNBvBG-Gn*n4B+ZYGyt;MxO2b8v{`Bc1ccm$M|5U_~<6z$x@PWjJlI6B^B761A>AX8JH%| zmegQ!Fq?c-(u~QKWAa}~Q!(?&G5RbZ4FS%c1_42?!Cn7P1y_@g_{(C?mQ# IQObi207+UMdH?_b diff --git a/tests/data/acpi/x86/q35/DSDT.pvpanic-isa b/tests/data/acpi/x86/q35/DSDT.pvpanic-isa index 7a8e568315a43f1fa98068d8e78995c98064fb91..6ad42873e91c80cef5a42224cb4d31936dad59b4 100644 GIT binary patch delta 76 zcmZ4G)ZxVC66_Mfp~%3%_;Mo`yCkQ-U3{=pd~}n?WGTrwMvcjpk_t==%##;LYB2d4 gPd+7S#^%B1930{}xkpNmi6LRKuaxxWJgGfw010Fie*gdg delta 108 zcmeBhTIIy$66_M9rO3d*Xtj}xU6Rw$F+SKSKDxTqweHNNdUP0Qtck6951J diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count b/tests/data/acpi/x86/q35/DSDT.thread-count index 08f5d5f54bcb61235b98fc85bb814046dd038c13..a24b04cbdbf09383b933a42a2a15182545543a87 100644 GIT binary patch delta 71 zcmV-N0J#5?W$|PRL{mgmaWVh^0k5$N2N?>DTLE8FLSI{gNRt~GUI8eRdKn-B00Wbu d87KmZP_xS!vjPH029tmsHIvL78nf&irxpI;78U>i delta 110 zcmeyEGC7sYCD>Gcqtu zo+GKj)E>0?nB-%qo diff --git a/tests/data/acpi/x86/q35/DSDT.thread-count2 b/tests/data/acpi/x86/q35/DSDT.thread-count2 index d29a7108f82110ce9f9b4e006501215d41c5420a..3a0cb8c581c8cc630a2ec21712b7f8b75fcad1c8 100644 GIT binary patch delta 100 zcmeC_V0zWg#N`s~67s5llNU;A zFg0gvJ}nu~!{os>Sx3!ON@{Y9J_|^3fU~DTK#*&&bAYQKV*wX`!sJ8^(ames;$s1w CTO4Nq delta 126 zcmaFW&eYSv#N`s~62jBMz`*FVk&9iDtD}@HKG-Qfy2*F4v}7Ek?&K;-1$O6vpkPJ@ zrpa?8HJBQ*HXoCW=izkaa1IV}>&YlJVL9W4I aos*;0q}dj-7I5(TqsHV)Nd+bb=E(~rHJE&j gC!dluWAk8h4i0gg+#{vO#E>xAS4w(wp43(j0Oqn2?EnA( delta 108 zcmZp4TkOW=66_M9tjxf`xML$1yCkQhV|=hvd~}oVWGTrwM%~Gkk_zn30YSlx3`~<} zOKLEAnNB_`X~yKrG5N2gshIiX7=0Fyh5%0PFk|4*&oF delta 108 zcmbR3w%3izCD*k8bjqEG-$ws4=-pQh|wqdGbO@ z4W_oun@>xgW@Yzaa}Ex1 J&CCjWGytMf9?Ad! delta 135 zcmbO`k#YY-MlP3NmyrD)3=E928@bpexw;Q9#RogZM>qLSmX?fT)SX-} zgQ@@F=F^g=dD%VKoP$FgIVU%2h)=fC5R+j@SRgeyMxO1bj(qsVVzaPi| delta 135 zcmZph&a`4W6PHV{OUR0D1_nm`ja=-KTrKa};)9*yqnmsuOH0Nv>Q1hbRA6@w2nuFo zV46HfQiG}I@#bTar+L|2Ih=z-94Be$Su!LnFrOTw&jQjH;OuD-5ab#R*0DeeQ`psy Tv4D#|VIgY)7w_hW>PwRWojNEz diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index 512d40665d..dfb8523c8b 100644 --- a/tests/qtest/bios-tables-test-allowed-diff.h +++ b/tests/qtest/bios-tables-test-allowed-diff.h @@ -1,42 +1 @@ /* List of comma-separated changed AML files to ignore */ -"tests/data/acpi/x86/pc/DSDT", -"tests/data/acpi/x86/pc/DSDT.acpierst", -"tests/data/acpi/x86/pc/DSDT.acpihmat", -"tests/data/acpi/x86/pc/DSDT.bridge", -"tests/data/acpi/x86/pc/DSDT.cphp", -"tests/data/acpi/x86/pc/DSDT.dimmpxm", -"tests/data/acpi/x86/pc/DSDT.hpbridge", -"tests/data/acpi/x86/pc/DSDT.hpbrroot", -"tests/data/acpi/x86/pc/DSDT.ipmikcs", -"tests/data/acpi/x86/pc/DSDT.memhp", -"tests/data/acpi/x86/pc/DSDT.nohpet", -"tests/data/acpi/x86/pc/DSDT.numamem", -"tests/data/acpi/x86/pc/DSDT.roothp", -"tests/data/acpi/x86/q35/DSDT", -"tests/data/acpi/x86/q35/DSDT.acpierst", -"tests/data/acpi/x86/q35/DSDT.acpihmat", -"tests/data/acpi/x86/q35/DSDT.acpihmat-noinitiator", -"tests/data/acpi/x86/q35/DSDT.applesmc", -"tests/data/acpi/x86/q35/DSDT.bridge", -"tests/data/acpi/x86/q35/DSDT.core-count", -"tests/data/acpi/x86/q35/DSDT.core-count2", -"tests/data/acpi/x86/q35/DSDT.cphp", -"tests/data/acpi/x86/q35/DSDT.cxl", -"tests/data/acpi/x86/q35/DSDT.dimmpxm", -"tests/data/acpi/x86/q35/DSDT.ipmibt", -"tests/data/acpi/x86/q35/DSDT.ipmismbus", -"tests/data/acpi/x86/q35/DSDT.ivrs", -"tests/data/acpi/x86/q35/DSDT.memhp", -"tests/data/acpi/x86/q35/DSDT.mmio64", -"tests/data/acpi/x86/q35/DSDT.multi-bridge", -"tests/data/acpi/x86/q35/DSDT.noacpihp", -"tests/data/acpi/x86/q35/DSDT.nohpet", -"tests/data/acpi/x86/q35/DSDT.numamem", -"tests/data/acpi/x86/q35/DSDT.pvpanic-isa", -"tests/data/acpi/x86/q35/DSDT.thread-count", -"tests/data/acpi/x86/q35/DSDT.thread-count2", -"tests/data/acpi/x86/q35/DSDT.tis.tpm12", -"tests/data/acpi/x86/q35/DSDT.tis.tpm2", -"tests/data/acpi/x86/q35/DSDT.type4-count", -"tests/data/acpi/x86/q35/DSDT.viot", -"tests/data/acpi/x86/q35/DSDT.xapic", From patchwork Wed Nov 27 13:57:43 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887052 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6DD11D6ACD7 for ; Wed, 27 Nov 2024 13:58:15 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYF-0003hR-Ak; Wed, 27 Nov 2024 08:57:55 -0500 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 1tGIYC-0003gi-6j for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:52 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYA-0007jA-Sm for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:51 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715870; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=Xa+LddK6R1hQSg+kwz9wRZYAduyZ+dc2oJ7aKe9ONYs=; b=ahpj6Iwil/+lV644u44qy3T+abI+tsO1KPWeMzhQoMdb+35C0XhBvdw+12xlHLa4YV5NjE 07Fm7qwHRkDPdvsDmLHce8VGQykSAnEHxZzSZAYMnGeXe9rYYsgIHv5QkhDxjE2QhYgkvo R00li03phxV5h+dSalrH3PxgzzlqtG0= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-377-ToDpg7qSMLayc-eq0MK1nA-1; Wed, 27 Nov 2024 08:57:47 -0500 X-MC-Unique: ToDpg7qSMLayc-eq0MK1nA-1 X-Mimecast-MFC-AGG-ID: ToDpg7qSMLayc-eq0MK1nA Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-382450d158fso3295855f8f.1 for ; Wed, 27 Nov 2024 05:57:47 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715866; x=1733320666; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=Xa+LddK6R1hQSg+kwz9wRZYAduyZ+dc2oJ7aKe9ONYs=; b=eu7hMT6tyfsjb7U97WuJTqQRBU9QSlW+MiShlNQsxClWJQTrX0ybLYNw3Zity2RK4K wMtM93VybirDL01SIzoyr+2ASzsNsbT4RTfD7Al6/0C5/aN1T+YWq1+a27n0yd57U2dX 2nbpYL8spAHd+7CUUMR97T18f6pW1onheghDj3F3bvX0/vNzc+QUFoTAvVKlZ50n2sJF R9THkGgdrdO+EsVTpSvJ//UgoQq/fHIowCzDujYz2qZuIyr5DN5nP//LV2OXybOhXbsO rYKoXhrO0SMHNePQxhRmspO37ejY910XXTrb2ALxHj5fOPYppCQXz1Rzst9834Lj58LF n3ag== X-Gm-Message-State: AOJu0YwuEvkxAqNl5WoBYXMddRdIkBl9exQKaltl12rMLhpEk+LsInsv 5RR41BzdlPPXskJCy9aL2DIjyFRZu3HbepFGgKGRu8rLIErGNoUjimusg//pSaadZIjXZl87zD1 3WpRJ2dOP9DM3wIojkoSIDGEV/wR5Nno+9LE6QO+OtejoTZT7e6+raH8doJh/MG9xLqnPOjBKQS NuoaiG3ZilJoZ39jZmLaTQn5MNN1zjP1Vh X-Gm-Gg: ASbGncu7VdWutwAQNHGf8DczTP0459yHJYAL5DyyZ1HhJerle2y8yR1JuYomg0Pv0FD xlSejo6Fy0jmcyOb5maMBK3nAnhELH7ih6zf/elHw+jzY+0pctDm479tB6+WQrPgFOIlh5e0yYU bxvxTiJKrvccBC3KTUAS1OnD+Q+lkauIqGX2BRMMkQVFtQDUEQbt+/tzDdr9kgdEIcXT+uSksPE 9FnHSM2eWrJYa/HfA70Imo01+3f7Q+a0+0K3YoAhZ+L X-Received: by 2002:a05:6000:1866:b0:382:4a98:da57 with SMTP id ffacd0b85a97d-385c6ed75afmr2839345f8f.47.1732715866266; Wed, 27 Nov 2024 05:57:46 -0800 (PST) X-Google-Smtp-Source: AGHT+IG2h/4bv3bZBmCd0Px+rc0BS1nvvkRJchOVpoYq9Dwa5vQQlIpT5VHSx4BSvHgIT7zP0j8z8w== X-Received: by 2002:a05:6000:1866:b0:382:4a98:da57 with SMTP id ffacd0b85a97d-385c6ed75afmr2839325f8f.47.1732715865825; Wed, 27 Nov 2024 05:57:45 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-385c91f66c6sm944377f8f.72.2024.11.27.05.57.44 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:45 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:43 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Fan Ni Subject: [PULL 06/13] hw/cxl: Check for zero length features in cmd_features_set_feature() Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron Zero length data for features doesn't make any sense so exclude that case early. This fixes the undefined behavior reported by coverity for a zero length memcpy(). Resolves CID 1564900 and 1564901 Reported-by: Peter Maydell Signed-off-by: Jonathan Cameron Message-Id: <20241108175814.1248278-1-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- hw/cxl/cxl-mailbox-utils.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/hw/cxl/cxl-mailbox-utils.c b/hw/cxl/cxl-mailbox-utils.c index 2d4d62c454..ce9aa18364 100644 --- a/hw/cxl/cxl-mailbox-utils.c +++ b/hw/cxl/cxl-mailbox-utils.c @@ -1288,6 +1288,10 @@ static CXLRetCode cmd_features_set_feature(const struct cxl_cmd *cmd, set_feat_info->data_offset = hdr->offset; bytes_to_copy = len_in - sizeof(CXLSetFeatureInHeader); + if (bytes_to_copy == 0) { + return CXL_MBOX_INVALID_PAYLOAD_LENGTH; + } + if (qemu_uuid_is_equal(&hdr->uuid, &patrol_scrub_uuid)) { if (hdr->version != CXL_MEMDEV_PS_SET_FEATURE_VERSION) { return CXL_MBOX_UNSUPPORTED; From patchwork Wed Nov 27 13:57:46 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887053 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6B4DDD6ACD6 for ; Wed, 27 Nov 2024 13:58:15 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYF-0003hU-PH; Wed, 27 Nov 2024 08:57:55 -0500 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 1tGIYE-0003h4-IX for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:54 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYC-0007jM-Iw for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:57:54 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715872; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=GMcl0MSS++Dl9hhYbEW3wwttoqPTslrQ++mjFH0r/Vg=; b=Ja0P3bpC1c61WflXKQAmNmke2zZ4a1OAD65pHjcC01ai0gH8DYIUnAPfwto8sYtDWeJm5K n/neGASjX9GAFz4HHSOwcCcoGPmH/vvTFucbqrhlQ1bWRmQn30KzHt7msRL8VCV33M0q0M LIqDOnjrxp4CkOrXi1QCkvBCdmVuX34= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-279-pRluDRE9NnO35IuYxP2itA-1; Wed, 27 Nov 2024 08:57:51 -0500 X-MC-Unique: pRluDRE9NnO35IuYxP2itA-1 X-Mimecast-MFC-AGG-ID: pRluDRE9NnO35IuYxP2itA Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-38242a78f3eso3542006f8f.0 for ; Wed, 27 Nov 2024 05:57:50 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715869; x=1733320669; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=GMcl0MSS++Dl9hhYbEW3wwttoqPTslrQ++mjFH0r/Vg=; b=Yr1sYPinP9GLrI1nvZ3FIeOkb70H9/RY2USS2TSxdyX1Gku6/ZR1aWLzdFXyy9xvuP MasicR+VhcpqEL/6x8GSRdOtskpfvWpm/iIG0iPHs0nuxfeqV94A6VLBeTNZyQ0q7MNR tGmysOqknEUMwrZAUmmv6ahDsU/1VFsCpHtvWnSCANfM+bWK7GgzuTAzHCqMuIWjnX4n gkq1JEnGclycLEYvkWx7/pASzk/OZapqxqZvUcO+tow6SWxj9l3rb+ylW97EMxeVS5Z1 WhQNydp6gbSxthjp+KfLbw+UkhumWXPV299gXNwM4K5ulA/xYsYYdSthBHU3vdnI4S55 MyNQ== X-Gm-Message-State: AOJu0Ywwc3w280+1Pi4sx4FzUiuuy9/GOXNXQlSL5Fi9EpIkbKNTcaAF AEGtTKeahZUDfh79v4t42yXFgRiD2WV+TeGO7m5T/CrTaOBj+7xpYs1LMuao756K6CFjEnmN64j jfDRAMb/SEZ1tKPaaNnIB8GLCqBfrG8U8hoR+AhMMvaAEtOJxXJHHx09qRRL68jI9g1A1xiFe9Q pNToYvuiFqlOB0TF2m4ob+CdZXnXQpfsBP X-Gm-Gg: ASbGncswr9av94z0G1LugrPmOp9Rhu2uzweDuXrnHsERY+xs3YRz2AQIWGSUo95quzD 1IJxyc+Y5cA3+hMkWgywHfULrz4QKhqWc1ij15PORICliL8RW8ytV8iWATHzdGIZUiRsM0yJgHe ht88rvraMI2F9Q/WoUe8RJey41O5QWvr1vIFfR7OyZbLgn/8dkqLukMCRjDAJsPSbZire4eROSO PG+FqpBEsczxVoK7Sw6/WJPX08VuSXxycdqJBllIHD5 X-Received: by 2002:a05:6000:1844:b0:37d:5352:c83f with SMTP id ffacd0b85a97d-385c6ebbab6mr2837470f8f.17.1732715869334; Wed, 27 Nov 2024 05:57:49 -0800 (PST) X-Google-Smtp-Source: AGHT+IF8zVZTYASQLLdwmEL8OSpzrpitTSPR32eerp6fYb0e3OyPz92snGztFChROzwkCqFK2gGwOw== X-Received: by 2002:a05:6000:1844:b0:37d:5352:c83f with SMTP id ffacd0b85a97d-385c6ebbab6mr2837441f8f.17.1732715869002; Wed, 27 Nov 2024 05:57:49 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbc3a3asm16319437f8f.66.2024.11.27.05.57.47 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:48 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:46 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Vladimir Sementsov-Ogievskiy , Paolo Bonzini , Daniel =?utf-8?b?UC4gQmVycmFuZ8Op?= , Eduardo Habkost , Eric Blake , Markus Armbruster Subject: [PULL 07/13] qapi: fix device-sync-config since-version Message-ID: References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Vladimir Sementsov-Ogievskiy Actually it comes in 9.2, not 9.1. Fixes: 3f98408e2e ("qapi: introduce device-sync-config") Signed-off-by: Vladimir Sementsov-Ogievskiy Message-Id: <20241108071957.727286-1-vsementsov@yandex-team.ru> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- qapi/qdev.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qapi/qdev.json b/qapi/qdev.json index 2a581129c9..25cbcf977b 100644 --- a/qapi/qdev.json +++ b/qapi/qdev.json @@ -182,7 +182,7 @@ # # @unstable: The command is experimental. # -# Since: 9.1 +# Since: 9.2 ## { 'command': 'device-sync-config', 'features': [ 'unstable' ], From patchwork Wed Nov 27 13:57:49 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887062 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BAC94D6ACD9 for ; Wed, 27 Nov 2024 14:00:16 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIZW-0006w1-FK; Wed, 27 Nov 2024 08:59:14 -0500 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 1tGIZT-0006vO-Ct for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:59:11 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIZP-00084u-WB for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:59:10 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715947; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=7mZYQc/0Beq9quMGIXTqw5aSICjDsmWEiV1q8R4CPFA=; b=FIxAq9OgEItPDA9C+znYRYVXNPGPY8VRnal19lMoGlaBXERzHbnIaOMVWmC0ZvsJ7uEfsx RDhxDle9gRnolDjoquayA8YduWAuRPXkLmV0jYGvTUE6rp5F20c7ZruIxIWwGJmaTrNkXR 3F1bOy8NhOGDOsXGspU0PqoAqjryL/s= Received: from mail-wr1-f69.google.com (mail-wr1-f69.google.com [209.85.221.69]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-241-piJIlEVBMraM6AiKnoUdlw-1; Wed, 27 Nov 2024 08:57:54 -0500 X-MC-Unique: piJIlEVBMraM6AiKnoUdlw-1 X-Mimecast-MFC-AGG-ID: piJIlEVBMraM6AiKnoUdlw Received: by mail-wr1-f69.google.com with SMTP id ffacd0b85a97d-3824395a677so3621017f8f.0 for ; Wed, 27 Nov 2024 05:57:53 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715872; x=1733320672; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=7mZYQc/0Beq9quMGIXTqw5aSICjDsmWEiV1q8R4CPFA=; b=IrHOn5Z0R81K0U1jM2HXSINzqtAxIvnSkMOo8dXXZn5oUJMncXk39LskJxyNelrv8U PEVMhUCj+EelDpLLHCEzdxY5nY+CJqjPhDv2BmYcQmCC5WuUjWBI1c4meYRgu8v5E3uI 0z9wde/TRlV3wXJvEvMW6Wv6DArekR3Bc/MpYTaBy/cg0qTy9hBaWjwMvH9a/6rqEj/V P6LZ1vA6iYgnTNZB1Zid7Y8K+gg0D8/6aOKqVhrzD5l2vZbvJYHjz3K4lNB3xohztG0U 30m2UkjYANtvx+iDqLWQUCeVfZlTe3N/KgVenb1pGR0A5Bkryg2XlKq1rztocTAb7mas ec5w== X-Gm-Message-State: AOJu0Yz4o78nfMF8umUi4Gr666lu18c5AIH851K3SR9JP/ZX5ZTdYkBh lG5qQj5K6FrYEA2G20S52kMSZFCO0SVxZbhJdUUije9NcTygLJpYCyKjPlEw+l/UZuTVCkRqi96 sJ3bqpOwIax6GVxGRpnSLMEtZ9Pi68iIkJfgQoud6nYclavqGG+LcHioeGiHRj29l0sRaxVNoBP Mmmibl0bvSj68k9wJqMq2W/s52Tgw4fkWg X-Gm-Gg: ASbGncuBC7mrvGFxEoSjvYAb9CPNTRWSLCs4Eqa0ZNEuIiHG01KPAgnvoGgf2ZaiDkt ZVh0JVghUng4wOHozdLuDl7NY9AsIBf9FkfNwkBtlEYl/TEBEwuMNbO/1VzdD7wWFHgQiO6o71/ dRTdojv26e/I5iEY8Z0yDqBkbc1QHmPHQya5flliNJ3k7OLoQzMxOGRpx8SaGpugVi4pn/ZB7kq jzUGNtUYd+CciO0hpGcSBlW9igXpksyXRl3/5F1tXog X-Received: by 2002:a05:6000:1545:b0:382:4804:27e4 with SMTP id ffacd0b85a97d-385c6ec0fa6mr2321483f8f.35.1732715872183; Wed, 27 Nov 2024 05:57:52 -0800 (PST) X-Google-Smtp-Source: AGHT+IHi3DXzRF9BpFMwyjTFSlr1FNlAAUH95J4n+5FAsT/XzxwAzNKKlX+y6JVEu6N85cEIwgNYsA== X-Received: by 2002:a05:6000:1545:b0:382:4804:27e4 with SMTP id ffacd0b85a97d-385c6ec0fa6mr2321453f8f.35.1732715871787; Wed, 27 Nov 2024 05:57:51 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fad627bsm16397555f8f.12.2024.11.27.05.57.50 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:51 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:49 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Daniel P =?utf-8?b?LiBCZXJy?= =?utf-8?b?YW5nw6k=?= , Igor Mammedov , Ani Sinha Subject: [PULL 08/13] hw/acpi: Fix size of HID in build_append_srat_acpi_device_handle() Message-ID: <13a4a6ba2bfd3cf9bd34a015929d93868d792de8.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron The size should always be 8 so hard code that. By coincidience the incorrect use of sizeof(char *) is 8 on 64 bit hosts, but was caught by CI testing with i686 as the host. Reported-by: Michael S. Tsirkin Closes: https://lore.kernel.org/qemu-devel/20241104110025-mutt-send-email-mst@kernel.org/ Signed-off-by: Jonathan Cameron Message-Id: <20241107123446.902801-2-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Daniel P. Berrangé --- hw/acpi/aml-build.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hw/acpi/aml-build.c b/hw/acpi/aml-build.c index 6a76626177..72282b173e 100644 --- a/hw/acpi/aml-build.c +++ b/hw/acpi/aml-build.c @@ -1960,7 +1960,7 @@ static void build_append_srat_acpi_device_handle(GArray *table_data, { assert(strlen(hid) == 8); /* Device Handle - ACPI */ - for (int i = 0; i < sizeof(hid); i++) { + for (int i = 0; i < 8; i++) { build_append_int_noprefix(table_data, hid[i], 1); } build_append_int_noprefix(table_data, uid, 4); From patchwork Wed Nov 27 13:57:51 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887061 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DE960D6ACD2 for ; Wed, 27 Nov 2024 14:00:10 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYO-0003j2-I7; Wed, 27 Nov 2024 08:58:04 -0500 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 1tGIYN-0003iZ-G7 for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:03 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYL-0007kL-7z for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:03 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715880; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Xc83yCJTjWUOlwbPIaTOBSqDgZomNzG9qaGscIC6VEA=; b=MxN4Kq6Yzltv0VlhgAZEvdMY+w0oLJ6bTCDAxYuDQu5sFoOh5ce2SlUQiG6wIG61jPlAvi ENvkBlpH4gD7oGMnvOiLMkngeI7MfdBV1ZnU8HpYqSWoo06mQ+8fNFg19QhtmLpePdSD/A yhp6/TdyaxhB7H6lzvVqWM7xKefVsp8= Received: from mail-wr1-f71.google.com (mail-wr1-f71.google.com [209.85.221.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-82-MRM3YBsRP2qndmx7lr4qCw-1; Wed, 27 Nov 2024 08:57:56 -0500 X-MC-Unique: MRM3YBsRP2qndmx7lr4qCw-1 X-Mimecast-MFC-AGG-ID: MRM3YBsRP2qndmx7lr4qCw Received: by mail-wr1-f71.google.com with SMTP id ffacd0b85a97d-38256bf4828so481746f8f.1 for ; Wed, 27 Nov 2024 05:57:56 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715874; x=1733320674; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=Xc83yCJTjWUOlwbPIaTOBSqDgZomNzG9qaGscIC6VEA=; b=VSIM0NYyCOtWgPJNYWma8+tIsJ5+Fa24YISWnrAbRlHmsgOijoX28crTksrxOM9h5Y RHFj8CEmmPGzP7WB+QBjfgc3AF17ApAlbY6prIGQGCiuBSeZSLI9w8khFCHncKwkWQMR dvP5RpT2lvPjKoIYc5g5OwD6/YdLX0pec0OF+1sMk69p7BjMYpoBgHboYd5Hkn3TMQv/ aIrzLiWDRLkJTWTQ91m79Bq/DewrAAuuyb32TUOGki3JlPi8T3nKYdgEzWFOax9B8Oma 3ed2TycBG9dluaV/OIhuC6fYXq8vMGvOlnRAT3S9sc7rrC3iVtqPfE/qap8+RxW0A1Dl jwjg== X-Gm-Message-State: AOJu0Yw/03BiqEtIN/C11G397pRK4cOw/s/Lnbr0IEV5Osx8yW29W823 R8eOuU6I+TydXkAuuatl47ZvUis+Au8RMjJI2n97f9cJDYurSpd0K1MPqdm2pqhiCJ81dwZFHLC bA1o/LO5qkslsRLkbYjKAp9/Pe8kfrph723H/Lx/nSz1B+TvmxR8gKYIC3x6c0ZHSxsEGjjYfrV Jq9cKF24V6jwkOiFeYfR5uJSSP6SOupwdZ X-Gm-Gg: ASbGncuI0DqVXXMKdqFLb4LJ6EWebgQPShwGOOrndGhEhagqG2nJo0ETv6StJv9opET g/Jb5qeCSu2+0WbiSuk/B+rNknCsRDWs5LptIPKHRTxLS9gPjExc8lcZ2IJvSOCD8QBfwwcbAWj dBfS5MHxwIemRr9WChO8fMeCgAVvuGlT04/1dC3orczs80Evxxi4Ah4O36p21v2RUkcZlUFiycz kosZ1cAz14fMYhCdXF1LYXe9v9TmylsYcnCnLIEqT+1 X-Received: by 2002:a05:6000:1ac8:b0:37d:3b31:7a9d with SMTP id ffacd0b85a97d-385bfb14bbemr7145654f8f.23.1732715874746; Wed, 27 Nov 2024 05:57:54 -0800 (PST) X-Google-Smtp-Source: AGHT+IHP2/hT+XDbnBycDKK19hbZRDr0/bQzKTJLnwPEO3Nm/VEPZW6jj1sUYvvtpxeTTQrmWUh37g== X-Received: by 2002:a05:6000:1ac8:b0:37d:3b31:7a9d with SMTP id ffacd0b85a97d-385bfb14bbemr7145618f8f.23.1732715874407; Wed, 27 Nov 2024 05:57:54 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbc40a1sm16382593f8f.69.2024.11.27.05.57.52 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:53 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:51 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Daniel P =?utf-8?b?LiBCZXJy?= =?utf-8?b?YW5nw6k=?= , Paolo Bonzini , Eduardo Habkost , Eric Blake , Markus Armbruster Subject: [PULL 09/13] qapi/qom: Change Since entry for AcpiGenericPortProperties to 9.2 Message-ID: <37a14f24b4ee696def42d7629598ab6da7399f65.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.133.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.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, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron This feature was only applied during the 9.2 cycle, so reflect that rather than 9.1. Reported-by: Daniel P. Berrangé Closes: https://lore.kernel.org/qemu-devel/ZyngEiwmYeZ-DvCy@redhat.com/ Signed-off-by: Jonathan Cameron Message-Id: <20241107123446.902801-3-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Daniel P. Berrangé --- qapi/qom.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qapi/qom.json b/qapi/qom.json index a8beeabf1f..28ce24cd8d 100644 --- a/qapi/qom.json +++ b/qapi/qom.json @@ -877,7 +877,7 @@ # complex as there may be interleaving across multiple devices # and shared links in the path. # -# Since: 9.1 +# Since: 9.2 ## { 'struct': 'AcpiGenericPortProperties', 'data': { 'pci-bus': 'str', From patchwork Wed Nov 27 13:57:54 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887060 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id B5B43D6ACD9 for ; Wed, 27 Nov 2024 14:00:02 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYO-0003it-1e; Wed, 27 Nov 2024 08:58:04 -0500 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 1tGIYM-0003iL-6m for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:02 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYK-0007kF-O9 for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:01 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715880; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=zSBqlGSMGO71ouqzAy5nufmGT2K8poJn0RVdXU6iYP0=; b=bzFbGOihZLNFdu6NOJ93Q8VEoQa7aShb9F9z2DKXs88kmHrcUpw1yDMVu0jf1dW7m07frW oT9fPmsMwr2qNEmG6CeEGXVyyCPijbGh03GPOLN1IIvTYvWAkl+Sf2pDfJ6F2zI2OCXO8t BY28s+Du1Vypdq5vyFUHTF0Adi3fj7U= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-665--JdigCxeNEqeDF_Ue7Oyzg-1; Wed, 27 Nov 2024 08:57:58 -0500 X-MC-Unique: -JdigCxeNEqeDF_Ue7Oyzg-1 X-Mimecast-MFC-AGG-ID: -JdigCxeNEqeDF_Ue7Oyzg Received: by mail-wr1-f70.google.com with SMTP id ffacd0b85a97d-38243a4ba7cso3612087f8f.1 for ; Wed, 27 Nov 2024 05:57:58 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715877; x=1733320677; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=zSBqlGSMGO71ouqzAy5nufmGT2K8poJn0RVdXU6iYP0=; b=jb92QiWFa/dPNJZCzHF6f6URrjmj07KpRtavcqJ+rsWNh9KUN4c7Yh+aAtwz211yio bZM7PkJOdHuX0auDRpBQkI6r38tGU6iPaq5sUx6w23MqzsWRRCseN4qxRXzJuiHyt7+R QRn+uh+2xQOf8SmoJ7ZsAXbJQpJeSqXMpbfL0RIQa8dO7+gPy8Beb6KXjwX2sfNlHFyf 0D43dBsM+jlS+c7pB02508aIL1AxComdeClQo9UN/hX4GSPYrDaMON22fzm9ccdSr2av ESsVD2lalghBcevB3hEKlPqKGJtz4ZJPCJzb1V7mvpRnhPWa3R01HFEvuJvfeP9yfPcs uSAQ== X-Gm-Message-State: AOJu0YwySFH229Nf50cmgyZeeq4ys3WQok6w/7SupsRaNY1uv0HoZ8cN 1jWo4pH54VyAphiM3YPqkaA84TnWP1bKJf/6ZFocxjzEzVw1O6HvqVWcT/JXRd8pMah1Nh0CtUp Qd1Ey3Qgp+LELsqtLWQhYWErrpmCJnjYRRHQ607vZ2kOOP4zeYwQ5a+cI/8al2dizg1hE8Ac/zH vQTROe/qJySuIDS0+n/rGnZ9DMirVNOlGR X-Gm-Gg: ASbGncsSEQsGLWDydcdkIb4YbrW2Iocaib9ZfQ3IF+aXUfjXxXb4ymkRprTqbWIuTpj AcLHzd48fYpeN7ei5WHU6x8/1gtJ4Y/pmiu6XCE5pFPxnegak+wgYfeDngOQc/Wh1i6J2qzIet+ RkL/+hPxTaYgqsPRwAqUVFgx17/aftEMSUq1cvT4fw/4uQE/0bqTjXNC/vgYcH2+41yU+wgIhCr owxSuJ2rbiajC+Rvi4f0BLqNUMgjxPXryLcvjPnBOoc X-Received: by 2002:a5d:6f06:0:b0:382:383e:850e with SMTP id ffacd0b85a97d-385c6ebcb69mr2665392f8f.13.1732715877323; Wed, 27 Nov 2024 05:57:57 -0800 (PST) X-Google-Smtp-Source: AGHT+IFg/2F7t+g1Gf9CMhMX1i0I+XZTOpGkQP/Q+oLvE+8IB528S/TwUgNV3COpO8AxqVw/qU+CVA== X-Received: by 2002:a5d:6f06:0:b0:382:383e:850e with SMTP id ffacd0b85a97d-385c6ebcb69mr2665371f8f.13.1732715876933; Wed, 27 Nov 2024 05:57:56 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbedce3sm16353473f8f.97.2024.11.27.05.57.55 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:56 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:54 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Igor Mammedov , Ani Sinha Subject: [PULL 10/13] bios-tables-test: Allow for new acpihmat-generic-x test data. Message-ID: <2cb740932c567b57f6f51c9ead036a1d31d3e620.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron The test to be added exercises many corner cases of the SRAT and HMAT table generation. Signed-off-by: Jonathan Cameron Message-Id: <20241107123446.902801-4-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test-allowed-diff.h | 5 +++++ tests/data/acpi/x86/q35/APIC.acpihmat-generic-x | 0 tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x | 0 tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x | 0 tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x | 0 tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x | 0 6 files changed, 5 insertions(+) create mode 100644 tests/data/acpi/x86/q35/APIC.acpihmat-generic-x create mode 100644 tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x create mode 100644 tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x create mode 100644 tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x create mode 100644 tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index dfb8523c8b..3c0967078f 100644 --- a/tests/qtest/bios-tables-test-allowed-diff.h +++ b/tests/qtest/bios-tables-test-allowed-diff.h @@ -1 +1,6 @@ /* List of comma-separated changed AML files to ignore */ +"tests/data/acpi/x86/q35/APIC.acpihmat-generic-x", +"tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x", +"tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x", +"tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x", +"tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x", diff --git a/tests/data/acpi/x86/q35/APIC.acpihmat-generic-x b/tests/data/acpi/x86/q35/APIC.acpihmat-generic-x new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x b/tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x b/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x b/tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x new file mode 100644 index 0000000000..e69de29bb2 diff --git a/tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x b/tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x new file mode 100644 index 0000000000..e69de29bb2 From patchwork Wed Nov 27 13:57:57 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887058 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 99BF5D6ACD2 for ; Wed, 27 Nov 2024 13:59:45 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYQ-0003jT-Nk; Wed, 27 Nov 2024 08:58:06 -0500 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 1tGIYQ-0003jK-0G for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:06 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYO-0007l2-Al for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:05 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715883; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=PVkuzJ3PZWHU81/L5qS9M1rthMb+FEyxqRXv2Sgjhfs=; b=Urax264/p8e6G9YaXB82si6eLaJMoKQTEK9gCP7foJR770mNk6oLIil6K9w2/WA/kJfy1z SPltw8pdojtpnPp3jBpDLbaH0mjM3XqcFd+JeB8cdSeb2pVK32zqULRD2ecsWGTxuW/kZY SkzYT2+invNqkmOzXgFHXTQE9VvS2Bg= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-694-7BZsZw3zP4izATAdwo09Vg-1; Wed, 27 Nov 2024 08:58:01 -0500 X-MC-Unique: 7BZsZw3zP4izATAdwo09Vg-1 X-Mimecast-MFC-AGG-ID: 7BZsZw3zP4izATAdwo09Vg Received: by mail-wm1-f71.google.com with SMTP id 5b1f17b1804b1-434a90febb8so7993075e9.1 for ; Wed, 27 Nov 2024 05:58:01 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715880; x=1733320680; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=PVkuzJ3PZWHU81/L5qS9M1rthMb+FEyxqRXv2Sgjhfs=; b=VbzUBGeR67SX1hhYoIKP7unls3+SA08Xinsw+QqEJg0YOsnpT1zsqWSEvRoBxlNaDv /I8juKlN8uvOKwhvZyIICHWvSg51UWSBgWPyqJuw5cG+2DjHKiD2/bmN1FHQRCb4yZyH +b9B7Zz6l9ALMGra7En6gFF8GAz8/eCk9NFKGgO1N1Or7lFvdl635UOQpoablsLTcAhV ad54Jz5njE0xPS5MAPYlbTFdYZMYBFs/L+9qd86PpuRqZMK2lXmjawxbPsrJ9iNAV2mE numoF3OkedLrH9sLFiohd3qwmMIJZBAz8vH6P0SXngRSJbw8EwlvF2d6GOCy07DsxOv1 7PdA== X-Gm-Message-State: AOJu0Yx+t2lRPU9CQwB7z7w0j10Tgy4N4U3Ghq7S6bpaWnz4oVvG0RSw FtY3Pia9xWj3+Vlzk6/szNfo6WpaNnrCDrNmojXEJLSroO+OVdm8dZJunB1QQREcMoky/VjrgST K7+lGqJAbK932LTFwb3NBEWSJ+0wsiuK77+V5+RwlEkdz6e1zjrphyWaTyLvrn6ACAV3GB8tAue qHUs/uE/btzYptsFazO8RcxNi+i4Iy/Mk0 X-Gm-Gg: ASbGnctidP5d7Ofr6YfZR1lZIh+sVGxsLHHOiZQ5vlVwsko+gk5EtgQRTFXJhGlyJ00 YBu837yX5oWemq2BGdJ8oq4Iv1ZOqMSn26ypOoEN8oIqe53jqKjxu+BkOY+mAIgm/rGUMCy0tdW qN/Cg92iUg56ABM1fQfimY77elJMKEaVKDvKomAUr551Y5dGvMcIrXTdvobxUUvGke4WdTca/9R cy3f+rIJHqOch1YDdwh2VrcrGLv+EDnzJUwXrvdl6X5 X-Received: by 2002:a05:6000:2cc:b0:382:498a:9cf5 with SMTP id ffacd0b85a97d-385c6ebb6d9mr2753746f8f.14.1732715879917; Wed, 27 Nov 2024 05:57:59 -0800 (PST) X-Google-Smtp-Source: AGHT+IFYo73Vhc5gZnsNQ89iDBTlPAYABUSyYi/dd+/lr+ioOv/d3JeBQh530VXrw8FJEgu2iKzXyw== X-Received: by 2002:a05:6000:2cc:b0:382:498a:9cf5 with SMTP id ffacd0b85a97d-385c6ebb6d9mr2753717f8f.14.1732715879435; Wed, 27 Nov 2024 05:57:59 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-434aa74f18bsm22809315e9.4.2024.11.27.05.57.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:57:58 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:57 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Igor Mammedov , Ani Sinha Subject: [PULL 11/13] bios-tables-test: Add complex SRAT / HMAT test for GI GP Message-ID: <18afed7402038b772dc9e9250181bcaddda864ab.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.133.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.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, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron Add a test with 6 nodes to exercise most interesting corner cases of SRAT and HMAT generation including the new Generic Initiator and Generic Port Affinity structures. More details of the set up in the following patch adding the table data. Signed-off-by: Jonathan Cameron Message-Id: <20241107123446.902801-5-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test.c | 97 ++++++++++++++++++++++++++++++++++ 1 file changed, 97 insertions(+) diff --git a/tests/qtest/bios-tables-test.c b/tests/qtest/bios-tables-test.c index e79f3a03df..16d0ffbdf6 100644 --- a/tests/qtest/bios-tables-test.c +++ b/tests/qtest/bios-tables-test.c @@ -1936,6 +1936,101 @@ static void test_acpi_q35_tcg_acpi_hmat_noinitiator(void) free_test_data(&data); } +/* Test intended to hit corner cases of SRAT and HMAT */ +static void test_acpi_q35_tcg_acpi_hmat_generic_x(void) +{ + test_data data = {}; + + data.machine = MACHINE_Q35; + data.arch = "x86"; + data.variant = ".acpihmat-generic-x"; + test_acpi_one(" -machine hmat=on,cxl=on" + " -smp 3,sockets=3" + " -m 128M,maxmem=384M,slots=2" + " -device pcie-root-port,chassis=1,id=pci.1" + " -device pci-testdev,bus=pci.1," + "multifunction=on,addr=00.0" + " -device pci-testdev,bus=pci.1,addr=00.1" + " -device pci-testdev,bus=pci.1,id=gidev,addr=00.2" + " -device pxb-cxl,bus_nr=64,bus=pcie.0,id=cxl.1" + " -object memory-backend-ram,size=64M,id=ram0" + " -object memory-backend-ram,size=64M,id=ram1" + " -numa node,nodeid=0,cpus=0,memdev=ram0" + " -numa node,nodeid=1" + " -object acpi-generic-initiator,id=gi0,pci-dev=gidev,node=1" + " -numa node,nodeid=2" + " -object acpi-generic-port,id=gp0,pci-bus=cxl.1,node=2" + " -numa node,nodeid=3,cpus=1" + " -numa node,nodeid=4,memdev=ram1" + " -numa node,nodeid=5,cpus=2" + " -numa hmat-lb,initiator=0,target=0,hierarchy=memory," + "data-type=access-latency,latency=10" + " -numa hmat-lb,initiator=0,target=0,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=800M" + " -numa hmat-lb,initiator=0,target=2,hierarchy=memory," + "data-type=access-latency,latency=100" + " -numa hmat-lb,initiator=0,target=2,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=0,target=4,hierarchy=memory," + "data-type=access-latency,latency=100" + " -numa hmat-lb,initiator=0,target=4,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=0,target=5,hierarchy=memory," + "data-type=access-latency,latency=200" + " -numa hmat-lb,initiator=0,target=5,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=400M" + " -numa hmat-lb,initiator=1,target=0,hierarchy=memory," + "data-type=access-latency,latency=500" + " -numa hmat-lb,initiator=1,target=0,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=100M" + " -numa hmat-lb,initiator=1,target=2,hierarchy=memory," + "data-type=access-latency,latency=50" + " -numa hmat-lb,initiator=1,target=2,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=400M" + " -numa hmat-lb,initiator=1,target=4,hierarchy=memory," + "data-type=access-latency,latency=50" + " -numa hmat-lb,initiator=1,target=4,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=800M" + " -numa hmat-lb,initiator=1,target=5,hierarchy=memory," + "data-type=access-latency,latency=500" + " -numa hmat-lb,initiator=1,target=5,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=100M" + " -numa hmat-lb,initiator=3,target=0,hierarchy=memory," + "data-type=access-latency,latency=20" + " -numa hmat-lb,initiator=3,target=0,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=400M" + " -numa hmat-lb,initiator=3,target=2,hierarchy=memory," + "data-type=access-latency,latency=80" + " -numa hmat-lb,initiator=3,target=2,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=3,target=4,hierarchy=memory," + "data-type=access-latency,latency=80" + " -numa hmat-lb,initiator=3,target=4,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=3,target=5,hierarchy=memory," + "data-type=access-latency,latency=20" + " -numa hmat-lb,initiator=3,target=5,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=400M" + " -numa hmat-lb,initiator=5,target=0,hierarchy=memory," + "data-type=access-latency,latency=20" + " -numa hmat-lb,initiator=5,target=0,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=400M" + " -numa hmat-lb,initiator=5,target=2,hierarchy=memory," + "data-type=access-latency,latency=80" + " -numa hmat-lb,initiator=5,target=4,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=5,target=4,hierarchy=memory," + "data-type=access-latency,latency=80" + " -numa hmat-lb,initiator=5,target=2,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=200M" + " -numa hmat-lb,initiator=5,target=5,hierarchy=memory," + "data-type=access-latency,latency=10" + " -numa hmat-lb,initiator=5,target=5,hierarchy=memory," + "data-type=access-bandwidth,bandwidth=800M", + &data); + free_test_data(&data); +} + #ifdef CONFIG_POSIX static void test_acpi_erst(const char *machine, const char *arch) { @@ -2414,6 +2509,8 @@ int main(int argc, char *argv[]) qtest_add_func("acpi/q35/nohpet", test_acpi_q35_tcg_nohpet); qtest_add_func("acpi/q35/acpihmat-noinitiator", test_acpi_q35_tcg_acpi_hmat_noinitiator); + qtest_add_func("acpi/q35/acpihmat-genericx", + test_acpi_q35_tcg_acpi_hmat_generic_x); /* i386 does not support memory hotplug */ if (strcmp(arch, "i386")) { From patchwork Wed Nov 27 13:57:59 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887064 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 04F76D6ACD9 for ; Wed, 27 Nov 2024 14:00:26 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYV-0003kG-9N; Wed, 27 Nov 2024 08:58:11 -0500 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 1tGIYS-0003jl-TN for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:08 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.133.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYQ-0007lM-Eu for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:08 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715885; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=KhZA6ORaO2duS0AgwjGZ6tlQJYQfNk0S+QGZU86VpQM=; b=H8mja4V3Rmctu6ZJovBKJBDESp+7SuXGHrKAuE/3Rb1HmQPQgVXq/IVgSihnrL/1hgVlxV +KThkY8e/0lmi4OCVcq6yfW+OqX/UEjFVrt3Y8c8I5zLSV0nNksUkVgVYoNPGe8zDAZpf9 fPVtEjB7hid4JvXb4oK1hxHm2GPjBcY= Received: from mail-wr1-f70.google.com (mail-wr1-f70.google.com [209.85.221.70]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-164-4PatTNZ-OH-mad5crYClLg-1; Wed, 27 Nov 2024 08:58:04 -0500 X-MC-Unique: 4PatTNZ-OH-mad5crYClLg-1 X-Mimecast-MFC-AGG-ID: 4PatTNZ-OH-mad5crYClLg Received: by mail-wr1-f70.google.com with SMTP id ffacd0b85a97d-382342b66f5so3881363f8f.1 for ; Wed, 27 Nov 2024 05:58:04 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715883; x=1733320683; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=KhZA6ORaO2duS0AgwjGZ6tlQJYQfNk0S+QGZU86VpQM=; b=AUMpiGnyzxHeBloeQGZt3EF0r/vODfkNeKQM4nRZaUZpb+6Ubk+oDIs3MCe0SuguHW EuDPwyepNNhcWlftYTV3MWa/78EL0fX2Ywtpholipwp5ptemg6OtCuN90/GljcVlSu/8 kyIHYKOh6Jjl7nC+GGyYYgUUU5YQSnaD/fcZQfZSvEflho3rf+Igro3ze/fZMpUvYijm MaQ7tws/CwnfXJthwoFvewDe/XUn73jAZCHVYJp6BkbsWLWwI4TVE42PLtyczeyEf70t x0kOl6bdSG0y+YoeGm+TlTU5hAn2f8hqqHxNFfZWQcmc2jtGNSDyKaQ8atUWMgIhVPrw Le2g== X-Gm-Message-State: AOJu0YyqMrO6lL/IeoTeA4Zwe+/wc7u6AolPzvApFE8bC92DlvPiv0sS mpROd8BSnAkRnHJHgaADC2sOlvT5z4obaWlhqE1TAZotCNSfavccEYVu4n3L2XUXuhazZu86qq/ uvVh7nnkK4zFztEKNdU0wY4pw40c2MyNIWu1AbyL0v9I4NLRzqHH6uSxk9uj8JQxXiuQP2i9hs4 2szp6LM2azchbDW89b4lEXn2lNjngBwlV4 X-Gm-Gg: ASbGncvzFQ9Q7DHJ8proPGo2jvrqJdMS4V+VKREqh/Rp00QCAFBcyQ54DGS+tyii3Ni 5VcImRerhrDAUQ4zdv8bUiGBKee8YqeE6Q5hrllXgdFvEghUkDjHaQ07XvewygXbS9eHgFachTZ FUrwAScAyDVyY6D/UuLM+C9UDJavetFbJAJLXXgyavEo/bXj39feYRCudDnQLCDOvnKBoZ5QYiz yVIptgibogITHDpGXYTvmrJWPJtVWb6pqng4XxrRRCm X-Received: by 2002:a05:6000:1545:b0:382:49f9:74bc with SMTP id ffacd0b85a97d-385c6edb224mr2127297f8f.41.1732715882673; Wed, 27 Nov 2024 05:58:02 -0800 (PST) X-Google-Smtp-Source: AGHT+IEQmrKcq5JaJoDZOYw9p+666j1t64477UWYM33Izq40Lk5HTponQB1otj+LotlHOLs1QBynnA== X-Received: by 2002:a05:6000:1545:b0:382:49f9:74bc with SMTP id ffacd0b85a97d-385c6edb224mr2127264f8f.41.1732715882100; Wed, 27 Nov 2024 05:58:02 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-3825fbe901esm16617807f8f.87.2024.11.27.05.58.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:58:01 -0800 (PST) Date: Wed, 27 Nov 2024 08:57:59 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Jonathan Cameron , Igor Mammedov , Ani Sinha Subject: [PULL 12/13] bios-tables-test: Add data for complex numa test (GI, GP etc) Message-ID: <5d525294626ebdc2741c19167217ec79a6c2cc08.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.133.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.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, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H3=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Jonathan Cameron Given this is a new configuration, there are affects on APIC, CEDT and DSDT, but the key elements are in SRAT (plus related data in HMAT). The configuration has node to exercise many different combinations. 0) CPUs + Memory 1) GI only 2) GP only 3) CPUS only 4) Memory only 5) CPUs + HP memory GI node, GP Node, Memory only node, hotplug memory only node, latency and bandwidth such that in Linux Access0 (any initiator) and Access1 (CPU initiators only) given different answers. Following cropped to remove details of each entry. [000h 0000 004h] Signature : "SRAT" [System Resource Affinity Table] ... [030h 0048 001h] Subtable Type : 00 [Processor Local APIC/SAPIC Affinity] ... [032h 0050 001h] Proximity Domain Low(8) : 00 [033h 0051 001h] Apic ID : 00 ... [040h 0064 001h] Subtable Type : 00 [Processor Local APIC/SAPIC Affinity] ... [042h 0066 001h] Proximity Domain Low(8) : 03 [043h 0067 001h] Apic ID : 01 ... [050h 0080 001h] Subtable Type : 00 [Processor Local APIC/SAPIC Affinity] ... [052h 0082 001h] Proximity Domain Low(8) : 05 [053h 0083 001h] Apic ID : 02 ... [060h 0096 001h] Subtable Type : 01 [Memory Affinity] ... [062h 0098 004h] Proximity Domain : 00000000 ... [068h 0104 008h] Base Address : 0000000000000000 [070h 0112 008h] Address Length : 00000000000A0000 ... [088h 0136 001h] Subtable Type : 01 [Memory Affinity] ... [08Ah 0138 004h] Proximity Domain : 00000000 ... [090h 0144 008h] Base Address : 0000000000100000 [098h 0152 008h] Address Length : 0000000003F00000 ... [0B0h 0176 001h] Subtable Type : 01 [Memory Affinity] ... [0B2h 0178 004h] Proximity Domain : 00000004 ... [0B8h 0184 008h] Base Address : 0000000004000000 [0C0h 0192 008h] Address Length : 0000000004000000 ... some zero length entries follow... [1A0h 0416 001h] Subtable Type : 05 [Generic Initiator Affinity] [1A1h 0417 001h] Length : 20 [1A2h 0418 001h] Reserved1 : 00 [1A3h 0419 001h] Device Handle Type : 01 [1A4h 0420 004h] Proximity Domain : 00000001 [1A8h 0424 010h] Device Handle : 00 00 01 02 00 00 00 00 00 00 00 00 00 00 00 00 [1B8h 0440 004h] Flags (decoded below) : 00000001 Enabled : 1 Architectural Transactions : 0 [1BCh 0444 004h] Reserved2 : 00000000 [1C0h 0448 001h] Subtable Type : 06 [Generic Port Affinity] [1C1h 0449 001h] Length : 20 [1C2h 0450 001h] Reserved1 : 00 [1C3h 0451 001h] Device Handle Type : 00 [1C4h 0452 004h] Proximity Domain : 00000002 [1C8h 0456 010h] Device Handle : 41 43 50 49 30 30 31 36 40 00 00 00 00 00 00 00 [1D8h 0472 004h] Flags (decoded below) : 00000001 Enabled : 1 Architectural Transactions : 0 [1DCh 0476 004h] Reserved2 : 00000000 [1E0h 0480 001h] Subtable Type : 01 [Memory Affinity] ... [1E2h 0482 004h] Proximity Domain : 00000005 ... [1E8h 0488 008h] Base Address : 0000000100000000 [1F0h 0496 008h] Address Length : 0000000090000000 Example block from HMAT: [0F0h 0240 002h] Structure Type : 0001 [System Locality Latency and Bandwidth Information] [0F2h 0242 002h] Reserved : 0000 [0F4h 0244 004h] Length : 00000078 [0F8h 0248 001h] Flags (decoded below) : 00 Memory Hierarchy : 0 Use Minimum Transfer Size : 0 Non-sequential Transfers : 0 [0F9h 0249 001h] Data Type : 03 [0FAh 0250 001h] Minimum Transfer Size : 00 [0FBh 0251 001h] Reserved1 : 00 [0FCh 0252 004h] Initiator Proximity Domains # : 00000004 [100h 0256 004h] Target Proximity Domains # : 00000006 [104h 0260 004h] Reserved2 : 00000000 [108h 0264 008h] Entry Base Unit : 0000000000000004 [110h 0272 004h] Initiator Proximity Domain List : 00000000 [114h 0276 004h] Initiator Proximity Domain List : 00000001 [118h 0280 004h] Initiator Proximity Domain List : 00000003 [11Ch 0284 004h] Initiator Proximity Domain List : 00000005 [120h 0288 004h] Target Proximity Domain List : 00000000 [124h 0292 004h] Target Proximity Domain List : 00000001 [128h 0296 004h] Target Proximity Domain List : 00000002 [12Ch 0300 004h] Target Proximity Domain List : 00000003 [130h 0304 004h] Target Proximity Domain List : 00000004 [134h 0308 004h] Target Proximity Domain List : 00000005 [138h 0312 002h] Entry : 00C8 [13Ah 0314 002h] Entry : 0000 [13Ch 0316 002h] Entry : 0032 [13Eh 0318 002h] Entry : 0000 [140h 0320 002h] Entry : 0032 [142h 0322 002h] Entry : 0064 [144h 0324 002h] Entry : 0019 [146h 0326 002h] Entry : 0000 [148h 0328 002h] Entry : 0064 [14Ah 0330 002h] Entry : 0000 [14Ch 0332 002h] Entry : 00C8 [14Eh 0334 002h] Entry : 0019 [150h 0336 002h] Entry : 0064 [152h 0338 002h] Entry : 0000 [154h 0340 002h] Entry : 0032 [156h 0342 002h] Entry : 0000 [158h 0344 002h] Entry : 0032 [15Ah 0346 002h] Entry : 0064 [15Ch 0348 002h] Entry : 0064 [15Eh 0350 002h] Entry : 0000 [160h 0352 002h] Entry : 0032 [162h 0354 002h] Entry : 0000 [164h 0356 002h] Entry : 0032 [166h 0358 002h] Entry : 00C8 Note the zeros represent entries where the target node has no memory. These could be surpressed but it isn't 'wrong' to provide them and it is (probably) permissible under ACPI to hotplug memory into these nodes later. Signed-off-by: Jonathan Cameron Message-Id: <20241107123446.902801-6-Jonathan.Cameron@huawei.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin --- tests/qtest/bios-tables-test-allowed-diff.h | 5 ----- tests/data/acpi/x86/q35/APIC.acpihmat-generic-x | Bin 0 -> 136 bytes tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x | Bin 0 -> 68 bytes tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x | Bin 0 -> 12565 bytes tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x | Bin 0 -> 360 bytes tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x | Bin 0 -> 520 bytes 6 files changed, 5 deletions(-) diff --git a/tests/data/acpi/x86/q35/APIC.acpihmat-generic-x b/tests/data/acpi/x86/q35/APIC.acpihmat-generic-x index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..317ddb3fbed94e4f49a87976fdc7f23b1a6c3fdc 100644 GIT binary patch literal 136 zcmXxb%Mn0O3`XITcnlXkq!sRl6*D%L%2Ae5RD#JhHu=utPrpp@0J43UEbqR4{U|?Xhb@F%i2v%^42yj+VP*7lGU|;~TK{Nw{0)qoc4VVoE6CiAe2mn)- B2LS*8 literal 0 HcmV?d00001 diff --git a/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x b/tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..497706c9742a9ea5396d6c9c4cc1cc2a4a530339 100644 GIT binary patch literal 12565 zcmcIqU2GdycD|QC(r`GEM%3uXvTPAf(qyw+hmwD?{n^Ik3`J7nP-6~dD;T33$#yHr zDKd5JBHF-!L<3nz4{Z~!7wsDzaCiH*`%v_)sDbvOeKSxL4N#y!ANsP7T@>pAMOd8s zL)}ABz#6;N19j)lIp;gy%$<8??)@C!X-qE>LVsLXx66LEu<6;^Y&J^?A<%~ZW+icp z1P4x`)THTHf(D0a)^-Y|f@XcoDQuMJU+jcG2*U^83+Yxww!bu=Zf`}8pM={4j2>=B zx(*FXBmY9zw5H8lajm&gafnqNZLa%v;^-c+otsOYTC%+} zIi;I{`)`+ivAA&U2R9$&pMU3H|KRAJp0J&p%SZQg&30~Hir^>wB-GD`9rKI!E`0>< z?A%gk>Welz&HGF^6NS+8%-~%RYN>nAT1yrk-yQ2*ZSRq2GEPa@ng4<(@Z#Nn`cMDL zciqotf99U~d)w`XozbVedaN5p`r(t1kZ#ynXz%HxQ6?E9VUoaKy1T^sX1?NV4zq$-5&VS0Hna*U zk6lk^c~Gylb-_BLs68@*nyM;>j_tw&q?R3AOQz@0bH)gK5D zZS&ipc@=wwSVq&U*fiRKBeQjb!u2)npRuv&)f--B?W2QCI+5=6NVT?BB)m*HpVA)XrIsbQWP=9xk~Q;25@^Gso$I^wA#o;v2K zW1eZmGmUtrG0!ySX&{~k;%Q)>2Ie`2c#a{SW0>a{<~fddjw7DqnCCd=Ie~aiAf6ML z=LF_yBAzDVX=0ux<~fOYP9mO@nCB$snL#`=h-U`#%wV2Vi02gIIfZ#nVV)M^X(65# z=4oM`(}?FZ;yI0ZPGg=ki02IAIfHr5V4kyx=Pcqmi+Rpso^y!j9O5~LdCp;;XP9SR z%$jGIXF*JMXSk<*dXQ+l37o@bHgS?qZhdoCc)1?0JaJr}U& zIplc`d7i_b=dkB_<|$|2^UPDu@#ndxoa4`PPdUdgBFsgExri|rB_m33$o;DAB=BRz*s5@RZbI%u|-O81s~-P(ZmUtHe-1 z6_hyhlo`aCr&>U{Df5H^$}?i+KEXU?1_|aVFD?{NC3z^I3eP0snM6FHfGRwpfGRvk z5YG|B6AGxp6AGxpa}@C$MLeN^DmsKOHpsKV1gJPpJX3aG*p3aG+!4DlR8JfVOpJfVOp zJjW5wal{h}sKOHpsKRps@tiks_=vYs_>jfJf{&)D4+^Y zD4+_@8N_o2@q_}Z@Pq=Y@SH_FXAw^*pbAeYpbF19#B&bugaWGYgaWGYJi|Pd0$TDb zY5^_z6}5nt{E94t^T>0aVMJXtlPMmn$*G1YdvAnqJ+XoYAK~8z-rFk`SU3Lehkg zePT^C{YIX2AqB5~J#S4mor+H$L0dmyJrl5?0TyJS8A!xb4ov+0hdC6Umy#lotDFDdXP245P0FP#Ek67c0g_|aE@FDvk6245b4FP{Qm4q9%a8t%k%Hig4Y4GG8l zhK4)w+*mlAH6$GG8yfD!b1;p=O%Dmj`-X-)F|KEDxS1j0c;C=)C&uY44mUd_9Ph*7 zOjqN}|3$6r6&rH&Q?ckD9F)C=OAJC@e4QqV;LwEQsT;+bllAIOGizOCl_!FpFoLeF z?}oY=c8t&4A!&!{Uaw0!G?%UvaAb(N=B)^EKNm+S3bz39tqVUF0SIAgZAku87tAo}3pA7=00{piDo z_ix|*hy>L3s%zq*8PoL={mS^tpvFE44!k<7*?(o|MsWBR9OOyX_KG!XS86`B>+TI| z7guxevqDMSwp~XH4l`gGxBX%@OR~0;Z)7c_S*caDh(Kr1T7B;p+ZeFD4W6W6vy_80 z__j*YddqICydUf6?H%%|oy+BRp-G{+wA)Fyck~#vmv^o5i0wD>so=F+?<7?Izzenm z2D|-YHG&P2_TUL%7U>gV@aBm#)wOPDxk+IAYCv=_RB<&oG`>}uv3CBYNk{Vy1nEM&wRJ!-#Bs=^B6D4(Zd|dORXwbbAn!F&^x+U8ghqbgF-0F3;g}%^+Gx8)~nE}mb0$6*~|uq-x^|{C)aeh=r^eg&xs39 zfeRa+!Qtyetm7_Bo$h+^M%G>T?W{G$ibyBbhMWjgh;%?*w)L4Gaecd{oQp^pyawlb zy)S?FX$T%s2mw1&ZI0epc|W8POXb;2$7=I*pcfuWIJzIwFnAM!bgA{5E))SJZ1k5q zgau5u87DqtGV9RF&$)KpPstRdt>)IRUY@+9=Z?Nfh!Gci#QndoVRIDrN5!8w?Cp~y zaMZ`$O0CM98Mjg^^X6|dz)pjA$9f8VaQ;3VQ`R}jI_ZOc545UquG(=%+t5ukw|;54zeV1s3{-(7P-OHj zE$g~lsWp$lULSY!o1!^WE!LV5Aj9d1rIFukMvy0*5_bsP`pv7Ct=C;|)1`wqAcD~; zZ)#}0<*xgU#0jHa7}Vz-^*4%v#Xc+<0kEs)m9*fW0zX?3HmS2ajep=yxH3sr+e;i?9i0arChQ=w{*$qQA3MB%Ci znE_WdNK>I|kjV>GgGAw~2AKg@HAqvTYLLkbRf9y~ss@<>S2ajep=yxH!=?-$U2Ln2 zs~ThmT-6{=g{na&FH{W@g{vB523*x3O@*pKCNESC5{0W8Bu}ntkfuV_Ad?rW28qH| z4Kf3+YLKQv)gY6HO&NtJS2f5CxT-;#3RQzlUZ@%*3RgAA47jR6nhI5eOkSuOBnnqG zNS<8PAWem;K_)L$4HAW`8e|4s)gVoUszD|%R1Fe^s~ThmT-6{=g{na&FH{W@g{vB5 z23*x3O@*pKCNESC5{0W8WCmQ-AWem;K_)L$4HAW`8e|4s)gVoUszD|%R1Fe^s~Thm zT-6{=g{na&FH{W@g{vB523*x3O@*pKCNESC66IG_HOLIOszLG;ss@?7P&G)NT-6{m z;Hm~`DpURHzzc@

65T8ZUdgjR(i5Y~DYvdY%FUWmjt|$2`N-1e?R$cT`@-DGb?6U(l zYTj)+zH4-HyvG0NfA0SL-+%na*M7A3*G78%PwSmjdyjM{y)yrtX-&n8v`%_3 zE3>~CK`KA~BMJDP*P8z2W=2csF+IkPuD}`KVgAnXZ3tQyS8m#T^>*xf(cQ@9a?3+d%%quw zML;vZ%-&hUwsWJ_h;`DR>`?M4P~SbR9keF= zrQ_QWhf!wvE5|*Hu#{V=UytdbZkTq0f0!Kp_B(O7rCz@8L2J3{HI`)T1Yh={$kx=yO7!FS$8k-&aKIuj+v{%MCrR1Y;Sisf@t1L; zT`-x?8})|W&n7JTOzPB*OiW@d0Cl4*>uG literal 0 HcmV?d00001 diff --git a/tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x b/tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..b45838adb7d304f8a36c38c90d89f3629ec48353 100644 GIT binary patch literal 520 zcmWFzatz^MVqjn_cJg=j2v%^42yj+VP*7lGU|;~TK{N=%fdD$6nGsc8r~%gr z1za!&in$1N0#Nx6%rJ$h=CQzpVGJ0JrVgeIKS0=v9}JW_Rs{xV_%bpfyAT#8Yzhnv iAPGn308ax0Lo)}cBA7g*1}j_}RBQrF03N^$3=9B2oeTj0 literal 0 HcmV?d00001 diff --git a/tests/qtest/bios-tables-test-allowed-diff.h b/tests/qtest/bios-tables-test-allowed-diff.h index 3c0967078f..dfb8523c8b 100644 --- a/tests/qtest/bios-tables-test-allowed-diff.h +++ b/tests/qtest/bios-tables-test-allowed-diff.h @@ -1,6 +1 @@ /* List of comma-separated changed AML files to ignore */ -"tests/data/acpi/x86/q35/APIC.acpihmat-generic-x", -"tests/data/acpi/x86/q35/CEDT.acpihmat-generic-x", -"tests/data/acpi/x86/q35/DSDT.acpihmat-generic-x", -"tests/data/acpi/x86/q35/HMAT.acpihmat-generic-x", -"tests/data/acpi/x86/q35/SRAT.acpihmat-generic-x", From patchwork Wed Nov 27 13:58:02 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Michael S. Tsirkin" X-Patchwork-Id: 13887059 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 20A77D6ACD2 for ; Wed, 27 Nov 2024 13:59:51 +0000 (UTC) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1tGIYV-0003kQ-Ay; Wed, 27 Nov 2024 08:58:11 -0500 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 1tGIYT-0003k1-KE for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:09 -0500 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1tGIYS-0007lZ-69 for qemu-devel@nongnu.org; Wed, 27 Nov 2024 08:58:09 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1732715887; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=NYnMOw7bKRrTuXSKTx/j7+jLrUGgRw/r9cXH2IkbZbo=; b=c9kJfgha7eSb6zFqfkG9i+UIANH1+6sFOzwK9DxQfbZ8oB3U4/1rTwkKeoCUxSkgH5u+b1 0VGVSS8h2ZC8e75UdVnLeR/n0gXKQP+f9OnWdLa4OyXjxb77Nx1PvU3I76Cv8EbIQNB535 6xSzxeTgUrWOAXS3P64mNfD2hPTzO6o= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-589-APhiqstKOcqrHC0OTa2yqw-1; Wed, 27 Nov 2024 08:58:06 -0500 X-MC-Unique: APhiqstKOcqrHC0OTa2yqw-1 X-Mimecast-MFC-AGG-ID: APhiqstKOcqrHC0OTa2yqw Received: by mail-wm1-f71.google.com with SMTP id 5b1f17b1804b1-4349df2d87dso34711365e9.2 for ; Wed, 27 Nov 2024 05:58:06 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1732715885; x=1733320685; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=NYnMOw7bKRrTuXSKTx/j7+jLrUGgRw/r9cXH2IkbZbo=; b=VFc89fpFgIqL4+Wlk/K6zxJ+fK2IXgnTUvipK0wbidxgZBobkWWuQpB0IXqMzdvcGY 3VzRWAZkh80Xty+ItCPrjuPSTH/U2OTfn8E/xdCr/LfVSR42WYqeBVeXBBhVcllvwnD4 eVtVQNJ5tXBZjJxadEojH4PTSLFqVHZpGRcqvYba4ZmKFRmN4tstop0IFg95MzV9DR0G gJOBiZJKPVzi/JO7jD+AVI8xPNE3nAyM3B7kTOuQbcgQ+IHJwCqFZRcJ44Yf3QQfhVHP wx+7eCpXva16bnT+tLxlmNxFy02AZ6NUQBOQITdod3eZAT8jW2PNyekeXVCfYSKGAMOX jZJw== X-Gm-Message-State: AOJu0YylNhZI3CxQtN7INu2+maaUd4MzkAHzUudLJitjXk4FmqQxhw9Z nofrxtHNk4/AKnpQ2JEaeN6E5wLXy7szpMp/zPuFVG1YGftRCO7q9hLOPxIVfm7Jk7rll/+OAEs 7gYQpwHgcGM0vIEo8+ASCGDd+SPsZp7m7ksJSU2m/rKrIFWCONBURZl3kf+18cq8DRe2X8Q4Rjp a+xNDMFYlfhsSzGx5+EylatbnvqHqya6cP X-Gm-Gg: ASbGncsBiN3tg/4EpnTG+NzfOLjn0NJOkZNKxwA56haWQSY/phncB1GAgR+NWT/di8t Cdx/d+OnFKSR/JznQV32Wj+79Sofy8KeTyatLS80+69YkgOMPHrEQtiUEFACMgG5EFntPDshaTx V/2M0ZifWOY4c827L4c17oh/0mmbudP968piGg+Sq1W43BRHRsSy8OccdDsmyrhCp2/tRI/8quR VZWUMrn8Wi/2Rq0T90E62BQo4Mx4vk+h+nVKgBaFCdB X-Received: by 2002:a05:600c:1d20:b0:434:a5c2:53c1 with SMTP id 5b1f17b1804b1-434a9de8bd4mr27435765e9.23.1732715884997; Wed, 27 Nov 2024 05:58:04 -0800 (PST) X-Google-Smtp-Source: AGHT+IFZ7zdAiUKzcmAmlMRToDZqxbL1+MAKAJicumhr7HFMIF0eXutGc3lu9Vrwr79QzQjsa/hh7g== X-Received: by 2002:a05:600c:1d20:b0:434:a5c2:53c1 with SMTP id 5b1f17b1804b1-434a9de8bd4mr27435575e9.23.1732715884659; Wed, 27 Nov 2024 05:58:04 -0800 (PST) Received: from redhat.com ([2a02:14f:1ec:7d4b:8b11:bed7:9e00:8df7]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-434aa7f79c8sm21870955e9.40.2024.11.27.05.58.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 27 Nov 2024 05:58:04 -0800 (PST) Date: Wed, 27 Nov 2024 08:58:02 -0500 From: "Michael S. Tsirkin" To: qemu-devel@nongnu.org Cc: Peter Maydell , Prasad Pandit , Stefano Garzarella Subject: [PULL 13/13] vhost: fail device start if iotlb update fails Message-ID: <571bdc97b83646dfd3746ec56fb2f70bca55b9a2.1732715823.git.mst@redhat.com> References: MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-Mailer: git-send-email 2.27.0.106.g8ac3dc51b1 X-Mutt-Fcc: =sent Received-SPF: pass client-ip=170.10.129.124; envelope-from=mst@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.931, RCVD_IN_VALIDITY_RPBL_BLOCKED=0.001, RCVD_IN_VALIDITY_SAFE_BLOCKED=0.001, 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+qemu-devel=archiver.kernel.org@nongnu.org Sender: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org From: Prasad Pandit While starting a vhost device, updating iotlb entries via 'vhost_device_iotlb_miss' may return an error. qemu-kvm: vhost_device_iotlb_miss: 700871,700871: Fail to update device iotlb Fail device start when such an error occurs. Signed-off-by: Prasad Pandit Message-Id: <20241107113247.46532-1-ppandit@redhat.com> Reviewed-by: Michael S. Tsirkin Signed-off-by: Michael S. Tsirkin Reviewed-by: Stefano Garzarella --- hw/virtio/vhost.c | 13 ++++++++++++- 1 file changed, 12 insertions(+), 1 deletion(-) diff --git a/hw/virtio/vhost.c b/hw/virtio/vhost.c index 76f9b2aaad..c40f48ac4d 100644 --- a/hw/virtio/vhost.c +++ b/hw/virtio/vhost.c @@ -2095,11 +2095,22 @@ int vhost_dev_start(struct vhost_dev *hdev, VirtIODevice *vdev, bool vrings) * vhost-kernel code requires for this.*/ for (i = 0; i < hdev->nvqs; ++i) { struct vhost_virtqueue *vq = hdev->vqs + i; - vhost_device_iotlb_miss(hdev, vq->used_phys, true); + r = vhost_device_iotlb_miss(hdev, vq->used_phys, true); + if (r) { + goto fail_iotlb; + } } } vhost_start_config_intr(hdev); return 0; +fail_iotlb: + if (vhost_dev_has_iommu(hdev) && + hdev->vhost_ops->vhost_set_iotlb_callback) { + hdev->vhost_ops->vhost_set_iotlb_callback(hdev, false); + } + if (hdev->vhost_ops->vhost_dev_start) { + hdev->vhost_ops->vhost_dev_start(hdev, false); + } fail_start: if (vrings) { vhost_dev_set_vring_enable(hdev, false);