Thread (4 messages) 4 messages, 3 authors, 2012-02-14

Re: [PATCH] KEYS: Enable the compat keyctl wrapper on s390x

From: Christian Borntraeger <hidden>
Date: 2012-02-14 15:07:43
Also in: linux-s390, lkml

On 14/02/12 15:50, David Howells wrote:
Enable the compat keyctl wrapper on s390x so that 32-bit s390 userspace can
call the keyctl() syscall.

There's an s390x assembly wrapper that truncates all the register values to
32-bits and this then calls compat_sys_keyctl() - but the latter only exists if
CONFIG_KEYS_COMPAT is enabled, and the s390 Kconfig doesn't enable it.

Without this patch, 32-bit calls to the keyctl() syscall are given an ENOSYS
error:

	[root@devel4 ~]# keyctl show
	Session Keyring
	-3: key inaccessible (Function not implemented)

Signed-off-by: David Howells <dhowells@redhat.com>
Acked-by: dan@danny.cz
cc: Carsten Otte <redacted>
cc: Christian Borntraeger <redacted>
cc: linux-s390@vger.kernel.org
cc: stable@vger.kernel.org
Patch looks sane.
Reviewed-by: Christian Borntraeger <redacted>

Do you want to push this change via Martins s390 tree or via other
trees?
quoted hunk ↗ jump to hunk
---

 arch/s390/Kconfig |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig
index d172758..6d99a5f 100644
--- a/arch/s390/Kconfig
+++ b/arch/s390/Kconfig
@@ -227,6 +227,9 @@ config COMPAT
 config SYSVIPC_COMPAT
 	def_bool y if COMPAT && SYSVIPC

+config KEYS_COMPAT
+	def_bool y if COMPAT && KEYS
+
 config AUDIT_ARCH
 	def_bool y
  
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help