Hi Gerald, sorry for the delay. Answers inline.
On Fri, Jun 16, 2017 at 07:33:01PM +0200, Gerald Schaefer wrote: > Seems pretty straightforward, so > Reviewed-by: Gerald Schaefer <gerald.schae...@de.ibm.com> Thanks, I add it to the patch. > With generic_device_group() returning NULL in case the allocation failed, > this part of iommu_group_get_for_dev() would then happily dereference the > NULL pointer, because IS_ERR(group) would be false: Yeah, you are right, this is a bug. I'll send a patch to fix this shortly. I don't remember whether there was a good reason to drop the error values in the device_group call-backs. Probably there is none and I can just pass the error value up instead of NULLing them out. Joerg