All of lore.kernel.org
 help / color / mirror / Atom feed
From: Deepak Saxena <dsaxena@plexity.net>
To: "Amit D. Chaudhary" <amit_c@comcast.net>
Cc: linux-kernel@vger.kernel.org
Subject: Re: MAX_DMA_ADDRESS in include/asm/asm-i386/dma.h (2.6.x and 2.4.x)
Date: Fri, 16 Jul 2004 15:28:59 -0700	[thread overview]
Message-ID: <20040716222859.GA21647@plexity.net> (raw)
In-Reply-To: <40F852AE.8060703@comcast.net>

On Jul 16 2004, at 15:11, Amit D. Chaudhary was caught saying:
> Deepak,
> 
> I am missing what you are directing me to.
> 
> If it is,
> pci_alloc_consistent(), linux-2.4.25/arch/i386/kernel/pci-dma.c
> dma_alloc_coherent(), linux-2.6.8-rc1/arch/i386/kernel/pci-dma.c
> 
> They internally seem to __get_free_pages()

Correct, but take a second look at the code (2.6):

        void *ret;
        /* ignore region specifiers */
        gfp &= ~(__GFP_DMA | __GFP_HIGHMEM);

        if (dev == NULL || (dev->coherent_dma_mask < 0xffffffff))
                gfp |= GFP_DMA;

        ret = (void *)__get_free_pages(gfp, get_order(size));

It uses GFP_DMA iff your coherent_dma_mask is != 0xffffffff.  Assuming
your device can address a the full 32-bit PCI address space, you
need to set the coherent_dma_mask appropriately and you will get
buffers from all addressable lowmem. I don't do much x86, so not
sure how you go about allocating highmem DMA buffers.

> The memory need not be page size, as a matter of fact, using a large 
> consecutive block, for example using alloc_bootmem_low() during kernel 
> bootup, will simplify the data transfer and result in no internal 
> fragmentation, it does introduce inflexibility in changing the size and 
> other issues.

If you are using alloc_bootmem_low(), all you should have to do after
allocating the memory is call pci_dma_map_single()/map_sg() to get PCI-DMA 
addresses. You still should have no reason to touch MAX_DMA_ADDRESS.

~Deepak

-- 
Deepak Saxena - dsaxena at plexity dot net - http://www.plexity.net/

"Unlike me, many of you have accepted the situation of your imprisonment and
 will die here like rotten cabbages." - Number 6

  reply	other threads:[~2004-07-16 22:29 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-07-16 21:37 MAX_DMA_ADDRESS in include/asm/asm-i386/dma.h (2.6.x and 2.4.x) Amit D. Chaudhary
2004-07-16 21:47 ` Deepak Saxena
2004-07-16 22:11   ` Amit D. Chaudhary
2004-07-16 22:28     ` Deepak Saxena [this message]
2004-07-16 23:36       ` Amit D. Chaudhary
2004-08-05 19:00         ` Amit D. Chaudhary

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=20040716222859.GA21647@plexity.net \
    --to=dsaxena@plexity.net \
    --cc=amit_c@comcast.net \
    --cc=linux-kernel@vger.kernel.org \
    /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.