Re: [Xen-devel] [PATCH 1/2] Xen: enabling emulated MSI injection

[Wei Liu <liuw@xxxxxxxxx>]

On Thu, May 26, 2011 at 11:21 AM, Tian, Kevin <kevin.tian@xxxxxxxxx> wrote:
>> From: Wei Liu
>> Sent: Thursday, May 26, 2011 11:09 AM
>>
>> commit 176dc2a26b4b9dd0fe30fab3b168722766218245
>> Author: Wei Liu <liuw@xxxxxxxxx>
>> Date: Â Thu May 26 10:23:01 2011 +0800
>>
>> Â Â x86: Add a new operation in HVMOP to inject emulated MSI.
>>
>> Â Â The original vmsi_deliver is renamed to vmsi_deliver_irq. New
>
> in your patch you mean vmsi_deliver_pirq which is inconsistent.
>
>> Â Â vmsi_deliver is dedicated to the actually delivering.
>>
>> Â Â Signed-off-by: Wei Liu <liuw@xxxxxxxxx>
>>
>> --- a/xen/include/public/hvm/hvm_op.h
>> +++ b/xen/include/public/hvm/hvm_op.h
>> @@ -82,11 +82,24 @@ typedef enum {
>> Â Â ÂHVMMEM_mmio_dm, Â Â Â Â Â Â/* Reads and write go to the device
>> model */
>> Â} hvmmem_type_t;
>>
>> +/* MSI injection for emulated devices */
>> +#define HVMOP_inj_msi     6
>> +struct xen_hvm_inj_msi {
>> + Â Â/* Domain to be injected */
>> +  Âdomid_t  domid;
>> + Â Â/* Address (0xfeeXXXXX) */
>> + Â Âuint64_t Âaddr;
>> + Â Â/* Data -- lower 32 bits */
>> + Â Âuint32_t Âdata;
>> +};
>> +typedef struct xen_hvm_inj_msi xen_hvm_inj_msi_t;
>> +DEFINE_XEN_GUEST_HANDLE(xen_hvm_inj_msi_t);
>> +
>> Â/* Following tools-only interfaces may change in future. */
>> Â#if defined(__XEN__) || defined(__XEN_TOOLS__)
>>
>> Â/* Track dirty VRAM. */
>> -#define HVMOP_track_dirty_vram  Â6
>> +#define HVMOP_track_dirty_vram  Â7
>> Âstruct xen_hvm_track_dirty_vram {
>> Â Â Â/* Domain to be tracked. */
>> Â Â Âdomid_t Âdomid;
>
> This breaks backward API compatibility.
>
> Thanks
> Kevin
>

Stefano suggests this rename.

The only function that invokes vmsi_deliver() is hvm_pci_msi_assert(),
IIRC. That has been taken care of.

-- 
Best regards
Wei Liu
Twitter: @iliuw
Site: http://liuw.name

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel