All of lore.kernel.org
 help / color / mirror / Atom feed
From: James Bottomley <James.Bottomley@SteelEye.com>
To: linux-arch@vger.kernel.org
Subject: Proposal for new generic device API: dma_get_required_mask()
Date: 17 Jun 2004 09:08:54 -0500	[thread overview]
Message-ID: <1087481335.2090.28.camel@mulgrave> (raw)

[This is for notice only.  There's a thread on linux-kernel with the
same contents, lets hold the debate in public]

Background:

We have a large number of devices in scsi: aacraid, aic7xxx, qla1280,
qla2xxx which can all do full 64 bit DMA, but which pay a performance
penalty for using the larger descriptors (aic7xxx is stranger in that it
has three modes of operation: 32 bit, 39 bit and 64 bit each with an
increasing performance penalty).

What all these devices would like to do is instead of simply trying the
64 bit mask first and having the platform accept it, even if it only has
< 4GB of memory, they'd like to be able to have the platform tell them,
given my current dma mask setting, what's the actual number of bits you
need me to DMA to.

This is precisely what the API would do.  It would return a bit mask
(never over the current dma_mask) that the platform considers optimal. 
The platform has complete freedom in this: it may return a mask covering
the total physical memory, or a mask covering all the bits it needs
setting for some weird numa scheme.

If the driver decides to use the mask, it would do another
dma_set_mask() to confirm it (this gives the platform the opportunity if
it so chooses to return a mask that doesn't quite cover memory, but
would be more optimal...say for platforms that have all memory under 4GB
bar one small chunk at 64GB or something).

Once the driver has the platform's optimal mask, it can use this to
decide on the correct descriptor size.

Comments?

James

             reply	other threads:[~2004-06-17 14:08 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-17 14:08 James Bottomley [this message]
  -- strict thread matches above, loose matches on Subject: below --
2004-06-17 14:52 Proposal for new generic device API: dma_get_required_mask() Salyzyn, Mark
2004-06-17 14:08 James Bottomley
2004-06-17 14:51 ` Meelis Roos
2004-06-17 15:28 ` Matthew Wilcox
2004-06-17 20:12   ` James Bottomley
2004-06-18  0:46 ` Krzysztof Halasa
2004-06-18  1:45   ` James Bottomley
2004-06-18 23:07     ` Krzysztof Halasa
2004-06-19 15:00       ` James Bottomley
2004-06-19 23:39         ` Krzysztof Halasa
2004-06-20 16:56           ` James Bottomley
2004-06-18  9:21   ` Russell King
2004-06-18 23:10     ` Krzysztof Halasa
2004-06-19 20:22       ` Russell King
2004-06-20  0:00         ` Krzysztof Halasa
2004-06-20 19:47           ` Russell King
2004-06-23 19:32             ` Krzysztof Halasa
2004-06-18  5:59 ` Jeremy Higdon
2004-06-18 14:19   ` James Bottomley

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=1087481335.2090.28.camel@mulgrave \
    --to=james.bottomley@steeleye.com \
    --cc=linux-arch@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.