Google
  Web www.spinics.net

RE: [PATCH 01/02] ia64: Move the macro definitions related to MSI to one header file.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]


Agree, it should go to Tony's Tree in theory, but we have to push it into Avi's tree first to fix an build issue.  So please help to commit the patch to kvm.git with Tony's Ack, Thanks!
Xiantao

-----Original Message-----
From: Marcelo Tosatti [mailto:mtosatti@xxxxxxxxxx] 
Sent: Thursday, February 19, 2009 7:03 AM
To: Zhang, Xiantao
Cc: Luck, Tony; linux-ia64@xxxxxxxxxxxxxxx; Avi Kivity; kvm@xxxxxxxxxxxxxxx; kvm-ia64@xxxxxxxxxxxxxxx
Subject: Re: [PATCH 01/02] ia64: Move the macro definitions related to MSI to one header file.


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 <tony.luck@xxxxxxxxx>
> 
> --
> To unsubscribe from this list: send the line "unsubscribe kvm" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe kvm-ia64" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Home]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]    [Yosemite Photos]    [Video Projectors]     [PDAs]     [Free Online Dating]     [Hacking TiVo]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Devices]     [Big List of Linux Books]     [16.7MP]

  Powered by Linux