Skip to content

Commit 3a2ffd3

Browse files
jpoimboejgunthorpe
authored andcommitted
iommu: Convert unreachable() to BUG()
Bare unreachable() should be avoided as it generates undefined behavior, e.g. falling through to the next function. Use BUG() instead so the error is defined. Fixes the following warnings: drivers/iommu/dma-iommu.o: warning: objtool: iommu_dma_sw_msi+0x92: can't find jump dest instruction at .text+0x54d5 vmlinux.o: warning: objtool: iommu_dma_get_msi_page() falls through to next function __iommu_dma_unmap() Link: https://patch.msgid.link/r/0c801ae017ec078cacd39f8f0898fc7780535f85.1743053325.git.jpoimboe@kernel.org Reported-by: Randy Dunlap <[email protected]> Closes: https://lore.kernel.org/[email protected] Reported-by: Paul E. McKenney <[email protected]> Closes: https://lore.kernel.org/5dd1f35e-8ece-43b7-ad6d-86d02d2718f6@paulmck-laptop Fixes: 6aa63a4 ("iommu: Sort out domain user data") Signed-off-by: Josh Poimboeuf <[email protected]> Signed-off-by: Jason Gunthorpe <[email protected]>
1 parent 6fc85bb commit 3a2ffd3

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/iommu/dma-iommu.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1762,7 +1762,7 @@ static size_t cookie_msi_granule(const struct iommu_domain *domain)
17621762
case IOMMU_COOKIE_DMA_MSI:
17631763
return PAGE_SIZE;
17641764
default:
1765-
unreachable();
1765+
BUG();
17661766
};
17671767
}
17681768

@@ -1774,7 +1774,7 @@ static struct list_head *cookie_msi_pages(const struct iommu_domain *domain)
17741774
case IOMMU_COOKIE_DMA_MSI:
17751775
return &domain->msi_cookie->msi_page_list;
17761776
default:
1777-
unreachable();
1777+
BUG();
17781778
};
17791779
}
17801780

0 commit comments

Comments
 (0)