From: Jan Beulich Date: Tue, 13 Jul 2021 08:16:18 +0000 (+0200) Subject: IOMMU: correct parsing of "quarantine=scratch-page" X-Git-Tag: archive/raspbian/4.16.0+51-g0941d6cb-1+rpi1~2^2~42^2~318 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=ec6e56359553e82fefa877fd221479847ffc93dd;p=xen.git 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 --- 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