Revert "Linux: normalize error return for emulated syscalls"
authorSimon Wilkinson <sxw@your-file-system.com>
Tue, 15 Mar 2011 00:06:19 +0000 (00:06 +0000)
committerDerrick Brashear <shadow@dementia.org>
Tue, 15 Mar 2011 02:27:09 +0000 (19:27 -0700)
commitff2933a122ddf9421ebcacdc7e4ec5f44333f894
tree985b2d2babaac763f2f7ea29d482f1a05398adb5
parentc78694fdb76e9606a06cf834b80cbba827b8ff2b
Revert "Linux: normalize error return for emulated syscalls"

This reverts commit 0bc837f68a72ba1f75d940cc5dd057774d9f36bb.

Sadly, this change fixed setpag(), but broke all of the pioctls. The
problem is actually a little more nuanced than we at first thought.
What's happening is yet another case of Linux's special handling of
negative return values. When an ioctl handler returns a negative
return code to the kernel, it does errno = -code, and sets the
return code to -1. If you pass it a postive return code, however,
it just returns that straight to the application.

The pioctl code gets this right. However, the setpag code doesn't,
and so tries to return postive values, which is why ioctl appears
to be returning the error code in the return value, not in the
errno.

Change-Id: I192ff45ad15b72a493a3c9c98546b026761dd95f
Reviewed-on: http://gerrit.openafs.org/4222
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Tested-by: Marc Dionne <marc.c.dionne@gmail.com>
Reviewed-by: Marc Dionne <marc.c.dionne@gmail.com>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
src/sys/glue.c