From mboxrd@z Thu Jan  1 00:00:00 1970
From: Christoph Hellwig <hch@infradead.org>
Subject: Re: [PATCH 01/17] kernel: add bl_list
Date: Sat, 16 Oct 2010 12:28:25 -0400
Message-ID: <20101016162825.GA20086@infradead.org>
References: <1285762729-17928-1-git-send-email-david@fromorbit.com>
 <1285762729-17928-2-git-send-email-david@fromorbit.com>
 <20100929215208.b0206460.akpm@linux-foundation.org>
 <20101016075521.GJ19147@amd>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: Andrew Morton <akpm@linux-foundation.org>,
	Dave Chinner <david@fromorbit.com>,
	linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org
To: Nick Piggin <npiggin@kernel.dk>
Return-path: <linux-kernel-owner@vger.kernel.org>
Content-Disposition: inline
In-Reply-To: <20101016075521.GJ19147@amd>
Sender: linux-kernel-owner@vger.kernel.org
List-Id: linux-fsdevel.vger.kernel.org

On Sat, Oct 16, 2010 at 06:55:21PM +1100, Nick Piggin wrote:
> > > +static inline void INIT_HLIST_BL_NODE(struct hlist_bl_node *h)
> > > +{
> > > +	h->next = NULL;
> > > +	h->pprev = NULL;
> > > +}
> > 
> > No need to shout.
> 
> Just following the rest of the lists.

Yes, just following hlist is logical.  Anyway, I don't think this
or the poisoning value really matter very much.