diff options
author | Joerg Roedel <jroedel@suse.de> | 2017-06-28 06:45:31 -0400 |
---|---|---|
committer | Joerg Roedel <jroedel@suse.de> | 2017-06-28 07:29:45 -0400 |
commit | 7f7a2304aabc4a8102bbbbeed2ec9eaee4a480c2 (patch) | |
tree | ab046f0bf6391cdc57f081537702231c03330bff | |
parent | aaffaa8a3b5950c47e5f7573c34bc47de8894a18 (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.c | 14 |
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 | */ |
916 | struct iommu_group *generic_device_group(struct device *dev) | 916 | struct 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 | /** |