about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJoseph Myers <joseph@codesourcery.com>2017-05-08 16:44:54 +0000
committerJoseph Myers <joseph@codesourcery.com>2017-05-08 16:44:54 +0000
commit1721145f0341d70a6d7807b172c5eb400b508fc0 (patch)
tree61dadb8a2fd3785448329e189d1ca39c8cd621d2
parent1278ed2c645cc31fc5a71febf309bbec9a9d9f90 (diff)
downloadglibc-1721145f0341d70a6d7807b172c5eb400b508fc0.tar.gz
glibc-1721145f0341d70a6d7807b172c5eb400b508fc0.tar.xz
glibc-1721145f0341d70a6d7807b172c5eb400b508fc0.zip
Remove __ASSUME_PROC_PID_TASK_COMM.
This patch removes the __ASSUME_PROC_PID_TASK_COMM macro, and
associated conditional code (in a testcase), now that 3.2 is the
global minimum Linux kernel version supported.

Tested for x86_64.

	* sysdeps/unix/sysv/linux/kernel-features.h
	(__ASSUME_PROC_PID_TASK_COMM): Remove macro.
	* sysdeps/unix/sysv/linux/tst-setgetname.c: Do not include
	<kernel-features.h>.
	(do_test) [!__ASSUME_PROC_PID_TASK_COMM]: Remove conditional code.
-rw-r--r--ChangeLog6
-rw-r--r--sysdeps/unix/sysv/linux/kernel-features.h6
-rw-r--r--sysdeps/unix/sysv/linux/tst-setgetname.c13
3 files changed, 6 insertions, 19 deletions
diff --git a/ChangeLog b/ChangeLog
index 7386ce96df..49378f6609 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,11 @@
 2017-05-08  Joseph Myers  <joseph@codesourcery.com>
 
+	* sysdeps/unix/sysv/linux/kernel-features.h
+	(__ASSUME_PROC_PID_TASK_COMM): Remove macro.
+	* sysdeps/unix/sysv/linux/tst-setgetname.c: Do not include
+	<kernel-features.h>.
+	(do_test) [!__ASSUME_PROC_PID_TASK_COMM]: Remove conditional code.
+
 	* sysdeps/unix/sysv/linux/x86_64/kernel-features.h
 	(__ASSUME_GETCPU_SYSCALL): Remove macro.
 
diff --git a/sysdeps/unix/sysv/linux/kernel-features.h b/sysdeps/unix/sysv/linux/kernel-features.h
index b981466f34..d837c221fb 100644
--- a/sysdeps/unix/sysv/linux/kernel-features.h
+++ b/sysdeps/unix/sysv/linux/kernel-features.h
@@ -98,12 +98,6 @@
 # define __ASSUME_RECVMMSG	1
 #endif
 
-/* Support for /proc/self/task/$tid/comm and /proc/$pid/task/$tid/comm was
-   added in 2.6.33.  */
-#if __LINUX_KERNEL_VERSION >= 0x020621
-# define __ASSUME_PROC_PID_TASK_COMM	1
-#endif
-
 /* statfs fills in f_flags since 2.6.36.  */
 #if __LINUX_KERNEL_VERSION >= 0x020624
 # define __ASSUME_STATFS_F_FLAGS	1
diff --git a/sysdeps/unix/sysv/linux/tst-setgetname.c b/sysdeps/unix/sysv/linux/tst-setgetname.c
index 5acd614117..0ab35e60f9 100644
--- a/sysdeps/unix/sysv/linux/tst-setgetname.c
+++ b/sysdeps/unix/sysv/linux/tst-setgetname.c
@@ -23,7 +23,6 @@
 #include <unistd.h>
 #include <fcntl.h>
 #include <errno.h>
-#include <kernel-features.h>
 
 /* New name of process.  */
 #define NEW_NAME "setname"
@@ -101,18 +100,6 @@ do_test (int argc, char **argv)
     {
       res = get_self_comm (gettid (), name_check, TASK_COMM_LEN);
 
-#ifndef __ASSUME_PROC_PID_TASK_COMM
-      /* On this first test we look for ENOENT to be returned from
-         get_self_comm to indicate that the kernel is older than
-         2.6.33 and doesn't contain comm within the proc structure.
-         In that case we skip the entire test.  */
-      if (res == ENOENT)
-	{
-	  printf ("SKIP: The kernel does not have /proc/self/task/%%lu/comm.\n");
-	  return 0;
-	}
-#endif
-
       if (res == 0)
        {
          if (strncmp (name, name_check, strlen (BIG_NAME)) == 0)