)]}'
{
  "log": [
    {
      "commit": "36c8b586896f60cb91a4fd526233190b34316baf",
      "tree": "003246e1e676de33703daa979b3e3109ca202a89",
      "parents": [
        "48f24c4da1ee7f3f22289cb85e8b8a73e4df4db5"
      ],
      "author": {
        "name": "Ingo Molnar",
        "email": "mingo@elte.hu",
        "time": "Mon Jul 03 00:25:41 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Mon Jul 03 15:27:11 2006 -0700"
      },
      "message": "[PATCH] sched: cleanup, remove task_t, convert to struct task_struct\n\ncleanup: remove task_t and convert all the uses to struct task_struct. I\nintroduced it for the scheduler anno and it was a mistake.\n\nConversion was mostly scripted, the result was reviewed and all\nsecondary whitespace and style impact (if any) was fixed up by hand.\n\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "d5f70c00ad24cd1158d3678b44ff969b4c971d49",
      "tree": "b4b71a71b413250fb24cb2d83cbbfd7507da9efc",
      "parents": [
        "281de339ceb822ca6c04d4373ecb9a45c1890ce4"
      ],
      "author": {
        "name": "Oleg Nesterov",
        "email": "oleg@tv-sign.ru",
        "time": "Mon Jun 26 00:26:07 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Mon Jun 26 09:58:27 2006 -0700"
      },
      "message": "[PATCH] coredump: kill ptrace related stuff\n\nWith this patch zap_process() sets SIGNAL_GROUP_EXIT while sending SIGKILL to\nthe thread group.  This means that a TASK_TRACED task\n\n\t1. Will be awakened by signal_wake_up(1)\n\n\t2. Can\u0027t sleep again via ptrace_notify()\n\n\t3. Can\u0027t go to do_signal_stop() after return\n\t   from ptrace_stop() in get_signal_to_deliver()\n\nSo we can remove all ptrace related stuff from coredump path.\n\nSigned-off-by: Oleg Nesterov \u003coleg@tv-sign.ru\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "df26c40e567356caeefe2861311e19c54444d917",
      "tree": "415527677e85e8b612b916f6fda1645a6207d8e2",
      "parents": [
        "778c1144771f0064b6f51bee865cceb0d996f2f9"
      ],
      "author": {
        "name": "Eric W. Biederman",
        "email": "ebiederm@xmission.com",
        "time": "Mon Jun 26 00:25:59 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Mon Jun 26 09:58:26 2006 -0700"
      },
      "message": "[PATCH] proc: Cleanup proc_fd_access_allowed\n\nIn process of getting proc_fd_access_allowed to work it has developed a few\nwarts.  In particular the special case that always allows introspection and\nthe special case to allow inspection of kernel threads.\n\nThe special case for introspection is needed for /proc/self/mem.\n\nThe special case for kernel threads really should be overridable\nby security modules.\n\nSo consolidate these checks into ptrace.c:may_attach().\n\nThe check to always allow introspection is trivial.\n\nThe check to allow access to kernel threads, and zombies is a little\ntrickier.  mem_read and mem_write already verify an mm exists so it isn\u0027t\nneeded twice.  proc_fd_access_allowed only doesn\u0027t want a check to verify\ntask-\u003emm exits, s it prevents all access to kernel threads.  So just move\nthe task-\u003emm check into ptrace_attach where it is needed for practical\nreasons.\n\nI did a quick audit and none of the security modules in the kernel seem to\ncare if they are passed a task without an mm into security_ptrace.  So the\nabove move should be safe and it allows security modules to come up with\nmore restrictive policy.\n\nSigned-off-by: Eric W. Biederman \u003cebiederm@xmission.com\u003e\nCc: Stephen Smalley \u003csds@tycho.nsa.gov\u003e\nCc: Chris Wright \u003cchrisw@sous-sol.org\u003e\nCc: James Morris \u003cjmorris@namei.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "f358166a9405e4f1d8e50d8f415c26d95505b6de",
      "tree": "7f5349d9e8b59633b9f8a7bd9bcd45ace1d21a41",
      "parents": [
        "0e44dc383787b472a7f13564c6bd8a44cc07d408"
      ],
      "author": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Thu May 11 11:08:49 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Thu May 11 11:08:49 2006 -0700"
      },
      "message": "ptrace_attach: fix possible deadlock schenario with irqs\n\nEric Biederman points out that we can\u0027t take the task_lock while holding\ntasklist_lock for writing, because another CPU that holds the task lock\nmight take an interrupt that then tries to take tasklist_lock for writing.\n\nWhich would be a nasty deadlock, with one CPU spinning forever in an\ninterrupt handler (although admittedly you need to really work at\ntriggering it ;)\n\nSince the ptrace_attach() code is special and very unusual, just make it\nbe extra careful, and use trylock+repeat to avoid the possible deadlock.\n\nCc: Oleg Nesterov \u003coleg@tv-sign.ru\u003e\nCc: Eric W. Biederman \u003cebiederm@xmission.com\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "f5b40e363ad6041a96e3da32281d8faa191597b9",
      "tree": "5f35453cff7acd5df5622a108381bbd011a383c5",
      "parents": [
        "5528e568a760442e0ec8fd2dea1f0791875a066b"
      ],
      "author": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Sun May 07 10:49:33 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Sun May 07 10:49:33 2006 -0700"
      },
      "message": "Fix ptrace_attach()/ptrace_traceme()/de_thread() race\n\nThis holds the task lock (and, for ptrace_attach, the tasklist_lock)\nover the actual attach event, which closes a race between attacking to a\nthread that is either doing a PTRACE_TRACEME or getting de-threaded.\n\nThanks to Oleg Nesterov for reminding me about this, and Chris Wright\nfor noticing a lost return value in my first version.\n\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "e57a5059846e55d82b86d96dde40e988598601b3",
      "tree": "85b0f48406b37b46261735cbdd66e379977a0327",
      "parents": [
        "c06511d12d720b23c8dffff23004f0a888698f20"
      ],
      "author": {
        "name": "Roland McGrath",
        "email": "roland@redhat.com",
        "time": "Wed Apr 12 16:30:20 2006 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Fri Apr 14 08:59:13 2006 -0700"
      },
      "message": "[PATCH] fix non-leader exec under ptrace\n\nThis reverts most of commit 30e0fca6c1d7d26f3f2daa4dd2b12c51dadc778a.\nIt broke the case of non-leader MT exec when ptraced.\nI think the bug it was intended to fix was already addressed by commit\n788e05a67c343fa22f2ae1d3ca264e7f15c25eaf.\n\nSigned-off-by: Roland McGrath \u003croland@redhat.com\u003e\nAcked-by: Oleg Nesterov \u003coleg@tv-sign.ru\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "524223ca8142d593124bde66f3ffa1deb6f56c06",
      "tree": "881d27c034459b931d8ba3adc0f1d2c8777fbf76",
      "parents": [
        "9ba025f10885758975fbbc2292a5b9e7cb8026a8"
      ],
      "author": {
        "name": "Eric Sesterhenn",
        "email": "snakebyte@gmx.de",
        "time": "Sun Apr 02 13:43:40 2006 +0200"
      },
      "committer": {
        "name": "Adrian Bunk",
        "email": "bunk@stusta.de",
        "time": "Sun Apr 02 13:43:40 2006 +0200"
      },
      "message": "BUG_ON() Conversion in kernel/ptrace.c\n\nthis changes if() BUG(); constructs to BUG_ON() which is\ncleaner, contains unlikely() and can better optimized away.\n\nSigned-off-by: Eric Sesterhenn \u003csnakebyte@gmx.de\u003e\nSigned-off-by: Adrian Bunk \u003cbunk@stusta.de\u003e\n"
    },
    {
      "commit": "9b678ece42893b53aae5ed7cb8d7cb261cacb72c",
      "tree": "2fc3cdcdc2940127885219b8d4142f8317b46bbd",
      "parents": [
        "8fafabd86f1b75ed3cc6a6ffbe6c3e53e3d8457d"
      ],
      "author": {
        "name": "Oleg Nesterov",
        "email": "oleg@tv-sign.ru",
        "time": "Tue Mar 28 16:11:05 2006 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Tue Mar 28 18:36:41 2006 -0800"
      },
      "message": "[PATCH] don\u0027t use REMOVE_LINKS/SET_LINKS for reparenting\n\nThere are places where kernel uses REMOVE_LINKS/SET_LINKS while changing\nprocess\u0027s -\u003eparent.  Use add_parent/remove_parent instead, they don\u0027t abuse\nof global process list.\n\nSigned-off-by: Oleg Nesterov \u003coleg@tv-sign.ru\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "5ecfbae093f0c37311e89b29bfc0c9d586eace87",
      "tree": "eabd0a145af64e26c900578c95175ab313828661",
      "parents": [
        "dadac81b1b86196fcc48fb87620403c4a7174f06"
      ],
      "author": {
        "name": "Oleg Nesterov",
        "email": "oleg@tv-sign.ru",
        "time": "Wed Feb 15 22:50:10 2006 +0300"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Wed Feb 15 11:05:43 2006 -0800"
      },
      "message": "[PATCH] fix zap_thread\u0027s ptrace related problems\n\n1. The tracee can go from ptrace_stop() to do_signal_stop()\n   after __ptrace_unlink(p).\n\n2. It is unsafe to __ptrace_unlink(p) while p-\u003eparent may wait\n   for tasklist_lock in ptrace_detach().\n\nSigned-off-by: Oleg Nesterov \u003coleg@tv-sign.ru\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nCc: Eric W. Biederman \u003cebiederm@xmission.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "16bf134840da3920ded1290973c56ec214636f12",
      "tree": "1aacddb873bd383030a65e13e78c2ced38edfa23",
      "parents": [
        "d98c7a09843621f1b145ca5ae8ed03ff04085edb"
      ],
      "author": {
        "name": "Hugh Dickins",
        "email": "hugh@veritas.com",
        "time": "Tue Feb 14 13:52:59 2006 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Tue Feb 14 16:09:33 2006 -0800"
      },
      "message": "[PATCH] compound page: no access_process_vm check\n\nThe PageCompound check before access_process_vm\u0027s set_page_dirty_lock is no\nlonger necessary, so remove it.  But leave the PageCompound checks in\nbio_set_pages_dirty, dio_bio_complete and nfs_free_user_pages: at least some\nof those were introduced as a little optimization on hugetlb pages.\n\nSigned-off-by: Hugh Dickins \u003chugh@veritas.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "c59ede7b78db329949d9cdcd7064e22d357560ef",
      "tree": "f9dc9d464fdad5bfd464d983e77c1af031389dda",
      "parents": [
        "e16885c5ad624a6efe1b1bf764e075d75f65a788"
      ],
      "author": {
        "name": "Randy.Dunlap",
        "email": "rdunlap@xenotime.net",
        "time": "Wed Jan 11 12:17:46 2006 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Wed Jan 11 18:42:13 2006 -0800"
      },
      "message": "[PATCH] move capable() to capability.h\n\n- Move capable() from sched.h to capability.h;\n\n- Use \u003clinux/capability.h\u003e where capable() is used\n\t(in include/, block/, ipc/, kernel/, a few drivers/,\n\tmm/, security/, \u0026 sound/;\n\tmany more drivers/ to go)\n\nSigned-off-by: Randy Dunlap \u003crdunlap@xenotime.net\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "6b9c7ed84837753a436415097063232422e29a35",
      "tree": "6ad59a7bebcec359e08b3a211701781db819450d",
      "parents": [
        "6b34350f490b2c8508717541ec1fd2bbaadded94"
      ],
      "author": {
        "name": "Christoph Hellwig",
        "email": "hch@lst.de",
        "time": "Sun Jan 08 01:02:33 2006 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Sun Jan 08 20:13:51 2006 -0800"
      },
      "message": "[PATCH] use ptrace_get_task_struct in various places\n\nThe ptrace_get_task_struct() helper that I added as part of the ptrace\nconsolidation is useful in variety of places that currently opencode it.\nSwitch them to the common helpers.\n\nAdd a ptrace_traceme() helper that needs to be explicitly called, and simplify\nthe ptrace_get_task_struct() interface.  We don\u0027t need the request argument\nnow, and we return the task_struct directly, using ERR_PTR() for error\nreturns.  It\u0027s a bit more code in the callers, but we have two sane routines\nthat do one thing well now.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "5bd0190bf3d7e53043a048e809ffa29d41b9d6ac",
      "tree": "c6fba8d5a532ecfa53dc97a3841d13d972964854",
      "parents": [
        "df69a60dc6afc2936d79054d30b481c1fd9720e5"
      ],
      "author": {
        "name": "David Gibson",
        "email": "david@gibson.dropbear.id.au",
        "time": "Tue Nov 29 19:34:32 2005 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Tue Nov 29 19:47:03 2005 -0800"
      },
      "message": "[PATCH] Fix crash when ptrace poking hugepage areas\n\nset_page_dirty() will not cope with being handed a page * which is part of\na compound page, but not the master page in that compound page.  This case\ncan occur via access_process_vm() if you attemp to write to another\nprocess\u0027s hugepage memory area using ptrace() (causing an oops or hang).\n\nThis patch fixes the bug by only calling set_page_dirty() from\naccess_process_vm() if the page is not a compound page.  We already use a\nsimilar fix in bio_set_pages_dirty() for the case of direct io to\nhugepages.\n\nSigned-off-by: David Gibson \u003cdavid@gibson.dropbear.id.au\u003e\nAcked-by: William Irwin \u003cwli@holomorphy.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "005f18dfd0ed86c39716277b61dfb4bd2af91059",
      "tree": "8425228dc52bb38a5fe85d52afaecacca2f3a6fd",
      "parents": [
        "ab767201881fec073157986c314485ab26caa4a0"
      ],
      "author": {
        "name": "Christoph Hellwig",
        "email": "hch@lst.de",
        "time": "Sun Nov 13 16:06:33 2005 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Sun Nov 13 18:14:11 2005 -0800"
      },
      "message": "[PATCH] fix task_struct leak in ptrace\n\nWhen ptrace_attach fails we need to drop the task_struct reference.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "28d838cc4dfea980cb6eda0a7409cbf91889ca74",
      "tree": "4bbf42063718ac7744ca67ef9f67adce17bfbb5c",
      "parents": [
        "ac111bfaa6b0b3c0edc63c27bd9617d6b08851ff"
      ],
      "author": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Wed Nov 09 11:33:07 2005 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Wed Nov 09 11:33:07 2005 -0800"
      },
      "message": "Fix ptrace self-attach rule\n\nBefore we did CLONE_THREAD, the way to check whether we were attaching\nto ourselves was to just check \"current \u003d\u003d task\", but with CLONE_THREAD\nwe should check that the thread group ID matches instead.\n\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "481bed454247538e9f57d4ea37b153ccba24ba7b",
      "tree": "bb4198296962c08dbf52e8f377dc27206f621640",
      "parents": [
        "db73e9aa99bf093427b79877f9475392724fd5e5"
      ],
      "author": {
        "name": "Christoph Hellwig",
        "email": "hch@lst.de",
        "time": "Mon Nov 07 00:59:47 2005 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Mon Nov 07 07:53:42 2005 -0800"
      },
      "message": "[PATCH] consolidate sys_ptrace()\n\nThe sys_ptrace boilerplate code (everything outside the big switch\nstatement for the arch-specific requests) is shared by most architectures.\nThis patch moves it to kernel/ptrace.c and leaves the arch-specific code as\narch_ptrace.\n\nSome architectures have a too different ptrace so we have to exclude them.\nThey continue to keep their implementations.  For sh64 I had to add a\nsh64_ptrace wrapper because it does some initialization on the first call.\nFor um I removed an ifdefed SUBARCH_PTRACE_SPECIAL block, but\nSUBARCH_PTRACE_SPECIAL isn\u0027t defined anywhere in the tree.\n\nSigned-off-by: Christoph Hellwig \u003chch@lst.de\u003e\nAcked-by: Paul Mackerras \u003cpaulus@samba.org\u003e\nAcked-by: Ralf Baechle \u003cralf@linux-mips.org\u003e\nAcked-By: David Howells \u003cdhowells@redhat.com\u003e\nAcked-by: Russell King \u003crmk+kernel@arm.linux.org.uk\u003e\nAcked-by: Paul Mundt \u003clethal@linux-sh.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "30e0fca6c1d7d26f3f2daa4dd2b12c51dadc778a",
      "tree": "945004584995e3af816125245bbb2436cc8ebc9e",
      "parents": [
        "68860ec10bcc07ab4f89f9d940e3b77ae5ca13b3"
      ],
      "author": {
        "name": "Andrea Arcangeli",
        "email": "andrea@suse.de",
        "time": "Sun Oct 30 15:02:38 2005 -0800"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Sun Oct 30 17:37:22 2005 -0800"
      },
      "message": "[PATCH] ptrace/coredump/exit_group deadlock\n\nI could seldom reproduce a deadlock with a task not killable in T state\n(TASK_STOPPED, not TASK_TRACED) by attaching a NPTL threaded program to\ngdb, by segfaulting the task and triggering a core dump while some other\ntask is executing exit_group and while one task is in ptrace_attached\nTASK_STOPPED state (not TASK_TRACED yet).  This originated from a gdb\nbugreport (the fact gdb was segfaulting the task wasn\u0027t a kernel bug), but\nI just incidentally noticed the gdb bug triggered a real kernel bug as\nwell.\n\nMost threads hangs in exit_mm because the core_dumping is still going, the\ncore dumping hangs because the stopped task doesn\u0027t exit, the stopped task\ncan\u0027t wakeup because it has SIGNAL_GROUP_EXIT set, hence the deadlock.\n\nTo me it seems that the problem is that the force_sig_specific(SIGKILL) in\nzap_threads is a noop if the task has PF_PTRACED set (like in this case\nbecause gdb is attached).  The __ptrace_unlink does nothing because the\nsignal-\u003eflags is set to SIGNAL_GROUP_EXIT|SIGNAL_STOP_DEQUEUED (verified).\n\nThe above info also shows that the stopped task hit a race and got the stop\nsignal (presumably by the ptrace_attach, only the attach, state is still\nTASK_STOPPED and gdb hangs waiting the core before it can set it to\nTASK_TRACED) after one of the thread invoked the core dump (it\u0027s the core\ndump that sets signal-\u003eflags to SIGNAL_GROUP_EXIT).\n\nSo beside the fact nobody would wakeup the task in __ptrace_unlink (the\nstate is _not_ TASK_TRACED), there\u0027s a secondary problem in the signal\nhandling code, where a task should ignore the ptrace-sigstops as long as\nSIGNAL_GROUP_EXIT is set (or the wakeup in __ptrace_unlink path wouldn\u0027t be\nenough).\n\nSo I attempted to make this patch that seems to fix the problem.  There\nwere various ways to fix it, perhaps you prefer a different one, I just\nopted to the one that looked safer to me.\n\nI also removed the clearing of the stopped bits from the zap_other_threads\n(zap_other_threads was safe unlike zap_threads).  I don\u0027t like useless\ncode, this whole NPTL signal/ptrace thing is already unreadable enough and\nfull of corner cases without confusing useless code into it to make it even\nless readable.  And if this code is really needed, then you may want to\nexplain why it\u0027s not being done in the other paths that sets\nSIGNAL_GROUP_EXIT at least.\n\nEven after this patch I still wonder who serializes the read of\np-\u003eptrace in zap_threads.\n\nPatch is called ptrace-core_dump-exit_group-deadlock-1.\n\nThis was the trace I\u0027ve got:\n\ntest          T ffff81003e8118c0     0 14305      1         14311 14309 (NOTLB)\nffff810058ccdde8 0000000000000082 000001f4000037e1 ffff810000000013\n       00000000000000f8 ffff81003e811b00 ffff81003e8118c0 ffff810011362100\n       0000000000000012 ffff810017ca4180\nCall Trace:\u003cffffffff801317ed\u003e{try_to_wake_up+893} \u003cffffffff80141677\u003e{finish_stop+87}\n       \u003cffffffff8014367f\u003e{get_signal_to_deliver+1359} \u003cffffffff8010d3ad\u003e{do_signal+157}\n       \u003cffffffff8013deee\u003e{ptrace_check_attach+222} \u003cffffffff80111575\u003e{sys_ptrace+2293}\n       \u003cffffffff80131810\u003e{default_wake_function+0} \u003cffffffff80196399\u003e{sys_ioctl+73}\n       \u003cffffffff8010dd27\u003e{sysret_signal+28} \u003cffffffff8010e00f\u003e{ptregscall_common+103}\n\ntest          D ffff810011362100     0 14309      1         14305 14312 (NOTLB)\nffff810053c81cf8 0000000000000082 0000000000000286 0000000000000001\n       0000000000000195 ffff810011362340 ffff810011362100 ffff81002e338040\n       ffff810001e0ca80 0000000000000001\nCall Trace:\u003cffffffff801317ed\u003e{try_to_wake_up+893} \u003cffffffff8044677d\u003e{wait_for_completion+173}\n       \u003cffffffff80131810\u003e{default_wake_function+0} \u003cffffffff80137435\u003e{exit_mm+149}\n       \u003cffffffff801381af\u003e{do_exit+479} \u003cffffffff80138d0c\u003e{do_group_exit+252}\n       \u003cffffffff801436db\u003e{get_signal_to_deliver+1451} \u003cffffffff8010d3ad\u003e{do_signal+157}\n       \u003cffffffff8013deee\u003e{ptrace_check_attach+222} \u003cffffffff80140850\u003e{specific_send_sig_info+2\n\n       \u003cffffffff8014208a\u003e{force_sig_info+186} \u003cffffffff804479a0\u003e{do_int3+112}\n       \u003cffffffff8010e308\u003e{retint_signal+61}\ntest          D ffff81002e338040     0 14311      1         14716 14305 (NOTLB)\nffff81005ca8dcf8 0000000000000082 0000000000000286 0000000000000001\n       0000000000000120 ffff81002e338280 ffff81002e338040 ffff8100481cb740\n       ffff810001e0ca80 0000000000000001\nCall Trace:\u003cffffffff801317ed\u003e{try_to_wake_up+893} \u003cffffffff8044677d\u003e{wait_for_completion+173}\n       \u003cffffffff80131810\u003e{default_wake_function+0} \u003cffffffff80137435\u003e{exit_mm+149}\n       \u003cffffffff801381af\u003e{do_exit+479} \u003cffffffff80142d0e\u003e{__dequeue_signal+558}\n       \u003cffffffff80138d0c\u003e{do_group_exit+252} \u003cffffffff801436db\u003e{get_signal_to_deliver+1451}\n       \u003cffffffff8010d3ad\u003e{do_signal+157} \u003cffffffff8013deee\u003e{ptrace_check_attach+222}\n       \u003cffffffff80140850\u003e{specific_send_sig_info+208} \u003cffffffff8014208a\u003e{force_sig_info+186}\n       \u003cffffffff804479a0\u003e{do_int3+112} \u003cffffffff8010e308\u003e{retint_signal+61}\n\ntest          D ffff810017ca4180     0 14312      1         14309 13882 (NOTLB)\nffff81005d15fcb8 0000000000000082 ffff81005d15fc58 ffffffff80130816\n       0000000000000897 ffff810017ca43c0 ffff810017ca4180 ffff81003e8118c0\n       0000000000000082 ffffffff801317ed\nCall Trace:\u003cffffffff80130816\u003e{activate_task+150} \u003cffffffff801317ed\u003e{try_to_wake_up+893}\n       \u003cffffffff8044677d\u003e{wait_for_completion+173} \u003cffffffff80131810\u003e{default_wake_function+0}\n       \u003cffffffff8018cdc3\u003e{do_coredump+819} \u003cffffffff80445f52\u003e{thread_return+82}\n       \u003cffffffff801436d4\u003e{get_signal_to_deliver+1444} \u003cffffffff8010d3ad\u003e{do_signal+157}\n       \u003cffffffff8013deee\u003e{ptrace_check_attach+222} \u003cffffffff80140850\u003e{specific_send_sig_info+2\n\n       \u003cffffffff804472e5\u003e{_spin_unlock_irqrestore+5} \u003cffffffff8014208a\u003e{force_sig_info+186}\n       \u003cffffffff804476ff\u003e{do_general_protection+159} \u003cffffffff8010e308\u003e{retint_signal+61}\n\nSigned-off-by: Andrea Arcangeli \u003candrea@suse.de\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Linus Torvalds \u003ctorvalds@osdl.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "ab8d11beb46f0bd0617e04205c01f5c1fe845b61",
      "tree": "33b5aa5b63268b8f4d1428d74547fd351bb37113",
      "parents": [
        "5e21ccb136047e556acf0fdf227cab5db05c1c25"
      ],
      "author": {
        "name": "Miklos Szeredi",
        "email": "miklos@szeredi.hu",
        "time": "Tue Sep 06 15:18:24 2005 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@g5.osdl.org",
        "time": "Wed Sep 07 16:57:43 2005 -0700"
      },
      "message": "[PATCH] remove duplicated code from proc and ptrace\n\nExtract common code used by ptrace_attach() and may_ptrace_attach()\ninto a separate function.\n\nSigned-off-by: Miklos Szeredi \u003cmiklos@szeredi.hu\u003e\nCc: \u003cviro@parcelfarce.linux.theplanet.co.uk\u003e\nCc: Christoph Hellwig \u003chch@lst.de\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "7ed20e1ad521b5f5df61bf6559ae60738e393741",
      "tree": "90931724e45eaedb3445314e8b94e78253642395",
      "parents": [
        "e5bdd883a189243541e7a132385580703b049102"
      ],
      "author": {
        "name": "Jesper Juhl",
        "email": "juhl-lkml@dif.dk",
        "time": "Sun May 01 08:59:14 2005 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@ppc970.osdl.org",
        "time": "Sun May 01 08:59:14 2005 -0700"
      },
      "message": "[PATCH] convert that currently tests _NSIG directly to use valid_signal()\n\nConvert most of the current code that uses _NSIG directly to instead use\nvalid_signal().  This avoids gcc -W warnings and off-by-one errors.\n\nSigned-off-by: Jesper Juhl \u003cjuhl-lkml@dif.dk\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "d59dd4620fb8d6422555a9e2b82a707718e68327",
      "tree": "78205657d24efcaa3f8314064951ac3814ed95a7",
      "parents": [
        "0d8d4d42f2d00eb65262b49f4edd4cf7ef4eb6fc"
      ],
      "author": {
        "name": "akpm@osdl.org",
        "email": "akpm@osdl.org",
        "time": "Sun May 01 08:58:47 2005 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@ppc970.osdl.org",
        "time": "Sun May 01 08:58:47 2005 -0700"
      },
      "message": "[PATCH] use smp_mb/wmb/rmb where possible\n\nReplace a number of memory barriers with smp_ variants.  This means we won\u0027t\ntake the unnecessary hit on UP machines.\n\nSigned-off-by: Anton Blanchard \u003canton@samba.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@osdl.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@osdl.org\u003e\n"
    },
    {
      "commit": "1da177e4c3f41524e886b7f1b8a0c1fc7321cac2",
      "tree": "0bba044c4ce775e45a88a51686b5d9f90697ea9d",
      "parents": [],
      "author": {
        "name": "Linus Torvalds",
        "email": "torvalds@ppc970.osdl.org",
        "time": "Sat Apr 16 15:20:36 2005 -0700"
      },
      "committer": {
        "name": "Linus Torvalds",
        "email": "torvalds@ppc970.osdl.org",
        "time": "Sat Apr 16 15:20:36 2005 -0700"
      },
      "message": "Linux-2.6.12-rc2\n\nInitial git repository build. I\u0027m not bothering with the full history,\neven though we have it. We can create a separate \"historical\" git\narchive of that later if we want to, and in the meantime it\u0027s about\n3.2GB when imported into git - space that would just make the early\ngit days unnecessarily complicated, when we don\u0027t have a lot of good\ninfrastructure for it.\n\nLet it rip!\n"
    }
  ]
}
