From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D5E8719D409; Tue, 7 May 2024 23:14:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715123642; cv=none; b=LxDf2/Biff8l5zzxkRB6UU/YieRT23pPh4FLbCgnSy7IEk5VE2Ggay1qAy9Ts1A1jn5LRD8v1H+Qatx+kIjPy3GozZws9HTfJCdOY8XYYKU5zs4b9miZTlUIMZr5pfOtn/qnJH1d+OINo9j+HKQGHcQeMDTUGqAswOABdNU3Zag= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1715123642; c=relaxed/simple; bh=Hs3S7Bam/o6E7QCw9UqHc74qAk572AAnODzDEDVeXjo=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=bNRGKj5kyvDieOxUvlKrDFp1xwfBkAvrGtjMWnOvcMPoBdg1U2bLtHSaDPt0DDpiHxmrr/RV4XWeBUbH3qk0HvnmOTzIVP28ATHdvI5v4n+jxrKkp0y96JcVW0a3y+CBYjjQ9eKZRAKWmJPF9W3Uo4T2Omhc4MVPqcZ2Ls7UziE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=pkZWDC8z; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="pkZWDC8z" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B5A45C3277B; Tue, 7 May 2024 23:14:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715123642; bh=Hs3S7Bam/o6E7QCw9UqHc74qAk572AAnODzDEDVeXjo=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=pkZWDC8z74R2b1n3eR2l8MG9dZvWmeo9XP3/ACMLmPbGGslvmPDBGt8WBQqRz7e9P vaeZmKK5SZVhpxNb9LI7jRvX/pbOzA19eOrY/f2ugIDlK5Bjhi3rYAB9b3XyEVGglu YKyuvXpJf/jrTNlt5M7UY+9NFPB/lVC7esiNnBJffEDUqIjTl4VeitR07BHogdtyNi gKWNHeW/2gCO1zVHl1t25uUltChcpNE5604GVOPrAzhsKyQ09vGmKEuu6R2L2PZtpg SwjYSnEcIr/UQ+lcrQU58nj5CiTM/8tDV823c92fda772WUFqzqP+vYCwS3lIcy9IY BRvrwzYl4RywA== From: Sasha Levin To: linux-kernel@vger.kernel.org, stable@vger.kernel.org Cc: Nilay Shroff , Christoph Hellwig , Sagi Grimberg , Chaitanya Kulkarni , Keith Busch , Sasha Levin , linux-nvme@lists.infradead.org Subject: [PATCH AUTOSEL 5.15 14/15] nvme: find numa distance only if controller has valid numa id Date: Tue, 7 May 2024 19:13:23 -0400 Message-ID: <20240507231333.394765-14-sashal@kernel.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240507231333.394765-1-sashal@kernel.org> References: <20240507231333.394765-1-sashal@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-stable: review X-Patchwork-Hint: Ignore X-stable-base: Linux 5.15.158 Content-Transfer-Encoding: 8bit From: Nilay Shroff [ Upstream commit 863fe60ed27f2c85172654a63c5b827e72c8b2e6 ] On system where native nvme multipath is configured and iopolicy is set to numa but the nvme controller numa node id is undefined or -1 (NUMA_NO_NODE) then avoid calculating node distance for finding optimal io path. In such case we may access numa distance table with invalid index and that may potentially refer to incorrect memory. So this patch ensures that if the nvme controller numa node id is -1 then instead of calculating node distance for finding optimal io path, we set the numa node distance of such controller to default 10 (LOCAL_DISTANCE). Link: https://lore.kernel.org/all/20240413090614.678353-1-nilay@linux.ibm.com/ Signed-off-by: Nilay Shroff Reviewed-by: Christoph Hellwig Reviewed-by: Sagi Grimberg Reviewed-by: Chaitanya Kulkarni Signed-off-by: Keith Busch Signed-off-by: Sasha Levin --- drivers/nvme/host/multipath.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/nvme/host/multipath.c b/drivers/nvme/host/multipath.c index 73eddb67f0d24..f8ad43b5f0690 100644 --- a/drivers/nvme/host/multipath.c +++ b/drivers/nvme/host/multipath.c @@ -190,7 +190,8 @@ static struct nvme_ns *__nvme_find_path(struct nvme_ns_head *head, int node) if (nvme_path_is_disabled(ns)) continue; - if (READ_ONCE(head->subsys->iopolicy) == NVME_IOPOLICY_NUMA) + if (ns->ctrl->numa_node != NUMA_NO_NODE && + READ_ONCE(head->subsys->iopolicy) == NVME_IOPOLICY_NUMA) distance = node_distance(node, ns->ctrl->numa_node); else distance = LOCAL_DISTANCE; -- 2.43.0