aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJoerg Roedel <jroedel@suse.de>2017-06-28 06:45:31 -0400
committerJoerg Roedel <jroedel@suse.de>2017-06-28 07:29:45 -0400
commit7f7a2304aabc4a8102bbbbeed2ec9eaee4a480c2 (patch)
treeab046f0bf6391cdc57f081537702231c03330bff
parentaaffaa8a3b5950c47e5f7573c34bc47de8894a18 (diff)
iommu: Return ERR_PTR() values from device_group call-backs
The generic device_group call-backs in iommu.c return NULL in case of error. Since they are getting ERR_PTR values from iommu_group_alloc(), just pass them up instead. Reported-by: Gerald Schaefer <gerald.schaefer@de.ibm.com> Signed-off-by: Joerg Roedel <jroedel@suse.de>
-rw-r--r--drivers/iommu/iommu.c14
1 files changed, 2 insertions, 12 deletions
diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index cf7ca7e70777..de09e1e35830 100644
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -915,13 +915,7 @@ static int get_pci_alias_or_group(struct pci_dev *pdev, u16 alias, void *opaque)
915 */ 915 */
916struct iommu_group *generic_device_group(struct device *dev) 916struct iommu_group *generic_device_group(struct device *dev)
917{ 917{
918 struct iommu_group *group; 918 return iommu_group_alloc();
919
920 group = iommu_group_alloc();
921 if (IS_ERR(group))
922 return NULL;
923
924 return group;
925} 919}
926 920
927/* 921/*
@@ -988,11 +982,7 @@ struct iommu_group *pci_device_group(struct device *dev)
988 return group; 982 return group;
989 983
990 /* No shared group found, allocate new */ 984 /* No shared group found, allocate new */
991 group = iommu_group_alloc(); 985 return iommu_group_alloc();
992 if (IS_ERR(group))
993 return NULL;
994
995 return group;
996} 986}
997 987
998/** 988/**