drm-shim: Return -EINVAL instead of abort()ing on unknown ioctls.

I had this as abort() in my original implementation since I was doing
drm-shim and my kernel driver in parallel based around using a SW
simulator, and I wanted to always update both, but it means that people's
new feature detection code can easily end up breaing their drm-shim
shader-db runs (such as intel's kernel_has_dynamic_config_support()
checking for -ENOENT instead of -EINVAL for a feature, which showed up on
my personal runner but not fd.o's for reasons I'm unclear on).

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5994>
This commit is contained in:
Eric Anholt 2020-07-20 11:21:36 -07:00 committed by Marge Bot
parent c77a414ec2
commit 75b1f3d39d
1 changed files with 1 additions and 1 deletions

View File

@ -261,7 +261,7 @@ drm_shim_ioctl(int fd, unsigned long request, void *arg)
nr, request);
}
abort();
return -EINVAL;
}
void