Commit 23d3a98c authored by Joerg Roedel's avatar Joerg Roedel

iommu/amd: Use BUG_ON instead of if () BUG()

Found by a coccicheck script.
Signed-off-by: default avatarJoerg Roedel <jroedel@suse.de>
parent 4160cd9e
...@@ -1835,8 +1835,8 @@ static void free_gcr3_table(struct protection_domain *domain) ...@@ -1835,8 +1835,8 @@ static void free_gcr3_table(struct protection_domain *domain)
free_gcr3_tbl_level2(domain->gcr3_tbl); free_gcr3_tbl_level2(domain->gcr3_tbl);
else if (domain->glx == 1) else if (domain->glx == 1)
free_gcr3_tbl_level1(domain->gcr3_tbl); free_gcr3_tbl_level1(domain->gcr3_tbl);
else if (domain->glx != 0) else
BUG(); BUG_ON(domain->glx != 0);
free_page((unsigned long)domain->gcr3_tbl); free_page((unsigned long)domain->gcr3_tbl);
} }
......
...@@ -356,8 +356,8 @@ static void free_pasid_states(struct device_state *dev_state) ...@@ -356,8 +356,8 @@ static void free_pasid_states(struct device_state *dev_state)
free_pasid_states_level2(dev_state->states); free_pasid_states_level2(dev_state->states);
else if (dev_state->pasid_levels == 1) else if (dev_state->pasid_levels == 1)
free_pasid_states_level1(dev_state->states); free_pasid_states_level1(dev_state->states);
else if (dev_state->pasid_levels != 0) else
BUG(); BUG_ON(dev_state->pasid_levels != 0);
free_page((unsigned long)dev_state->states); free_page((unsigned long)dev_state->states);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment