nvme: remove redundant BUILD_BUG_ON check

guixinliu1995 · keithbusch · commit 1e1c4bd16e38 · 2024-03-21T10:46:12.000-07:00
Remove redundant BUILD_BUG_ON check of struct nvme_dsm_range, it's
already checked in nvme_init_ctrl().

Signed-off-by: Guixin Liu &lt;kanie@linux.alibaba.com&gt;
Reviewed-by: Chaitanya Kulkarni &lt;kch@nvidia.com&gt;
Reviewed-by: Sagi Grimberg &lt;sagi@grimberg.me&gt;
Signed-off-by: Keith Busch &lt;kbusch@kernel.org&gt;
diff --git a/drivers/nvme/host/core.c b/drivers/nvme/host/core.c
@@ -1803,9 +1803,6 @@ static void nvme_config_discard(struct nvme_ns *ns, struct queue_limits *lim)
 {
 	struct nvme_ctrl *ctrl = ns->ctrl;
 
-	BUILD_BUG_ON(PAGE_SIZE / sizeof(struct nvme_dsm_range) <
-			NVME_DSM_MAX_RANGES);
-
 	if (ctrl->dmrsl && ctrl->dmrsl <= nvme_sect_to_lba(ns->head, UINT_MAX))
 		lim->max_hw_discard_sectors =
 			nvme_lba_to_sect(ns->head, ctrl->dmrsl);

Original file line number	Diff line number	Diff line change
`@@ -1803,9 +1803,6 @@ static void nvme_config_discard(struct nvme_ns ns, struct queue_limits lim)`
`1803`	`1803`	`{`
`1804`	`1804`	`struct nvme_ctrl *ctrl = ns->ctrl;`
`1805`	`1805`
`1806`		`- BUILD_BUG_ON(PAGE_SIZE / sizeof(struct nvme_dsm_range) <`
`1807`		`- NVME_DSM_MAX_RANGES);`
`1808`		`-`
`1809`	`1806`	`if (ctrl->dmrsl && ctrl->dmrsl <= nvme_sect_to_lba(ns->head, UINT_MAX))`
`1810`	`1807`	`lim->max_hw_discard_sectors =`
`1811`	`1808`	`nvme_lba_to_sect(ns->head, ctrl->dmrsl);`