All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: airlied@linux.ie, linux-kernel@vger.kernel.org,
	iommu@lists.linux-foundation.org, mingo@redhat.com,
	dwmw2@infradead.org
Subject: Re: IOMMU and graphics cards
Date: Thu, 7 May 2009 13:26:01 +0200	[thread overview]
Message-ID: <20090507112601.GJ28398@elte.hu> (raw)
In-Reply-To: <20090507201538U.fujita.tomonori@lab.ntt.co.jp>


* FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:

> On Thu, 7 May 2009 13:01:35 +0200
> Ingo Molnar <mingo@elte.hu> wrote:
> 
> > 
> > * FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> wrote:
> > 
> > > > The proprietary drivers make problems so far. For the ATI one I 
> > > > am in contact with the developers to try to fix it. But I can't 
> > > > do anything about the other proprietary driver I am aware of :-(
> > > 
> > > I don't know anything about the graphic drivers but are there any 
> > > other proprietary drivers except for ATI and AMD? Fixing only two 
> > > drivers to make the majority happy?
> > 
> > ATI ~== AMD these days. The problem is Nvidia cards stuck into AMD 
> > systems.
> 
> Oops, I meant ATI and Nvidia.
> 
> 
> > I guess refusing those DMA accesses (and printing something 
> > meaningful and relentlessly honest so that the user knows where the 
> > problem comes from) is the proper solution.
> 
> Refusing DMA means that we will break these broken drivers. That's 
> what David and I like. Yeah, telling users explicitly who is to 
> blame for the problem is even better.

If it means a non-working Xorg, then there might be nothing on the 
screen to report - just a seemingly hung box and people will blame 
the kernel. It might be better to just turn off all things IOMMU at 
that point, but still allow things to continue.

... which might break good drivers that relied on the IOMMU sorting 
out 32-bit DMA space limitations.

So it would be nice to make the failure mode somehow nicer. I.e. 
test how this affects the nvidia driver. Is there something on the 
screen to see? Could we print an URL to the Noveau driver perhaps 
too?

A bad solution in general is to crash/hang a significant proportion 
of Linux boxes. That is a foot-in-own-mouth masochistic excercise 
mostly, it creates a stigma for the IOMMU code, not for nvidia. So 
the failure mode has to be well thought out.

	Ingo

  reply	other threads:[~2009-05-07 11:26 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-28 15:05 IOMMU and graphics cards Joerg Roedel
2009-04-28 15:28 ` David Woodhouse
2009-04-28 16:04   ` Joerg Roedel
2009-05-07  7:47     ` FUJITA Tomonori
2009-05-07 10:01       ` Joerg Roedel
2009-05-07 10:22         ` David Woodhouse
2009-05-07 10:42           ` FUJITA Tomonori
2009-05-07 12:15           ` Joerg Roedel
2009-05-07 12:59             ` FUJITA Tomonori
2009-05-07 14:37               ` Joerg Roedel
2009-05-07 17:04             ` Christoph Hellwig
2009-05-07 18:36           ` David Miller
2009-05-08  9:35             ` Joerg Roedel
2009-07-06 12:26               ` David Woodhouse
2009-07-06 13:11                 ` Joerg Roedel
2009-07-06 14:18                   ` David Woodhouse
2009-07-06 21:35                     ` Dave Airlie
2009-07-06 22:00                       ` Dave Airlie
2009-07-07  8:54                         ` Joerg Roedel
2009-07-07 15:24                         ` Duran, Leo
2009-07-07 15:33                           ` Duran, Leo
2009-07-07 15:36                             ` David Woodhouse
2009-07-07  8:59                       ` Joerg Roedel
2009-07-07  9:16                         ` David Woodhouse
2009-07-07  9:05                       ` David Woodhouse
2009-05-07 10:23         ` FUJITA Tomonori
2009-05-07 11:01           ` Ingo Molnar
2009-05-07 11:14             ` FUJITA Tomonori
2009-05-07 11:26               ` Ingo Molnar [this message]
2009-05-07 12:00           ` Joerg Roedel
2009-05-07 10:55         ` Ingo Molnar

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=20090507112601.GJ28398@elte.hu \
    --to=mingo@elte.hu \
    --cc=airlied@linux.ie \
    --cc=dwmw2@infradead.org \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=iommu@lists.linux-foundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@redhat.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.