From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932526Ab1AKR6v (ORCPT ); Tue, 11 Jan 2011 12:58:51 -0500 Received: from relay3.sgi.com ([192.48.152.1]:57673 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S932274Ab1AKR6t (ORCPT ); Tue, 11 Jan 2011 12:58:49 -0500 Subject: Re: [announce] vfs-scale git tree update From: Alex Elder Reply-To: aelder@sgi.com To: Linus Torvalds Cc: Nick Piggin , Al Viro , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org In-Reply-To: References: <20110107075831.GA14915@amd> <1294763679.2435.72.camel@doink> Content-Type: text/plain; charset="UTF-8" Date: Tue, 11 Jan 2011 11:57:48 -0600 Message-ID: <1294768668.2435.177.camel@doink> Mime-Version: 1.0 X-Mailer: Evolution 2.30.3 Content-Transfer-Encoding: 7bit X-OriginalArrivalTime: 11 Jan 2011 17:57:49.0337 (UTC) FILETIME=[0F43F890:01CBB1B9] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 2011-01-11 at 08:51 -0800, Linus Torvalds wrote: > On Tue, Jan 11, 2011 at 8:34 AM, Alex Elder wrote: > > > > FYI, when using this code, as merged by Linus, I hit the > > BUG_ON() at the beginning of d_set_d_op() when it's called > > by autofs4_dir_mkdir(). I managed to work around it by > > just commenting out those BUG_ON() calls but it's something > > that ought to get addressed properly. > > Yeah, removing the BUG_ON() isn't the right thing to do - it means > that autofs4 is obviously setting the dentry ops twice for the same > dentry. > > Possibly the thing could be relaxed to allow setting the _same_ d_op > pointer, ie do something like > > if (dentry->d_op == op) > return; > > at the top of that function. But looking at it, I don't think that > fixes the autofs4 issue. That's easy enough, but it seems everybody else ensures this gets done just once per dentry, and it would be nice to preserve that "tightness" if possible. > The fact that autofs4 does "d_add()" before it sets the d_ops (or > other dentry state, for that matter) looks a bit scary. To me that > smells like it might get a dentry lookup hit before it's actually > fully done. Agreed. > Does it make any difference if you move the various d_add() calls down > to the end of the functions to when the "dentry" has really been > instantiated? Looking at it quickly, I don't think that would matter for the case at hand. I.e., that might be safer but it doesn't address the fact that these fields are getting initialized multiple times. -Alex