Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760612Ab2EIRtA (ORCPT ); Wed, 9 May 2012 13:49:00 -0400 Received: from mail-pz0-f46.google.com ([209.85.210.46]:54024 "EHLO mail-pz0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754963Ab2EIRs6 (ORCPT ); Wed, 9 May 2012 13:48:58 -0400 Date: Wed, 9 May 2012 10:48:53 -0700 From: Tejun Heo To: Alan Stern Cc: "Eric W. Biederman" , Peter Zijlstra , Kernel development list Subject: Re: Lockdep false positive in sysfs Message-ID: <20120509174853.GF24636@google.com> References: <20120509174114.GE24636@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1544 Lines: 41 On Wed, May 09, 2012 at 01:47:34PM -0400, Alan Stern wrote: > On Wed, 9 May 2012, Tejun Heo wrote: > > > Hello, > > > > On Tue, May 08, 2012 at 02:53:11PM -0400, Alan Stern wrote: > > > On Mon, 7 May 2012, Tejun Heo wrote: > > > > > > > On Mon, May 07, 2012 at 05:51:52PM -0400, Alan Stern wrote: > > > > > I guess in the end it's a question of balance. Which has more > > > > > overhead, adding a few function calls here and there, or adding a new > > > > > flags field to every struct attribute? > > > > > > > > Yes, and there are different types of overheads. I'm happy to trade > > > > some runtime memory overhead under debugging mode for lower code > > > > complexity. Lock proving is pretty expensive anyway. I don't think > > > > there's much point in trying to optimize some bytes from struct > > > > attributes. > > > > > > Okay, then what do you think about this approach? It does seem smaller > > > and simpler than the previous attempt. > > > > > > And I did try to avoid unnecessary bloat; if lockdep isn't being used > > > then the extra attribute flag isn't present. > > > > Yeap, looks good to me. > > Unless there are any objections from Eric or Peter in the next few > days, I'll submit it. Can I add your Acked-by? Sure. Thanks for the persistence. -- tejun -- 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/