From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marcelo Tosatti Subject: Re: [PATCH 01/02] ia64: Move the macro definitions related to MSI to one header file. Date: Wed, 18 Feb 2009 20:02:58 -0300 Message-ID: <20090218230258.GA28402@amt.cnet> References: <706158FABBBA044BAD4FE898A02E4BC2241A31DA@pdsmsx503.ccr.corp.intel.com> <57C9024A16AD2D4C97DC78E552063EA361791E04@orsmsx505.amr.corp.intel.com> <706158FABBBA044BAD4FE898A02E4BC22422A91C@pdsmsx503.ccr.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "Luck, Tony" , "linux-ia64@vger.kernel.org" , Avi Kivity , "kvm@vger.kernel.org" , "kvm-ia64@vger.kernel.org" To: "Zhang, Xiantao" Return-path: Received: from mx2.redhat.com ([66.187.237.31]:33646 "EHLO mx2.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751580AbZBRXDd (ORCPT ); Wed, 18 Feb 2009 18:03:33 -0500 Content-Disposition: inline In-Reply-To: <706158FABBBA044BAD4FE898A02E4BC22422A91C@pdsmsx503.ccr.corp.intel.com> Sender: kvm-owner@vger.kernel.org List-ID: Looks good, should go through Tony's tree I believe. On Wed, Feb 18, 2009 at 10:17:56AM +0800, Zhang, Xiantao wrote: > Thanks, Tony! It should not break anything due to no changes about code logic. :) > > Avi, > Could you help to commit the patches with Tony's Ack ? Thanks! > Xiantao > > > Luck, Tony wrote: > >> For supporting kvm's MSI, we have to move some macros > >> from ia64_msi.c out to avoide duplicate them. In addition, > >> to keep them consistent with x86's , I also changed some > >> macros' name. How do you think of the patch ? If you > >> agree to the changes, could you add your Sign-off-by to > >> the patch, and Avi may check-in it to kvm.git first to > >> fix an emergent build issue for kvm/ia64. Thanks! > > > > Looks OK to me (I didn't test it, or even build it ... so I hope you > > did!). > > > > Acked-by: Tony Luck > > -- > To unsubscribe from this list: send the line "unsubscribe kvm" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html