From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail138.messagelabs.com (mail138.messagelabs.com [216.82.249.35]) by kanga.kvack.org (Postfix) with SMTP id 6EFBF6B003D for ; Fri, 3 Apr 2009 12:22:09 -0400 (EDT) Date: Fri, 3 Apr 2009 09:22:17 -0700 From: Chris Wright Subject: Re: [PATCH 5/4] update ksm userspace interfaces Message-ID: <20090403162217.GB13121@x200.localdomain> References: <49D23CD1.9090208@codemonkey.ws> <20090331162525.GU9137@random.random> <49D24A02.6070000@codemonkey.ws> <20090402012215.GE1117@x200.localdomain> <49D424AF.3090806@codemonkey.ws> <20090402053114.GF1117@x200.localdomain> <49D4BE64.8020508@redhat.com> <49D5E1EE.6030707@redhat.com> <49D5E9B5.1020101@redhat.com> <49D5EE1F.4060009@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <49D5EE1F.4060009@redhat.com> Sender: owner-linux-mm@kvack.org To: Gerd Hoffmann Cc: Izik Eidus , Chris Wright , Anthony Liguori , Andrea Arcangeli , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, linux-mm@kvack.org, avi@redhat.com, riel@redhat.com, jeremy@goop.org, mtosatti@redhat.com, hugh@veritas.com, corbet@lwn.net, yaniv@redhat.com, dmonakhov@openvz.org List-ID: * Gerd Hoffmann (kraxel@redhat.com) wrote: > mmput() call was in ->release() callback, ->release() in turn never was > called because the kernel didn't zap the mappings because of the > reference ... Don't have this issue. That mmput() is not tied to zapping mappings, rather zapping files. IOW, I think you're saying exit_mmap() wasn't running due to your get_task_mm() (quite understandable, you still hold a ref), whereas this ref is tied to exit_files(). So do_exit would do: exit_mm mmput <-- not dropped yet exit_files ->release mmput <-- dropped here thanks, -chris -- 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/ . Don't email: email@kvack.org