From mboxrd@z Thu Jan 1 00:00:00 1970 From: Nick Piggin Subject: Re: [PATCH 11/46] fs: dcache scale hash Date: Mon, 13 Dec 2010 16:05:58 +1100 Message-ID: <20101213050558.GA8624@amd> References: <3eb32695435ae6c5fd1601467d78b560b5058e2b.1290852959.git.npiggin@kernel.dk> <20101209060911.GB8259@dastard> <20101209062801.GA3749@amd> <20101209081756.GE8259@dastard> <20101209234258.GB9925@dastard> <20101210023520.GC3331@amd> <20101210090126.GH8259@dastard> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Nick Piggin , Nick Piggin , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org To: Dave Chinner Return-path: Content-Disposition: inline In-Reply-To: <20101210090126.GH8259@dastard> Sender: linux-kernel-owner@vger.kernel.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Dec 10, 2010 at 08:01:26PM +1100, Dave Chinner wrote: > > I would prefer more in-depth review than from someone who doesn't know > > d_lock protects d_flags, > > Your implication about my competence is incorrect and entirely > inappropriate. Ad hominen attacks don't improve your argument or > encourage other people to review your code. I'll also just add that if you don't like ad-hominen attacks, you shouldn't have made implications about my integrity and honesty by accusing me of wanting a rubber stamp, rather than a real review. Which was before I suggested that you were confused about d_flags locking, you'll note.