alsa-devel.alsa-project.org archive mirror
 help / color / mirror / Atom feed
From: David Woodhouse <dwmw2@infradead.org>
To: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Cc: alsa-devel@alsa-project.org, linux-usb@vger.kernel.org,
	konrad.wilk@oracle.com, tiwai@suse.de, gregkh@suse.de,
	clemens@ladisch.de, linux-kernel@vger.kernel.org,
	chrisw@sous-sol.org, iommu@lists.linux-foundation.org,
	andi@firstfloor.org, stern@rowland.harvard.edu, pedrib@gmail.com,
	akpm@linux-foundation.org
Subject: Re: USB transfer_buffer allocations on 64bit systems
Date: Mon, 10 May 2010 10:21:05 +0100	[thread overview]
Message-ID: <1273483265.372.3383.camel@macbook.infradead.org> (raw)
In-Reply-To: <20100510115048E.fujita.tomonori@lab.ntt.co.jp>

On Mon, 2010-05-10 at 11:50 +0900, FUJITA Tomonori wrote:
> On Fri, 7 May 2010 10:51:10 -0400 (EDT)
> Alan Stern <stern@rowland.harvard.edu> wrote:
> 
> > On Fri, 7 May 2010, Daniel Mack wrote:
> > 
> > > > At least the audio class and ua101 drivers don't do this and fill the
> > > > buffers before they are submitted.
> > > 
> > > Gnaa, you're right. I _thought_ my code does it the way I described, but
> > > what I wrote is how I _wanted_ to do it, not how it's currently done. I
> > > have a plan to change this in the future.
> > > 
> > > So unfortunately, that doesn't explain it either. Sorry for the noise.
> > 
> > At one point we tried an experiment, printing out the buffer and DMA 
> > addresses.  I don't recall seeing anything obviously wrong, but if an 
> > IOMMU was in use then that might not mean anything.  Is it possible 
> > that the IOMMU mappings sometimes get messed up for addresses above 4 
> > GB?
> 
> You mean that an IOMMU could allocate an address above 4GB wrongly? If
> so, IIRC, all the IOMMU implementations use dev->dma_mask and
> dev->coherent_dma_mask properly. And the DMA address space of the
> majority of IOMMUs are limited less than 4GB.

The Intel IOMMU code will use dev->dma_mask and dev->coherent_dma_mask
properly. It is not limited to 4GiB, but it will tend to give virtual
DMA addresses below 4GiB even when a device is capable of more; it'll
only give out higher addresses when the address space below 4GiB is
exhausted.

-- 
dwmw2

  reply	other threads:[~2010-05-10  9:21 UTC|newest]

Thread overview: 92+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <g2h74fd948d1004141820ladc1941eo732d059dd678df0a@mail.gmail.com>
     [not found] ` <Pine.LNX.4.44L0.1004151114490.1562-100000@iolanthe.rowland.org>
     [not found]   ` <t2r74fd948d1004191716l49dedc2p25a27da39bcc614a@mail.gmail.com>
2010-05-07  7:48     ` USB transfer_buffer allocations on 64bit systems Daniel Mack
2010-05-07  9:47       ` Clemens Ladisch
2010-05-07 10:24         ` Daniel Mack
2010-05-07 14:51           ` [alsa-devel] " Alan Stern
2010-05-10  2:50             ` FUJITA Tomonori
2010-05-10  9:21               ` David Woodhouse [this message]
     [not found]                 ` <1273483265.372.3383.camel-uXGAPMMVk8bAQYKIod7YupZV94DADvEd@public.gmane.org>
2010-05-10 14:58                   ` Alan Stern
     [not found]                     ` <Pine.LNX.4.44L0.1005101049100.1626-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-05-11  1:06                       ` FUJITA Tomonori
     [not found]                         ` <20100511100637D.fujita.tomonori-Zyj7fXuS5i5L9jVzuh4AOg@public.gmane.org>
2010-05-11 14:00                           ` Alan Stern
2010-05-11 14:22                             ` FUJITA Tomonori
2010-05-11 14:24                             ` Konrad Rzeszutek Wilk
2010-05-11 14:38                               ` FUJITA Tomonori
2010-05-11 15:04                                 ` Alan Stern
2010-05-11 15:34                                   ` FUJITA Tomonori
     [not found]           ` <20100507102408.GM30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-05-10 14:31             ` Konrad Rzeszutek Wilk
     [not found]         ` <4BE3E1B9.5020602-P6GI/4k7KOmELgA04lAiVw@public.gmane.org>
2010-05-07 11:42           ` Oliver Neukum
     [not found]             ` <201005071342.34923.oneukum-l3A5Bk7waGM@public.gmane.org>
2010-05-07 11:47               ` Oliver Neukum
2010-05-07 11:58                 ` Daniel Mack
     [not found]                   ` <20100507115810.GN30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-05-07 14:45                     ` [alsa-devel] " Alan Stern
2010-04-07  9:06 Daniel Mack
2010-04-07 14:59 ` Alan Stern
2010-04-07 15:11   ` Daniel Mack
2010-04-07 15:31     ` Greg KH
2010-04-07 15:35       ` Daniel Mack
2010-04-07 15:51         ` Greg KH
     [not found]           ` <20100407155122.GA13974-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org>
2010-04-07 16:04             ` Alan Stern
2010-04-08  6:09         ` Oliver Neukum
     [not found]           ` <201004080809.11756.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2010-04-08 11:07             ` Daniel Mack
