The patch titled
     make-good_sigevent-non-static fix
has been added to the -mm tree.  Its filename is
     make-good_sigevent-non-static-fix.patch

*** Remember to use Documentation/SubmitChecklist when testing your code ***

See http://www.zip.com.au/~akpm/linux/patches/stuff/added-to-mm.txt to find
out what to do about this

------------------------------------------------------
Subject: make-good_sigevent-non-static fix
From: Sébastien Dugué <[EMAIL PROTECTED]>

  Could you replace make-good_sigevent-non-static.patch with this
patch, as it fixes some problems reported by Oleg Nesterov.

  Oleg, chime in if you still see something I missed.

  Hopefully I will have gotten it right this time.

  Sorry for the hassle.

  Sébastien.


                Make good_sigevent() non-static and rename it

  Move good_sigevent() from posix-timers.c to signal.c where it belongs,
clean it up, rename it to sigevent_find_task() to better describe what the
function does and make it non-static so that it can be used by other
 subsystems.

Signed-off-by: Sébastien Dugué <[EMAIL PROTECTED]>
Cc: Bharata B Rao <[EMAIL PROTECTED]>
Cc: Christoph Hellwig <[EMAIL PROTECTED]>
Cc: Suparna Bhattacharya <[EMAIL PROTECTED]>
Cc: Zach Brown <[EMAIL PROTECTED]>
Cc: Oleg Nesterov <[EMAIL PROTECTED]>
Cc: Badari Pulavarty <[EMAIL PROTECTED]>
Cc: Benjamin LaHaise <[EMAIL PROTECTED]>
Cc: Jean Pierre Dion <[EMAIL PROTECTED]>
Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
---

 kernel/signal.c |   16 ++++++++++------
 1 files changed, 10 insertions(+), 6 deletions(-)

diff -puN kernel/signal.c~make-good_sigevent-non-static-fix kernel/signal.c
--- a/kernel/signal.c~make-good_sigevent-non-static-fix
+++ a/kernel/signal.c
@@ -1097,18 +1097,22 @@ int group_send_sig_info(int sig, struct 
  */
 struct task_struct * sigevent_find_task(sigevent_t * event)
 {
-       struct task_struct *task = NULL;
+       struct task_struct *task = current->group_leader;
 
-       if (event->sigev_signo <= 0 || event->sigev_signo > SIGRTMAX)
-               return NULL;
+       if ((event->sigev_notify & ~SIGEV_THREAD_ID) != SIGEV_NONE) {
+               if (event->sigev_signo <= 0 || event->sigev_signo > SIGRTMAX)
+                       return NULL;
+       }
+
+       if (event->sigev_notify & SIGEV_THREAD_ID) {
+               if ((event->sigev_notify & ~SIGEV_THREAD_ID) != SIGEV_SIGNAL)
+                       return NULL;
 
-       if ((event->sigev_notify & SIGEV_THREAD_ID ) == SIGEV_THREAD_ID) {
                task = find_task_by_pid(event->sigev_notify_thread_id);
 
                if (!task || task->tgid != current->tgid)
                        task = NULL;
-       } else if (event->sigev_notify == SIGEV_SIGNAL)
-               task = current->group_leader;
+       }
 
        return task;
 }
_

Patches currently in -mm which might be from [EMAIL PROTECTED] are

rework-compat_sys_io_submit.patch
fix-aioh-includes.patch
fix-access_ok-checks.patch
make-good_sigevent-non-static.patch
make-good_sigevent-non-static-fix.patch
make-__sigqueue_free-and.patch
aio-completion-signal-notification.patch
add-listio-syscall-support.patch

-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to