Commit 967747bb authored by Arnd Bergmann's avatar Arnd Bergmann
Browse files

uaccess: remove CONFIG_SET_FS 



There are no remaining callers of set_fs(), so CONFIG_SET_FS
can be removed globally, along with the thread_info field and
any references to it.

This turns access_ok() into a cheaper check against TASK_SIZE_MAX.

As CONFIG_SET_FS is now gone, drop all remaining references to
set_fs()/get_fs(), mm_segment_t, user_addr_max() and uaccess_kernel().

Acked-by: Sam Ravnborg <sam@ravnborg.org> # for sparc32 changes
Acked-by: default avatar"Eric W. Biederman" <ebiederm@xmission.com>
Tested-by: Sergey Matyukevich <sergey.matyukevich@synopsys.com> # for arc changes
Acked-by: Stafford Horne <shorne@gmail.com> # [openrisc, asm-generic]
Acked-by: default avatarDinh Nguyen <dinguyen@kernel.org>
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
parent e5ef21d1

arch/Kconfig

+0 −3
Original line number Diff line number Diff line
@@ -24,9 +24,6 @@ config KEXEC_ELF 
config HAVE_IMA_KEXEC

	bool



config SET_FS

	bool



config HOTPLUG_SMT

	bool

arch/alpha/Kconfig

+0 −1
Original line number Diff line number Diff line
@@ -35,7 +35,6 @@ config ALPHA 
	select OLD_SIGSUSPEND

	select CPU_NO_EFFICIENT_FFS if !ALPHA_EV67

	select MMU_GATHER_NO_RANGE

	select SET_FS

	select SPARSEMEM_EXTREME if SPARSEMEM

	select ZONE_DMA

	help

arch/alpha/include/asm/processor.h

+0 −4
Original line number Diff line number Diff line
@@ -26,10 +26,6 @@ 
#define TASK_UNMAPPED_BASE \

  ((current->personality & ADDR_LIMIT_32BIT) ? 0x40000000 : TASK_SIZE / 2)



typedef struct {

	unsigned long seg;

} mm_segment_t;



/* This is dead.  Everything has been moved to thread_info.  */

struct thread_struct { };

#define INIT_THREAD  { }

arch/alpha/include/asm/thread_info.h

+0 −2
Original line number Diff line number Diff line
@@ -19,7 +19,6 @@ struct thread_info { 
	unsigned int		flags;		/* low level flags */

	unsigned int		ieee_state;	/* see fpu.h */



	mm_segment_t		addr_limit;	/* thread address space */

	unsigned		cpu;		/* current CPU */

	int			preempt_count; /* 0 => preemptable, <0 => BUG */

	unsigned int		status;		/* thread-synchronous flags */
@@ -35,7 +34,6 @@ struct thread_info { 
#define INIT_THREAD_INFO(tsk)			\

{						\

	.task		= &tsk,			\

	.addr_limit	= KERNEL_DS,		\

	.preempt_count	= INIT_PREEMPT_COUNT,	\

}

arch/alpha/include/asm/uaccess.h

+0 −19
Original line number Diff line number Diff line
@@ -2,26 +2,7 @@ 
#ifndef __ALPHA_UACCESS_H

#define __ALPHA_UACCESS_H



/*

 * The fs value determines whether argument validity checking should be

 * performed or not.  If get_fs() == USER_DS, checking is performed, with

 * get_fs() == KERNEL_DS, checking is bypassed.

 *

 * Or at least it did once upon a time.  Nowadays it is a mask that

 * defines which bits of the address space are off limits.  This is a

 * wee bit faster than the above.

 *

 * For historical reasons, these macros are grossly misnamed.

 */



#define KERNEL_DS	((mm_segment_t) { 0UL })

#define USER_DS		((mm_segment_t) { -0x40000000000UL })



#define get_fs()  (current_thread_info()->addr_limit)

#define set_fs(x) (current_thread_info()->addr_limit = (x))



#include <asm-generic/access_ok.h>



/*

 * These are the main single-value transfer routines.  They automatically

 * use the right size if we just have the right pointer type.