diff options
| author | Thomas Gleixner <tglx@linutronix.de> | 2012-05-03 09:02:47 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-08-20 08:26:28 -0700 |
| commit | 0a2436eff9888b8fff8b7a5cf6bae49826615a7e (patch) | |
| tree | e6001f3b0b82738e54aaf7c1f89a128465bcb730 /arch | |
| parent | a42efb79d54d9a13c8f68df122c832bca08b74ae (diff) | |
frv: Use correct size for task_struct allocation
commit cce4517f33384c3794c759e206cc8e1bb6df146b upstream.
alloc_task_struct_node() allocates THREAD_SIZE and maintains some
weird refcount in the allocated memory. This never blew up as
task_struct size on 32bit machines was always less than THREAD_SIZE
Allocate just sizeof(struct task_struct) and get rid of the magic
refcounting.
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Acked-by: David Howells <dhowells@redhat.com>
Link: http://lkml.kernel.org/r/20120503085033.898475542@linutronix.de
Cc: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'arch')
| -rw-r--r-- | arch/frv/kernel/process.c | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/arch/frv/kernel/process.c b/arch/frv/kernel/process.c index 3941cbc91ff6..40630b782b73 100644 --- a/arch/frv/kernel/process.c +++ b/arch/frv/kernel/process.c @@ -46,17 +46,12 @@ EXPORT_SYMBOL(pm_power_off); struct task_struct *alloc_task_struct_node(int node) { - struct task_struct *p = kmalloc_node(THREAD_SIZE, GFP_KERNEL, node); - - if (p) - atomic_set((atomic_t *)(p+1), 1); - return p; + return kmalloc_node(sizeof(task_struct), GFP_KERNEL, node); } void free_task_struct(struct task_struct *p) { - if (atomic_dec_and_test((atomic_t *)(p+1))) - kfree(p); + kfree(p); } static void core_sleep_idle(void) |