On Tue, 27 May 2014, Michael Ellerman wrote: > On Mon, 2014-05-26 at 20:41 -0400, Dave Jones wrote: > > On Mon, May 26, 2014 at 10:32:01PM +1000, Michael Ellerman wrote: > > > Some syscalls return ENOSYS depending on their arguments. We don't want > > > to stop calling them just because we hit one of those cases. Add a flag > > > to specify this behaviour so we don't have to keep special-casing those > > > calls in mkcall(). > > > > I was hopeful this list wouldn't grow, but that doesn't seem to be > > the case. Begrudgingly, I applied this. It's going to be a lot > > cleaner to maintain if people keep doing this. > > Yeah it's annoying for sure, maybe perf will be the last one, but at least > there's a clean way to handle it if not. As the author of the man page that you probably got the perf ENOSYS info from, I have to put out there that perf_event_open() has really inconsistent and confusing error return values, and they vary among the various architectures. I'd hope other syscalls do better, but it's really easy to slip in weird return values in unexpected places. I somewhat try to follow the perf_event ABI and watch for stuff like this (if only to keep the manpage up to date) but I missed the ENOSYS commit at the time (especially since it's not possible to trigger on x86 where I do most of my testing). Vince -- To unsubscribe from this list: send the line "unsubscribe trinity" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
