Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758906AbZAWRlJ (ORCPT ); Fri, 23 Jan 2009 12:41:09 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1755591AbZAWRkz (ORCPT ); Fri, 23 Jan 2009 12:40:55 -0500 Received: from bombadil.infradead.org ([18.85.46.34]:45813 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754851AbZAWRky (ORCPT ); Fri, 23 Jan 2009 12:40:54 -0500 Subject: Re: [PATCH] x86,mm: fix pte_free() From: Peter Zijlstra To: Ingo Molnar Cc: Linus Torvalds , Nick Piggin , Hugh Dickins , Thomas Gleixner , Andrew Morton , L-K , linux-mm , David Howells In-Reply-To: <20090123173421.GA30980@elte.hu> References: <1232728669.4826.143.camel@laptop> <20090123173421.GA30980@elte.hu> Content-Type: text/plain Date: Fri, 23 Jan 2009 18:39:46 +0100 Message-Id: <1232732387.4850.1.camel@laptop> Mime-Version: 1.0 X-Mailer: Evolution 2.24.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 632 Lines: 15 On Fri, 2009-01-23 at 18:34 +0100, Ingo Molnar wrote: > So i agree with the fix, but the patch does not look right: shouldnt that > be pgtable_page_dtor(pte), so that we get ->mapping cleared via > pte_lock_deinit()? (which i guess your intention was here - this probably > wont even build) Yeah, I somehow fudged it, already send out a better one. -- One of them days I guess :-( -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/