kernel/signal.c: fix style and typos etc.
PATCH 1/2 kernel/signal.c: fix style and typos etc.
From: Randy Dunlap <[email protected]>
General coding style and comment fixes; no code changes:
- Use multi-line-comment coding style.
- Put some function signatures completely on one line.
- Hyphenate some words.
- Spell Posix as POSIX.
- Correct typos & spellos in some comments.
- Drop trailing whitespace.
- End sentences with periods.
Signed-off-by Randy Dunlap <[email protected]>
--- kernel/signal.c | 90 ++++++++++++++++++++++++---------------------- 1 file changed, 48 insertions(+), 42 deletions(-)
kernel/signal.c +48 -42
@@ -226,7 +226,7 @@ static inline void print_dropped_signal(/* * allocate a new signal queue record * - this may be called without locks if and only if t == current, otherwise an -* appopriate lock must be held to stop the target task from exiting +* appropriate lock must be held to stop the target task from exiting */ static struct sigqueue *__sigqueue_alloc (int sig,struct task_struct *t,gfp_t flags,int override_rlimit)
@@ -375,15 +375,15 @@ int unhandled_signal(struct task_structreturn !tracehook_consider_fatal_signal (tsk, sig); } - -/* Notify the system that a driver wants to block all signals for this +/* +* Notify the system that a driver wants to block all signals for this * process, and wants to be notified if any signals at all were to be * sent/acted upon. If the notifier routine returns non-zero, then the * signal will be acted upon after all. If the notifier routine returns 0, * then then signal will be blocked. Only one block per process is * allowed. priv is a pointer to private data that the notifier routine -* can use to determine if the signal should be blocked or not. */ - +* can use to determine if the signal should be blocked or not. +*/ void block_all_signals (int (*notifier)(void *priv),void *priv,sigset_t *mask) {
@@ -434,9 +434,10 @@ still_pending:copy_siginfo (info, &first->info);__sigqueue_free (first); }else { -/* Ok, it wasn't in the queue. This must be -a fast-pathed signal or we must have been -out of queue space. So zero out the info. +/* +* Ok, it wasn't in the queue. This must be +* a fast-pathed signal or we must have been +* out of queue space. So zero out the info. */ info->si_signo = sig; info->si_errno =0 ;
@@ -468,7 +469,7 @@ static int __dequeue_signal(struct sigpe }/* -* Dequeue a signal and return the element to the caller, which is +* Dequeue a signal and return the element to the caller, which is * expected to free it. * * All callers have to hold the siglock.
@@ -490,7 +491,7 @@ int dequeue_signal(struct task_struct *t* itimers are process shared and we restart periodic * itimers in the signal delivery path to prevent DoS * attacks in the high resolution timer case. This is -* compliant with the old way of self restarting +* compliant with the old way of self-restarting * itimers, as the SIGALRM is a legacy signal and only * queued once. Changing the restart behaviour to * restart the timer in the signal dequeue path is
@@ -909,14 +910,15 @@ static int __send_signal(int sig, structif (info ==SEND_SIG_FORCED ) goto out_set; -/* Real-time signals must be queued if sent by sigqueue, or -some other real-time mechanism. It is implementation -defined whether kill() does so. We attempt to do so, on -the principle of least surprise, but since kill is not -allowed to fail with EAGAIN when low on memory we just -make sure at least one signal gets delivered and don't -pass on the info struct. */ - +/* +* Real-time signals must be queued if sent by sigqueue, or +* some other real-time mechanism. It is implementation +* defined whether kill() does so. We attempt to do so, on +* the principle of least surprise, but since kill is not +* allowed to fail with EAGAIN when low on memory we just +* make sure at least one signal gets delivered and don't +* pass on the info struct. +*/ if (sig <SIGRTMIN ) override_rlimit = (is_si_special (info) || info->si_code >=0 );else
@@ -1187,8 +1189,7 @@ retry:return error; } -int -kill_proc_info (int sig,struct siginfo *info,pid_t pid) +int kill_proc_info (int sig,struct siginfo *info,pid_t pid) {int error;rcu_read_lock ();
@@ -1285,8 +1286,7 @@ static int kill_something_info(int sig,* These are for backward compatibility with the rest of the kernel source. */ -int -send_sig_info (int sig,struct siginfo *info,struct task_struct *p) +int send_sig_info (int sig,struct siginfo *info,struct task_struct *p) {/* * Make sure legacy kernel users don't send in bad values
@@ -1354,7 +1354,7 @@ EXPORT_SYMBOL(kill_pid);* These functions support sending signals using preallocated sigqueue * structures. This is needed "because realtime applications cannot * afford to lose notifications of asynchronous events, like timer -* expirations or I/O completions". In the case of Posix Timers +* expirations or I/O completions". In the case of POSIX Timers * we allocate the sigqueue structure from the timer_create. If this * allocation fails we are able to report the failure to the application * with an EAGAIN error.
@@ -1539,7 +1539,7 @@ static void do_notify_parent_cldstop(str info.si_signo =SIGCHLD ; info.si_errno =0 ;/* -* see comment in do_notify_parent() abot the following 3 lines +* see comment in do_notify_parent() about the following 4 lines */ rcu_read_lock (); info.si_pid =task_pid_nr_ns (tsk, parent->nsproxy->pid_ns);
@@ -1597,7 +1597,7 @@ static inline int may_ptrace_stop(void) }/* -* Return nonzero if there is a SIGKILL that should be waking us up. +* Return non-zero if there is a SIGKILL that should be waking us up. * Called with the siglock held. */ static int sigkill_pending (struct task_struct *tsk)
@@ -1721,7 +1721,7 @@ void ptrace_notify(int exit_code)/* * This performs the stopping for SIGSTOP and other stop signals. * We have to stop all threads in the thread group. -* Returns nonzero if we've actually stopped and released the siglock. +* Returns non-zero if we've actually stopped and released the siglock. * Returns zero if we didn't stop and still hold the siglock. */ static int do_signal_stop (int signr)
@@ -1809,10 +1809,12 @@ static int ptrace_signal(int signr, sigi current->exit_code =0 ; -/* Update the siginfo structure if the signal has -changed. If the debugger wanted something -specific in the siginfo structure then it should -have updated *info via PTRACE_SETSIGINFO. */ +/* +* Update the siginfo structure if the signal has +* changed. If the debugger wanted something +* specific in the siginfo structure then it should +* have updated *info via PTRACE_SETSIGINFO. +*/ if (signr != info->si_signo) { info->si_signo = signr; info->si_errno =0 ;
@@ -2020,7 +2022,8 @@ void exit_signals(struct task_struct *tsif (!signal_pending (tsk)) goto out; -/* It could be that __group_complete_signal() choose us to +/* +* It could be that __group_complete_signal() choose us to * notify about group-wide signal. Another thread should be * woken now to take the signal since we will not. */
@@ -2169,7 +2172,7 @@ long do_sigpending(void __user *set, uns out:return error; -} +} SYSCALL_DEFINE2(rt_sigpending, sigset_t __user *, set, size_t, sigsetsize) {
@@ -2219,9 +2222,9 @@ int copy_siginfo_to_user(siginfo_t __use err |=__put_user (from->si_trapno, &to->si_trapno);#endif #ifdef BUS_MCEERR_AO -/* +/* * Other callers might not initialize the si_lsb field, -* so check explicitely for the right codes here. +* so check explicitly for the right codes here. */ if (from->si_code ==BUS_MCEERR_AR || from->si_code ==BUS_MCEERR_AO ) err |=__put_user (from->si_addr_lsb, &to->si_addr_lsb);
@@ -2266,7 +2269,7 @@ SYSCALL_DEFINE4(rt_sigtimedwait, const sif (copy_from_user (&these, uthese,sizeof (these)))return -EFAULT ; - +/* * Invert the set of allowed signals to get those we * want to block.
@@ -2291,9 +2294,11 @@ SYSCALL_DEFINE4(rt_sigtimedwait, const s + (ts.tv_sec || ts.tv_nsec));if (timeout) { -/* None ready -- temporarily unblock those we're +/* +* None ready -- temporarily unblock those we're * interested while we are sleeping in so that we'll -* be awakened when they arrive. */ +* be awakened when they arrive. +*/ current->real_blocked = current->blocked;sigandsets (¤t->blocked, ¤t->blocked, &these);recalc_sigpending ();
@@ -2531,12 +2536,11 @@ do_sigaltstack (const stack_t __user *us error = -EINVAL ;/* -* -* Note - this code used to test ss_flags incorrectly +* Note - this code used to test ss_flags incorrectly: * old code may have been written using ss_flags==0 * to mean ss_flags==SS_ONSTACK (as this was the only * way that worked) - this fix preserves that older -* mechanism +* mechanism. */ if (ss_flags !=SS_DISABLE && ss_flags !=SS_ONSTACK && ss_flags !=0 ) goto out;
@@ -2578,8 +2582,10 @@ SYSCALL_DEFINE1(sigpending, old_sigset_t#endif #ifdef __ARCH_WANT_SYS_SIGPROCMASK -/* Some platforms have their own version with special arguments others -support only sys_rt_sigprocmask. */ +/* +* Some platforms have their own version with special arguments; +* others support only sys_rt_sigprocmask. +*/ SYSCALL_DEFINE3(sigprocmask, int, how, old_sigset_t __user *, set, old_sigset_t __user *, oset)