From ec6e56359553e82fefa877fd221479847ffc93dd Mon Sep 17 00:00:00 2001 From: Jan Beulich Date: Tue, 13 Jul 2021 10:16:18 +0200 Subject: [PATCH] IOMMU: correct parsing of "quarantine=scratch-page" During the multiple renames of the sub-option I apparently forgot to update the left side of the &&, and this pretty consistently. Fixes: 980d6acf1517 ("IOMMU: make DMA containment of quarantined devices optional") Reported-by: Andrew Cooper Signed-off-by: Jan Beulich Reviewed-by: Paul Durrant --- xen/drivers/passthrough/iommu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/xen/drivers/passthrough/iommu.c b/xen/drivers/passthrough/iommu.c index ab2ae61b11..fd2578d30a 100644 --- a/xen/drivers/passthrough/iommu.c +++ b/xen/drivers/passthrough/iommu.c @@ -82,7 +82,7 @@ static int __init parse_iommu_param(const char *s) #ifdef CONFIG_HAS_PCI else if ( (val = parse_boolean("quarantine", s, ss)) >= 0 ) iommu_quarantine = val; - else if ( ss == s + 15 && !strncmp(s, "quarantine=scratch-page", 23) ) + else if ( ss == s + 23 && !strncmp(s, "quarantine=scratch-page", 23) ) iommu_quarantine = IOMMU_quarantine_scratch_page; #endif #ifdef CONFIG_X86 -- 2.30.2