All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ting-Wei Lan <lantw44@gmail.com>
To: Andrew Cooper <andrew.cooper3@citrix.com>,
	"Tian, Kevin" <kevin.tian@intel.com>,
	"xen-devel@lists.xen.org" <xen-devel@lists.xen.org>
Cc: "Zhang, Yang Z" <yang.z.zhang@intel.com>,
	Ting-Wei Lan <lantw44@gmail.com>, Jan Beulich <jbeulich@suse.com>
Subject: Re: [PATCH] VT-d: add iommu=igfx_off option to workaround graphics issues
Date: Mon, 20 Jul 2015 18:44:28 +0800	[thread overview]
Message-ID: <1437389068.21078.5.camel@gmail.com> (raw)
In-Reply-To: <55ACAF80.4000608@citrix.com>

於 一,2015-07-20 於 09:21 +0100,Andrew Cooper 提到:
> On 20/07/2015 02:28, Tian, Kevin wrote:
> > > From: Ting-Wei Lan [mailto:lantw44@gmail.com]
> > > Sent: Saturday, July 18, 2015 3:06 AM
> > > 
> > > When using Linux >= 3.19 (commit 47591df) as dom0 on some Intel 
> > > Ironlake
> > > devices, It is possible to encounter graphics issues that make 
> > > screen
> > > unreadable or crash the system. It was reported in freedesktop 
> > > bugzilla:
> > > 
> > > https://bugs.freedesktop.org/show_bug.cgi?id=90037
> > > 
> > > As we still cannot find a proper fix for this problem, this patch 
> > > adds
> > > iommu=igfx_off option that is similar to Linux 
> > > intel_iommu=igfx_off for
> > > users to manually workaround the problem.
> > > 
> > > Signed-off-by: Ting-Wei Lan <lantw44@gmail.com>
> > Since igfx works before, I'd think a more proper fix should be on 
> > the
> > bisected Linux commit or i915 to have two working correctly 
> > together.
> > Otherwise this patch is just hiding problem.
> 
> The linux commit is the one which actually fixes PAT support for 
> Linux
> under Xen.
> 
> It will cause the i915 driver to actually get WC mappings when it 
> asks
> for them.

I made a new i915 bug report because the old one seems to be ignored:
https://bugs.freedesktop.org/show_bug.cgi?id=91400

> 
> > There is one possible usage to do selective IOMMU disable other 
> > than
> > global "iommu=off" switch. Then making this option general would
> > be better than igfx_off, e.g. based on BDF. But I'm not sure how it
> > is useful in reality.
> 
> It is curious that just disabling the IOMMU appears to fix the 
> problem. 

Yes, disabling IOMMU (iommu=off or iommu=igfx_off added by this patch)
avoids the problem.

A similar problem also happens on Linux >= 3.7 without using Xen, and
it can also be avoided using intel_iommu=igfx_off.

> Are there any errata you are aware of on this class of system?

With Xen:
https://bugs.freedesktop.org/show_bug.cgi?id=90037
http://lists.xenproject.org/archives/html/xen-devel/2015
-06/msg02236.html
https://bugs.freedesktop.org/show_bug.cgi?id=91400

Without Xen:
https://bugs.freedesktop.org/show_bug.cgi?id=91127

> 
> ~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2015-07-20 10:44 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-07-17 19:05 [PATCH] VT-d: add iommu=igfx_off option to workaround graphics issues Ting-Wei Lan
2015-07-17 19:36 ` Andrew Cooper
2015-07-18  8:46   ` 藍挺瑋
2015-07-19 15:53 ` Julien Grall
2015-07-20  8:27   ` Andrew Cooper
2015-07-20 10:19     ` Julien Grall
2015-07-20  1:28 ` Tian, Kevin
2015-07-20  8:21   ` Andrew Cooper
2015-07-20 10:44     ` Ting-Wei Lan [this message]
2015-07-21  0:57     ` Tian, Kevin
2015-07-21  6:56       ` Jan Beulich
2015-07-21  7:05         ` Tian, Kevin
2015-07-21  7:16           ` Jan Beulich
2015-07-21  7:23             ` Tian, Kevin
2015-07-21  7:33               ` Jan Beulich
2015-07-23 16:41                 ` Ting-Wei Lan
2015-07-20  8:46 ` Jan Beulich
2015-07-25 16:57   ` [PATCH v2] VT-d: add iommu=igfx " Ting-Wei Lan
2015-07-26 16:47     ` Andrew Cooper
2015-07-31  1:26     ` Tian, Kevin
2015-07-31  8:37       ` Ting-Wei Lan
2015-08-04  2:00         ` Tian, Kevin
2015-08-05  9:11           ` [PATCH v3] " Ting-Wei Lan
2015-08-05 12:18             ` Andrew Cooper
2015-08-05 13:35               ` Wei Liu
2015-08-05 17:10                 ` [PATCH v4] " Ting-Wei Lan
2015-08-06  0:49                   ` Tian, Kevin
2015-08-06  8:25                     ` Wei Liu
2015-08-06  9:28                       ` Ian Campbell
2015-07-20 12:12 ` [PATCH] VT-d: add iommu=igfx_off " Andrew Cooper
2015-07-20 12:24   ` Jan Beulich
2015-07-20 12:34     ` Andrew Cooper
2015-07-20 13:55       ` Jan Beulich
2015-07-20 14:12         ` Andrew Cooper
2015-07-20 14:25           ` Jan Beulich
2015-07-21  1:15     ` Tian, Kevin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1437389068.21078.5.camel@gmail.com \
    --to=lantw44@gmail.com \
    --cc=andrew.cooper3@citrix.com \
    --cc=jbeulich@suse.com \
    --cc=kevin.tian@intel.com \
    --cc=xen-devel@lists.xen.org \
    --cc=yang.z.zhang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.