All of lore.kernel.org
 help / color / mirror / Atom feed
From: Waiman Long <waiman.long@hp.com>
To: Al Viro <viro@ZenIV.linux.org.uk>
Cc: Jeff Layton <jlayton@redhat.com>,
	Miklos Szeredi <mszeredi@suse.cz>, Ingo Molnar <mingo@redhat.com>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org,
	Peter Zijlstra <peterz@infradead.org>,
	Steven Rostedt <rostedt@goodmis.org>,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Andi Kleen <andi@firstfloor.org>,
	"Chandramouleeswaran, Aswin" <aswin@hp.com>,
	"Norton, Scott J" <scott.norton@hp.com>
Subject: Re: [PATCH v5 00/12] Lockless update of reference count protected by spinlock
Date: Fri, 05 Jul 2013 11:29:10 -0400	[thread overview]
Message-ID: <51D6E646.6020703@hp.com> (raw)
In-Reply-To: <20130705150224.GS4165@ZenIV.linux.org.uk>

On 07/05/2013 11:02 AM, Al Viro wrote:
> On Fri, Jul 05, 2013 at 10:47:33AM -0400, Waiman Long wrote:
>> v4->v5:
>>   - Add a d_count() helper for readonly access of reference count and
>>     change all references to d_count outside of dcache.c, dcache.h
>>     and namei.c to use d_count().
> Sigh...  You are breaking bisectability again.  This helper should
> be introduced *first*, along with conversion to it.  Then you can
> do the rest on top of that.
>
> I've just pushed such commit into vfs.git#for-linus; please, do the rest
> on top of it.  And keep it bisectable, i.e. so that at any intermediate
> point the tree would build and work.

I am sorry. I didn't change anything in the dentry structure in patch 3. 
So putting patches 4-11 on top of it won't break the build.

Regards,
Longman

  reply	other threads:[~2013-07-05 15:29 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-05 14:47 [PATCH v5 00/12] Lockless update of reference count protected by spinlock Waiman Long
2013-07-05 14:47 ` [PATCH v5 01/12] spinlock: A new lockref structure for lockless update of refcount Waiman Long
2013-07-05 18:59   ` Thomas Gleixner
2013-07-08 14:21     ` Waiman Long
2013-07-15 14:41       ` Thomas Gleixner
2013-07-15 21:24         ` Waiman Long
2013-07-15 23:47           ` Thomas Gleixner
2013-07-16  1:40             ` Waiman Long
2013-07-05 14:47 ` [PATCH v5 02/12] spinlock: Enable x86 architecture to do lockless refcount update Waiman Long
2013-07-05 20:19   ` Thomas Gleixner
2013-07-05 14:47 ` [PATCH v5 03/12] dcache: rename d_count field of dentry to d_refcount Waiman Long
2013-07-05 15:02 ` [PATCH v5 00/12] Lockless update of reference count protected by spinlock Al Viro
2013-07-05 15:29   ` Waiman Long [this message]
2013-07-05 15:31     ` Waiman Long
2013-07-05 17:54     ` Al Viro
2013-07-05 18:56       ` Waiman Long
2013-07-05 20:33 ` Thomas Gleixner
2013-07-08 14:22   ` Waiman Long

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51D6E646.6020703@hp.com \
    --to=waiman.long@hp.com \
    --cc=andi@firstfloor.org \
    --cc=aswin@hp.com \
    --cc=benh@kernel.crashing.org \
    --cc=jlayton@redhat.com \
    --cc=linux-fsdevel@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.com \
    --cc=mszeredi@suse.cz \
    --cc=peterz@infradead.org \
    --cc=rostedt@goodmis.org \
    --cc=scott.norton@hp.com \
    --cc=tglx@linutronix.de \
    --cc=torvalds@linux-foundation.org \
    --cc=viro@ZenIV.linux.org.uk \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.