Mailing List Archive: Xen: Devel

libxl: disable msitranslate by default

Index | Next | Previous | View Threaded

stefano.stabellini at eu

Jun 27, 2012, 6:56 AM

Post #1 of 5 (72 views)
Permalink

libxl: disable msitranslate by default

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>

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
http://lists.xen.org/xen-devel

Ian.Campbell at citrix

Jun 27, 2012, 7:59 AM

Post #2 of 5 (78 views)
Permalink

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.

>
> 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
http://lists.xen.org/xen-devel

stefano.stabellini at eu

Jun 27, 2012, 8:25 AM

Post #3 of 5 (72 views)
Permalink

Re: libxl: disable msitranslate by default [In reply to]

On Wed, 27 Jun 2012, Ian Campbell wrote:
> 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?

nope

> 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
http://lists.xen.org/xen-devel

Ian.Jackson at eu

Jun 28, 2012, 7:42 AM

Post #4 of 5 (73 views)
Permalink

Re: libxl: disable msitranslate by default [In reply to]

Stefano Stabellini writes ("Re: [Xen-devel] libxl: disable msitranslate by default"):
> On Wed, 27 Jun 2012, Ian Campbell wrote:
> > 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?
>
> nope

Thanks.

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

I'll apply the patch as-is but perhaps a followup to change it to a
defbool would be good ?

Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel [at] lists
http://lists.xen.org/xen-devel

Ian.Jackson at eu

Jun 28, 2012, 7:50 AM

Post #5 of 5 (71 views)
Permalink

Re: libxl: disable msitranslate by default [In reply to]

Ian Campbell writes ("Re: [Xen-devel] libxl: disable msitranslate by default"):
> 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>

Committed-by: Ian Jackson <ian.jackson [at] eu>

_______________________________________________
Xen-devel mailing list
Xen-devel [at] lists
http://lists.xen.org/xen-devel

Index | Next | Previous | View Threaded

Interested in having your list archived? Contact Gossamer Threads