)]}' { "log": [ { "commit": "80d26af89a7249aa5475467000322163c60cdd72", "tree": "e611dcf9d6962c81186ede5e94570a730accc222", "parents": [ "6aa9707099c4b25700940eb3d016f16c4434360d" ], "author": { "name": "Mandeep Singh Baines", "email": "msb@chromium.org", "time": "Wed Feb 27 17:03:20 2013 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Feb 27 19:10:11 2013 -0800" }, "message": "coredump: use a freezable_schedule for the coredump_finish wait\n\nPrevents hung_task detector from panicing the machine. This is also\nneeded to prevent this wait from blocking suspend.\n\n(It doesnt\u0027 currently block suspend but it would once the next\npatch in this series is applied.)\n\n[yongjun_wei@trendmicro.com.cn: kernel/exit.c: remove duplicated include]\nSigned-off-by: Mandeep Singh Baines \u003cmsb@chromium.org\u003e\nCc: Ben Chan \u003cbenchan@chromium.org\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Rafael J. Wysocki \u003crjw@sisk.pl\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nSigned-off-by: Wei Yongjun \u003cyongjun_wei@trendmicro.com.cn\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6aa9707099c4b25700940eb3d016f16c4434360d", "tree": "6996f5789a3c0a09eea2fe57d318c1d387c24f3a", "parents": [ "c2c1b089b44b783bd50fae4bccaa6f367f92e492" ], "author": { "name": "Mandeep Singh Baines", "email": "msb@chromium.org", "time": "Wed Feb 27 17:03:18 2013 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Feb 27 19:10:11 2013 -0800" }, "message": "lockdep: check that no locks held at freeze time\n\nWe shouldn\u0027t try_to_freeze if locks are held. Holding a lock can cause a\ndeadlock if the lock is later acquired in the suspend or hibernate path\n(e.g. by dpm). Holding a lock can also cause a deadlock in the case of\ncgroup_freezer if a lock is held inside a frozen cgroup that is later\nacquired by a process outside that group.\n\n[akpm@linux-foundation.org: export debug_check_no_locks_held]\nSigned-off-by: Mandeep Singh Baines \u003cmsb@chromium.org\u003e\nCc: Ben Chan \u003cbenchan@chromium.org\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Rafael J. Wysocki \u003crjw@sisk.pl\u003e\nCc: Ingo Molnar \u003cmingo@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6fac4829ce0ef9b7f24369086ce5f0e9f38d37bc", "tree": "44da712b66d778c2fd701f9c96b00e108530e1a9", "parents": [ "3f4724ea85b7d9055a9976fa8f30b471bdfbca93" ], "author": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Tue Nov 13 14:20:55 2012 +0100" }, "committer": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Sun Jan 27 19:23:31 2013 +0100" }, "message": "cputime: Use accessors to read task cputime stats\n\nThis is in preparation for the full dynticks feature. While\nremotely reading the cputime of a task running in a full\ndynticks CPU, we\u0027ll need to do some extra-computation. This\nway we can account the time it spent tickless in userspace\nsince its last cputime snapshot.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nCc: Ingo Molnar \u003cmingo@kernel.org\u003e\nCc: Li Zhong \u003czhong@linux.vnet.ibm.com\u003e\nCc: Namhyung Kim \u003cnamhyung.kim@lge.com\u003e\nCc: Paul E. McKenney \u003cpaulmck@linux.vnet.ibm.com\u003e\nCc: Paul Gortmaker \u003cpaul.gortmaker@windriver.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Thomas Gleixner \u003ctglx@linutronix.de\u003e\n" }, { "commit": "6a2b60b17b3e48a418695a94bd2420f6ab32e519", "tree": "54b7792fa68b8890f710fa6398b6ba8626a039a8", "parents": [ "9228ff90387e276ad67b10c0eb525c9d6a57d5e9", "98f842e675f96ffac96e6c50315790912b2812be" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 17 15:44:47 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Dec 17 15:44:47 2012 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm/user-namespace\n\nPull user namespace changes from Eric Biederman:\n \"While small this set of changes is very significant with respect to\n containers in general and user namespaces in particular. The user\n space interface is now complete.\n\n This set of changes adds support for unprivileged users to create user\n namespaces and as a user namespace root to create other namespaces.\n The tyranny of supporting suid root preventing unprivileged users from\n using cool new kernel features is broken.\n\n This set of changes completes the work on setns, adding support for\n the pid, user, mount namespaces.\n\n This set of changes includes a bunch of basic pid namespace\n cleanups/simplifications. Of particular significance is the rework of\n the pid namespace cleanup so it no longer requires sending out\n tendrils into all kinds of unexpected cleanup paths for operation. At\n least one case of broken error handling is fixed by this cleanup.\n\n The files under /proc/\u003cpid\u003e/ns/ have been converted from regular files\n to magic symlinks which prevents incorrect caching by the VFS,\n ensuring the files always refer to the namespace the process is\n currently using and ensuring that the ptrace_mayaccess permission\n checks are always applied.\n\n The files under /proc/\u003cpid\u003e/ns/ have been given stable inode numbers\n so it is now possible to see if different processes share the same\n namespaces.\n\n Through the David Miller\u0027s net tree are changes to relax many of the\n permission checks in the networking stack to allowing the user\n namespace root to usefully use the networking stack. Similar changes\n for the mount namespace and the pid namespace are coming through my\n tree.\n\n Two small changes to add user namespace support were commited here adn\n in David Miller\u0027s -net tree so that I could complete the work on the\n /proc/\u003cpid\u003e/ns/ files in this tree.\n\n Work remains to make it safe to build user namespaces and 9p, afs,\n ceph, cifs, coda, gfs2, ncpfs, nfs, nfsd, ocfs2, and xfs so the\n Kconfig guard remains in place preventing that user namespaces from\n being built when any of those filesystems are enabled.\n\n Future design work remains to allow root users outside of the initial\n user namespace to mount more than just /proc and /sys.\"\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/ebiederm/user-namespace: (38 commits)\n proc: Usable inode numbers for the namespace file descriptors.\n proc: Fix the namespace inode permission checks.\n proc: Generalize proc inode allocation\n userns: Allow unprivilged mounts of proc and sysfs\n userns: For /proc/self/{uid,gid}_map derive the lower userns from the struct file\n procfs: Print task uids and gids in the userns that opened the proc file\n userns: Implement unshare of the user namespace\n userns: Implent proc namespace operations\n userns: Kill task_user_ns\n userns: Make create_new_namespaces take a user_ns parameter\n userns: Allow unprivileged use of setns.\n userns: Allow unprivileged users to create new namespaces\n userns: Allow setting a userns mapping to your current uid.\n userns: Allow chown and setgid preservation\n userns: Allow unprivileged users to create user namespaces.\n userns: Ignore suid and sgid on binaries if the uid or gid can not be mapped\n userns: fix return value on mntns_install() failure\n vfs: Allow unprivileged manipulation of the mount namespace.\n vfs: Only support slave subtrees across different user namespaces\n vfs: Add a user namespace reference from struct mnt_namespace\n ...\n" }, { "commit": "9977d9b379cb77e0f67bd6f4563618106e58e11d", "tree": "0191accfddf578edb52c69c933d64521e3dce297", "parents": [ "cf4af01221579a4e895f43dbfc47598fbfc5a731", "541880d9a2c7871f6370071d55aa6662d329c51e" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Dec 12 12:22:13 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Dec 12 12:22:13 2012 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/signal\n\nPull big execve/kernel_thread/fork unification series from Al Viro:\n \"All architectures are converted to new model. Quite a bit of that\n stuff is actually shared with architecture trees; in such cases it\u0027s\n literally shared branch pulled by both, not a cherry-pick.\n\n A lot of ugliness and black magic is gone (-3KLoC total in this one):\n\n - kernel_thread()/kernel_execve()/sys_execve() redesign.\n\n We don\u0027t do syscalls from kernel anymore for either kernel_thread()\n or kernel_execve():\n\n kernel_thread() is essentially clone(2) with callback run before we\n return to userland, the callbacks either never return or do\n successful do_execve() before returning.\n\n kernel_execve() is a wrapper for do_execve() - it doesn\u0027t need to\n do transition to user mode anymore.\n\n As a result kernel_thread() and kernel_execve() are\n arch-independent now - they live in kernel/fork.c and fs/exec.c\n resp. sys_execve() is also in fs/exec.c and it\u0027s completely\n architecture-independent.\n\n - daemonize() is gone, along with its parts in fs/*.c\n\n - struct pt_regs * is no longer passed to do_fork/copy_process/\n copy_thread/do_execve/search_binary_handler/-\u003eload_binary/do_coredump.\n\n - sys_fork()/sys_vfork()/sys_clone() unified; some architectures\n still need wrappers (ones with callee-saved registers not saved in\n pt_regs on syscall entry), but the main part of those suckers is in\n kernel/fork.c now.\"\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/signal: (113 commits)\n do_coredump(): get rid of pt_regs argument\n print_fatal_signal(): get rid of pt_regs argument\n ptrace_signal(): get rid of unused arguments\n get rid of ptrace_signal_deliver() arguments\n new helper: signal_pt_regs()\n unify default ptrace_signal_deliver\n flagday: kill pt_regs argument of do_fork()\n death to idle_regs()\n don\u0027t pass regs to copy_process()\n flagday: don\u0027t pass regs to copy_thread()\n bfin: switch to generic vfork, get rid of pointless wrappers\n xtensa: switch to generic clone()\n openrisc: switch to use of generic fork and clone\n unicore32: switch to generic clone(2)\n score: switch to generic fork/vfork/clone\n c6x: sanitize copy_thread(), get rid of clone(2) wrapper, switch to generic clone()\n take sys_fork/sys_vfork/sys_clone prototypes to linux/syscalls.h\n mn10300: switch to generic fork/vfork/clone\n h8300: switch to generic fork/vfork/clone\n tile: switch to generic clone()\n ...\n\nConflicts:\n\tarch/microblaze/include/asm/Kbuild\n" }, { "commit": "c4144670fd9b34d6eae22c9f83751745898e8243", "tree": "da0bbda5707fafdc1d79eb2c851835c777182e89", "parents": [ "02232f8d2b22708f0651dc515544f4a7ef1e0224" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Oct 02 16:34:38 2012 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Nov 28 21:49:02 2012 -0500" }, "message": "kill daemonize()\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "e80d0a1ae8bb8fee0edd37427836f108b30f596b", "tree": "437db8b1643e39d3d114a7c0e3c913455e693b70", "parents": [ "a634f93335daa8f38180a0e576ccd68a73c36eaf" ], "author": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Wed Nov 21 16:26:44 2012 +0100" }, "committer": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Wed Nov 28 17:07:57 2012 +0100" }, "message": "cputime: Rename thread_group_times to thread_group_cputime_adjusted\n\nWe have thread_group_cputime() and thread_group_times(). The naming\ndoesn\u0027t provide enough information about the difference between\nthese two APIs.\n\nTo lower the confusion, rename thread_group_times() to\nthread_group_cputime_adjusted(). This name better suggests that\nit\u0027s a version of thread_group_cputime() that does some stabilization\non the raw cputime values. ie here: scale on top of CFS runtime\nstats and bound lower value for monotonicity.\n\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Ingo Molnar \u003cmingo@kernel.org\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nCc: Steven Rostedt \u003crostedt@goodmis.org\u003e\nCc: Paul Gortmaker \u003cpaul.gortmaker@windriver.com\u003e\n" }, { "commit": "af4b8a83add95ef40716401395b44a1b579965f4", "tree": "2f3f606b7327f74c1c1beb8a75886318c51c838a", "parents": [ "5e1182deb81ae8c68494017c4a8a71811659c870" ], "author": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Wed Aug 01 15:03:42 2012 -0700" }, "committer": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Mon Nov 19 05:59:12 2012 -0800" }, "message": "pidns: Wait in zap_pid_ns_processes until pid_ns-\u003enr_hashed \u003d\u003d 1\n\nLooking at pid_ns-\u003enr_hashed is a bit simpler and it works for\ndisjoint process trees that an unshare or a join of a pid_namespace\nmay create.\n\nAcked-by: \"Serge E. Hallyn\" \u003cserge@hallyn.com\u003e\nSigned-off-by: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\n" }, { "commit": "aab174f0df5d72d31caccf281af5f614fa254578", "tree": "2a172c5009c4ac8755e858593154c258ce7709a0", "parents": [ "ca41cc96b2813221b05af57d0355157924de5a07", "2bd2c1941f141ad780135ccc1cd08ca71a24f10a" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Oct 02 20:25:04 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Oct 02 20:25:04 2012 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs\n\nPull vfs update from Al Viro:\n\n - big one - consolidation of descriptor-related logics; almost all of\n that is moved to fs/file.c\n\n (BTW, I\u0027m seriously tempted to rename the result to fd.c. As it is,\n we have a situation when file_table.c is about handling of struct\n file and file.c is about handling of descriptor tables; the reasons\n are historical - file_table.c used to be about a static array of\n struct file we used to have way back).\n\n A lot of stray ends got cleaned up and converted to saner primitives,\n disgusting mess in android/binder.c is still disgusting, but at least\n doesn\u0027t poke so much in descriptor table guts anymore. A bunch of\n relatively minor races got fixed in process, plus an ext4 struct file\n leak.\n\n - related thing - fget_light() partially unuglified; see fdget() in\n there (and yes, it generates the code as good as we used to have).\n\n - also related - bits of Cyrill\u0027s procfs stuff that got entangled into\n that work; _not_ all of it, just the initial move to fs/proc/fd.c and\n switch of fdinfo to seq_file.\n\n - Alex\u0027s fs/coredump.c spiltoff - the same story, had been easier to\n take that commit than mess with conflicts. The rest is a separate\n pile, this was just a mechanical code movement.\n\n - a few misc patches all over the place. Not all for this cycle,\n there\u0027ll be more (and quite a few currently sit in akpm\u0027s tree).\"\n\nFix up trivial conflicts in the android binder driver, and some fairly\nsimple conflicts due to two different changes to the sock_alloc_file()\ninterface (\"take descriptor handling from sock_alloc_file() to callers\"\nvs \"net: Providing protocol type via system.sockprotoname xattr of\n/proc/PID/fd entries\" adding a dentry name to the socket)\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs: (72 commits)\n MAX_LFS_FILESIZE should be a loff_t\n compat: fs: Generic compat_sys_sendfile implementation\n fs: push rcu_barrier() from deactivate_locked_super() to filesystems\n btrfs: reada_extent doesn\u0027t need kref for refcount\n coredump: move core dump functionality into its own file\n coredump: prevent double-free on an error path in core dumper\n usb/gadget: fix misannotations\n fcntl: fix misannotations\n ceph: don\u0027t abuse d_delete() on failure exits\n hypfs: -\u003ed_parent is never NULL or negative\n vfs: delete surplus inode NULL check\n switch simple cases of fget_light to fdget\n new helpers: fdget()/fdput()\n switch o2hb_region_dev_write() to fget_light()\n proc_map_files_readdir(): don\u0027t bother with grabbing files\n make get_file() return its argument\n vhost_set_vring(): turn pollstart/pollstop into bool\n switch prctl_set_mm_exe_file() to fget_light()\n switch xfs_find_handle() to fget_light()\n switch xfs_swapext() to fget_light()\n ...\n" }, { "commit": "864bdb3b6cbd9911222543fef1cfe36f88183f44", "tree": "bd00e6293592a084ee0fe68e3acb93060f54ae68", "parents": [ "2be7fd55d44882c9818ed2d1dabc0f0abab5eeed" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 22 18:42:10 2012 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Sep 26 21:10:00 2012 -0400" }, "message": "new helper: daemonize_descriptors()\n\ndescriptor-related parts of daemonize, done right. As the\nresult we simplify the locking rules for -\u003efiles - we\nhold task_lock in *all* cases when we modify -\u003efiles.\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "7cf4dc3c8dbfdfde163d4636f621cf99a1f63bfb", "tree": "66ae2653acb5c5325cc4fc921bba53b6e6760983", "parents": [ "dcfadfa4ec5a12404a99ad6426871a6b03a62b37" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Aug 15 19:56:12 2012 -0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Sep 26 21:08:54 2012 -0400" }, "message": "move files_struct-related bits from kernel/exit.c to fs/file.c\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "5640f7685831e088fe6c2e1f863a6805962f8e81", "tree": "fb7660173338a45c27d610eb59ba20cf5c2b91b8", "parents": [ "b98b8babd6e3370fadb7c6eaacb00eb2f6344a6c" ], "author": { "name": "Eric Dumazet", "email": "edumazet@google.com", "time": "Sun Sep 23 23:04:42 2012 +0000" }, "committer": { "name": "David S. Miller", "email": "davem@davemloft.net", "time": "Mon Sep 24 16:31:37 2012 -0400" }, "message": "net: use a per task frag allocator\n\nWe currently use a per socket order-0 page cache for tcp_sendmsg()\noperations.\n\nThis page is used to build fragments for skbs.\n\nIts done to increase probability of coalescing small write() into\nsingle segments in skbs still in write queue (not yet sent)\n\nBut it wastes a lot of memory for applications handling many mostly\nidle sockets, since each socket holds one page in sk-\u003esk_sndmsg_page\n\nIts also quite inefficient to build TSO 64KB packets, because we need\nabout 16 pages per skb on arches where PAGE_SIZE \u003d 4096, so we hit\npage allocator more than wanted.\n\nThis patch adds a per task frag allocator and uses bigger pages,\nif available. An automatic fallback is done in case of memory pressure.\n\n(up to 32768 bytes per frag, thats order-3 pages on x86)\n\nThis increases TCP stream performance by 20% on loopback device,\nbut also benefits on other network devices, since 8x less frags are\nmapped on transmit and unmapped on tx completion. Alexander Duyck\nmentioned a probable performance win on systems with IOMMU enabled.\n\nIts possible some SG enabled hardware cant cope with bigger fragments,\nbut their ndo_start_xmit() should already handle this, splitting a\nfragment in sub fragments, since some arches have PAGE_SIZE\u003d65536\n\nSuccessfully tested on various ethernet devices.\n(ixgbe, igb, bnx2x, tg3, mellanox mlx4)\n\nSigned-off-by: Eric Dumazet \u003cedumazet@google.com\u003e\nCc: Ben Hutchings \u003cbhutchings@solarflare.com\u003e\nCc: Vijay Subramanian \u003csubramanian.vijay@gmail.com\u003e\nCc: Alexander Duyck \u003calexander.h.duyck@intel.com\u003e\nTested-by: Vijay Subramanian \u003csubramanian.vijay@gmail.com\u003e\nSigned-off-by: David S. Miller \u003cdavem@davemloft.net\u003e\n" }, { "commit": "8ded2bbc1845e19c771eb55209aab166ef011243", "tree": "9ab400cdd407a9426fdfcc3432d7c0e19a0607ed", "parents": [ "4cb38750d49010ae72e718d46605ac9ba5a851b4" ], "author": { "name": "Josh Boyer", "email": "jwboyer@redhat.com", "time": "Wed Jul 25 10:40:34 2012 -0400" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jul 26 13:36:43 2012 -0700" }, "message": "posix_types.h: Cleanup stale __NFDBITS and related definitions\n\nRecently, glibc made a change to suppress sign-conversion warnings in\nFD_SET (glibc commit ceb9e56b3d1). This uncovered an issue with the\nkernel\u0027s definition of __NFDBITS if applications #include\n\u003clinux/types.h\u003e after including \u003csys/select.h\u003e. A build failure would\nbe seen when passing the -Werror\u003dsign-compare and -D_FORTIFY_SOURCE\u003d2\nflags to gcc.\n\nIt was suggested that the kernel should either match the glibc\ndefinition of __NFDBITS or remove that entirely. The current in-kernel\nuses of __NFDBITS can be replaced with BITS_PER_LONG, and there are no\nuses of the related __FDELT and __FDMASK defines. Given that, we\u0027ll\ncontinue the cleanup that was started with commit 8b3d1cda4f5f\n(\"posix_types: Remove fd_set macros\") and drop the remaining unused\nmacros.\n\nAdditionally, linux/time.h has similar macros defined that expand to\nnothing so we\u0027ll remove those at the same time.\n\nReported-by: Jeff Law \u003claw@redhat.com\u003e\nSuggested-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nCC: \u003cstable@vger.kernel.org\u003e\nSigned-off-by: Josh Boyer \u003cjwboyer@redhat.com\u003e\n[ .. and fix up whitespace as per akpm ]\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "ed3e694d78cc75fa79bf29698631b146fd27aa35", "tree": "97dad6768546b489a3d09c42cef0738b57f4822e", "parents": [ "67d1214551e800f9fe7dc7c47a346d2df0fafed5" ], "author": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed Jun 27 11:31:24 2012 +0400" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Sun Jul 22 23:57:57 2012 +0400" }, "message": "move exit_task_work() past exit_files() et.al.\n\n... and get rid of PF_EXITING check in task_work_add().\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "50d75f8daead8a1f850c40a3b6c6575ab19b48cf", "tree": "1917eec9deadded31869372207772d9a3d3ca11a", "parents": [ "6347e90091041e34bea625370794c92f4ce71228" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 20 12:53:04 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 20 14:39:36 2012 -0700" }, "message": "pidns: find_new_reaper() can no longer switch to init_pid_ns.child_reaper\n\nfind_new_reaper() changes pid_ns-\u003echild_reaper, see add0d4df (\"pid_ns:\nzap_pid_ns_processes: fix the -\u003echild_reaper changing\").\n\nThe original reason has gone away after the previous patch, -\u003echildren\nlist must be empty after zap_pid_ns_processes().\n\nHowever now we can not switch to init_pid_ns.child_reaper.\n__unhash_process() relies on the \"-\u003echild_reaper \u003d\u003d parent\" check, but\nthis check does not work if the last exiting task is also the child\nreaper.\n\nAs Eric sugested, we can change __unhash_process() to use the parent\u0027s\npid_ns and remove this code.\n\nAlso, with this change we can move detach_pid(PIDTYPE_PID) back, where it\nwas before the previous fix.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nCc: Louis Rilling \u003clouis.rilling@kerlabs.com\u003e\nCc: Mike Galbraith \u003cefault@gmx.de\u003e\nAcked-by: Pavel Emelyanov \u003cxemul@parallels.com\u003e\nTested-by: Andrew Wagin \u003cavagin@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "6347e90091041e34bea625370794c92f4ce71228", "tree": "20614d4eedd2993248be3c3562177c7e00f24b68", "parents": [ "f39cdaebb89dc3e6dd4f3e75b6d4e87ef12190af" ], "author": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Wed Jun 20 12:53:03 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 20 14:39:36 2012 -0700" }, "message": "pidns: guarantee that the pidns init will be the last pidns process reaped\n\nToday we have a twofold bug. Sometimes release_task on pid \u003d\u003d 1 in a pid\nnamespace can run before other processes in a pid namespace have had\nrelease task called. With the result that pid_ns_release_proc can be\ncalled before the last proc_flus_task() is done using upid-\u003ens-\u003eproc_mnt,\nresulting in the use of a stale pointer. This same set of circumstances\ncan lead to waitpid(...) returning for a processes started with\nclone(CLONE_NEWPID) before the every process in the pid namespace has\nactually exited.\n\nTo fix this modify zap_pid_ns_processess wait until all other processes in\nthe pid namespace have exited, even EXIT_DEAD zombies.\n\nThe delay_group_leader and related tests ensure that the thread gruop\nleader will be the last thread of a process group to be reaped, or to\nbecome EXIT_DEAD and self reap. With the change to zap_pid_ns_processes\nwe get the guarantee that pid \u003d\u003d 1 in a pid namespace will be the last\ntask that release_task is called on.\n\nWith pid \u003d\u003d 1 being the last task to pass through release_task\npid_ns_release_proc can no longer be called too early nor can wait return\nbefore all of the EXIT_DEAD tasks in a pid namespace have exited.\n\nSigned-off-by: Eric W. Biederman \u003cebiederm@xmission.com\u003e\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Louis Rilling \u003clouis.rilling@kerlabs.com\u003e\nCc: Mike Galbraith \u003cefault@gmx.de\u003e\nAcked-by: Pavel Emelyanov \u003cxemul@parallels.com\u003e\nTested-by: Andrew Wagin \u003cavagin@gmail.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4fe7efdbdfb1c7e7a7f31decfd831c0f31d37091", "tree": "5d8fdf353ba3791ef10031ade2d93aa87aa179d3", "parents": [ "e0897d75f0b22e8c3a7287a48548c5686ef73447" ], "author": { "name": "Konstantin Khlebnikov", "email": "khlebnikov@openvz.org", "time": "Wed Jun 20 12:53:01 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 20 14:39:36 2012 -0700" }, "message": "mm: correctly synchronize rss-counters at exit/exec\n\ndo_exit() and exec_mmap() call sync_mm_rss() before mm_release() does\nput_user(clear_child_tid) which can update task-\u003erss_stat and thus make\nmm-\u003erss_stat inconsistent. This triggers the \"BUG:\" printk in check_mm().\n\nLet\u0027s fix this bug in the safest way, and optimize/cleanup this later.\n\nReported-by: Markus Trippelsdorf \u003cmarkus@trippelsdorf.de\u003e\nSigned-off-by: Konstantin Khlebnikov \u003ckhlebnikov@openvz.org\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nCc: Hugh Dickins \u003chughd@google.com\u003e\nCc: \u003cstable@vger.kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "48d212a2eecaca2e1875925837ad27b2f43f48a3", "tree": "fbd83c7b15856971f6d7297e1d23dc862ba2989c", "parents": [ "46edaedaf3842164281d0f86c41bc152f4b4d32e" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 07 17:54:07 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 07 17:54:07 2012 -0700" }, "message": "Revert \"mm: correctly synchronize rss-counters at exit/exec\"\n\nThis reverts commit 40af1bbdca47e5c8a2044039bb78ca8fd8b20f94.\n\nIt\u0027s horribly and utterly broken for at least the following reasons:\n\n - calling sync_mm_rss() from mmput() is fundamentally wrong, because\n there\u0027s absolutely no reason to believe that the task that does the\n mmput() always does it on its own VM. Example: fork, ptrace, /proc -\n you name it.\n\n - calling it *after* having done mmdrop() on it is doubly insane, since\n the mm struct may well be gone now.\n\n - testing mm against NULL before you call it is insane too, since a\nNULL mm there would have caused oopses long before.\n\n.. and those are just the three bugs I found before I decided to give up\nlooking for me and revert it asap. I should have caught it before I\neven took it, but I trusted Andrew too much.\n\nCc: Konstantin Khlebnikov \u003ckhlebnikov@openvz.org\u003e\nCc: Markus Trippelsdorf \u003cmarkus@trippelsdorf.de\u003e\nCc: Hugh Dickins \u003chughd@google.com\u003e\nCc: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "40af1bbdca47e5c8a2044039bb78ca8fd8b20f94", "tree": "70b5a21579f8b0137c09e54d523246d84be93430", "parents": [ "39caa0916ef27cf1da5026eb708a2b8413156f75" ], "author": { "name": "Konstantin Khlebnikov", "email": "khlebnikov@openvz.org", "time": "Thu Jun 07 14:21:14 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jun 07 14:43:55 2012 -0700" }, "message": "mm: correctly synchronize rss-counters at exit/exec\n\nmm-\u003erss_stat counters have per-task delta: task-\u003erss_stat. Before\nchanging task-\u003emm pointer the kernel must flush this delta with\nsync_mm_rss().\n\ndo_exit() already calls sync_mm_rss() to flush the rss-counters before\ncommitting the rss statistics into task-\u003esignal-\u003emaxrss, taskstats,\naudit and other stuff. Unfortunately the kernel does this before\ncalling mm_release(), which can call put_user() for processing\ntask-\u003eclear_child_tid. So at this point we can trigger page-faults and\ntask-\u003erss_stat becomes non-zero again. As a result mm-\u003erss_stat becomes\ninconsistent and check_mm() will print something like this:\n\n| BUG: Bad rss-counter state mm:ffff88020813c380 idx:1 val:-1\n| BUG: Bad rss-counter state mm:ffff88020813c380 idx:2 val:1\n\nThis patch moves sync_mm_rss() into mm_release(), and moves mm_release()\nout of do_exit() and calls it earlier. After mm_release() there should\nbe no pagefaults.\n\n[akpm@linux-foundation.org: tweak comment]\nSigned-off-by: Konstantin Khlebnikov \u003ckhlebnikov@openvz.org\u003e\nReported-by: Markus Trippelsdorf \u003cmarkus@trippelsdorf.de\u003e\nCc: Hugh Dickins \u003chughd@google.com\u003e\nCc: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: \u003cstable@vger.kernel.org\u003e\t\t[3.4.x]\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "fb21affa49204acd409328415b49bfe90136653c", "tree": "3535dbe0c0aad049a38cadfcffe78409397a1b32", "parents": [ "a00b6151a2ae4c52576c35d3998e144a993d50b8", "f23ca335462e3c84f13270b9e65f83936068ec2c" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 31 18:47:30 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 31 18:47:30 2012 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/signal\n\nPull second pile of signal handling patches from Al Viro:\n \"This one is just task_work_add() series + remaining prereqs for it.\n\n There probably will be another pull request from that tree this\n cycle - at least for helpers, to get them out of the way for per-arch\n fixes remaining in the tree.\"\n\nFix trivial conflict in kernel/irq/manage.c: the merge of Andrew\u0027s pile\nhad brought in commit 97fd75b7b8e0 (\"kernel/irq/manage.c: use the\npr_foo() infrastructure to prefix printks\") which changed one of the\npr_err() calls that this merge moves around.\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/signal:\n keys: kill task_struct-\u003ereplacement_session_keyring\n keys: kill the dummy key_replace_session_keyring()\n keys: change keyctl_session_to_parent() to use task_work_add()\n genirq: reimplement exit_irq_thread() hook via task_work_add()\n task_work_add: generic process-context callbacks\n avr32: missed _TIF_NOTIFY_RESUME on one of do_notify_resume callers\n parisc: need to check NOTIFY_RESUME when exiting from syscall\n move key_repace_session_keyring() into tracehook_notify_resume()\n TIF_NOTIFY_RESUME is defined on all targets now\n" }, { "commit": "168eeccbc956d2ec083c3a513f7706784ee0dc5f", "tree": "6fb95d3080fadd96ea37b0e8c1bd747a183f8f60", "parents": [ "43e13cc107cf6cd3c15fbe1cef849435c2223d50" ], "author": { "name": "Tim Bird", "email": "tim.bird@am.sony.com", "time": "Thu May 31 16:26:16 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 31 17:49:28 2012 -0700" }, "message": "stack usage: add pid to warning printk in check_stack_usage\n\nIn embedded systems, sometimes the same program (busybox) is the cause of\nmultiple warnings. Outputting the pid with the program name in the\nwarning printk helps distinguish which instances of a program are using\nthe stack most.\n\nThis is a small patch, but useful.\n\nSigned-off-by: Tim Bird \u003ctim.bird@am.sony.com\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Andrea Arcangeli \u003caarcange@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "43e13cc107cf6cd3c15fbe1cef849435c2223d50", "tree": "9a1b3aa79ec2e8374944c23cf57a73790a3fc6ce", "parents": [ "9b3c98cd663750c33434572ff76ba306505eba5a" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu May 31 16:26:16 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 31 17:49:28 2012 -0700" }, "message": "cred: remove task_is_dead() from __task_cred() validation\n\nCommit 8f92054e7ca1 (\"CRED: Fix __task_cred()\u0027s lockdep check and banner\ncomment\"):\n\n add the following validation condition:\n\n task-\u003eexit_state \u003e\u003d 0\n\n to permit the access if the target task is dead and therefore\n unable to change its own credentials.\n\nOK, but afaics currently this can only help wait_task_zombie() which calls\n__task_cred() without rcu lock.\n\nRemove this validation and change wait_task_zombie() to use task_uid()\ninstead. This means we do rcu_read_lock() only to shut up the lockdep,\nbut we already do the same in, say, wait_task_stopped().\n\ntask_is_dead() should die, task-\u003eexit_state !\u003d 0 means that this task has\npassed exit_notify(), only do_wait-like code paths should use this.\n\nUnfortunately, we can\u0027t kill task_is_dead() right now, it has already\nacquired buggy users in drivers/staging. The fix already exists.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nAcked-by: David Howells \u003cdhowells@redhat.com\u003e\nCc: Jiri Olsa \u003cjolsa@redhat.com\u003e\nCc: Paul E. McKenney \u003cpaulmck@linux.vnet.ibm.com\u003e\nCc: James Morris \u003cjmorris@namei.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4d1d61a6b203d957777d73fcebf19d90b038b5b2", "tree": "8b597fa00168ee59e3abf5ec8cf4e92f8fd28d8a", "parents": [ "e73f8959af0439d114847eab5a8a5ce48f1217c4" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Fri May 11 10:59:08 2012 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed May 23 22:11:12 2012 -0400" }, "message": "genirq: reimplement exit_irq_thread() hook via task_work_add()\n\nexit_irq_thread() and task-\u003eirq_thread are needed to handle the unexpected\n(and unlikely) exit of irq-thread.\n\nWe can use task_work instead and make this all private to\nkernel/irq/manage.c, cleanup plus micro-optimization.\n\n1. rename exit_irq_thread() to irq_thread_dtor(), make it\n static, and move it up before irq_thread().\n\n2. change irq_thread() to do task_work_add(irq_thread_dtor)\n at the start and task_work_cancel() before return.\n\n tracehook_notify_resume() can never play with kthreads,\n only do_exit()-\u003eexit_task_work() can call the callback\n and this is what we want.\n\n3. remove task_struct-\u003eirq_thread and the special hook\n in do_exit().\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nCc: David Howells \u003cdhowells@redhat.com\u003e\nCc: Richard Kuo \u003crkuo@codeaurora.org\u003e\nCc: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nCc: Alexander Gordeev \u003cagordeev@redhat.com\u003e\nCc: Chris Zankel \u003cchris@zankel.net\u003e\nCc: David Smith \u003cdsmith@redhat.com\u003e\nCc: \"Frank Ch. Eigler\" \u003cfche@redhat.com\u003e\nCc: Geert Uytterhoeven \u003cgeert@linux-m68k.org\u003e\nCc: Larry Woodman \u003clwoodman@redhat.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "e73f8959af0439d114847eab5a8a5ce48f1217c4", "tree": "47f056093590a5e5552e3a75f163e1f798063bda", "parents": [ "62366c88b29c5a32e1531142092f98eaf49b1103" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Fri May 11 10:59:07 2012 +1000" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Wed May 23 22:09:21 2012 -0400" }, "message": "task_work_add: generic process-context callbacks\n\nProvide a simple mechanism that allows running code in the (nonatomic)\ncontext of the arbitrary task.\n\nThe caller does task_work_add(task, task_work) and this task executes\ntask_work-\u003efunc() either from do_notify_resume() or from do_exit(). The\ncallback can rely on PF_EXITING to detect the latter case.\n\n\"struct task_work\" can be embedded in another struct, still it has \"void\n*data\" to handle the most common/simple case.\n\nThis allows us to kill the -\u003ereplacement_session_keyring hack, and\npotentially this can have more users.\n\nPerformance-wise, this adds 2 \"unlikely(!hlist_empty())\" checks into\ntracehook_notify_resume() and do_exit(). But at the same time we can\nremove the \"replacement_session_keyring !\u003d NULL\" checks from\narch/*/signal.c and exit_creds().\n\nNote: task_work_add/task_work_run abuses -\u003epi_lock. This is only because\nthis lock is already used by lookup_pi_state() to synchronize with\ndo_exit() setting PF_EXITING. Fortunately the scope of this lock in\ntask_work.c is really tiny, and the code is unlikely anyway.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: David Howells \u003cdhowells@redhat.com\u003e\nCc: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nCc: Richard Kuo \u003crkuo@codeaurora.org\u003e\nCc: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nCc: Alexander Gordeev \u003cagordeev@redhat.com\u003e\nCc: Chris Zankel \u003cchris@zankel.net\u003e\nCc: David Smith \u003cdsmith@redhat.com\u003e\nCc: \"Frank Ch. Eigler\" \u003cfche@redhat.com\u003e\nCc: Geert Uytterhoeven \u003cgeert@linux-m68k.org\u003e\nCc: Larry Woodman \u003clwoodman@redhat.com\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "8ca937a668d4be0b0c89c4fc9a912a5885ac06fe", "tree": "085d17219a0bbb7bd17eb60aab028256b2b5948d", "parents": [ "b38a86eb196d9402299919456fe3f28e490c76fa" ], "author": { "name": "Sasha Levin", "email": "levinsasha928@gmail.com", "time": "Thu May 17 23:31:39 2012 +0200" }, "committer": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Thu May 17 16:50:06 2012 -0600" }, "message": "cred: use correct cred accessor with regards to rcu read lock\n\nCommit \"userns: Convert setting and getting uid and gid system calls to use\nkuid and kgid has modified the accessors in wait_task_continued() and\nwait_task_stopped() to use __task_cred() instead of task_uid().\n\n__task_cred() assumes that we\u0027re inside a rcu read lock, which is untrue\nfor these two functions.\n\nModify it to use task_uid() instead.\n\nSigned-off-by: Sasha Levin \u003clevinsasha928@gmail.com\u003e\nSigned-off-by: Eric W. Biederman \u003cebiederm@xmission.com\u003e\n" }, { "commit": "a29c33f4e506e1dae7e0985b6328046535becbf8", "tree": "529720b8aefaf115b4be197e5d7628657cf5b4bc", "parents": [ "18815a18085364d8514c0d0c4c986776cb74272c" ], "author": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Tue Feb 07 18:51:01 2012 -0800" }, "committer": { "name": "Eric W. Biederman", "email": "ebiederm@xmission.com", "time": "Thu May 03 03:28:41 2012 -0700" }, "message": "userns: Convert setting and getting uid and gid system calls to use kuid and kgid\n\nConvert setregid, setgid, setreuid, setuid,\nsetresuid, getresuid, setresgid, getresgid, setfsuid, setfsgid,\ngetuid, geteuid, getgid, getegid,\nwaitpid, waitid, wait4.\n\nConvert userspace uids and gids into kuids and kgids before\nbeing placed on struct cred. Convert struct cred kuids and\nkgids into userspace uids and gids when returning them.\n\nSigned-off-by: Eric W. Biederman \u003cebiederm@xmission.com\u003e\n" }, { "commit": "a591afc01d9e48affbacb365558a31e53c85af45", "tree": "9bb91f4eb94ec69fc4706c4944788ec5f3586063", "parents": [ "820d41cf0cd0e94a5661e093821e2e5c6b36a9d8", "31796ac4e8f0e88f5c10f1ad6dab8f19bebe44a4" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 29 18:12:23 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 29 18:12:23 2012 -0700" }, "message": "Merge branch \u0027x86-x32-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip\n\nPull x32 support for x86-64 from Ingo Molnar:\n \"This tree introduces the X32 binary format and execution mode for x86:\n 32-bit data space binaries using 64-bit instructions and 64-bit kernel\n syscalls.\n\n This allows applications whose working set fits into a 32 bits address\n space to make use of 64-bit instructions while using a 32-bit address\n space with shorter pointers, more compressed data structures, etc.\"\n\nFix up trivial context conflicts in arch/x86/{Kconfig,vdso/vma.c}\n\n* \u0027x86-x32-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: (71 commits)\n x32: Fix alignment fail in struct compat_siginfo\n x32: Fix stupid ia32/x32 inversion in the siginfo format\n x32: Add ptrace for x32\n x32: Switch to a 64-bit clock_t\n x32: Provide separate is_ia32_task() and is_x32_task() predicates\n x86, mtrr: Use explicit sizing and padding for the 64-bit ioctls\n x86/x32: Fix the binutils auto-detect\n x32: Warn and disable rather than error if binutils too old\n x32: Only clear TIF_X32 flag once\n x32: Make sure TS_COMPAT is cleared for x32 tasks\n fs: Remove missed -\u003efds_bits from cessation use of fd_set structs internally\n fs: Fix close_on_exec pointer in alloc_fdtable\n x32: Drop non-__vdso weak symbols from the x32 VDSO\n x32: Fix coding style violations in the x32 VDSO code\n x32: Add x32 VDSO support\n x32: Allow x32 to be configured\n x32: If configured, add x32 system calls to system call tables\n x32: Handle process creation\n x32: Signal-related system calls\n x86: Add #ifdef CONFIG_COMPAT to \u003casm/sys_ia32.h\u003e\n ...\n" }, { "commit": "397a21f24d455982a8a6f9bc11b5f3326ce3c6ef", "tree": "ba92e73491599341db615f719df97cea6fdfaf6b", "parents": [ "ebec18a6d3aa1e7d84aab16225e87fd25170ec2b" ], "author": { "name": "Denys Vlasenko", "email": "vda.linux@googlemail.com", "time": "Fri Mar 23 15:01:54 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 23 16:58:32 2012 -0700" }, "message": "kernel/exit.c: if init dies, log a signal which killed it, if any\n\nI just received another user\u0027s pleas for help when their init\nmysteriously died. I again explained that they need to check whether it\ndied because of bad instruction, a segv, or something else. Which was\nan annoying detour into writing a trivial C program to spawn his init\nand print its exit code:\n\n http://lists.busybox.net/pipermail/busybox/2012-January/077172.html\n\nI hear you saying \"just test it under /bin/sh\". Well, the crashing init\n_was_ /bin/sh.\n\nWhich prompted me to make kernel do this first step automatically. We can\nprint exit code, which makes it possible to see that death was from e.g.\nSIGILL without writing test programs.\n\n[akpm@linux-foundation.org: add 0x to hex number output]\nSigned-off-by: Denys Vlasenko \u003cvda.linux@googlemail.com\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "ebec18a6d3aa1e7d84aab16225e87fd25170ec2b", "tree": "fd329dcd5173c252fc7aed64cab9c2a51575dcec", "parents": [ "953326cb60c1dff1bd3458d6468d16d75f2bcd61" ], "author": { "name": "Lennart Poettering", "email": "lennart@poettering.net", "time": "Fri Mar 23 15:01:54 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Mar 23 16:58:32 2012 -0700" }, "message": "prctl: add PR_{SET,GET}_CHILD_SUBREAPER to allow simple process supervision\n\nUserspace service managers/supervisors need to track their started\nservices. Many services daemonize by double-forking and get implicitly\nre-parented to PID 1. The service manager will no longer be able to\nreceive the SIGCHLD signals for them, and is no longer in charge of\nreaping the children with wait(). All information about the children is\nlost at the moment PID 1 cleans up the re-parented processes.\n\nWith this prctl, a service manager process can mark itself as a sort of\n\u0027sub-init\u0027, able to stay as the parent for all orphaned processes\ncreated by the started services. All SIGCHLD signals will be delivered\nto the service manager.\n\nReceiving SIGCHLD and doing wait() is in cases of a service-manager much\npreferred over any possible asynchronous notification about specific\nPIDs, because the service manager has full access to the child process\ndata in /proc and the PID can not be re-used until the wait(), the\nservice-manager itself is in charge of, has happened.\n\nAs a side effect, the relevant parent PID information does not get lost\nby a double-fork, which results in a more elaborate process tree and\n\u0027ps\u0027 output:\n\nbefore:\n # ps afx\n 253 ? Ss 0:00 /bin/dbus-daemon --system --nofork\n 294 ? Sl 0:00 /usr/libexec/polkit-1/polkitd\n 328 ? S 0:00 /usr/sbin/modem-manager\n 608 ? Sl 0:00 /usr/libexec/colord\n 658 ? Sl 0:00 /usr/libexec/upowerd\n 819 ? Sl 0:00 /usr/libexec/imsettings-daemon\n 916 ? Sl 0:00 /usr/libexec/udisks-daemon\n 917 ? S 0:00 \\_ udisks-daemon: not polling any devices\n\nafter:\n # ps afx\n 294 ? Ss 0:00 /bin/dbus-daemon --system --nofork\n 426 ? Sl 0:00 \\_ /usr/libexec/polkit-1/polkitd\n 449 ? S 0:00 \\_ /usr/sbin/modem-manager\n 635 ? Sl 0:00 \\_ /usr/libexec/colord\n 705 ? Sl 0:00 \\_ /usr/libexec/upowerd\n 959 ? Sl 0:00 \\_ /usr/libexec/udisks-daemon\n 960 ? S 0:00 | \\_ udisks-daemon: not polling any devices\n 977 ? Sl 0:00 \\_ /usr/libexec/packagekitd\n\nThis prctl is orthogonal to PID namespaces. PID namespaces are isolated\nfrom each other, while a service management process usually requires the\nservices to live in the same namespace, to be able to talk to each\nother.\n\nUsers of this will be the systemd per-user instance, which provides\ninit-like functionality for the user\u0027s login session and D-Bus, which\nactivates bus services on-demand. Both need init-like capabilities to\nbe able to properly keep track of the services they start.\n\nMany thanks to Oleg for several rounds of review and insights.\n\n[akpm@linux-foundation.org: fix comment layout and spelling]\n[akpm@linux-foundation.org: add lengthy code comment from Oleg]\nReviewed-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Lennart Poettering \u003clennart@poettering.net\u003e\nSigned-off-by: Kay Sievers \u003ckay.sievers@vrfy.org\u003e\nAcked-by: Valdis Kletnieks \u003cValdis.Kletnieks@vt.edu\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "95211279c5ad00a317c98221d7e4365e02f20836", "tree": "2ddc8625378d2915b8c96392f3cf6663b705ed55", "parents": [ "5375871d432ae9fc581014ac117b96aaee3cd0c7", "12724850e8064f64b6223d26d78c0597c742c65a" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 22 09:04:48 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Mar 22 09:04:48 2012 -0700" }, "message": "Merge branch \u0027akpm\u0027 (Andrew\u0027s patch-bomb)\n\nMerge first batch of patches from Andrew Morton:\n \"A few misc things and all the MM queue\"\n\n* emailed from Andrew Morton \u003cakpm@linux-foundation.org\u003e: (92 commits)\n memcg: avoid THP split in task migration\n thp: add HPAGE_PMD_* definitions for !CONFIG_TRANSPARENT_HUGEPAGE\n memcg: clean up existing move charge code\n mm/memcontrol.c: remove unnecessary \u0027break\u0027 in mem_cgroup_read()\n mm/memcontrol.c: remove redundant BUG_ON() in mem_cgroup_usage_unregister_event()\n mm/memcontrol.c: s/stealed/stolen/\n memcg: fix performance of mem_cgroup_begin_update_page_stat()\n memcg: remove PCG_FILE_MAPPED\n memcg: use new logic for page stat accounting\n memcg: remove PCG_MOVE_LOCK flag from page_cgroup\n memcg: simplify move_account() check\n memcg: remove EXPORT_SYMBOL(mem_cgroup_update_page_stat)\n memcg: kill dead prev_priority stubs\n memcg: remove PCG_CACHE page_cgroup flag\n memcg: let css_get_next() rely upon rcu_read_lock()\n cgroup: revert ss_id_lock to spinlock\n idr: make idr_get_next() good for rcu_read_lock()\n memcg: remove unnecessary thp check in page stat accounting\n memcg: remove redundant returns\n memcg: enum lru_list lru\n ...\n" }, { "commit": "05af2e104a0c282dcd9303431e1360750ba76de6", "tree": "cdd5876f2d17b26cc3ded7ef85d04d0e853e9b7e", "parents": [ "90481622d75715bfcb68501280a917dbfe516029" ], "author": { "name": "David Rientjes", "email": "rientjes@google.com", "time": "Wed Mar 21 16:34:13 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 17:54:59 2012 -0700" }, "message": "mm, counters: remove task argument to sync_mm_rss() and __sync_task_rss_stat()\n\nsync_mm_rss() can only be used for current to avoid race conditions in\niterating and clearing its per-task counters. Remove the task argument\nfor it and its helper function, __sync_task_rss_stat(), to avoid thinking\nit can be used safely for anything other than current.\n\nSigned-off-by: David Rientjes \u003crientjes@google.com\u003e\nAcked-by: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3556485f1595e3964ba539e39ea682acbb835cee", "tree": "7f5ee254f425b1427ac0059b5f347a307f8538a1", "parents": [ "b8716614a7cc2fc15ea2a518edd04755fb08d922", "09f61cdbb32a9d812c618d3922db533542736bb0" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 13:25:04 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 13:25:04 2012 -0700" }, "message": "Merge branch \u0027next\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security\n\nPull security subsystem updates for 3.4 from James Morris:\n \"The main addition here is the new Yama security module from Kees Cook,\n which was discussed at the Linux Security Summit last year. Its\n purpose is to collect miscellaneous DAC security enhancements in one\n place. This also marks a departure in policy for LSM modules, which\n were previously limited to being standalone access control systems.\n Chromium OS is using Yama, and I believe there are plans for Ubuntu,\n at least.\n\n This patchset also includes maintenance updates for AppArmor, TOMOYO\n and others.\"\n\nFix trivial conflict in \u003cnet/sock.h\u003e due to the jumo_label-\u003estatic_key\nrename.\n\n* \u0027next\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security: (38 commits)\n AppArmor: Fix location of const qualifier on generated string tables\n TOMOYO: Return error if fails to delete a domain\n AppArmor: add const qualifiers to string arrays\n AppArmor: Add ability to load extended policy\n TOMOYO: Return appropriate value to poll().\n AppArmor: Move path failure information into aa_get_name and rename\n AppArmor: Update dfa matching routines.\n AppArmor: Minor cleanup of d_namespace_path to consolidate error handling\n AppArmor: Retrieve the dentry_path for error reporting when path lookup fails\n AppArmor: Add const qualifiers to generated string tables\n AppArmor: Fix oops in policy unpack auditing\n AppArmor: Fix error returned when a path lookup is disconnected\n KEYS: testing wrong bit for KEY_FLAG_REVOKED\n TOMOYO: Fix mount flags checking order.\n security: fix ima kconfig warning\n AppArmor: Fix the error case for chroot relative path name lookup\n AppArmor: fix mapping of META_READ to audit and quiet flags\n AppArmor: Fix underflow in xindex calculation\n AppArmor: Fix dropping of allowed operations that are force audited\n AppArmor: Add mising end of structure test to caps unpacking\n ...\n" }, { "commit": "c7c66c0cb0c77b1a8edf09bca57d922312d58030", "tree": "77277103c5f16aa4dee64978a060933d92e14776", "parents": [ "9f3938346a5c1fa504647670edb5fea5756cfb00", "98e8bdafeb4728a6af7bbcbcc3984967d1cf2bc1" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 10:15:51 2012 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Mar 21 10:15:51 2012 -0700" }, "message": "Merge tag \u0027pm-for-3.4\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm\n\nPull power management updates for 3.4 from Rafael Wysocki:\n \"Assorted extensions and fixes including:\n\n * Introduction of early/late suspend/hibernation device callbacks.\n * Generic PM domains extensions and fixes.\n * devfreq updates from Axel Lin and MyungJoo Ham.\n * Device PM QoS updates.\n * Fixes of concurrency problems with wakeup sources.\n * System suspend and hibernation fixes.\"\n\n* tag \u0027pm-for-3.4\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm: (43 commits)\n PM / Domains: Check domain status during hibernation restore of devices\n PM / devfreq: add relation of recommended frequency.\n PM / shmobile: Make MTU2 driver use pm_genpd_dev_always_on()\n PM / shmobile: Make CMT driver use pm_genpd_dev_always_on()\n PM / shmobile: Make TMU driver use pm_genpd_dev_always_on()\n PM / Domains: Introduce \"always on\" device flag\n PM / Domains: Fix hibernation restore of devices, v2\n PM / Domains: Fix handling of wakeup devices during system resume\n sh_mmcif / PM: Use PM QoS latency constraint\n tmio_mmc / PM: Use PM QoS latency constraint\n PM / QoS: Make it possible to expose PM QoS latency constraints\n PM / Sleep: JBD and JBD2 missing set_freezable()\n PM / Domains: Fix include for PM_GENERIC_DOMAINS\u003dn case\n PM / Freezer: Remove references to TIF_FREEZE in comments\n PM / Sleep: Add more wakeup source initialization routines\n PM / Hibernate: Enable usermodehelpers in hibernate() error path\n PM / Sleep: Make __pm_stay_awake() delete wakeup source timers\n PM / Sleep: Fix race conditions related to wakeup source timer function\n PM / Sleep: Fix possible infinite loop during wakeup source destruction\n PM / Hibernate: print physical addresses consistently with other parts of kernel\n ...\n" }, { "commit": "b6e238dceed36891cc633167afe7151f1f3d83c5", "tree": "eef4928c1a0cb1b56eb20f8d0ee592709d533f10", "parents": [ "e636825346b36a07ccfc8e30946d52855e21f681" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Mar 19 17:03:41 2012 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Mar 20 14:16:50 2012 -0700" }, "message": "exit_signal: fix the \"parent has changed security domain\" logic\n\nexit_notify() changes -\u003eexit_signal if the parent already did exec.\nThis doesn\u0027t really work, we are not going to send the signal now\nif there is another live thread or the exiting task is traced. The\nparent can exec before the last dies or the tracer detaches.\n\nMove this check into do_notify_parent() which actually sends the\nsignal.\n\nThe user-visible change is that we do not change -\u003eexit_signal,\nand thus the exiting task is still \"clone children\" for\ndo_wait()-\u003eeligible_child(__WCLONE). Hopefully this is fine, the\ncurrent logic is racy anyway.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e636825346b36a07ccfc8e30946d52855e21f681", "tree": "128c37b3102223a95ce03b856766c33e4d077114", "parents": [ "5f8aadd8b9966d71a77bba52b9d499cc2f38269f" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Mar 19 17:03:22 2012 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Mar 20 14:16:50 2012 -0700" }, "message": "exit_signal: simplify the \"we have changed execution domain\" logic\n\nexit_notify() checks \"tsk-\u003eself_exec_id !\u003d tsk-\u003eparent_exec_id\"\nto handle the \"we have changed execution domain\" case.\n\nWe can change do_thread() to always set -\u003eexit_signal \u003d SIGCHLD\nand remove this check to simplify the code.\n\nWe could change setup_new_exec() instead, this looks more logical\nbecause it increments -\u003eself_exec_id. But note that de_thread()\nalready resets -\u003eexit_signal if it changes the leader, let\u0027s keep\nboth changes close to each other.\n\nNote that we change -\u003eexit_signal lockless, this changes the rules.\nThereafter -\u003eexit_signal is not stable under tasklist but this is\nfine, the only possible change is OLDSIG -\u003e SIGCHLD. This can race\nwith eligible_child() but the race is harmless. We can race with\nreparent_leader() which changes our -\u003eexit_signal in parallel, but\nit does the same change to SIGCHLD.\n\nThe noticeable user-visible change is that the execing task is not\n\"visible\" to do_wait()-\u003eeligible_child(__WCLONE) right after exec.\nTo me this looks more logical, and this is consistent with mt case.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "5234ffb9f74cfa8993d174782bc861dd9b7b5bfb", "tree": "c3933cd6177cde81a99f0bcdadb333b1233e007d", "parents": [ "05d74efa3c72a5c40b0edeb15856c0230126313b" ], "author": { "name": "Alexander Gordeev", "email": "agordeev@redhat.com", "time": "Fri Mar 09 14:59:59 2012 +0100" }, "committer": { "name": "Thomas Gleixner", "email": "tglx@linutronix.de", "time": "Fri Mar 09 17:19:09 2012 +0100" }, "message": "genirq: Get rid of unnecessary IRQTF_DIED flag\n\nCurrently IRQTF_DIED flag is set when a IRQ thread handler calls do_exit()\nBut also PF_EXITING per process flag gets set when a thread exits. This\nfix eliminates the duplicate by using PF_EXITING flag.\n\nAlso, there is a race condition in exit_irq_thread(). In case a thread\u0027s\nbit is cleared in desc-\u003ethreads_oneshot (and the IRQ line gets unmasked),\nbut before IRQTF_DIED flag is set, a new interrupt might come in and set\njust cleared bit again, this time forever. This fix throws IRQTF_DIED flag\naway, eliminating the race as a result.\n\n[ tglx: Test THREAD_EXITING first as suggested by Oleg ]\n\nReported-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Alexander Gordeev \u003cagordeev@redhat.com\u003e\nLink: http://lkml.kernel.org/r/20120309135958.GD2114@dhcp-26-207.brq.redhat.com\nSigned-off-by: Thomas Gleixner \u003ctglx@linutronix.de\u003e\n" }, { "commit": "643161ace2a7624fd0106ede12ae43bcbbfc1de0", "tree": "56fb5d4af5c5e46da8cfe3c613a84f1402a60d41", "parents": [ "743c5bc210f45b728a246da65fd1a3160566d34d", "37f08be11be9a7d9351fb1b9b408259519a126f3" ], "author": { "name": "Rafael J. Wysocki", "email": "rjw@sisk.pl", "time": "Sun Mar 04 23:11:14 2012 +0100" }, "committer": { "name": "Rafael J. Wysocki", "email": "rjw@sisk.pl", "time": "Sun Mar 04 23:11:14 2012 +0100" }, "message": "Merge branch \u0027pm-sleep\u0027\n\n* pm-sleep:\n PM / Freezer: Remove references to TIF_FREEZE in comments\n PM / Sleep: Add more wakeup source initialization routines\n PM / Hibernate: Enable usermodehelpers in hibernate() error path\n PM / Sleep: Make __pm_stay_awake() delete wakeup source timers\n PM / Sleep: Fix race conditions related to wakeup source timer function\n PM / Sleep: Fix possible infinite loop during wakeup source destruction\n PM / Hibernate: print physical addresses consistently with other parts of kernel\n PM: Add comment describing relationships between PM callbacks to pm.h\n PM / Sleep: Drop suspend_stats_update()\n PM / Sleep: Make enter_state() in kernel/power/suspend.c static\n PM / Sleep: Unify kerneldoc comments in kernel/power/suspend.c\n PM / Sleep: Remove unnecessary label from suspend_freeze_processes()\n PM / Sleep: Do not check wakeup too often in try_to_freeze_tasks()\n PM / Sleep: Initialize wakeup source locks in wakeup_source_add()\n PM / Hibernate: Refactor and simplify freezer_test_done\n PM / Hibernate: Thaw kernel threads in hibernation_snapshot() in error/test path\n PM / Freezer / Docs: Document the beauty of freeze/thaw semantics\n PM / Suspend: Avoid code duplication in suspend statistics update\n PM / Sleep: Introduce generic callbacks for new device PM phases\n PM / Sleep: Introduce \"late suspend\" and \"early resume\" of devices\n" }, { "commit": "37f08be11be9a7d9351fb1b9b408259519a126f3", "tree": "7edf402614ae9f36a6e9226b380072cec969c847", "parents": [ "8671bbc1bd0442ef0eab27f7d56216431c490820" ], "author": { "name": "Marcos Paulo de Souza", "email": "marcos.mage@gmail.com", "time": "Tue Feb 21 23:57:47 2012 +0100" }, "committer": { "name": "Rafael J. Wysocki", "email": "rjw@sisk.pl", "time": "Sun Mar 04 23:08:54 2012 +0100" }, "message": "PM / Freezer: Remove references to TIF_FREEZE in comments\n\nThis patch removes all the references in the code about the TIF_FREEZE\nflag removed by commit a3201227f803ad7fd43180c5195dbe5a2bf998aa\n\n freezer: make freezing() test freeze conditions in effect instead of TIF_FREEZE\n\nThere still are some references to TIF_FREEZE in\nDocumentation/power/freezing-of-tasks.txt, but it looks like that\ndocumentation needs more thorough work to reflect how the new\nfreezer works, and hence merely removing the references to TIF_FREEZE\nwon\u0027t really help. So I have not touched that part in this patch.\n\nSuggested-by: Srivatsa S. Bhat \u003csrivatsa.bhat@linux.vnet.ibm.com\u003e\nSigned-off-by: Marcos Paulo de Souza \u003cmarcos.mage@gmail.com\u003e\nSigned-off-by: Rafael J. Wysocki \u003crjw@sisk.pl\u003e\n" }, { "commit": "1fd36adcd98c14d2fd97f545293c488775cb2823", "tree": "c13ab1934a15aebe0d81601d910ce5a3c6fa2c6f", "parents": [ "1dce27c5aa6770e9d195f2bb7db1db3d4dde5591" ], "author": { "name": "David Howells", "email": "dhowells@redhat.com", "time": "Thu Feb 16 17:49:54 2012 +0000" }, "committer": { "name": "H. Peter Anvin", "email": "hpa@zytor.com", "time": "Sun Feb 19 10:30:57 2012 -0800" }, "message": "Replace the fd_sets in struct fdtable with an array of unsigned longs\n\nReplace the fd_sets in struct fdtable with an array of unsigned longs and then\nuse the standard non-atomic bit operations rather than the FD_* macros.\n\nThis:\n\n (1) Removes the abuses of struct fd_set:\n\n (a) Since we don\u0027t want to allocate a full fd_set the vast majority of the\n \t time, we actually, in effect, just allocate a just-big-enough array of\n \t unsigned longs and cast it to an fd_set type - so why bother with the\n \t fd_set at all?\n\n (b) Some places outside of the core fdtable handling code (such as\n \t SELinux) want to look inside the array of unsigned longs hidden inside\n \t the fd_set struct for more efficient iteration over the entire set.\n\n (2) Eliminates the use of FD_*() macros in the kernel completely.\n\n (3) Permits the __FD_*() macros to be deleted entirely where not exposed to\n userspace.\n\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nLink: http://lkml.kernel.org/r/20120216174954.23314.48147.stgit@warthog.procyon.org.uk\nSigned-off-by: H. Peter Anvin \u003chpa@zytor.com\u003e\nCc: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\n" }, { "commit": "4040153087478993cbf0809f444400a3c808074c", "tree": "2dc7af85b0cf930f1656553bd38410b8c16601a6", "parents": [ "191c542442fdf53cc3c496c00be13367fd9cd42d" ], "author": { "name": "Al Viro", "email": "viro@ftp.linux.org.uk", "time": "Mon Feb 13 03:58:52 2012 +0000" }, "committer": { "name": "James Morris", "email": "jmorris@namei.org", "time": "Tue Feb 14 10:45:42 2012 +1100" }, "message": "security: trim security.h\n\nTrim security.h\n\nSigned-off-by: Al Viro \u003cviro@zeniv.linux.org.uk\u003e\nSigned-off-by: James Morris \u003cjmorris@namei.org\u003e\n" }, { "commit": "b5740f4b2cb3503b436925eb2242bc3d75cd3dfe", "tree": "c7e00c7be916f5fdc01109257752fc408b2b2d97", "parents": [ "71325960d16cd68ea0e22a8da15b2495b0f363f7" ], "author": { "name": "Yasunori Goto", "email": "y-goto@jp.fujitsu.com", "time": "Tue Jan 17 17:40:31 2012 +0900" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Fri Jan 27 11:55:36 2012 +0100" }, "message": "sched: Fix ancient race in do_exit()\n\ntry_to_wake_up() has a problem which may change status from TASK_DEAD to\nTASK_RUNNING in race condition with SMI or guest environment of virtual\nmachine. As a result, exited task is scheduled() again and panic occurs.\n\nHere is the sequence how it occurs:\n\n ----------------------------------+-----------------------------\n |\n CPU A | CPU B\n ----------------------------------+-----------------------------\n\nTASK A calls exit()....\n\ndo_exit()\n\n exit_mm()\n down_read(mm-\u003emmap_sem);\n\n rwsem_down_failed_common()\n\n set TASK_UNINTERRUPTIBLE\n set waiter.task \u003c\u003d task A\n list_add to sem-\u003ewait_list\n :\n raw_spin_unlock_irq()\n (I/O interruption occured)\n\n __rwsem_do_wake(mmap_sem)\n\n list_del(\u0026waiter-\u003elist);\n waiter-\u003etask \u003d NULL\n wake_up_process(task A)\n try_to_wake_up()\n (task is still\n TASK_UNINTERRUPTIBLE)\n p-\u003eon_rq is still 1.)\n\n ttwu_do_wakeup()\n (*A)\n :\n (I/O interruption handler finished)\n\n if (!waiter.task)\n schedule() is not called\n due to waiter.task is NULL.\n\n tsk-\u003estate \u003d TASK_RUNNING\n\n :\n check_preempt_curr();\n :\n task-\u003estate \u003d TASK_DEAD\n (*B)\n \u003c--- set TASK_RUNNING (*C)\n\n schedule()\n (exit task is running again)\n BUG_ON() is called!\n --------------------------------------------------------\n\nThe execution time between (*A) and (*B) is usually very short,\nbecause the interruption is disabled, and setting TASK_RUNNING at (*C)\nmust be executed before setting TASK_DEAD.\n\nHOWEVER, if SMI is interrupted between (*A) and (*B),\n(*C) is able to execute AFTER setting TASK_DEAD!\nThen, exited task is scheduled again, and BUG_ON() is called....\n\nIf the system works on guest system of virtual machine, the time\nbetween (*A) and (*B) may be also long due to scheduling of hypervisor,\nand same phenomenon can occur.\n\nBy this patch, do_exit() waits for releasing task-\u003epi_lock which is used\nin try_to_wake_up(). It guarantees the task becomes TASK_DEAD after\nwaking up.\n\nSigned-off-by: Yasunori Goto \u003cy-goto@jp.fujitsu.com\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nCc: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\nCc: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nLink: http://lkml.kernel.org/r/20120117174031.3118.E1E9C6FF@jp.fujitsu.com\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "f429ee3b808118591d1f3cdf3c0d0793911a5677", "tree": "96d848f5f677d96758ecd2aee5eb6931b75bf218", "parents": [ "22b4eb5e3174efb49791c62823d0cccc35394c36", "c158a35c8a681cf68d36f22f058f9f5466386c71" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 17 16:06:51 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 17 16:41:31 2012 -0800" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/audit\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/viro/audit: (29 commits)\n audit: no leading space in audit_log_d_path prefix\n audit: treat s_id as an untrusted string\n audit: fix signedness bug in audit_log_execve_info()\n audit: comparison on interprocess fields\n audit: implement all object interfield comparisons\n audit: allow interfield comparison between gid and ogid\n audit: complex interfield comparison helper\n audit: allow interfield comparison in audit rules\n Kernel: Audit Support For The ARM Platform\n audit: do not call audit_getname on error\n audit: only allow tasks to set their loginuid if it is -1\n audit: remove task argument to audit_set_loginuid\n audit: allow audit matching on inode gid\n audit: allow matching on obj_uid\n audit: remove audit_finish_fork as it can\u0027t be called\n audit: reject entry,always rules\n audit: inline audit_free to simplify the look of generic code\n audit: drop audit_set_macxattr as it doesn\u0027t do anything\n audit: inline checks for not needing to collect aux records\n audit: drop some potentially inadvisable likely notations\n ...\n\nUse evil merge to fix up grammar mistakes in Kconfig file.\n\nBad speling and horrible grammar (and copious swearing) is to be\nexpected, but let\u0027s keep it to commit messages and comments, rather than\nexpose it to users in config help texts or printouts.\n" }, { "commit": "a4ff8dba7d8ce5ceb43fb27df66292251cc73bdc", "tree": "2c89a0a7a7dad853a2c2ec70417ef2f3f5a04fd4", "parents": [ "38cdce53daa0408a61fe6d86fe48f31515c9b840" ], "author": { "name": "Eric Paris", "email": "eparis@redhat.com", "time": "Tue Jan 03 14:23:07 2012 -0500" }, "committer": { "name": "Al Viro", "email": "viro@zeniv.linux.org.uk", "time": "Tue Jan 17 16:16:58 2012 -0500" }, "message": "audit: inline audit_free to simplify the look of generic code\n\nmake the conditional a static inline instead of doing it in generic code.\n\nSigned-off-by: Eric Paris \u003ceparis@redhat.com\u003e\n" }, { "commit": "9402c95f34a66e81eba473a2f7267bbae5a1dee2", "tree": "c46628e702d625a737fedbd9de88138a94c93d1f", "parents": [ "80bf007f20b16272f210e0803f739f5606cff59d" ], "author": { "name": "Joe Perches", "email": "joe@perches.com", "time": "Thu Jan 12 17:17:17 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Jan 12 20:13:03 2012 -0800" }, "message": "treewide: remove useless NORET_TYPE macro and uses\n\nIt\u0027s a very old and now unused prototype marking so just delete it.\n\nNeaten panic pointer argument style to keep checkpatch quiet.\n\nSigned-off-by: Joe Perches \u003cjoe@perches.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nCc: Haavard Skinnemoen \u003chskinnemoen@gmail.com\u003e\nCc: Hans-Christian Egtvedt \u003cegtvedt@samfundet.no\u003e\nCc: Tony Luck \u003ctony.luck@intel.com\u003e\nCc: Fenghua Yu \u003cfenghua.yu@intel.com\u003e\nAcked-by: Geert Uytterhoeven \u003cgeert@linux-m68k.org\u003e\nAcked-by: Ralf Baechle \u003cralf@linux-mips.org\u003e\nCc: Benjamin Herrenschmidt \u003cbenh@kernel.crashing.org\u003e\nCc: Paul Mackerras \u003cpaulus@samba.org\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nCc: Heiko Carstens \u003cheiko.carstens@de.ibm.com\u003e\nCc: Chris Metcalf \u003ccmetcalf@tilera.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "001a541ea9163ace5e8243ee0e907ad80a4c0ec2", "tree": "a76225046369c440de93739add9823f5ea060245", "parents": [ "40ba587923ae67090d9f141c1d3c951be5c1420e", "bc31b86a5923fad5f3fbb6192f767f410241ba27" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 10 16:59:59 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 10 16:59:59 2012 -0800" }, "message": "Merge branch \u0027writeback-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/wfg/linux\n\n* \u0027writeback-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/wfg/linux:\n writeback: move MIN_WRITEBACK_PAGES to fs-writeback.c\n writeback: balanced_rate cannot exceed write bandwidth\n writeback: do strict bdi dirty_exceeded\n writeback: avoid tiny dirty poll intervals\n writeback: max, min and target dirty pause time\n writeback: dirty ratelimit - think time compensation\n btrfs: fix dirtied pages accounting on sub-page writes\n writeback: fix dirtied pages accounting on redirty\n writeback: fix dirtied pages accounting on sub-page writes\n writeback: charge leaked page dirties to active tasks\n writeback: Include all dirty inodes in background writeback\n" }, { "commit": "eb59c505f8a5906ad2e053d14fab50eb8574fd6f", "tree": "c6e875adc12b481b916e847e8f80b8881a0fb02c", "parents": [ "1619ed8f60959829d070d8f39cd2f8ca0e7135ce", "c233523b3d392e530033a7587d7970dc62a02361" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jan 08 13:10:57 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Sun Jan 08 13:10:57 2012 -0800" }, "message": "Merge branch \u0027pm-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm\n\n* \u0027pm-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm: (76 commits)\n PM / Hibernate: Implement compat_ioctl for /dev/snapshot\n PM / Freezer: fix return value of freezable_schedule_timeout_killable()\n PM / shmobile: Allow the A4R domain to be turned off at run time\n PM / input / touchscreen: Make st1232 use device PM QoS constraints\n PM / QoS: Introduce dev_pm_qos_add_ancestor_request()\n PM / shmobile: Remove the stay_on flag from SH7372\u0027s PM domains\n PM / shmobile: Don\u0027t include SH7372\u0027s INTCS in syscore suspend/resume\n PM / shmobile: Add support for the sh7372 A4S power domain / sleep mode\n PM: Drop generic_subsys_pm_ops\n PM / Sleep: Remove forward-only callbacks from AMBA bus type\n PM / Sleep: Remove forward-only callbacks from platform bus type\n PM: Run the driver callback directly if the subsystem one is not there\n PM / Sleep: Make pm_op() and pm_noirq_op() return callback pointers\n PM/Devfreq: Add Exynos4-bus device DVFS driver for Exynos4210/4212/4412.\n PM / Sleep: Merge internal functions in generic_ops.c\n PM / Sleep: Simplify generic system suspend callbacks\n PM / Hibernate: Remove deprecated hibernation snapshot ioctls\n PM / Sleep: Fix freezer failures due to racy usermodehelper_is_disabled()\n ARM: S3C64XX: Implement basic power domain support\n PM / shmobile: Use common always on power domain governor\n ...\n\nFix up trivial conflict in fs/xfs/xfs_buf.c due to removal of unused\nXBT_FORCE_SLEEP bit\n" }, { "commit": "0db49b72bce26341274b74fd968501489a361ae3", "tree": "cdb076827aefb38d719d4c42f8ef291c36072fa8", "parents": [ "35b740e4662ef386f0c60e1b60aaf5b44db9914c", "1ac9bc6943edf7d181b4b1cc734981350d4f6bae" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 06 08:33:28 2012 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jan 06 08:44:54 2012 -0800" }, "message": "Merge branch \u0027sched-core-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip\n\n* \u0027sched-core-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip: (40 commits)\n sched/tracing: Add a new tracepoint for sleeptime\n sched: Disable scheduler warnings during oopses\n sched: Fix cgroup movement of waking process\n sched: Fix cgroup movement of newly created process\n sched: Fix cgroup movement of forking process\n sched: Remove cfs bandwidth period check in tg_set_cfs_period()\n sched: Fix load-balance lock-breaking\n sched: Replace all_pinned with a generic flags field\n sched: Only queue remote wakeups when crossing cache boundaries\n sched: Add missing rcu_dereference() around -\u003ereal_parent usage\n [S390] fix cputime overflow in uptime_proc_show\n [S390] cputime: add sparse checking and cleanup\n sched: Mark parent and real_parent as __rcu\n sched, nohz: Fix missing RCU read lock\n sched, nohz: Set the NOHZ_BALANCE_KICK flag for idle load balancer\n sched, nohz: Fix the idle cpu check in nohz_idle_balance\n sched: Use jump_labels for sched_feat\n sched/accounting: Fix parameter passing in task_group_account_field\n sched/accounting: Fix user/system tick double accounting\n sched/accounting: Re-use scheduler statistics for the root cgroup\n ...\n\nFix up conflicts in\n - arch/ia64/include/asm/cputime.h, include/asm-generic/cputime.h\n\tusecs_to_cputime64() vs the sparse cleanups\n - kernel/sched/fair.c, kernel/time/tick-sched.c\n\tscheduler changes in multiple branches\n" }, { "commit": "50b8d257486a45cba7b65ca978986ed216bbcc10", "tree": "0bcbb7eefaaf3da47a23c259e91baf414319ff96", "parents": [ "8d9cbf8240023f5b6d19f2106aff3d7077fc4a3b" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jan 04 17:29:02 2012 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jan 04 15:01:59 2012 -0800" }, "message": "ptrace: partially fix the do_wait(WEXITED) vs EXIT_DEAD-\u003eEXIT_ZOMBIE race\n\nTest-case:\n\n\tint main(void)\n\t{\n\t\tint pid, status;\n\n\t\tpid \u003d fork();\n\t\tif (!pid) {\n\t\t\tfor (;;) {\n\t\t\t\tif (!fork())\n\t\t\t\t\treturn 0;\n\t\t\t\tif (waitpid(-1, \u0026status, 0) \u003c 0) {\n\t\t\t\t\tprintf(\"ERR!! wait: %m\\n\");\n\t\t\t\t\treturn 0;\n\t\t\t\t}\n\t\t\t}\n\t\t}\n\n\t\tassert(ptrace(PTRACE_ATTACH, pid, 0,0) \u003d\u003d 0);\n\t\tassert(waitpid(-1, NULL, 0) \u003d\u003d pid);\n\n\t\tassert(ptrace(PTRACE_SETOPTIONS, pid, 0,\n\t\t\t\t\tPTRACE_O_TRACEFORK) \u003d\u003d 0);\n\n\t\tdo {\n\t\t\tptrace(PTRACE_CONT, pid, 0, 0);\n\t\t\tpid \u003d waitpid(-1, NULL, 0);\n\t\t} while (pid \u003e 0);\n\n\t\treturn 1;\n\t}\n\nIt fails because -\u003ereal_parent sees its child in EXIT_DEAD state\nwhile the tracer is going to change the state back to EXIT_ZOMBIE\nin wait_task_zombie().\n\nThe offending commit is 823b018e which moved the EXIT_DEAD check,\nbut in fact we should not blame it. The original code was not\ncorrect as well because it didn\u0027t take ptrace_reparented() into\naccount and because we can\u0027t really trust -\u003eptrace.\n\nThis patch adds the additional check to close this particular\nrace but it doesn\u0027t solve the whole problem. We simply can\u0027t\nrely on -\u003eptrace in this case, it can be cleared if the tracer\nis multithreaded by the exiting -\u003eparent.\n\nI think we should kill EXIT_DEAD altogether, we should always\nremove the soon-to-be-reaped child from -\u003echildren or at least\nwe should never do the DEAD-\u003eZOMBIE transition. But this is too\ncomplex for 3.2.\n\nReported-and-tested-by: Denys Vlasenko \u003cvda.linux@googlemail.com\u003e\nTested-by: Lukasz Michalik \u003clmi@ift.uni.wroc.pl\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: \u003cstable@kernel.org\u003e\t\t[3.0+]\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "54848d73f9f254631303d6eab9b976855988b266", "tree": "9fb4b7e564f2c0df88d0bde2f482b9b7efc847fa", "parents": [ "1bc36b6426ae49139e9f56491db76b95921454d7" ], "author": { "name": "Wu Fengguang", "email": "fengguang.wu@intel.com", "time": "Tue Apr 05 13:21:19 2011 -0600" }, "committer": { "name": "Wu Fengguang", "email": "fengguang.wu@intel.com", "time": "Sun Dec 18 14:20:20 2011 +0800" }, "message": "writeback: charge leaked page dirties to active tasks\n\nIt\u0027s a years long problem that a large number of short-lived dirtiers\n(eg. gcc instances in a fast kernel build) may starve long-run dirtiers\n(eg. dd) as well as pushing the dirty pages to the global hard limit.\n\nThe solution is to charge the pages dirtied by the exited gcc to the\nother random dirtying tasks. It sounds not perfect, however should\nbehave good enough in practice, seeing as that throttled tasks aren\u0027t\nactually running so those that are running are more likely to pick it up\nand get throttled, therefore promoting an equal spread.\n\nRandy: fix compile error: \u0027dirty_throttle_leaks\u0027 undeclared in exit.c\n\nAcked-by: Jan Kara \u003cjack@suse.cz\u003e\nAcked-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nSigned-off-by: Randy Dunlap \u003crdunlap@xenotime.net\u003e\nSigned-off-by: Wu Fengguang \u003cfengguang.wu@intel.com\u003e\n" }, { "commit": "648616343cdbe904c585a6c12e323d3b3c72e46f", "tree": "514bce1b52663db4ab5662b637c764cf3c2ed1eb", "parents": [ "55b02d2f4445ad625213817a1736bf2884d32547" ], "author": { "name": "Martin Schwidefsky", "email": "schwidefsky@de.ibm.com", "time": "Thu Dec 15 14:56:09 2011 +0100" }, "committer": { "name": "Martin Schwidefsky", "email": "schwidefsky@de.ibm.com", "time": "Thu Dec 15 14:56:19 2011 +0100" }, "message": "[S390] cputime: add sparse checking and cleanup\n\nMake cputime_t and cputime64_t nocast to enable sparse checking to\ndetect incorrect use of cputime. Drop the cputime macros for simple\nscalar operations. The conversion macros are still needed.\n\nSigned-off-by: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\n" }, { "commit": "a585042f7b933539a0b6bc63650c2d49ffb2e55d", "tree": "bb93d05050e04c4bcd53fdf535239e6312fb62e2", "parents": [ "a5be2d0d1a8746e7be5210e3d6b904455000443c" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 21 12:32:23 2011 -0800" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Mon Nov 21 12:32:23 2011 -0800" }, "message": "freezer: remove racy clear_freeze_flag() and set PF_NOFREEZE on dead tasks\n\nclear_freeze_flag() in exit_mm() is racy. Freezing can start\nafterwards. Remove it. Skipping freezer for exiting task will be\nproperly implemented later.\n\nAlso, freezable() was testing exit_state directly to make system\nfreezer ignore dead tasks. Let the exiting task set PF_NOFREEZE after\nentering TASK_DEAD instead.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "c9f01245b6a7d77d17deaa71af10f6aca14fa24e", "tree": "13ffde591a5bcefba39cb6393f09b27f1ebc1a30", "parents": [ "7b0d44fa49b1dcfdcf4897f12ddd12ddeab1a9d7" ], "author": { "name": "David Rientjes", "email": "rientjes@google.com", "time": "Mon Oct 31 17:07:15 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Oct 31 17:30:45 2011 -0700" }, "message": "oom: remove oom_disable_count\n\nThis removes mm-\u003eoom_disable_count entirely since it\u0027s unnecessary and\ncurrently buggy. The counter was intended to be per-process but it\u0027s\ncurrently decremented in the exit path for each thread that exits, causing\nit to underflow.\n\nThe count was originally intended to prevent oom killing threads that\nshare memory with threads that cannot be killed since it doesn\u0027t lead to\nfuture memory freeing. The counter could be fixed to represent all\nthreads sharing the same mm, but it\u0027s better to remove the count since:\n\n - it is possible that the OOM_DISABLE thread sharing memory with the\n victim is waiting on that thread to exit and will actually cause\n future memory freeing, and\n\n - there is no guarantee that a thread is disabled from oom killing just\n because another thread sharing its mm is oom disabled.\n\nSigned-off-by: David Rientjes \u003crientjes@google.com\u003e\nReported-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Ying Han \u003cyinghan@google.com\u003e\nCc: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b34a6b1da371ed8af1221459a18c67970f7e3d53", "tree": "5addc850de13623b172395b9d0d7d670930fa6b3", "parents": [ "d40dcdb0172a1ba853464983a059fb45e0aaf61a" ], "author": { "name": "Vasiliy Kulikov", "email": "segoon@openwall.com", "time": "Tue Jul 26 16:08:48 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jul 26 16:49:44 2011 -0700" }, "message": "ipc: introduce shm_rmid_forced sysctl\n\nAdd support for the shm_rmid_forced sysctl. If set to 1, all shared\nmemory objects in current ipc namespace will be automatically forced to\nuse IPC_RMID.\n\nThe POSIX way of handling shmem allows one to create shm objects and\ncall shmdt(), leaving shm object associated with no process, thus\nconsuming memory not counted via rlimits.\n\nWith shm_rmid_forced\u003d1 the shared memory object is counted at least for\none process, so OOM killer may effectively kill the fat process holding\nthe shared memory.\n\nIt obviously breaks POSIX - some programs relying on the feature would\nstop working. So set shm_rmid_forced\u003d1 only if you\u0027re sure nobody uses\n\"orphaned\" memory. Use shm_rmid_forced\u003d0 by default for compatability\nreasons.\n\nThe feature was previously impemented in -ow as a configure option.\n\n[akpm@linux-foundation.org: fix documentation, per Randy]\n[akpm@linux-foundation.org: fix warning]\n[akpm@linux-foundation.org: readability/conventionality tweaks]\n[akpm@linux-foundation.org: fix shm_rmid_forced/shm_forced_rmid confusion, use standard comment layout]\nSigned-off-by: Vasiliy Kulikov \u003csegoon@openwall.com\u003e\nCc: Randy Dunlap \u003crdunlap@xenotime.net\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nCc: \"Serge E. Hallyn\" \u003cserge.hallyn@canonical.com\u003e\nCc: Daniel Lezcano \u003cdaniel.lezcano@free.fr\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Alan Cox \u003calan@lxorguk.ukuu.org.uk\u003e\nCc: Solar Designer \u003csolar@openwall.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d3ec4844d449cf7af9e749f73ba2052fb7b72fc2", "tree": "c515913e85f7e50878c83da2a88bc5a7269d087c", "parents": [ "0003230e8200699860f0b10af524dc47bf8aecad", "df2e301fee3c2c2a87592151397ad7699bb14c37" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 25 13:56:39 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 25 13:56:39 2011 -0700" }, "message": "Merge branch \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial\n\n* \u0027for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/jikos/trivial: (43 commits)\n fs: Merge split strings\n treewide: fix potentially dangerous trailing \u0027;\u0027 in #defined values/expressions\n uwb: Fix misspelling of neighbourhood in comment\n net, netfilter: Remove redundant goto in ebt_ulog_packet\n trivial: don\u0027t touch files that are removed in the staging tree\n lib/vsprintf: replace link to Draft by final RFC number\n doc: Kconfig: `to be\u0027 -\u003e `be\u0027\n doc: Kconfig: Typo: square -\u003e squared\n doc: Konfig: Documentation/power/{pm \u003d\u003e apm-acpi}.txt\n drivers/net: static should be at beginning of declaration\n drivers/media: static should be at beginning of declaration\n drivers/i2c: static should be at beginning of declaration\n XTENSA: static should be at beginning of declaration\n SH: static should be at beginning of declaration\n MIPS: static should be at beginning of declaration\n ARM: static should be at beginning of declaration\n rcu: treewide: Do not use rcu_read_lock_held when calling rcu_dereference_check\n Update my e-mail address\n PCIe ASPM: forcedly -\u003e forcibly\n gma500: push through device driver tree\n ...\n\nFix up trivial conflicts:\n - arch/arm/mach-ep93xx/dma-m2p.c (deleted)\n - drivers/gpio/gpio-ep93xx.c (renamed and context nearby)\n - drivers/net/r8169.c (just context changes)\n" }, { "commit": "096a705bbc080a4041636d07514560da8d78acbe", "tree": "38c3c01225709ffa53419083ea6332f8a72610de", "parents": [ "fea80311a939a746533a6d7e7c3183729d6a3faf", "5757a6d76cdf6dda2a492c09b985c015e86779b1" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 25 10:33:36 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Mon Jul 25 10:33:36 2011 -0700" }, "message": "Merge branch \u0027for-3.1/core\u0027 of git://git.kernel.dk/linux-block\n\n* \u0027for-3.1/core\u0027 of git://git.kernel.dk/linux-block: (24 commits)\n block: strict rq_affinity\n backing-dev: use synchronize_rcu_expedited instead of synchronize_rcu\n block: fix patch import error in max_discard_sectors check\n block: reorder request_queue to remove 64 bit alignment padding\n CFQ: add think time check for group\n CFQ: add think time check for service tree\n CFQ: move think time check variables to a separate struct\n fixlet: Remove fs_excl from struct task.\n cfq: Remove special treatment for metadata rqs.\n block: document blk_plug list access\n block: avoid building too big plug list\n compat_ioctl: fix make headers_check regression\n block: eliminate potential for infinite loop in blkdev_issue_discard\n compat_ioctl: fix warning caused by qemu\n block: flush MEDIA_CHANGE from drivers on close(2)\n blk-throttle: Make total_nr_queued unsigned\n block: Add __attribute__((format(printf...) and fix fallout\n fs/partitions/check.c: make local symbols static\n block:remove some spare spaces in genhd.c\n block:fix the comment error in blkdev.h\n ...\n" }, { "commit": "8209f53d79444747782a28520187abaf689761f2", "tree": "726270ea29e037f026d77a99787b9d844531ac42", "parents": [ "22a3b9771117d566def0150ea787fcc95f16e724", "eac1b5e57d7abc836e78fd3fbcf77dbeed01edc9" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jul 22 15:06:50 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Jul 22 15:06:50 2011 -0700" }, "message": "Merge branch \u0027ptrace\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc\n\n* \u0027ptrace\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc: (39 commits)\n ptrace: do_wait(traced_leader_killed_by_mt_exec) can block forever\n ptrace: fix ptrace_signal() \u0026\u0026 STOP_DEQUEUED interaction\n connector: add an event for monitoring process tracers\n ptrace: dont send SIGSTOP on auto-attach if PT_SEIZED\n ptrace: mv send-SIGSTOP from do_fork() to ptrace_init_task()\n ptrace_init_task: initialize child-\u003ejobctl explicitly\n has_stopped_jobs: s/task_is_stopped/SIGNAL_STOP_STOPPED/\n ptrace: make former thread ID available via PTRACE_GETEVENTMSG after PTRACE_EVENT_EXEC stop\n ptrace: wait_consider_task: s/same_thread_group/ptrace_reparented/\n ptrace: kill real_parent_is_ptracer() in in favor of ptrace_reparented()\n ptrace: ptrace_reparented() should check same_thread_group()\n redefine thread_group_leader() as exit_signal \u003e\u003d 0\n do not change dead_task-\u003eexit_signal\n kill task_detached()\n reparent_leader: check EXIT_DEAD instead of task_detached()\n make do_notify_parent() __must_check, update the callers\n __ptrace_detach: avoid task_detached(), check do_notify_parent()\n kill tracehook_notify_death()\n make do_notify_parent() return bool\n ptrace: s/tracehook_tracer_task()/ptrace_parent()/\n ...\n" }, { "commit": "961c4675c75112717705fa5c0c53cb9664051479", "tree": "d424199346b5dc1c5d27fe920a66a6ea11989d89", "parents": [ "bb188d7e64deb0e9cf13a99f44ae0065de5352d6" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu Jul 07 21:33:54 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Sun Jul 17 20:23:50 2011 +0200" }, "message": "has_stopped_jobs: s/task_is_stopped/SIGNAL_STOP_STOPPED/\n\nhas_stopped_jobs() naively checks task_is_stopped(group_leader). This\nwas always wrong even without ptrace, group_leader can be dead. And\ngiven that ptrace can change the state to TRACED this is wrong even\nin the single-threaded case.\n\nChange the code to check SIGNAL_STOP_STOPPED and simplify the code,\nretval + break/continue doesn\u0027t make this trivial code more readable.\n\nWe could probably add the usual \"|| signal-\u003egroup_stop_count\" check\nbut I don\u0027t think this makes sense, the task can start the group-stop\nright after the check anyway.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "4aede84b33d6beb401136a3deca0651ae07c5e99", "tree": "831266dbb15227584da5011ae4bb1e5038a69677", "parents": [ "a07405b7802691d29ab3b23bdc76ee6d006aad0b" ], "author": { "name": "Justin TerAvest", "email": "teravest@google.com", "time": "Tue Jul 12 08:31:45 2011 +0200" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Jul 12 08:35:10 2011 +0200" }, "message": "fixlet: Remove fs_excl from struct task.\n\nfs_excl is a poor man\u0027s priority inheritance for filesystems to hint to\nthe block layer that an operation is important. It was never clearly\nspecified, not widely adopted, and will not prevent starvation in many\ncases (like across cgroups).\n\nfs_excl was introduced with the time sliced CFQ IO scheduler, to\nindicate when a process held FS exclusive resources and thus needed\na boost.\n\nIt doesn\u0027t cover all file systems, and it was never fully complete.\nLets kill it.\n\nSigned-off-by: Justin TerAvest \u003cteravest@google.com\u003e\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "b7e9c223be8ce335e30f2cf6ba588e6a4092275c", "tree": "2d1e3b75606abc18df7ad65e51ac3f90cd68b38d", "parents": [ "c172d82500a6cf3c32d1e650722a1055d72ce858", "e3bbfa78bab125f58b831b5f7f45b5a305091d72" ], "author": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Jul 11 14:15:48 2011 +0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Mon Jul 11 14:15:55 2011 +0200" }, "message": "Merge branch \u0027master\u0027 into for-next\n\nSync with Linus\u0027 tree to be able to apply pending patches that\nare based on newer code already present upstream.\n" }, { "commit": "d8bf4ca9ca9576548628344c9725edd3786e90b1", "tree": "df338f50a5af6bc3651bd863b79fa91e6b1e9e20", "parents": [ "eb032b9837a958e21ca000358a5bde5e17192ddb" ], "author": { "name": "Michal Hocko", "email": "mhocko@suse.cz", "time": "Fri Jul 08 14:39:41 2011 +0200" }, "committer": { "name": "Jiri Kosina", "email": "jkosina@suse.cz", "time": "Fri Jul 08 22:21:58 2011 +0200" }, "message": "rcu: treewide: Do not use rcu_read_lock_held when calling rcu_dereference_check\n\nSince ca5ecddf (rcu: define __rcu address space modifier for sparse)\nrcu_dereference_check use rcu_read_lock_held as a part of condition\nautomatically so callers do not have to do that as well.\n\nSigned-off-by: Michal Hocko \u003cmhocko@suse.cz\u003e\nAcked-by: Paul E. McKenney \u003cpaulmck@linux.vnet.ibm.com\u003e\nSigned-off-by: Jiri Kosina \u003cjkosina@suse.cz\u003e\n" }, { "commit": "479bf98c1c29b40d86e40a4e6e4944c2f03d9493", "tree": "259bda55b4e49d764e8768e5149f755c8897ab5c", "parents": [ "bb3696da89743d580f869142d0a6e6ba9b7fe89a" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Fri Jun 24 17:34:39 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:11 2011 +0200" }, "message": "ptrace: wait_consider_task: s/same_thread_group/ptrace_reparented/\n\nwait_consider_task() checks same_thread_group(parent, real_parent),\nthis is the open-coded ptrace_reparented().\n\n__ptrace_detach() remains the only function which has to check this by\nhand, although we could reorganize the code to delay __ptrace_unlink.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "e550f14dc6322e794d4e70825f63c9c99177ae8b", "tree": "64f918a0d70f82842211a3c0d54ca658877b5b5d", "parents": [ "0976a03e5ce8ec346e985f21046d7a75bb7fdffd" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 23:09:54 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:09 2011 +0200" }, "message": "kill task_detached()\n\nUpadate the last user of task_detached(), wait_task_zombie(), to\nuse thread_group_leader() and kill task_detached().\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "0976a03e5ce8ec346e985f21046d7a75bb7fdffd", "tree": "347ab2fa51689226d734b288f330b5ad16997d8e", "parents": [ "8677347378044ab564470bced2275520efb3670d" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 23:09:39 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:09 2011 +0200" }, "message": "reparent_leader: check EXIT_DEAD instead of task_detached()\n\nChange reparent_leader() to check -\u003eexit_state instead of -\u003eexit_signal,\nthis matches the similar EXIT_DEAD check in wait_consider_task() and\nallows us to cleanup the do_notify_parent/task_detached logic.\n\ntask_detached() was really needed during reparenting before 9cd80bbb\n\"do_wait() optimization: do not place sub-threads on -\u003echildren list\"\nto filter out the sub-threads. After this change task_detached(p) can\nonly be true if p is the dead group_leader and its parent ignores\nSIGCHLD, in this case the caller of do_notify_parent() is going to\nreap this task and it should set EXIT_DEAD.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "8677347378044ab564470bced2275520efb3670d", "tree": "61baa72f24bb12eadd6956fb0abeb7eabda46b88", "parents": [ "9843a1e977977986d0a4c1000f2229b032572534" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 23:09:09 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:09 2011 +0200" }, "message": "make do_notify_parent() __must_check, update the callers\n\nChange other callers of do_notify_parent() to check the value it\nreturns, this makes the subsequent task_detached() unnecessary.\nMark do_notify_parent() as __must_check.\n\nUse thread_group_leader() instead of !task_detached() to check\nif we need to notify the real parent in wait_task_zombie().\n\nRemove the stale comment in release_task(). \"just for sanity\" is\nno longer true, we have to set EXIT_DEAD to avoid the races with\ndo_wait().\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "45cdf5cc0703c537194588c63d53bad1f2539d36", "tree": "579573d2cbe054d767a9f6b575b88836586fb7ce", "parents": [ "53c8f9f199b239668e6b1a907735ee323a0d1ccd" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu Jun 23 19:06:50 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:08 2011 +0200" }, "message": "kill tracehook_notify_death()\n\nKill tracehook_notify_death(), reimplement the logic in its caller,\nexit_notify().\n\nAlso, change the exec_id\u0027s check to use thread_group_leader() instead\nof task_detached(), this is more clear. This logic only applies to\nthe exiting leader, a sub-thread must never change its exit_signal.\n\nNote: when the traced group leader exits the exit_signal-or-SIGCHLD\nlogic looks really strange:\n\n\t- we notify the tracer even if !thread_group_empty() but\n\t do_wait(WEXITED) can\u0027t work until all threads exit\n\n\t- if the tracer is real_parent, it is not clear why can\u0027t\n\t we use -\u003eexit_signal event if !thread_group_empty()\n\n-v2: do not try to fix the 2nd oddity to avoid the subtle behavior\n change mixed with reorganization, suggested by Tejun.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nReviewed-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "53c8f9f199b239668e6b1a907735ee323a0d1ccd", "tree": "785afb9c72a2df27cd6802cf311df18711c59b76", "parents": [ "06d984737bac0545fe20bb5447ee488b95adb531" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 23:08:18 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Mon Jun 27 20:30:08 2011 +0200" }, "message": "make do_notify_parent() return bool\n\n- change do_notify_parent() to return a boolean, true if the task should\n be reaped because its parent ignores SIGCHLD.\n\n- update the only caller which checks the returned value, exit_notify().\n\nThis temporary uglifies exit_notify() even more, will be cleanuped by\nthe next change.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "a288eecce5253cc1565d400a52b9b476a157e040", "tree": "a933dd8c9791be83b7a67a33502c79e7b441960e", "parents": [ "f3c04b934d429b1ace21866f011b66de328c0dc9" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jun 17 16:50:37 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 19:26:28 2011 +0200" }, "message": "ptrace: kill trivial tracehooks\n\nAt this point, tracehooks aren\u0027t useful to mainline kernel and mostly\njust add an extra layer of obfuscation. Although they have comments,\nwithout actual in-kernel users, it is difficult to tell what are their\nassumptions and they\u0027re actually trying to achieve. To mainline\nkernel, they just aren\u0027t worth keeping around.\n\nThis patch kills the following trivial tracehooks.\n\n* Ones testing whether task is ptraced. Replace with -\u003eptrace test.\n\n\ttracehook_expect_breakpoints()\n\ttracehook_consider_ignored_signal()\n\ttracehook_consider_fatal_signal()\n\n* ptrace_event() wrappers. Call directly.\n\n\ttracehook_report_exec()\n\ttracehook_report_exit()\n\ttracehook_report_vfork_done()\n\n* ptrace_release_task() wrapper. Call directly.\n\n\ttracehook_finish_release_task()\n\n* noop\n\n\ttracehook_prepare_release_task()\n\ttracehook_report_death()\n\nThis doesn\u0027t introduce any behavior change.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Christoph Hellwig \u003chch@infradead.org\u003e\nCc: Martin Schwidefsky \u003cschwidefsky@de.ibm.com\u003e\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "d21142ece414ce1088cfcae760689aa60d6fee80", "tree": "9f97b4518cd06fe695476fc6cc4ff9ed8d5bdd58", "parents": [ "544b2c91a9f14f9565af1972203438b7f49afd48" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Jun 17 16:50:34 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed Jun 22 19:26:27 2011 +0200" }, "message": "ptrace: kill task_ptrace()\n\ntask_ptrace(task) simply dereferences task-\u003eptrace and isn\u0027t even used\nconsistently only adding confusion. Kill it and directly access\n-\u003eptrace instead.\n\nThis doesn\u0027t introduce any behavior change.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "544b2c91a9f14f9565af1972203438b7f49afd48", "tree": "38615eeed1e50580a2341b5a9d15c98793d33c2d", "parents": [ "fb1d910c178ba0c5bc32d3e5a9e82e05b7aad3cd" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Tue Jun 14 11:20:18 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu Jun 16 21:41:54 2011 +0200" }, "message": "ptrace: implement PTRACE_LISTEN\n\nThe previous patch implemented async notification for ptrace but it\nonly worked while trace is running. This patch introduces\nPTRACE_LISTEN which is suggested by Oleg Nestrov.\n\nIt\u0027s allowed iff tracee is in STOP trap and puts tracee into\nquasi-running state - tracee never really runs but wait(2) and\nptrace(2) consider it to be running. While ptracer is listening,\ntracee is allowed to re-enter STOP to notify an async event.\nListening state is cleared on the first notification. Ptracer can\nalso clear it by issuing INTERRUPT - tracee will re-trap into STOP\nwith listening state cleared.\n\nThis allows ptracer to monitor group stop state without running tracee\n- use INTERRUPT to put tracee into STOP trap, issue LISTEN and then\nwait(2) to wait for the next group stop event. When it happens,\nPTRACE_GETSIGINFO provides information to determine the current state.\n\nTest program follows.\n\n #define PTRACE_SEIZE\t\t0x4206\n #define PTRACE_INTERRUPT\t0x4207\n #define PTRACE_LISTEN\t\t0x4208\n\n #define PTRACE_SEIZE_DEVEL\t0x80000000\n\n static const struct timespec ts1s \u003d { .tv_sec \u003d 1 };\n\n int main(int argc, char **argv)\n {\n\t pid_t tracee, tracer;\n\t int i;\n\n\t tracee \u003d fork();\n\t if (!tracee)\n\t\t while (1)\n\t\t\t pause();\n\n\t tracer \u003d fork();\n\t if (!tracer) {\n\t\t siginfo_t si;\n\n\t\t ptrace(PTRACE_SEIZE, tracee, NULL,\n\t\t\t (void *)(unsigned long)PTRACE_SEIZE_DEVEL);\n\t\t ptrace(PTRACE_INTERRUPT, tracee, NULL, NULL);\n\t repeat:\n\t\t waitid(P_PID, tracee, NULL, WSTOPPED);\n\n\t\t ptrace(PTRACE_GETSIGINFO, tracee, NULL, \u0026si);\n\t\t if (!si.si_code) {\n\t\t\t printf(\"tracer: SIG %d\\n\", si.si_signo);\n\t\t\t ptrace(PTRACE_CONT, tracee, NULL,\n\t\t\t\t (void *)(unsigned long)si.si_signo);\n\t\t\t goto repeat;\n\t\t }\n\t\t printf(\"tracer: stopped\u003d%d signo\u003d%d\\n\",\n\t\t\t si.si_signo !\u003d SIGTRAP, si.si_signo);\n\t\t if (si.si_signo !\u003d SIGTRAP)\n\t\t\t ptrace(PTRACE_LISTEN, tracee, NULL, NULL);\n\t\t else\n\t\t\t ptrace(PTRACE_CONT, tracee, NULL, NULL);\n\t\t goto repeat;\n\t }\n\n\t for (i \u003d 0; i \u003c 3; i++) {\n\t\t nanosleep(\u0026ts1s, NULL);\n\t\t printf(\"mother: SIGSTOP\\n\");\n\t\t kill(tracee, SIGSTOP);\n\t\t nanosleep(\u0026ts1s, NULL);\n\t\t printf(\"mother: SIGCONT\\n\");\n\t\t kill(tracee, SIGCONT);\n\t }\n\t nanosleep(\u0026ts1s, NULL);\n\n\t kill(tracer, SIGKILL);\n\t kill(tracee, SIGKILL);\n\t return 0;\n }\n\nThis is identical to the program to test TRAP_NOTIFY except that\ntracee is PTRACE_LISTEN\u0027d instead of PTRACE_CONT\u0027d when group stopped.\nThis allows ptracer to monitor when group stop ends without running\ntracee.\n\n # ./test-listen\n tracer: stopped\u003d0 signo\u003d5\n mother: SIGSTOP\n tracer: SIG 19\n tracer: stopped\u003d1 signo\u003d19\n mother: SIGCONT\n tracer: stopped\u003d0 signo\u003d5\n tracer: SIG 18\n mother: SIGSTOP\n tracer: SIG 19\n tracer: stopped\u003d1 signo\u003d19\n mother: SIGCONT\n tracer: stopped\u003d0 signo\u003d5\n tracer: SIG 18\n mother: SIGSTOP\n tracer: SIG 19\n tracer: stopped\u003d1 signo\u003d19\n mother: SIGCONT\n tracer: stopped\u003d0 signo\u003d5\n tracer: SIG 18\n\n-v2: Moved JOBCTL_LISTENING check in wait_task_stopped() into\n task_stopped_code() as suggested by Oleg.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nCc: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "733eda7ac316cd4e550fa096e4ed42356dc546e7", "tree": "6522f47608fb97314a7b615b1cb1d6e2c58952c8", "parents": [ "37573e8c718277103f61f03741bdc5606d31b07e" ], "author": { "name": "KAMEZAWA Hiroyuki", "email": "kamezawa.hiroyu@jp.fujitsu.com", "time": "Wed Jun 15 15:08:43 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Jun 15 20:04:01 2011 -0700" }, "message": "memcg: clear mm-\u003eowner when last possible owner leaves\n\nThe following crash was reported:\n\n\u003e Call Trace:\n\u003e [\u003cffffffff81139792\u003e] mem_cgroup_from_task+0x15/0x17\n\u003e [\u003cffffffff8113a75a\u003e] __mem_cgroup_try_charge+0x148/0x4b4\n\u003e [\u003cffffffff810493f3\u003e] ? need_resched+0x23/0x2d\n\u003e [\u003cffffffff814cbf43\u003e] ? preempt_schedule+0x46/0x4f\n\u003e [\u003cffffffff8113afe8\u003e] mem_cgroup_charge_common+0x9a/0xce\n\u003e [\u003cffffffff8113b6d1\u003e] mem_cgroup_newpage_charge+0x5d/0x5f\n\u003e [\u003cffffffff81134024\u003e] khugepaged+0x5da/0xfaf\n\u003e [\u003cffffffff81078ea0\u003e] ? __init_waitqueue_head+0x4b/0x4b\n\u003e [\u003cffffffff81133a4a\u003e] ? add_mm_counter.constprop.5+0x13/0x13\n\u003e [\u003cffffffff81078625\u003e] kthread+0xa8/0xb0\n\u003e [\u003cffffffff814d13e8\u003e] ? sub_preempt_count+0xa1/0xb4\n\u003e [\u003cffffffff814d5664\u003e] kernel_thread_helper+0x4/0x10\n\u003e [\u003cffffffff814ce858\u003e] ? retint_restore_args+0x13/0x13\n\u003e [\u003cffffffff8107857d\u003e] ? __init_kthread_worker+0x5a/0x5a\n\nWhat happens is that khugepaged tries to charge a huge page against an mm\nwhose last possible owner has already exited, and the memory controller\ncrashes when the stale mm-\u003eowner is used to look up the cgroup to charge.\n\nmm-\u003eowner has never been set to NULL with the last owner going away, but\nnobody cared until khugepaged came along.\n\nEven then it wasn\u0027t a problem because the final mmput() on an mm was\nforced to acquire and release mmap_sem in write-mode, preventing an\nexiting owner to go away while the mmap_sem was held, and until \"692e0b3\nmm: thp: optimize memcg charge in khugepaged\", the memory cgroup charge\nwas protected by mmap_sem in read-mode.\n\nInstead of going back to relying on the mmap_sem to enforce lifetime of a\ntask, this patch ensures that mm-\u003eowner is properly set to NULL when the\nlast possible owner is exiting, which the memory controller can handle\njust fine.\n\n[akpm@linux-foundation.org: tweak comments]\nSigned-off-by: Hugh Dickins \u003chughd@google.com\u003e\nSigned-off-by: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nSigned-off-by: Johannes Weiner \u003channes@cmpxchg.org\u003e\nReported-by: Hugh Dickins \u003chughd@google.com\u003e\nReported-by: Dave Jones \u003cdavej@redhat.com\u003e\nReviewed-by: Andrea Arcangeli \u003caarcange@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3ed4c0583daa34dedb568b26ff99e5a7b58db612", "tree": "a531d4cc94acaa58fe0600cf83da9fb8b77f6e50", "parents": [ "ad9471752ebae25daa133b4e5d9299809c35e155", "bd715d9a4f13f87bad5526c2cd41370949473b16" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 20 13:33:21 2011 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri May 20 13:33:21 2011 -0700" }, "message": "Merge branch \u0027ptrace\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc\n\n* \u0027ptrace\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/oleg/misc: (41 commits)\n signal: trivial, fix the \"timespec declared inside parameter list\" warning\n job control: reorganize wait_task_stopped()\n ptrace: fix signal-\u003ewait_chldexit usage in task_clear_group_stop_trapping()\n signal: sys_sigprocmask() needs retarget_shared_pending()\n signal: cleanup sys_sigprocmask()\n signal: rename signandsets() to sigandnsets()\n signal: do_sigtimedwait() needs retarget_shared_pending()\n signal: introduce do_sigtimedwait() to factor out compat/native code\n signal: sys_rt_sigtimedwait: simplify the timeout logic\n signal: cleanup sys_rt_sigprocmask()\n x86: signal: sys_rt_sigreturn() should use set_current_blocked()\n x86: signal: handle_signal() should use set_current_blocked()\n signal: sigprocmask() should do retarget_shared_pending()\n signal: sigprocmask: narrow the scope of -\u003esiglock\n signal: retarget_shared_pending: optimize while_each_thread() loop\n signal: retarget_shared_pending: consider shared/unblocked signals only\n signal: introduce retarget_shared_pending()\n ptrace: ptrace_check_attach() should not do s/STOPPED/TRACED/\n signal: Turn SIGNAL_STOP_DEQUEUED into GROUP_STOP_DEQUEUED\n signal: do_signal_stop: Remove the unneeded task_clear_group_stop_pending()\n ...\n" }, { "commit": "19e274630c9e23a84d5940af83cf5db35103f968", "tree": "334344b87c92cf9d9ec6ea04151e1bf65742c58f", "parents": [ "40ae717d1e78d982bd469b2013a4cbf4ec1ca434" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Thu May 12 10:47:23 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Fri May 13 18:56:02 2011 +0200" }, "message": "job control: reorganize wait_task_stopped()\n\nwait_task_stopped() tested task_stopped_code() without acquiring\nsiglock and, if stop condition existed, called wait_task_stopped() and\ndirectly returned the result. This patch moves the initial\ntask_stopped_code() testing into wait_task_stopped() and make\nwait_consider_task() fall through to wait_task_continue() on 0 return.\n\nThis is for the following two reasons.\n\n* Because the initial task_stopped_code() test is done without\n acquiring siglock, it may race against SIGCONT generation. The\n stopped condition might have been replaced by continued state by the\n time wait_task_stopped() acquired siglock. This may lead to\n unexpected failure of WNOHANG waits.\n\n This reorganization addresses this single race case but there are\n other cases - TASK_RUNNING -\u003e TASK_STOPPED transition and EXIT_*\n transitions.\n\n* Scheduled ptrace updates require changes to the initial test which\n would fit better inside wait_task_stopped().\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nReviewed-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "bf26c018490c2fce7fe9b629083b96ce0e6ad019", "tree": "ff595f6268cb6fc675beb6accf6abb0701829443", "parents": [ "f4929bd37208540c2c6f416e9035ff1938f2dbc6" ], "author": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Thu Apr 07 16:53:20 2011 +0200" }, "committer": { "name": "Frederic Weisbecker", "email": "fweisbec@gmail.com", "time": "Mon Apr 25 17:28:24 2011 +0200" }, "message": "ptrace: Prepare to fix racy accesses on task breakpoints\n\nWhen a task is traced and is in a stopped state, the tracer\nmay execute a ptrace request to examine the tracee state and\nget its task struct. Right after, the tracee can be killed\nand thus its breakpoints released.\nThis can happen concurrently when the tracer is in the middle\nof reading or modifying these breakpoints, leading to dereferencing\na freed pointer.\n\nHence, to prepare the fix, create a generic breakpoint reference\nholding API. When a reference on the breakpoints of a task is\nheld, the breakpoints won\u0027t be released until the last reference\nis dropped. After that, no more ptrace request on the task\u0027s\nbreakpoints can be serviced for the tracer.\n\nReported-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Frederic Weisbecker \u003cfweisbec@gmail.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nCc: Will Deacon \u003cwill.deacon@arm.com\u003e\nCc: Prasad \u003cprasad@linux.vnet.ibm.com\u003e\nCc: Paul Mundt \u003clethal@linux-sh.org\u003e\nCc: v2.6.33.. \u003cstable@kernel.org\u003e\nLink: http://lkml.kernel.org/r/1302284067-7860-2-git-send-email-fweisbec@gmail.com\n" }, { "commit": "e46bc9b6fd65bc9f406a4211fbf95683cc9c2937", "tree": "57046f6b2f4674a0c9048ab1ad1ff50fae7e373a", "parents": [ "2b9accbee563f535046ff2cd382d0acaa92e130c", "321fb561971ba0f10ce18c0f8a4b9fbfc7cef4b9" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu Apr 07 20:44:11 2011 +0200" }, "committer": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Thu Apr 07 20:44:11 2011 +0200" }, "message": "Merge branch \u0027ptrace\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc into ptrace\n" }, { "commit": "25985edcedea6396277003854657b5f3cb31a628", "tree": "f026e810210a2ee7290caeb737c23cb6472b7c38", "parents": [ "6aba74f2791287ec407e0f92487a725a25908067" ], "author": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Wed Mar 30 22:57:33 2011 -0300" }, "committer": { "name": "Lucas De Marchi", "email": "lucas.demarchi@profusion.mobi", "time": "Thu Mar 31 11:26:23 2011 -0300" }, "message": "Fix common misspellings\n\nFixes generated by \u0027codespell\u0027 and manually reviewed.\n\nSigned-off-by: Lucas De Marchi \u003clucas.demarchi@profusion.mobi\u003e\n" }, { "commit": "45cb24a1da53beb70f09efccc0373f6a47a9efe0", "tree": "f27b2a6ca76f9b4e17f2adddfcf27b3d988152ed", "parents": [ "9b84cca2564b9a5b2d064fb44d2a55a5b44473a0" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "message": "job control: Allow access to job control events through ptracees\n\nCurrently a real parent can\u0027t access job control stopped/continued\nevents through a ptraced child. This utterly breaks job control when\nthe children are ptraced.\n\nFor example, if a program is run from an interactive shell and then\nstrace(1) attaches to it, pressing ^Z would send SIGTSTP and strace(1)\nwould notice it but the shell has no way to tell whether the child\nentered job control stop and thus can\u0027t tell when to take over the\nterminal - leading to awkward lone ^Z on the terminal.\n\nBecause the job control and ptrace stopped states are independent,\nthere is no reason to prevent real parents from accessing the stopped\nstate regardless of ptrace. The continued state isn\u0027t separate but\nptracers don\u0027t have any use for them as ptracees can never resume\nwithout explicit command from their ptracers, so as long as ptracers\ndon\u0027t consume it, it should be fine.\n\nAlthough this is a behavior change, because the previous behavior is\nutterly broken when viewed from real parents and the change is only\nvisible to real parents, I don\u0027t think it\u0027s necessary to make this\nbehavior optional.\n\nOne situation to be careful about is when a task from the real\nparent\u0027s group is ptracing. The parent group is the recipient of both\nptrace and job control stop events and one stop can be reported as\nboth job control and ptrace stops. As this can break the current\nptrace users, suppress job control stopped events for these cases.\n\nIf a real parent ptracer wants to know about both job control and\nptrace stops, it can create a separate process to serve the role of\nreal parent.\n\nNote that this only updates wait(2) side of things. The real parent\ncan access the states via wait(2) but still is not properly notified\n(woken up and delivered signal). Test case polls wait(2) with WNOHANG\nto work around. Notification will be updated by future patches.\n\nTest case follows.\n\n #include \u003cstdio.h\u003e\n #include \u003cunistd.h\u003e\n #include \u003ctime.h\u003e\n #include \u003cerrno.h\u003e\n #include \u003csys/types.h\u003e\n #include \u003csys/ptrace.h\u003e\n #include \u003csys/wait.h\u003e\n\n int main(void)\n {\n\t const struct timespec ts100ms \u003d { .tv_nsec \u003d 100000000 };\n\t pid_t tracee, tracer;\n\t siginfo_t si;\n\t int i;\n\n\t tracee \u003d fork();\n\t if (tracee \u003d\u003d 0) {\n\t\t while (1) {\n\t\t\t printf(\"tracee: SIGSTOP\\n\");\n\t\t\t raise(SIGSTOP);\n\t\t\t nanosleep(\u0026ts100ms, NULL);\n\t\t\t printf(\"tracee: SIGCONT\\n\");\n\t\t\t raise(SIGCONT);\n\t\t\t nanosleep(\u0026ts100ms, NULL);\n\t\t }\n\t }\n\n\t waitid(P_PID, tracee, \u0026si, WSTOPPED | WNOHANG | WNOWAIT);\n\n\t tracer \u003d fork();\n\t if (tracer \u003d\u003d 0) {\n\t\t nanosleep(\u0026ts100ms, NULL);\n\t\t ptrace(PTRACE_ATTACH, tracee, NULL, NULL);\n\n\t\t for (i \u003d 0; i \u003c 11; i++) {\n\t\t\t si.si_pid \u003d 0;\n\t\t\t waitid(P_PID, tracee, \u0026si, WSTOPPED);\n\t\t\t if (si.si_pid \u0026\u0026 si.si_code \u003d\u003d CLD_TRAPPED)\n\t\t\t\t ptrace(PTRACE_CONT, tracee, NULL,\n\t\t\t\t\t (void *)(long)si.si_status);\n\t\t }\n\t\t printf(\"tracer: EXITING\\n\");\n\t\t return 0;\n\t }\n\n\t while (1) {\n\t\t si.si_pid \u003d 0;\n\t\t waitid(P_PID, tracee, \u0026si,\n\t\t\t WSTOPPED | WCONTINUED | WEXITED | WNOHANG);\n\t\t if (si.si_pid)\n\t\t\t printf(\"mommy : WAIT status\u003d%02d code\u003d%02d\\n\",\n\t\t\t\t si.si_status, si.si_code);\n\t\t nanosleep(\u0026ts100ms, NULL);\n\t }\n\t return 0;\n }\n\nBefore the patch, while ptraced, the parent can\u0027t see any job control\nevents.\n\n tracee: SIGSTOP\n mommy : WAIT status\u003d19 code\u003d05\n tracee: SIGCONT\n tracee: SIGSTOP\n tracee: SIGCONT\n tracee: SIGSTOP\n tracee: SIGCONT\n tracee: SIGSTOP\n tracer: EXITING\n mommy : WAIT status\u003d19 code\u003d05\n ^C\n\nAfter the patch,\n\n tracee: SIGSTOP\n mommy : WAIT status\u003d19 code\u003d05\n tracee: SIGCONT\n mommy : WAIT status\u003d18 code\u003d06\n tracee: SIGSTOP\n mommy : WAIT status\u003d19 code\u003d05\n tracee: SIGCONT\n mommy : WAIT status\u003d18 code\u003d06\n tracee: SIGSTOP\n mommy : WAIT status\u003d19 code\u003d05\n tracee: SIGCONT\n mommy : WAIT status\u003d18 code\u003d06\n tracee: SIGSTOP\n tracer: EXITING\n mommy : WAIT status\u003d19 code\u003d05\n ^C\n\n-v2: Oleg pointed out that wait(2) should be suppressed for the real\n parent\u0027s group instead of only the real parent task itself.\n Updated accordingly.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "9b84cca2564b9a5b2d064fb44d2a55a5b44473a0", "tree": "376808993d708f23830ff069fdf157e99bfb7516", "parents": [ "823b018e5b1196d810790559357447948f644548" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "message": "job control: Fix ptracer wait(2) hang and explain notask_error clearing\n\nwait(2) and friends allow access to stopped/continued states through\nzombies, which is required as the states are process-wide and should\nbe accessible whether the leader task is alive or undead.\nwait_consider_task() implements this by always clearing notask_error\nand going through wait_task_stopped/continued() for unreaped zombies.\n\nHowever, while ptraced, the stopped state is per-task and as such if\nthe ptracee became a zombie, there\u0027s no further stopped event to\nlisten to and wait(2) and friends should return -ECHILD on the tracee.\n\nFix it by clearing notask_error only if WCONTINUED | WEXITED is set\nfor ptraced zombies. While at it, document why clearing notask_error\nis safe for each case.\n\nTest case follows.\n\n #include \u003cstdio.h\u003e\n #include \u003cunistd.h\u003e\n #include \u003cpthread.h\u003e\n #include \u003ctime.h\u003e\n #include \u003csys/types.h\u003e\n #include \u003csys/ptrace.h\u003e\n #include \u003csys/wait.h\u003e\n\n static void *nooper(void *arg)\n {\n\t pause();\n\t return NULL;\n }\n\n int main(void)\n {\n\t const struct timespec ts1s \u003d { .tv_sec \u003d 1 };\n\t pid_t tracee, tracer;\n\t siginfo_t si;\n\n\t tracee \u003d fork();\n\t if (tracee \u003d\u003d 0) {\n\t\t pthread_t thr;\n\n\t\t pthread_create(\u0026thr, NULL, nooper, NULL);\n\t\t nanosleep(\u0026ts1s, NULL);\n\t\t printf(\"tracee exiting\\n\");\n\t\t pthread_exit(NULL);\t/* let subthread run */\n\t }\n\n\t tracer \u003d fork();\n\t if (tracer \u003d\u003d 0) {\n\t\t ptrace(PTRACE_ATTACH, tracee, NULL, NULL);\n\t\t while (1) {\n\t\t\t if (waitid(P_PID, tracee, \u0026si, WSTOPPED) \u003c 0) {\n\t\t\t\t perror(\"waitid\");\n\t\t\t\t break;\n\t\t\t }\n\t\t\t ptrace(PTRACE_CONT, tracee, NULL,\n\t\t\t\t (void *)(long)si.si_status);\n\t\t }\n\t\t return 0;\n\t }\n\n\t waitid(P_PID, tracer, \u0026si, WEXITED);\n\t kill(tracee, SIGKILL);\n\t return 0;\n }\n\nBefore the patch, after the tracee becomes a zombie, the tracer\u0027s\nwaitid(WSTOPPED) never returns and the program doesn\u0027t terminate.\n\n tracee exiting\n ^C\n\nAfter the patch, tracee exiting triggers waitid() to fail.\n\n tracee exiting\n waitid: No child processes\n\n-v2: Oleg pointed out that exited in addition to continued can happen\n for ptraced dead group leader. Clear notask_error for ptraced\n child on WEXITED too.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "823b018e5b1196d810790559357447948f644548", "tree": "a7bb20bc487a08943c25e789c51c790aef14d169", "parents": [ "408a37de6c95832a4880a88a742f89f0cc554d06" ], "author": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Wed Mar 23 10:37:01 2011 +0100" }, "message": "job control: Small reorganization of wait_consider_task()\n\nMove EXIT_DEAD test in wait_consider_task() above ptrace check. As\nptraced tasks can\u0027t be EXIT_DEAD, this change doesn\u0027t cause any\nbehavior change. This is to prepare for further changes.\n\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\n" }, { "commit": "73c101011926c5832e6e141682180c4debe2cf45", "tree": "b8eeb521a7833cb198d6f39d5a931d820e2a663f", "parents": [ "a488e74976bf0a9bccecdd094378394942dacef1" ], "author": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Tue Mar 08 13:19:51 2011 +0100" }, "committer": { "name": "Jens Axboe", "email": "jaxboe@fusionio.com", "time": "Thu Mar 10 08:45:54 2011 +0100" }, "message": "block: initial patch for on-stack per-task plugging\n\nThis patch adds support for creating a queuing context outside\nof the queue itself. This enables us to batch up pieces of IO\nbefore grabbing the block device queue lock and submitting them to\nthe IO scheduler.\n\nThe context is created on the stack of the process and assigned in\nthe task structure, so that we can auto-unplug it if we hit a schedule\nevent.\n\nThe current queue plugging happens implicitly if IO is submitted to\nan empty device, yet callers have to remember to unplug that IO when\nthey are going to wait for it. This is an ugly API and has caused bugs\nin the past. Additionally, it requires hacks in the vm (-\u003esync_page()\ncallback) to handle that logic. By switching to an explicit plugging\nscheme we make the API a lot nicer and can get rid of the -\u003esync_page()\nhack in the vm.\n\nSigned-off-by: Jens Axboe \u003cjaxboe@fusionio.com\u003e\n" }, { "commit": "42776163e13a56ea3096edff7a5df95408e80eb4", "tree": "92f17bb5dadc7261b2d9238244cd8d4cb6c1bfd7", "parents": [ "edb2877f4a62647e36e20839a786f94d688a06ed", "3d03e2ea74103a50c23d6ab1906cf73399c0dafb" ], "author": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 11 11:02:13 2011 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Jan 11 11:02:13 2011 -0800" }, "message": "Merge branch \u0027perf-fixes-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip\n\n* \u0027perf-fixes-for-linus\u0027 of git://git.kernel.org/pub/scm/linux/kernel/git/tip/linux-2.6-tip: (28 commits)\n perf session: Fix infinite loop in __perf_session__process_events\n perf evsel: Support perf_evsel__open(cpus \u003e 1 \u0026\u0026 threads \u003e 1)\n perf sched: Use PTHREAD_STACK_MIN to avoid pthread_attr_setstacksize() fail\n perf tools: Emit clearer message for sys_perf_event_open ENOENT return\n perf stat: better error message for unsupported events\n perf sched: Fix allocation result check\n perf, x86: P4 PMU - Fix unflagged overflows handling\n dynamic debug: Fix build issue with older gcc\n tracing: Fix TRACE_EVENT power tracepoint creation\n tracing: Fix preempt count leak\n tracepoint: Add __rcu annotation\n tracing: remove duplicate null-pointer check in skb tracepoint\n tracing/trivial: Add missing comma in TRACE_EVENT comment\n tracing: Include module.h in define_trace.h\n x86: Save rbp in pt_regs on irq entry\n x86, dumpstack: Fix unused variable warning\n x86, NMI: Clean-up default_do_nmi()\n x86, NMI: Allow NMI reason io port (0x61) to be processed on any CPU\n x86, NMI: Remove DIE_NMI_IPI\n x86, NMI: Add priorities to handlers\n ...\n" }, { "commit": "0b3fcf178deefd7b64154c2c0760a2c63df0b74f", "tree": "b84509c9ac5935ac41f6fc9c444bc406cd2c54d8", "parents": [ "23a2f3ab46596d9fd0b0e592d2101bea90970594" ], "author": { "name": "Stephane Eranian", "email": "eranian@google.com", "time": "Mon Jan 03 18:20:01 2011 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Fri Jan 07 15:08:50 2011 +0100" }, "message": "perf_events: Move code around to prepare for cgroup\n\nIn particular this patch move perf_event_exit_task() before\ncgroup_exit() to allow for cgroup support. The cgroup_exit()\nfunction detaches the cgroups attached to a task.\n\nOther movements include hoisting some definitions and inlines\nat the top of perf_event.c\n\nSigned-off-by: Stephane Eranian \u003ceranian@google.com\u003e\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nLKML-Reference: \u003c4d22058b.cdace30a.4657.ffff95b1@mx.google.com\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "909ea96468096b07fbb41aaf69be060d92bd9271", "tree": "a7e015edd96b5f674874fe78cdd889769e130a2a", "parents": [ "780f36d8b3fa9572f731d4fb85067b2e45e6f993" ], "author": { "name": "Christoph Lameter", "email": "cl@linux.com", "time": "Wed Dec 08 16:22:55 2010 +0100" }, "committer": { "name": "Tejun Heo", "email": "tj@kernel.org", "time": "Fri Dec 17 15:07:19 2010 +0100" }, "message": "core: Replace __get_cpu_var with __this_cpu_read if not used for an address.\n\n__get_cpu_var() can be replaced with this_cpu_read and will then use a\nsingle read instruction with implied address calculation to access the\ncorrect per cpu instance.\n\nHowever, the address of a per cpu variable passed to __this_cpu_read()\ncannot be determined (since it\u0027s an implied address conversion through\nsegment prefixes). Therefore apply this only to uses of __get_cpu_var\nwhere the address of the variable is not used.\n\nCc: Pekka Enberg \u003cpenberg@cs.helsinki.fi\u003e\nCc: Hugh Dickins \u003chughd@google.com\u003e\nCc: Thomas Gleixner \u003ctglx@linutronix.de\u003e\nAcked-by: H. Peter Anvin \u003chpa@zytor.com\u003e\nSigned-off-by: Christoph Lameter \u003ccl@linux.com\u003e\nSigned-off-by: Tejun Heo \u003ctj@kernel.org\u003e\n" }, { "commit": "33dd94ae1ccbfb7bf0fb6c692bc3d1c4269e6177", "tree": "c00535d8fe9b29be84b02d057e41cd63d7fe3b6b", "parents": [ "a0b0f58cdd32ab363a600a294ddaa90f0c32de8c" ], "author": { "name": "Nelson Elhage", "email": "nelhage@ksplice.com", "time": "Thu Dec 02 14:31:21 2010 -0800" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu Dec 02 14:51:16 2010 -0800" }, "message": "do_exit(): make sure that we run with get_fs() \u003d\u003d USER_DS\n\nIf a user manages to trigger an oops with fs set to KERNEL_DS, fs is not\notherwise reset before do_exit(). do_exit may later (via mm_release in\nfork.c) do a put_user to a user-controlled address, potentially allowing\na user to leverage an oops into a controlled write into kernel memory.\n\nThis is only triggerable in the presence of another bug, but this\npotentially turns a lot of DoS bugs into privilege escalations, so it\u0027s\nworth fixing. I have proof-of-concept code which uses this bug along\nwith CVE-2010-3849 to write a zero to an arbitrary kernel address, so\nI\u0027ve tested that this is not theoretical.\n\nA more logical place to put this fix might be when we know an oops has\noccurred, before we call do_exit(), but that would involve changing\nevery architecture, in multiple places.\n\nLet\u0027s just stick it in do_exit instead.\n\n[akpm@linux-foundation.org: update code comment]\nSigned-off-by: Nelson Elhage \u003cnelhage@ksplice.com\u003e\nCc: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nCc: \u003cstable@kernel.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "e0a70217107e6f9844628120412cb27bb4cea194", "tree": "3c805ee6cfe97afee77b226e0cf5d36135e82dda", "parents": [ "b312e131cb88b83b6edc7a017b0249b4ea176edc" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Fri Nov 05 16:53:42 2010 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Fri Nov 05 14:16:03 2010 -0700" }, "message": "posix-cpu-timers: workaround to suppress the problems with mt exec\n\nposix-cpu-timers.c correctly assumes that the dying process does\nposix_cpu_timers_exit_group() and removes all !CPUCLOCK_PERTHREAD\ntimers from signal-\u003ecpu_timers list.\n\nBut, it also assumes that timer-\u003eit.cpu.task is always the group\nleader, and thus the dead -\u003etask means the dead thread group.\n\nThis is obviously not true after de_thread() changes the leader.\nAfter that almost every posix_cpu_timer_ method has problems.\n\nIt is not simple to fix this bug correctly. First of all, I think\nthat timer-\u003eit.cpu should use struct pid instead of task_struct.\nAlso, the locking should be reworked completely. In particular,\ntasklist_lock should not be used at all. This all needs a lot of\nnontrivial and hard-to-test changes.\n\nChange __exit_signal() to do posix_cpu_timers_exit_group() when\nthe old leader dies during exec. This is not the fix, just the\ntemporary hack to hide the problem for 2.6.37 and stable. IOW,\nthis is obviously wrong but this is what we currently have anyway:\ncpu timers do not work after mt exec.\n\nIn theory this change adds another race. The exiting leader can\ndetach the timers which were attached to the new leader. However,\nthe window between de_thread() and release_task() is small, we\ncan pretend that sys_timer_create() was called before de_thread().\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d16e15f5b029fc7d03540ba0e5fb23b0abb0ebe0", "tree": "382a6779f4eee476d86538372f0f6586d64ddb8b", "parents": [ "895021552d6ffe8a4d076cb5c4b1e700c33e96e1" ], "author": { "name": "Namhyung Kim", "email": "namhyung@gmail.com", "time": "Wed Oct 27 15:34:10 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Oct 27 18:03:13 2010 -0700" }, "message": "exit: add lock context annotation on find_new_reaper()\n\nfind_new_reaper() releases and regrabs tasklist_lock but was missing\nproper annotations. Add it. This remove following sparse warning:\n\n warning: context imbalance in \u0027find_new_reaper\u0027 - unexpected unlock\n\nSigned-off-by: Namhyung Kim \u003cnamhyung@gmail.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "3d5992d2ac7dc09aed8ab537cba074589f0f0a52", "tree": "0fbb51026e8aa13ddd7a6516c04471ceb97bc277", "parents": [ "0f4d208f1975f16f269134cee5f44c1f048581da" ], "author": { "name": "Ying Han", "email": "yinghan@google.com", "time": "Tue Oct 26 14:21:23 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Oct 26 16:52:05 2010 -0700" }, "message": "oom: add per-mm oom disable count\n\nIt\u0027s pointless to kill a task if another thread sharing its mm cannot be\nkilled to allow future memory freeing. A subsequent patch will prevent\nkills in such cases, but first it\u0027s necessary to have a way to flag a task\nthat shares memory with an OOM_DISABLE task that doesn\u0027t incur an\nadditional tasklist scan, which would make select_bad_process() an O(n^2)\nfunction.\n\nThis patch adds an atomic counter to struct mm_struct that follows how\nmany threads attached to it have an oom_score_adj of OOM_SCORE_ADJ_MIN.\nThey cannot be killed by the kernel, so their memory cannot be freed in\noom conditions.\n\nThis only requires task_lock() on the task that we\u0027re operating on, it\ndoes not require mm-\u003emmap_sem since task_lock() pins the mm and the\noperation is atomic.\n\n[rientjes@google.com: changelog and sys_unshare() code]\n[rientjes@google.com: protect oom_disable_count with task_lock in fork]\n[rientjes@google.com: use old_mm for oom_disable_count in exec]\nSigned-off-by: Ying Han \u003cyinghan@google.com\u003e\nSigned-off-by: David Rientjes \u003crientjes@google.com\u003e\nCc: KAMEZAWA Hiroyuki \u003ckamezawa.hiroyu@jp.fujitsu.com\u003e\nCc: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nCc: Rik van Riel \u003criel@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4e231c7962ce711c7d8c2a4dc23ecd1e8fc28363", "tree": "05af30e5c2f1084d8e77c89f0e55beaafa2194c0", "parents": [ "1b9a644fece117cfa5474a2388d6b89d1baf8ddf" ], "author": { "name": "Peter Zijlstra", "email": "a.p.zijlstra@chello.nl", "time": "Thu Sep 09 21:01:59 2010 +0200" }, "committer": { "name": "Ingo Molnar", "email": "mingo@elte.hu", "time": "Thu Sep 09 21:07:09 2010 +0200" }, "message": "perf: Fix up delayed_put_task_struct()\n\nI missed a perf_event_ctxp user when converting it to an array. Pull this\nlast user into perf_event.c as well and fix it up.\n\nSigned-off-by: Peter Zijlstra \u003ca.p.zijlstra@chello.nl\u003e\nLKML-Reference: \u003cnew-submission\u003e\nSigned-off-by: Ingo Molnar \u003cmingo@elte.hu\u003e\n" }, { "commit": "f362b73244fb16ea4ae127ced1467dd8adaa7733", "tree": "6aa3e767b527157b532c0620f2e9ef4c8f131c45", "parents": [ "d7627467b7a8dd6944885290a03a07ceb28c10eb" ], "author": { "name": "Daniel J Blueman", "email": "daniel.blueman@gmail.com", "time": "Tue Aug 17 23:56:55 2010 +0100" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue Aug 17 18:07:43 2010 -0700" }, "message": "Fix unprotected access to task credentials in waitid()\n\nUsing a program like the following:\n\n\t#include \u003cstdlib.h\u003e\n\t#include \u003cunistd.h\u003e\n\t#include \u003csys/types.h\u003e\n\t#include \u003csys/wait.h\u003e\n\n\tint main() {\n\t\tid_t id;\n\t\tsiginfo_t infop;\n\t\tpid_t res;\n\n\t\tid \u003d fork();\n\t\tif (id \u003d\u003d 0) { sleep(1); exit(0); }\n\t\tkill(id, SIGSTOP);\n\t\talarm(1);\n\t\twaitid(P_PID, id, \u0026infop, WCONTINUED);\n\t\treturn 0;\n\t}\n\nto call waitid() on a stopped process results in access to the child task\u0027s\ncredentials without the RCU read lock being held - which may be replaced in the\nmeantime - eliciting the following warning:\n\n\t\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\n\t[ INFO: suspicious rcu_dereference_check() usage. ]\n\t---------------------------------------------------\n\tkernel/exit.c:1460 invoked rcu_dereference_check() without protection!\n\n\tother info that might help us debug this:\n\n\trcu_scheduler_active \u003d 1, debug_locks \u003d 1\n\t2 locks held by waitid02/22252:\n\t #0: (tasklist_lock){.?.?..}, at: [\u003cffffffff81061ce5\u003e] do_wait+0xc5/0x310\n\t #1: (\u0026(\u0026sighand-\u003esiglock)-\u003erlock){-.-...}, at: [\u003cffffffff810611da\u003e]\n\twait_consider_task+0x19a/0xbe0\n\n\tstack backtrace:\n\tPid: 22252, comm: waitid02 Not tainted 2.6.35-323cd+ #3\n\tCall Trace:\n\t [\u003cffffffff81095da4\u003e] lockdep_rcu_dereference+0xa4/0xc0\n\t [\u003cffffffff81061b31\u003e] wait_consider_task+0xaf1/0xbe0\n\t [\u003cffffffff81061d15\u003e] do_wait+0xf5/0x310\n\t [\u003cffffffff810620b6\u003e] sys_waitid+0x86/0x1f0\n\t [\u003cffffffff8105fce0\u003e] ? child_wait_callback+0x0/0x70\n\t [\u003cffffffff81003282\u003e] system_call_fastpath+0x16/0x1b\n\nThis is fixed by holding the RCU read lock in wait_task_continued() to ensure\nthat the task\u0027s current credentials aren\u0027t destroyed between us reading the\ncred pointer and us reading the UID from those credentials.\n\nFurthermore, protect wait_task_stopped() in the same way.\n\nWe don\u0027t need to keep holding the RCU read lock once we\u0027ve read the UID from\nthe credentials as holding the RCU read lock doesn\u0027t stop the target task from\nchanging its creds under us - so the credentials may be outdated immediately\nafter we\u0027ve read the pointer, lock or no lock.\n\nSigned-off-by: Daniel J Blueman \u003cdaniel.blueman@gmail.com\u003e\nSigned-off-by: David Howells \u003cdhowells@redhat.com\u003e\nAcked-by: Paul E. McKenney \u003cpaulmck@linux.vnet.ibm.com\u003e\nAcked-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c7e49c1488ab20342eaaf38f1ca35a207f4c051d", "tree": "43876a69cafbb49ce86f71ba5db88018ca7cc036", "parents": [ "13d7e3a2dba6a79589ed34dc0b9114d7b5ff9eab" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Tue Aug 10 18:03:07 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Wed Aug 11 08:59:19 2010 -0700" }, "message": "ptrace: optimize exit_ptrace() for the likely case\n\nexit_ptrace() takes tasklist_lock unconditionally. We need this lock to\navoid the race with ptrace_traceme(), it acts as a barrier.\n\nChange its caller, forget_original_parent(), to call exit_ptrace() under\ntasklist_lock. Change exit_ptrace() to drop and reacquire this lock if\nneeded.\n\nThis allows us to add the fastpath list_empty(ptraced) check. In the\nlikely no-tracees case exit_ptrace() just returns and we avoid the lock()\n+ unlock() sequence.\n\n\"Zhang, Yanmin\" \u003cyanmin_zhang@linux.intel.com\u003e suggested to add this\ncheck, and he reports that this change adds about 11% improvement in some\ntests.\n\nSuggested-and-tested-by: \"Zhang, Yanmin\" \u003cyanmin_zhang@linux.intel.com\u003e\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b3ac022cb9dc5883505a88b159d1b240ad1ef405", "tree": "bffa035303cbe3c5bde048ac3d3154fb57059e2d", "parents": [ "dd98acf74762764fbc4382a1d9a244f11a2658cc" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:24 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:47 2010 -0700" }, "message": "proc: turn signal_struct-\u003ecount into \"int nr_threads\"\n\nNo functional changes, just s/atomic_t count/int nr_threads/.\n\nWith the recent changes this counter has a single user, get_nr_threads()\nAnd, none of its callers need the really accurate number of threads, not\nto mention each caller obviously races with fork/exit. It is only used to\nreport this value to the user-space, except first_tid() uses it to avoid\nthe unnecessary while_each_thread() loop in the unlikely case.\n\nIt is a bit sad we need a word in struct signal_struct for this, perhaps\nwe can change get_nr_threads() to approximate the number of threads using\nsignal-\u003elive and kill -\u003enr_threads later.\n\n[akpm@linux-foundation.org: coding-style fixes]\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Alexey Dobriyan \u003cadobriyan@gmail.com\u003e\nCc: \"Eric W. Biederman\" \u003cebiederm@xmission.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "97101eb41d0d3c97543878ce40e0b8a8b2747ed7", "tree": "d33299c8d4787d3b578ea432fdaceb965a2acd97", "parents": [ "a705be6b5e8b05f2ae51536ec709de921960326c" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:20 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "exit: move taskstats_tgid_free() from __exit_signal() to free_signal_struct()\n\nMove taskstats_tgid_free() from __exit_signal() to free_signal_struct().\n\nThis way signal-\u003estats never points to nowhere and we can read -\u003estats\nlockless.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Balbir Singh \u003cbalbir@linux.vnet.ibm.com\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nCc: Veaceslav Falico \u003cvfalico@redhat.com\u003e\nCc: Stanislaw Gruszka \u003csgruszka@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d40e48e02f3785b9342ee4eb3d7cc9f12981b7f5", "tree": "05f900ae17f4727c0e8b8c2c84d41b1cee7c13b0", "parents": [ "b7b8ff6373d4b910af081f76888395e6df53249d" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:19 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "exit: __exit_signal: use thread_group_leader() consistently\n\nCleanup:\n\n- Add the boolean, group_dead \u003d thread_group_leader(), for clarity.\n\n- Do not test/set sig \u003d\u003d NULL to detect the all-dead case, use this\n boolean.\n\n- Pass this boolen to __unhash_process() and use it instead of another\n thread_group_leader() call which needs -\u003egroup_leader.\n\n This can be considered as microoptimization, but hopefully this also\n allows us do do other cleanups later.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Balbir Singh \u003cbalbir@linux.vnet.ibm.com\u003e\nCc: Roland McGrath \u003croland@redhat.com\u003e\nCc: Veaceslav Falico \u003cvfalico@redhat.com\u003e\nCc: Stanislaw Gruszka \u003csgruszka@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "b7b8ff6373d4b910af081f76888395e6df53249d", "tree": "128a1b2cf026fb8b9791eaefb6f79f872999a5c6", "parents": [ "4ada856fb0ee62f6fe3aac3de726deac0640d929" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:18 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "signals: kill the awful task_rq_unlock_wait() hack\n\nNow that task-\u003esignal can\u0027t go away we can revert the horrible hack added\nby ad474caca3e2a0550b7ce0706527ad5ab389a4d4 (\"fix for\naccount_group_exec_runtime(), make sure -\u003esignal can\u0027t be freed under\nrq-\u003elock\").\n\nAnd we can do more cleanups sched_stats.h/posix-cpu-timers.c later.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Alan Cox \u003calan@linux.intel.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4ada856fb0ee62f6fe3aac3de726deac0640d929", "tree": "8dd3efbc19f6e1f1dac2ff7afdcd9658fb08b814", "parents": [ "ea6d290ca34c4fd91b7348338c0cc7bdeff94a35" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:17 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "signals: clear signal-\u003etty when the last thread exits\n\nWhen the last thread exits signal-\u003etty is freed, but the pointer is not\ncleared and points to nowhere.\n\nThis is OK. Nobody should use signal-\u003etty lockless, and it is no longer\npossible to take -\u003esiglock. However this looks wrong even if correct, and\nthe nice OOPS is better than subtle and hard to find bugs.\n\nChange __exit_signal() to clear signal-\u003etty under -\u003esiglock.\n\nNote: __exit_signal() needs more cleanups. It should not check \"sig !\u003d\nNULL\" to detect the all-dead case and we have the same issues with\nsignal-\u003estats.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Alan Cox \u003calan@linux.intel.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nAcked-by: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "ea6d290ca34c4fd91b7348338c0cc7bdeff94a35", "tree": "6e9bd367650d9233c5b6cf1059845f17cb1bc460", "parents": [ "4dec2a91fd7e8815d730afbfdcf085cbf53433ac" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:16 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "signals: make task_struct-\u003esignal immutable/refcountable\n\nWe have a lot of problems with accessing task_struct-\u003esignal, it can\n\"disappear\" at any moment. Even current can\u0027t use its -\u003esignal safely\nafter exit_notify(). -\u003esiglock helps, but it is not convenient, not\nalways possible, and sometimes it makes sense to use task-\u003esignal even\nafter this task has already dead.\n\nThis patch adds the reference counter, sigcnt, into signal_struct. This\nreference is owned by task_struct and it is dropped in\n__put_task_struct(). Perhaps it makes sense to export\nget/put_signal_struct() later, but currently I don\u0027t see the immediate\nreason.\n\nRename __cleanup_signal() to free_signal_struct() and unexport it. With\nthe previous changes it does nothing except kmem_cache_free().\n\nChange __exit_signal() to not clear/free -\u003esignal, it will be freed when\nthe last reference to any thread in the thread group goes away.\n\nNote:\n\t- when the last thead exits signal-\u003etty can point to nowhere, see\n\t the next patch.\n\n\t- with or without this patch signal_struct-\u003ecount should go away,\n\t or at least it should be \"int nr_threads\" for fs/proc. This will\n\t be addressed later.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nCc: Alan Cox \u003calan@linux.intel.com\u003e\nCc: Ingo Molnar \u003cmingo@elte.hu\u003e\nCc: Peter Zijlstra \u003cpeterz@infradead.org\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4dec2a91fd7e8815d730afbfdcf085cbf53433ac", "tree": "cd33cc793abf532d4ca212d09c1e8d3154a8b530", "parents": [ "ffdf91856c812646aa276711fa7e3ad80b00764a" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:15 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "fork/exit: move tty_kref_put() outside of __cleanup_signal()\n\ntty_kref_put() has two callsites in copy_process() paths,\n\n\t1. if copy_process() suceeds it is called before we copy\n\t signal-\u003etty from parent\n\n\t2. otherwise it is called from __cleanup_signal() under\n\t bad_fork_cleanup_signal: label\n\nIn both cases tty_kref_put() is not right and unneeded because we don\u0027t\nhave the balancing tty_kref_get(). Fortunately, this is harmless because\nthis can only happen without CLONE_THREAD, and in this case signal-\u003etty\nmust be NULL.\n\nRemove tty_kref_put() from copy_process() and __cleanup_signal(), and\nchange another caller of __cleanup_signal(), __exit_signal(), to call\ntty_kref_put() by hand.\n\nI hope this change makes sense by itself, but it is also needed to make\n-\u003esignal refcountable.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Alan Cox \u003calan@linux.intel.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nCc: Greg KH \u003cgreg@kroah.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "4a5999429739844367d0f77a65efdd7db8202779", "tree": "fff1158fa43466b30294a7db2c4629ca14973770", "parents": [ "d344193a05da89c97e965da2c5cbf687d7385eae" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:12 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "exit: avoid sig-\u003ecount in __exit_signal() to detect the group-dead case\n\nChange __exit_signal() to check thread_group_leader() instead of\natomic_dec_and_test(\u0026sig-\u003ecount). This must be equivalent, the group\nleader must be released only after all other threads have exited and\npassed __exit_signal().\n\nHenceforth sig-\u003ecount is not actually used, except in fs/proc for\nget_nr_threads/etc.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nCc: Veaceslav Falico \u003cvfalico@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "d344193a05da89c97e965da2c5cbf687d7385eae", "tree": "6267b68c31c4547f1749acd35d2a5bfaff919839", "parents": [ "09faef11df8c559a23e2405d123cb2683733a79a" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:11 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:46 2010 -0700" }, "message": "exit: avoid sig-\u003ecount in de_thread/__exit_signal synchronization\n\nde_thread() and __exit_signal() use signal_struct-\u003ecount/notify_count for\nsynchronization. We can simplify the code and use -\u003enotify_count only.\nInstead of comparing these two counters, we can change de_thread() to set\n-\u003enotify_count \u003d nr_of_sub_threads, then change __exit_signal() to\ndec-and-test this counter and notify group_exit_task.\n\nNote that __exit_signal() checks \"notify_count \u003e 0\" just for symmetry with\nexit_notify(), we could just check it is !\u003d 0.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nCc: Veaceslav Falico \u003cvfalico@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "9c3391684415c9dca239130d9e433a60a4edf04b", "tree": "a8019b964c625cfeeda0e9d85bafa48991546212", "parents": [ "269b005a28e124a341df4adef2c3661cf7371fcc" ], "author": { "name": "Oleg Nesterov", "email": "oleg@redhat.com", "time": "Wed May 26 14:43:10 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Thu May 27 09:12:45 2010 -0700" }, "message": "exit: exit_notify() can trust signal-\u003enotify_count \u003c 0\n\nsignal_struct-\u003ecount in its current form must die.\n\n- it has no reasons to be atomic_t\n\n- it looks like a reference counter, but it is not\n\n- otoh, we really need to make task-\u003esignal refcountable, just look at\n the extremely ugly task_rq_unlock_wait() called from __exit_signals().\n\n- we should change the lifetime rules for task-\u003esignal, it should be\n pinned to task_struct. We have a lot of code which can be simplified\n after that.\n\n- it is not needed! while the code is correct, any usage of this\n counter is artificial, except fs/proc uses it correctly to show the\n number of threads.\n\nThis series removes the usage of sig-\u003ecount from exit pathes.\n\nThis patch:\n\nNow that Veaceslav changed copy_signal() to use zalloc(), exit_notify()\ncan just check notify_count \u003c 0 to ensure the execing sub-threads needs\nthe notification from us. No need to do other checks, notify_count !\u003d 0\nmust always mean -\u003egroup_exit_task !\u003d NULL is waiting for us.\n\nSigned-off-by: Oleg Nesterov \u003coleg@redhat.com\u003e\nAcked-by: Roland McGrath \u003croland@redhat.com\u003e\nCc: Veaceslav Falico \u003cvfalico@redhat.com\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" }, { "commit": "c0ff7453bb5c7c98e0885fb94279f2571946f280", "tree": "8bb2b169a5145f0496575dbd2f48bb4b1c83f819", "parents": [ "708c1bbc9d0c3e57f40501794d9b0eed29d10fce" ], "author": { "name": "Miao Xie", "email": "miaox@cn.fujitsu.com", "time": "Mon May 24 14:32:08 2010 -0700" }, "committer": { "name": "Linus Torvalds", "email": "torvalds@linux-foundation.org", "time": "Tue May 25 08:06:57 2010 -0700" }, "message": "cpuset,mm: fix no node to alloc memory when changing cpuset\u0027s mems\n\nBefore applying this patch, cpuset updates task-\u003emems_allowed and\nmempolicy by setting all new bits in the nodemask first, and clearing all\nold unallowed bits later. But in the way, the allocator may find that\nthere is no node to alloc memory.\n\nThe reason is that cpuset rebinds the task\u0027s mempolicy, it cleans the\nnodes which the allocater can alloc pages on, for example:\n\n(mpol: mempolicy)\n\ttask1\t\t\ttask1\u0027s mpol\ttask2\n\talloc page\t\t1\n\t alloc on node0? NO\t1\n\t\t\t\t1\t\tchange mems from 1 to 0\n\t\t\t\t1\t\trebind task1\u0027s mpol\n\t\t\t\t0-1\t\t set new bits\n\t\t\t\t0\t \t clear disallowed bits\n\t alloc on node1? NO\t0\n\t ...\n\tcan\u0027t alloc page\n\t goto oom\n\nThis patch fixes this problem by expanding the nodes range first(set newly\nallowed bits) and shrink it lazily(clear newly disallowed bits). So we\nuse a variable to tell the write-side task that read-side task is reading\nnodemask, and the write-side task clears newly disallowed nodes after\nread-side task ends the current memory allocation.\n\n[akpm@linux-foundation.org: fix spello]\nSigned-off-by: Miao Xie \u003cmiaox@cn.fujitsu.com\u003e\nCc: David Rientjes \u003crientjes@google.com\u003e\nCc: Nick Piggin \u003cnpiggin@suse.de\u003e\nCc: Paul Menage \u003cmenage@google.com\u003e\nCc: Lee Schermerhorn \u003clee.schermerhorn@hp.com\u003e\nCc: Hugh Dickins \u003chugh.dickins@tiscali.co.uk\u003e\nCc: Ravikiran Thirumalai \u003ckiran@scalex86.org\u003e\nCc: KOSAKI Motohiro \u003ckosaki.motohiro@jp.fujitsu.com\u003e\nCc: Christoph Lameter \u003ccl@linux-foundation.org\u003e\nCc: Andi Kleen \u003candi@firstfloor.org\u003e\nSigned-off-by: Andrew Morton \u003cakpm@linux-foundation.org\u003e\nSigned-off-by: Linus Torvalds \u003ctorvalds@linux-foundation.org\u003e\n" } ], "next": "b257c14ceb1194a6181144210056d38f22127189" }