diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-23 12:04:24 -0400 |
---|---|---|
committer | Eric W. Biederman <ebiederm@xmission.com> | 2019-05-29 10:31:43 -0400 |
commit | 2e1661d2673667d886cd40ad9f414cb6db48d8da (patch) | |
tree | 5a2c32eb8a5c575bfb5c7013f5d9d427f5c06c34 /arch/powerpc/platforms/cell/spufs/fault.c | |
parent | 91ca180dbdd687d45fe4aab055b02d29c91b90df (diff) |
signal: Remove the task parameter from force_sig_fault
As synchronous exceptions really only make sense against the current
task (otherwise how are you synchronous) remove the task parameter
from from force_sig_fault to make it explicit that is what is going
on.
The two known exceptions that deliver a synchronous exception to a
stopped ptraced task have already been changed to
force_sig_fault_to_task.
The callers have been changed with the following emacs regular expression
(with obvious variations on the architectures that take more arguments)
to avoid typos:
force_sig_fault[(]\([^,]+\)[,]\([^,]+\)[,]\([^,]+\)[,]\W+current[)]
->
force_sig_fault(\1,\2,\3)
Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>
Diffstat (limited to 'arch/powerpc/platforms/cell/spufs/fault.c')
-rw-r--r-- | arch/powerpc/platforms/cell/spufs/fault.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/arch/powerpc/platforms/cell/spufs/fault.c b/arch/powerpc/platforms/cell/spufs/fault.c index 971ac43b5d60..6634c0c5ed9e 100644 --- a/arch/powerpc/platforms/cell/spufs/fault.c +++ b/arch/powerpc/platforms/cell/spufs/fault.c | |||
@@ -44,22 +44,21 @@ static void spufs_handle_event(struct spu_context *ctx, | |||
44 | 44 | ||
45 | switch (type) { | 45 | switch (type) { |
46 | case SPE_EVENT_INVALID_DMA: | 46 | case SPE_EVENT_INVALID_DMA: |
47 | force_sig_fault(SIGBUS, BUS_OBJERR, NULL, current); | 47 | force_sig_fault(SIGBUS, BUS_OBJERR, NULL); |
48 | break; | 48 | break; |
49 | case SPE_EVENT_SPE_DATA_STORAGE: | 49 | case SPE_EVENT_SPE_DATA_STORAGE: |
50 | ctx->ops->restart_dma(ctx); | 50 | ctx->ops->restart_dma(ctx); |
51 | force_sig_fault(SIGSEGV, SEGV_ACCERR, (void __user *)ea, | 51 | force_sig_fault(SIGSEGV, SEGV_ACCERR, (void __user *)ea); |
52 | current); | ||
53 | break; | 52 | break; |
54 | case SPE_EVENT_DMA_ALIGNMENT: | 53 | case SPE_EVENT_DMA_ALIGNMENT: |
55 | /* DAR isn't set for an alignment fault :( */ | 54 | /* DAR isn't set for an alignment fault :( */ |
56 | force_sig_fault(SIGBUS, BUS_ADRALN, NULL, current); | 55 | force_sig_fault(SIGBUS, BUS_ADRALN, NULL); |
57 | break; | 56 | break; |
58 | case SPE_EVENT_SPE_ERROR: | 57 | case SPE_EVENT_SPE_ERROR: |
59 | force_sig_fault( | 58 | force_sig_fault( |
60 | SIGILL, ILL_ILLOPC, | 59 | SIGILL, ILL_ILLOPC, |
61 | (void __user *)(unsigned long) | 60 | (void __user *)(unsigned long) |
62 | ctx->ops->npc_read(ctx) - 4, current); | 61 | ctx->ops->npc_read(ctx) - 4); |
63 | break; | 62 | break; |
64 | } | 63 | } |
65 | } | 64 | } |