From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with ESMTP id 064426B002C for ; Tue, 11 Oct 2011 19:36:42 -0400 (EDT) Received: from wpaz37.hot.corp.google.com (wpaz37.hot.corp.google.com [172.24.198.101]) by smtp-out.google.com with ESMTP id p9BNacNb015093 for ; Tue, 11 Oct 2011 16:36:39 -0700 Received: from vcbfl17 (vcbfl17.prod.google.com [10.220.204.81]) by wpaz37.hot.corp.google.com with ESMTP id p9BNYKNx007440 (version=TLSv1/SSLv3 cipher=RC4-SHA bits=128 verify=NOT) for ; Tue, 11 Oct 2011 16:36:32 -0700 Received: by vcbfl17 with SMTP id fl17so12256vcb.7 for ; Tue, 11 Oct 2011 16:36:32 -0700 (PDT) Date: Tue, 11 Oct 2011 16:36:28 -0700 (PDT) From: David Rientjes Subject: Re: [patch v2] oom: thaw threads if oom killed thread is frozen before deferring In-Reply-To: <20111011063336.GA23284@tiehlicka.suse.cz> Message-ID: References: <20111011063336.GA23284@tiehlicka.suse.cz> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Michal Hocko Cc: KOSAKI Motohiro , Andrew Morton , Oleg Nesterov , KAMEZAWA Hiroyuki , "Rafael J. Wysocki" , linux-mm@kvack.org, Tejun Heo On Tue, 11 Oct 2011, Michal Hocko wrote: > The patch looks good but we still need other 2 patches > (http://comments.gmane.org/gmane.linux.kernel.mm/68578), right? > For the lguest patch, Rusty is the maintainer and has already acked the patch, so I think it should be merged through him. I don't see a need for the second patch since we'll now detect frozen oom killed tasks on retry and don't need to kill them directly when oom killed (it just adds additional, unnecessary code). > Anyway, I thought that we agreed on the other approach suggested by > Tejun (make frozen tasks oom killable without thawing). Even in that > case we want the first patch > (http://permalink.gmane.org/gmane.linux.kernel.mm/68576). If that's possible, then we can just add Tejun to add a follow-up patch to remove the thaw directly in the oom killer. I'm thinking that won't be possible for 3.2, though, so I don't know why we'd remove oom-thaw-threads-if-oom-killed-thread-is-frozen-before-deferring.patch from -mm? -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org