From mboxrd@z Thu Jan  1 00:00:00 1970
From: David Mosberger <davidm@napali.hpl.hp.com>
Date: Fri, 23 Jul 2004 19:34:19 +0000
Subject: Re: take2: [PATCH] Vector sharing (Large I/O system support)
Message-Id: <16641.26683.910827.567091@napali.hpl.hp.com>
List-Id: <linux-ia64.vger.kernel.org>
References: <40FE09B5.7020109@jp.fujitsu.com>
In-Reply-To: <40FE09B5.7020109@jp.fujitsu.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: linux-ia64@vger.kernel.org


  Kenji> This is the updated patch for vector sharing.  Summary of
  Kenji> changes: - Changed the name of some data structures.  -
  Kenji> Removed '__cacheline_aligned' from iosapic_intr_info

I'm mostly OK with the patch, but I don't like the list handling very
much.  For example, maintaining a separate "count" field for the list
just seems unnecessary and error-prone.  It would be nicer if you
could restructure the code such that you can use list_empty() instead.

	--david