[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH for-4.21 v3 1/2] x86/apic: Avoid infinite loop in io_apic_level_ack_pending()
- To: Jan Beulich <jbeulich@xxxxxxxx>
- From: Jason Andryuk <jason.andryuk@xxxxxxx>
- Date: Thu, 16 Oct 2025 18:07:52 -0400
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass (sender ip is 165.204.84.17) smtp.rcpttodomain=suse.com smtp.mailfrom=amd.com; dmarc=pass (p=quarantine sp=quarantine pct=100) action=none header.from=amd.com; dkim=none (message not signed); arc=none (0)
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=C3FMdvvl/y5XmTzmFHQ41vnWtANseuZf6tRbUKhBp9Y=; b=a7ewpoPH/lTe5igtzF1afiqwbDsjAVSDpCjhk/tNeE2CChHLhWyw6OoRVuvdCZkJuXNZESO4dKhCKz0fyBLpHKRlWc89PDFUqZW5LYK/1t6LaFWHr/gpHiY/M0R4QUTIojZcip90bTWBgyFYWIGzNrw8SguemwNXeEaucKR1R414b0/EgN4H/L9/FOPxc4Y1ZPTKU+6C0b3akTydZEyq2qKfzNkJD9BKTfXPNWJwn+A9yndWisbQYI1cAwn/J8x4z45g8eYH9FG23PEpRQ6SZ/PHbE0wGnqAlcGilF6aHgCO5vBMmjQ0NdlyEW8jfgSXKF09/GtC6Nrti+h5Ow9ENQ==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=R5cHVpO22PQvEbPkxIS6QiN0eluB7mwKbsARimUF9d5xmOxl7nwPIda1Qs0X7FcOBKE3Phtow4L4A27/UxHupDWsHiw3fYg8uuL3b0nmQ7qL+8lRabZSrunTrzJvZHLzKD+/fQjNBS1zAiDPfyK6eBjLVhnpp9AtWLoRUgCkRhXMSjl/fAQK8pdq/dya8t+FRpIW4GqJs/SFpmLLq/SDwUhVF95VKfzBWJS2ANT8COjvCxq2kRJR6ZqhVItv+JSip5IyAgg3F5b28/7vSez5m4Ur7mgtoT1hIe7yiN3gjdVpiJ491gHhPlJrn3tQ47IzaYjqpTrsEC0QOwit+7OQ/w==
- Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Thu, 16 Oct 2025 22:25:39 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
On 2025-10-16 02:47, Jan Beulich wrote:
On 15.10.2025 23:04, Jason Andryuk wrote:
io_apic_level_ack_pending() will end up in an infinite loop if
entry->pin == -1. entry does not change, so it will keep reading -1.
Convert to a proper for loop so that continue works. Add a new helper,
next_entry(), to handle advancing to the next irq_pin_list entry.
Noticed during code inspection. The infinite loop was not observed.
Fixes: f821102450a1 ("x86: IRQ Migration logic enhancement.")
Signed-off-by: Jason Andryuk <jason.andryuk@xxxxxxx>
Release-Acked-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
---
v3:
const on next_entry() parameter
Remove spaces inside for loop braces
Remove inner if (!entry) check
Expand commit message to state noticed during code inspection
v2:
continue (not break) for pin == -1.
I added the next_entry() helper since putting the expression in the for
loop is a little cluttered. The helper can also be re-used for other
instances within the file.
---
xen/arch/x86/io_apic.c | 17 +++++++++--------
1 file changed, 9 insertions(+), 8 deletions(-)
diff --git a/xen/arch/x86/io_apic.c b/xen/arch/x86/io_apic.c
index c384f10c1b..c35d611ecf 100644
--- a/xen/arch/x86/io_apic.c
+++ b/xen/arch/x86/io_apic.c
@@ -1586,20 +1586,24 @@ static int __init cf_check setup_ioapic_ack(const char
*s)
}
custom_param("ioapic_ack", setup_ioapic_ack);
+static struct irq_pin_list *next_entry(const struct irq_pin_list *entry)
+{
+ if ( !entry->next )
+ return NULL;
+
+ return irq_2_pin + entry->next;
+}
When replying to the v2 thread I hadn't spotted yet that a v3 was already
posted. As indicated, imo this name to too generic (now). I'd be happy to
make adjustments while committing, as long as we can agree on some less
generic name.
pin_list_next() works for me.
Thanks,
Jason
|
