Messages in this thread | | | Date | Thu, 1 Jul 2010 22:38:46 +0900 | From | Minchan Kim <> | Subject | Re: [PATCH 02/11] oom: oom_kill_process() doesn't select kthread child |
| |
On Thu, Jul 01, 2010 at 09:07:02AM +0900, KOSAKI Motohiro wrote: > > On Wed, Jun 30, 2010 at 06:27:52PM +0900, KOSAKI Motohiro wrote: > > > Now, select_bad_process() have PF_KTHREAD check, but oom_kill_process > > > doesn't. It mean oom_kill_process() may choose wrong task, especially, > > > when the child are using use_mm(). > > > > Is it possible child is kthread even though parent isn't kthread? > > Usually unhappen. but crappy driver can do any strange thing freely. > As I said, oom code should have conservative assumption as far as possible.
Okay. You change the check with oom_unkillable_task at last. The oom_unkillable_task is generic function so that the kthread check in oom_kill_process is tivial, I think.
Reviewed-by: Minchan Kim <minchan.kim@gmail.com>
> > >
-- Kind regards, Minchan Kim
| |