public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jesse Barnes <jbarnes@sgi.com>
To: linux-kernel@vger.kernel.org
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Subject: Re: Issues with /proc/bus/pci
Date: Mon, 22 Mar 2004 18:44:09 -0800	[thread overview]
Message-ID: <200403221844.09663.jbarnes@sgi.com> (raw)
In-Reply-To: <1080007613.22212.61.camel@gaston>

On Monday 22 March 2004 6:06 pm, Benjamin Herrenschmidt wrote:
> case with the HyperTransport host on the G5, though I could fix
> that in the long term, I suspect it may happen again elsewhere
> as they aren't required to show up as devices).

The same thing happens on Altix machines, our xio<->pci bridges don't show up.

> What do you think of dealing with that with a slight addition to
> the current ioctl's, basically adding a pair equivalent to
> PCIIOC_MMAP_IS_IO and PCIIOC_MMAP_IS_MEM that would be
> PCIIOC_MMAP_IS_HOST_IO and PCIIOC_MMAP_IS_HOST_MEM ? That would
> be, imho, the simplest solution, as far as userland is concerned,
> though it requires updating the implementation of pci_mmap_page_range()
> of all archs that implement it.

It's been awhile since I looked at this API, but this would allow userland to 
map legacy I/O or mem space for a given device using the above commands?

> Another simpler solution would be to consider that mapping outside
> of a device is only ever useful for legacy ISA IO space and simply
> fix the archs to allow an IO mapping of the IO region below 0x400
> using any device pci_dev (provided the region exist on a given host
> of course), since the value passed by userland is an absolute BAR
> address in bus space, that would work.

This would also work, but then on some platforms (like ia64) legacy space 
requires special treatment since target aborts can cause hard fails, so I'd 
prefer the previous approach since it would make setup for dealing with such 
platforms a bit more explicit.

Thanks,
Jesse

      parent reply	other threads:[~2004-03-23  2:44 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-03-23  2:06 Issues with /proc/bus/pci Benjamin Herrenschmidt
2004-03-23  2:31 ` David S. Miller
2004-03-23  2:40   ` Benjamin Herrenschmidt
2004-03-23  3:47     ` David S. Miller
2004-03-23  4:02       ` Benjamin Herrenschmidt
2004-03-23  4:12       ` Benjamin Herrenschmidt
2004-03-23  2:44 ` Jesse Barnes [this message]

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=200403221844.09663.jbarnes@sgi.com \
    --to=jbarnes@sgi.com \
    --cc=benh@kernel.crashing.org \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox