Commit 48cad41f authored by Michael Ellerman's avatar Michael Ellerman Committed by Paul Mackerras
Browse files

[POWERPC] spufs: Combine spufs_coredump_calls with spufs_calls 



Because spufs might be built as a module, we can't have other parts of the
kernel calling directly into it, we need stub routines that check first if the
module is loaded.

Currently we have two structures which hold callbacks for these stubs, the
syscalls are in spufs_calls and the coredump calls are in spufs_coredump_calls.
In both cases the logic for registering/unregistering is essentially the same,
so we can simplify things by combining the two.

Signed-off-by: default avatarMichael Ellerman <michael@ellerman.id.au>
Signed-off-by: default avatarJeremy Kerr <jk@ozlabs.org>
Acked-by: default avatarArnd Bergmann <arnd.bergmann@de.ibm.com>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent 78810ff6

arch/powerpc/platforms/cell/Makefile

+1 −1
Original line number Diff line number Diff line
@@ -19,7 +19,7 @@ spu-manage-$(CONFIG_PPC_CELLEB) += spu_manage.o 
spu-manage-$(CONFIG_PPC_CELL_NATIVE)	+= spu_manage.o



obj-$(CONFIG_SPU_BASE)			+= spu_callbacks.o spu_base.o \

					   spu_coredump.o spu_syscalls.o \

					   spu_syscalls.o \

					   $(spu-priv1-y) \

					   $(spu-manage-y) \

					   spufs/

arch/powerpc/platforms/cell/spu_coredump.c

deleted100644 → 0
+0 −83
Original line number Diff line number Diff line 
/*

 * SPU core dump code

 *

 * (C) Copyright 2006 IBM Corp.

 *

 * Author: Dwayne Grant McConnell <decimal@us.ibm.com>

 *

 * This program is free software; you can redistribute it and/or modify

 * it under the terms of the GNU General Public License as published by

 * the Free Software Foundation; either version 2, or (at your option)

 * any later version.

 *

 * This program is distributed in the hope that it will be useful,

 * but WITHOUT ANY WARRANTY; without even the implied warranty of

 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.	See the

 * GNU General Public License for more details.

 *

 * You should have received a copy of the GNU General Public License

 * along with this program; if not, write to the Free Software

 * Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.

 */



#include <linux/file.h>

#include <linux/module.h>

#include <linux/syscalls.h>



#include <asm/spu.h>



static struct spu_coredump_calls *spu_coredump_calls;

static DEFINE_MUTEX(spu_coredump_mutex);



int arch_notes_size(void)

{

	int ret;



	mutex_lock(&spu_coredump_mutex);



	if (spu_coredump_calls && try_module_get(spu_coredump_calls->owner)) {

		ret = spu_coredump_calls->arch_notes_size();

		module_put(spu_coredump_calls->owner);

	} else {

		ret = 0;

	}



	mutex_unlock(&spu_coredump_mutex);



	return ret;

}



void arch_write_notes(struct file *file)

{

	mutex_lock(&spu_coredump_mutex);

	if (spu_coredump_calls && try_module_get(spu_coredump_calls->owner)) {

		spu_coredump_calls->arch_write_notes(file);

		module_put(spu_coredump_calls->owner);

	}

	mutex_unlock(&spu_coredump_mutex);

}



int register_arch_coredump_calls(struct spu_coredump_calls *calls)

{

	int ret = 0;





	mutex_lock(&spu_coredump_mutex);

	if (spu_coredump_calls)

		ret = -EBUSY;

	else

		spu_coredump_calls = calls;

	mutex_unlock(&spu_coredump_mutex);

	return ret;

}

EXPORT_SYMBOL_GPL(register_arch_coredump_calls);



void unregister_arch_coredump_calls(struct spu_coredump_calls *calls)

{

	BUG_ON(spu_coredump_calls != calls);



	mutex_lock(&spu_coredump_mutex);

	spu_coredump_calls = NULL;

	mutex_unlock(&spu_coredump_mutex);

}

EXPORT_SYMBOL_GPL(unregister_arch_coredump_calls);

arch/powerpc/platforms/cell/spu_syscalls.c

+30 −0
Original line number Diff line number Diff line
@@ -2,6 +2,7 @@ 
 * SPU file system -- system call stubs

 *

 * (C) Copyright IBM Deutschland Entwicklung GmbH 2005

 * (C) Copyright 2006-2007, IBM Corporation

 *

 * Author: Arnd Bergmann <arndb@de.ibm.com>

 *
@@ -111,6 +112,35 @@ asmlinkage long sys_spu_run(int fd, __u32 __user *unpc, __u32 __user *ustatus) 
	return ret;

}



int arch_notes_size(void)

{

	struct spufs_calls *calls;

	int ret;



	calls = spufs_calls_get();

	if (!calls)

		return 0;



	ret = calls->coredump_extra_notes_size();



	spufs_calls_put(calls);



	return ret;

}



void arch_write_notes(struct file *file)

{

	struct spufs_calls *calls;



	calls = spufs_calls_get();

	if (!calls)

		return;



	calls->coredump_extra_notes_write(file);



	spufs_calls_put(calls);

}



int register_spu_syscalls(struct spufs_calls *calls)

{

	if (spufs_calls)

arch/powerpc/platforms/cell/spufs/coredump.c

+2 −8
Original line number Diff line number Diff line
@@ -122,7 +122,7 @@ static struct spu_context *coredump_next_context(int *fd) 
	return ctx;

}



static int spufs_arch_notes_size(void)

int spufs_coredump_extra_notes_size(void)

{

	struct spu_context *ctx;

	int size = 0, rc, fd;
@@ -185,7 +185,7 @@ out: 
	free_page((unsigned long)buf);

}



static void spufs_arch_write_notes(struct file *file)

void spufs_coredump_extra_notes_write(struct file *file)

{

	struct spu_context *ctx;

	int fd, j;
@@ -200,9 +200,3 @@ static void spufs_arch_write_notes(struct file *file) 
		spu_release_saved(ctx);

	}

}
 


struct spu_coredump_calls spufs_coredump_calls = {

	.arch_notes_size = spufs_arch_notes_size,

	.arch_write_notes = spufs_arch_write_notes,

	.owner = THIS_MODULE,

};

arch/powerpc/platforms/cell/spufs/inode.c

+0 −6
Original line number Diff line number Diff line
@@ -790,16 +790,11 @@ static int __init spufs_init(void) 
	ret = register_spu_syscalls(&spufs_calls);

	if (ret)

		goto out_fs;

	ret = register_arch_coredump_calls(&spufs_coredump_calls);

	if (ret)

		goto out_syscalls;



	spufs_init_isolated_loader();



	return 0;



out_syscalls:

	unregister_spu_syscalls(&spufs_calls);

out_fs:

	unregister_filesystem(&spufs_type);

out_sched:
@@ -815,7 +810,6 @@ static void __exit spufs_exit(void) 
{

	spu_sched_exit();

	spufs_exit_isolated_loader();

	unregister_arch_coredump_calls(&spufs_coredump_calls);

	unregister_spu_syscalls(&spufs_calls);

	unregister_filesystem(&spufs_type);

	kmem_cache_destroy(spufs_inode_cache);

CRA Git | Maintained and supported by SUSTech CRA and CCSE