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

["Tian, Kevin" <kevin.tian@xxxxxxxxx>]

> From: Wei Liu [mailto:liuw@xxxxxxxxx]
> Sent: Thursday, May 26, 2011 11:30 AM
> 
> 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.

the comment says vmsi_deliver_irq while your code has vmsi_deliver_pirq...

> >
> >> Â Â 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.
> 

You didn't catch my comment.

you change existing HVMOP command index which breaks backward
compatibility.

Thanks
Kevin

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