2010-04-07 15:55       ` Alan Stern
2010-04-07 16:16         ` Daniel Mack
2010-04-07 16:47           ` Alan Stern
2010-04-07 17:55           ` Takashi Iwai
2010-04-07 17:59             ` Daniel Mack
2010-04-07 18:06               ` Takashi Iwai
2010-04-07 19:13             ` Alan Stern
     [not found]               ` <Pine.LNX.4.44L0.1004071452560.5760-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-04-07 23:59                 ` Robert Hancock
2010-04-08  0:33               ` Greg KH
2010-04-09  0:01                 ` Robert Hancock
     [not found]                   ` <4BBE6E57.6020600-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2010-04-09 16:50                     ` Sarah Sharp
2010-04-09 23:38                       ` Robert Hancock
2010-04-10  8:34                         ` Daniel Mack
2010-04-07 17:52         ` Takashi Iwai
2010-04-07 15:46     ` Alan Stern
2010-04-08  6:12       ` Oliver Neukum
     [not found]         ` <201004080812.04419.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2010-04-08 16:59           ` Alan Stern
2010-04-08 21:24             ` Oliver Neukum
2010-04-08 22:20               ` Alan Stern
2010-04-09  6:04                 ` Oliver Neukum
     [not found]                   ` <201004090804.36213.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2010-04-09 14:41                     ` Alan Stern
     [not found]                       ` <Pine.LNX.4.44L0.1004091033150.1852-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-04-09 14:50                         ` Oliver Neukum
     [not found]                           ` <201004091650.31488.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2010-04-09 15:15                             ` Alan Stern
     [not found]                               ` <Pine.LNX.4.44L0.1004091114500.1852-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-04-09 20:51                                 ` Oliver Neukum
2010-04-09 21:21                                   ` Alan Stern
     [not found]   ` <Pine.LNX.4.44L0.1004071036060.1779-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-04-07 16:54     ` Oliver Neukum
     [not found]       ` <201004071854.55530.oliver-GvhC2dPhHPQdnm+yROfE0A@public.gmane.org>
2010-04-07 17:00         ` Daniel Mack
2010-04-07 23:55     ` Robert Hancock
     [not found]       ` <4BBD1B6F.3000205-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2010-04-08  2:10         ` Alan Stern
2010-04-08  7:30           ` Daniel Mack
     [not found]             ` <20100408073041.GO30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-08 16:57               ` Alan Stern
2010-04-08 17:17                 ` Pedro Ribeiro
     [not found]                 ` <Pine.LNX.4.44L0.1004081245330.1720-100000-IYeN2dnnYyZXsRXLowluHWD2FQJk+8+b@public.gmane.org>
2010-04-08 23:13                   ` Pedro Ribeiro
2010-04-09 16:01                     ` Alan Stern
2010-04-09 18:09                       ` Daniel Mack
     [not found]                         ` <20100409180942.GK30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-09 18:19                           ` Pedro Ribeiro
     [not found]                             ` <w2r74fd948d1004091119j9f33d8a6kc1824d9243abf38b-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-04-09 19:34                               ` Alan Stern
2010-04-09 20:14                                 ` Daniel Mack
2010-04-10 12:49                             ` Daniel Mack
     [not found]                               ` <20100410124912.GP30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-10 13:21                                 ` Pedro Ribeiro
2010-04-12  8:59   ` Andi Kleen
2010-04-12 11:14     ` Daniel Mack
     [not found]       ` <20100412111439.GU30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-12 11:53         ` Andi Kleen
2010-04-12 12:11           ` Pedro Ribeiro
     [not found]             ` <q2z74fd948d1004120511hebf19eaauc41ee9ed25dea19e-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-04-12 12:12               ` Andi Kleen
2010-04-12 12:32                 ` Daniel Mack
2010-04-12 12:47                   ` Andi Kleen
2010-04-12 12:54                     ` Daniel Mack
2010-04-12 15:43                       ` Andi Kleen
     [not found]                         ` <20100412154323.GP18855-qrUzlfsMFqo/4alezvVtWx2eb7JE58TQ@public.gmane.org>
2010-04-12 16:17                           ` Alan Stern
2010-04-12 16:29                             ` Andi Kleen
2010-04-12 16:57                               ` Alan Stern
2010-04-12 17:15                                 ` Daniel Mack
     [not found]                                   ` <20100412171507.GB30801-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-12 17:22                                     ` Andi Kleen
2010-04-12 17:56                                       ` Daniel Mack
2010-04-13 18:22                                 ` Daniel Mack
     [not found]                                   ` <20100413182233.GR30807-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-13 23:46                                     ` Pedro Ribeiro
2010-04-14 10:09                                       ` Daniel Mack
2010-04-14 10:47                                         ` Pedro Ribeiro
2010-04-14 11:02                                           ` Pedro Ribeiro
     [not found]                                           ` <t2w74fd948d1004140347k3447bffapb73856eacddfde55-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-04-14 14:08                                             ` Alan Stern
2010-04-14 16:36                                               ` Daniel Mack
     [not found]                                                 ` <20100414163637.GV30807-ahpEBR4enfnCULTFXS99ULNAH6kLmebB@public.gmane.org>
2010-04-14 17:21                                                   ` Pedro Ribeiro
2010-04-15  7:35                                                     ` Daniel Mack

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=1273483265.372.3383.camel@macbook.infradead.org \
    --to=dwmw2@infradead.org \
    --cc=akpm@linux-foundation.org \
    --cc=alsa-devel@alsa-project.org \
    --cc=andi@firstfloor.org \
    --cc=chrisw@sous-sol.org \
    --cc=clemens@ladisch.de \
    --cc=fujita.tomonori@lab.ntt.co.jp \
    --cc=gregkh@suse.de \
    --cc=iommu@lists.linux-foundation.org \
    --cc=konrad.wilk@oracle.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=pedrib@gmail.com \
    --cc=stern@rowland.harvard.edu \
    --cc=tiwai@suse.de \
    /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;
as well as URLs for NNTP newsgroup(s).