mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: Luming Yu <luming.yu@gmail.com>
Cc: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>,
	fenghua.yu@intel.com, tony.luck@intel.com,
	linux-kernel@vger.kernel.org, akpm@linux-foundation.org
Subject: Re: [RFC patch] init default dma_ops to prepare intel_iommu_init  failure
Date: Thu, 13 Aug 2009 09:49:33 +0100	[thread overview]
Message-ID: <1250153373.29180.225.camel@macbook.infradead.org> (raw)
In-Reply-To: <3877989d0908130111m1a7d8625ha84ed1732a7eae81@mail.gmail.com>

On Thu, 2009-08-13 at 16:11 +0800, Luming Yu wrote:
> I treat this as -32 or -33 stuff, because I don't see a real system is blocking
> on the problem. I found this problem when I was trying to disable some drhd
> for debugging a real iommu issue...that one has priority.

(The real bug you were chasing is fixed, isn't it? Or worked around, at
least, since the BIOS in question is still closed source and broken.)

But by talking about disabling _some_ drhd units, you've just reminded
me that the problem is far more interesting -- we can have the IOMMU
active for _some_ devices but not others.

I was toying with the idea of putting the intel_dma_ops into
dev->archdata for each pci device at boot or hotplug time, and leaving
the global variable alone. We can calculate the dev->iommu mapping at
that point too. And we can stop being quite so broken w.r.t. PCI
hotplug...

-- 
David Woodhouse                            Open Source Technology Centre
David.Woodhouse@intel.com                              Intel Corporation


  reply	other threads:[~2009-08-13  8:49 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-08-12  9:20 Luming Yu
2009-08-12  9:36 ` David Woodhouse
2009-08-12 18:17 ` Fenghua Yu
2009-08-12 18:20   ` David Woodhouse
2009-08-13  4:49   ` FUJITA Tomonori
2009-08-13  5:48     ` Luming Yu
2009-08-13  6:30       ` FUJITA Tomonori
2009-08-13  6:52       ` FUJITA Tomonori
2009-08-13  7:03         ` Luming Yu
2009-08-13  7:14           ` FUJITA Tomonori
2009-08-13  7:37             ` Luming Yu
2009-08-13  8:03               ` FUJITA Tomonori
2009-08-13  8:11                 ` Luming Yu
2009-08-13  8:49                   ` David Woodhouse [this message]
2009-08-13  9:00                     ` FUJITA Tomonori
2009-08-13  9:00                     ` Luming Yu
2009-08-13 17:08                       ` David Woodhouse
2009-08-13 17:30                         ` Yu, Fenghua
2009-08-14  5:44                           ` Luming Yu
2009-08-13  2:11 ` FUJITA Tomonori
2009-08-13  2:24   ` Luming Yu
2009-08-13  3:15     ` FUJITA Tomonori
2009-08-13  3:18       ` FUJITA Tomonori
2009-08-13  8:26         ` David Woodhouse
2009-08-13  8:35           ` FUJITA Tomonori

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=1250153373.29180.225.camel@macbook.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=akpm@linux-foundation.org \
    --cc=fenghua.yu@intel.com \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=linux-kernel@vger.kernel.org \
    --cc=luming.yu@gmail.com \
    --cc=tony.luck@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®