From mboxrd@z Thu Jan  1 00:00:00 1970
From: Peter Zijlstra <peterz@infradead.org>
Subject: Re: [PATCH 2/2] dentry: move to per-sb LRU locks
Date: Mon, 08 Aug 2011 15:02:24 +0200
Message-ID: <1312808544.10488.31.camel@twins>
References: <1312787021-11324-1-git-send-email-david@fromorbit.com>
	 <1312787021-11324-3-git-send-email-david@fromorbit.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Cc: viro@ZenIV.linux.org.uk, linux-kernel@vger.kernel.org,
	linux-fsdevel@vger.kernel.org
To: Dave Chinner <david@fromorbit.com>
Return-path: <linux-fsdevel-owner@vger.kernel.org>
Received: from merlin.infradead.org ([205.233.59.134]:38497 "EHLO
	merlin.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1753233Ab1HHNCj convert rfc822-to-8bit (ORCPT
	<rfc822;linux-fsdevel@vger.kernel.org>);
	Mon, 8 Aug 2011 09:02:39 -0400
In-Reply-To: <1312787021-11324-3-git-send-email-david@fromorbit.com>
Sender: linux-fsdevel-owner@vger.kernel.org
List-ID: <linux-fsdevel.vger.kernel.org>

On Mon, 2011-08-08 at 17:03 +1000, Dave Chinner wrote:
> +       /* s_dentry_lru_lock protects s_dentry_lru, s_nr_dentry_unused */
> +       spinlock_t              s_dentry_lru_lock ____cacheline_aligned_in_smp;
>         struct list_head        s_dentry_lru;   /* unused dentry lru */ 

Wouldn't it make sense to have both those on the same cacheline?