stefano.stabellini at eu
Jun 27, 2012, 8:25 AM
Post #3 of 5
On Wed, 27 Jun 2012, Ian Campbell wrote:
Re: libxl: disable msitranslate by default
[In reply to]
> On Wed, 2012-06-27 at 14:56 +0100, Stefano Stabellini wrote:
> > msitranslate is known to cause problems with some device drivers,
> > because it sets the real device in MSI mode while making the guest think
> > is actually in legacy interrupts mode. Some drivers are able to spot this
> > inconsistency and break (Nvidia drivers for example).
> > Disable msitranslate by default.
> > Signed-off-by: Stefano Stabellini <stefano.stabellini [at] eu>
> Acked-by: Ian Campbell <ian.campbell [at] citrix>
> I saw a qemu (trad) patch and I see I've got an xl patch further down my
> inbox. There is no sequencing requirement here, is there?
> I wonder if this field shouyld be a defbool instead of just a bool? I
> don't recall why I didn't transition it when I made the defbool stuff.
I think it could be a defbool
> > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c
> > index afa0af6..c80b9fb 100644
> > --- a/tools/libxl/xl_cmdimpl.c
> > +++ b/tools/libxl/xl_cmdimpl.c
> > @@ -562,7 +562,7 @@ static void parse_config_data(const char *config_source,
> > XLU_Config *config;
> > XLU_ConfigList *cpus, *vbds, *nics, *pcis, *cvfbs, *cpuids;
> > int pci_power_mgmt = 0;
> > - int pci_msitranslate = 1;
> > + int pci_msitranslate = 0;
> > int pci_permissive = 0;
> > int e;
Xen-devel mailing list
Xen-devel [at] lists