From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keir Fraser Subject: Re: [PATCH 1/2] Xen: enabling emulated MSI injection Date: Thu, 26 May 2011 10:10:51 +0100 Message-ID: References: <20110526084331.GB9790@whitby.uk.xensource.com> Mime-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20110526084331.GB9790@whitby.uk.xensource.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xensource.com Errors-To: xen-devel-bounces@lists.xensource.com To: Tim Deegan , Wei Liu Cc: "Tian, Kevin" , "xen-devel@lists.xensource.com" , Stefano Stabellini List-Id: xen-devel@lists.xenproject.org On 26/05/2011 09:43, "Tim Deegan" wrote: >> +/* MSI injection for emulated devices */ >> +#define HVMOP_inj_msi 16 >> +struct xen_hvm_inj_msi { >> + /* Domain to be injected */ >> + domid_t domid; >> + /* Address (0xfeeXXXXX) */ >> + uint64_t addr; >> + /* Data -- lower 32 bits */ >> + uint32_t data; >> +}; > > Please rearrange this so that it has the same size and layout on 32-bit > and 64-bit builds. You can do this most easily be simply swapping the second and third fields. -- Keir