diff options
| author | Mark Rutland <mark.rutland@arm.com> | 2018-07-11 14:56:36 +0100 |
|---|---|---|
| committer | Will Deacon <will.deacon@arm.com> | 2018-07-12 14:40:38 +0100 |
| commit | 3eb6f1f9e6364d5eca1ddf3c9e963c5916069190 (patch) | |
| tree | f1d514c5f95fe2e5b3412a486408657cd2d2342c /arch/arm64/kernel/signal.c | |
| parent | e87a4a92fba3721eb06ba8d061b550e09e3d063a (diff) | |
| download | lwn-3eb6f1f9e6364d5eca1ddf3c9e963c5916069190.tar.gz lwn-3eb6f1f9e6364d5eca1ddf3c9e963c5916069190.zip | |
arm64: consistently use unsigned long for thread flags
In do_notify_resume, we manipulate thread_flags as a 32-bit unsigned
int, whereas thread_info::flags is a 64-bit unsigned long, and elsewhere
(e.g. in the entry assembly) we manipulate the flags as a 64-bit
quantity.
For consistency, and to avoid problems if we end up with more than 32
flags, let's make do_notify_resume take the flags as a 64-bit unsigned
long.
Signed-off-by: Mark Rutland <mark.rutland@arm.com>
Reviewed-by: Dave Martin <dave.martin@arm.com>
Acked-by: Catalin Marinas <catalin.marinas@arm.com>
Cc: Will Deacon <will.deacon@arm.com>
Signed-off-by: Will Deacon <will.deacon@arm.com>
Diffstat (limited to 'arch/arm64/kernel/signal.c')
| -rw-r--r-- | arch/arm64/kernel/signal.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/arm64/kernel/signal.c b/arch/arm64/kernel/signal.c index e3b1d1b0aee8..5a8df96dc4ab 100644 --- a/arch/arm64/kernel/signal.c +++ b/arch/arm64/kernel/signal.c @@ -912,7 +912,7 @@ static void do_signal(struct pt_regs *regs) } asmlinkage void do_notify_resume(struct pt_regs *regs, - unsigned int thread_flags) + unsigned long thread_flags) { /* * The assembly code enters us with IRQs off, but it hasn't |