From mboxrd@z Thu Jan  1 00:00:00 1970
From: Amit Shah <amit.shah@redhat.com>
Subject: Re: [RFC][PATCH 0/2] Fix guest shared interrupt with in-kernel irqchip
Date: Fri, 3 Oct 2008 16:17:54 +0530
Message-ID: <200810031617.54294.amit.shah@redhat.com>
References: <D8078B8B3B09934AA9F8F2D5FB3F28CE08A3B9C7B0@pdsmsx502.ccr.corp.intel.com> <200810031306.53295.amit.shah@redhat.com> <20081003103754.GA29951@yukikaze>
Mime-Version: 1.0
Content-Type: text/plain;
  charset="iso-8859-1"
Content-Transfer-Encoding: 7bit
Cc: Avi Kivity <avi@redhat.com>, "Yang, Sheng" <sheng.yang@intel.com>,
	"'kvm@vger.kernel.org'" <kvm@vger.kernel.org>,
	sheng@linux.intel.com
To: Sheng Yang <yasker@gmail.com>
Return-path: <kvm-owner@vger.kernel.org>
Received: from mx2.redhat.com ([66.187.237.31]:45871 "EHLO mx2.redhat.com"
	rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
	id S1754641AbYJCKtQ (ORCPT <rfc822;kvm@vger.kernel.org>);
	Fri, 3 Oct 2008 06:49:16 -0400
In-Reply-To: <20081003103754.GA29951@yukikaze>
Content-Disposition: inline
Sender: kvm-owner@vger.kernel.org
List-ID: <kvm.vger.kernel.org>

* On Friday 03 Oct 2008 16:07:54 Sheng Yang wrote:
> > Just one thing: can you make the irq_source_id argument the 3rd one and
> > the 'level' as the last one? It's clearer to think of it that way.
>
> Yeah, put the irq_source_id at the end of parameter list is a little ugly.
> But I think it's better to keep "irq" and "level" together, so I move
> irq_souce_id to the 2nd place. :)

I think that's even better.