From: Grant Grundler <grundler@parisc-linux.org>
To: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Cc: Grant Grundler <grundler@parisc-linux.org>,
"David S. Miller" <davem@davemloft.net>,
linux-pci@atrey.karlin.mff.cuni.cz,
Linux Kernel list <linux-kernel@vger.kernel.org>,
Greg KH <greg@kroah.com>,
bjorn.helgaas@hp.com, "David S. Miller" <davem@redhat.com>
Subject: Re: pci-sysfs resource mmap broken (and PATCH)
Date: Thu, 28 Apr 2005 17:38:28 -0600 [thread overview]
Message-ID: <20050428233828.GI10171@colo.lackof.org> (raw)
In-Reply-To: <1114728447.7182.262.camel@gaston>
On Fri, Apr 29, 2005 at 08:47:27AM +1000, Benjamin Herrenschmidt wrote:
> > Well, if it's a device driver decision, I guess that's ok.
> > And the primary device driver happens to live in user space in X.org case.
>
> Agreed, but 1) Do you have an idea on how to expose this capability with
> the sysfs interface ? Adding ioctl's to it would suck big time :) and
I don't know enough about VM/TLB stuff to know the right answer.
I suspect the MAP_* attribute/hint needs to be passed in together
with the mmap call if any arch (ia64?) would return a different
virtual address depending the attribute (e.g cached vs uncached).
And write combining might be done in a "layer" below the CPU in
the HW hierarchy. e.g. PCI Host bus controller might combine writes
for some MMIO regions. I don't know if arch specific mmap support
can figure out which HW is the right one to enable write combining
in for a particular MMIO region or PCI device.
I generally don't work with graphics devices and only recently
started poking at infiniband support (128-512MB BAR depending
on card option) to understand really well how BAR is accessed/used.
> 2) It's still nice to have a "workaround" for existing X since the
> performance benefit is significant, but then, it's in arch code, so
> that's fine (and I could indeed limit it to VGA class devices as David
> suggests).
Yup.
thanks,
grant
>
> Ben.
>
next prev parent reply other threads:[~2005-04-28 23:36 UTC|newest]
Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-04-26 5:33 pci-sysfs resource mmap broken Benjamin Herrenschmidt
2005-04-26 6:09 ` Benjamin Herrenschmidt
2005-04-26 6:36 ` Greg KH
2005-04-26 9:24 ` Russell King
2005-04-26 16:30 ` Grant Grundler
2005-04-26 22:47 ` Benjamin Herrenschmidt
2005-04-27 3:55 ` Grant Grundler
2005-04-27 4:30 ` Benjamin Herrenschmidt
2005-04-27 4:28 ` David S. Miller
2005-04-27 4:39 ` Benjamin Herrenschmidt
2005-04-27 4:46 ` pci-sysfs resource mmap broken (and PATCH) Benjamin Herrenschmidt
2005-04-27 23:13 ` Benjamin Herrenschmidt
2005-04-28 5:33 ` Grant Grundler
2005-04-28 5:37 ` David S. Miller
2005-04-28 6:39 ` Benjamin Herrenschmidt
2005-04-28 6:50 ` David S. Miller
2005-04-28 7:21 ` Benjamin Herrenschmidt
2005-04-28 7:22 ` David S. Miller
2005-04-28 7:46 ` Benjamin Herrenschmidt
2005-04-28 15:11 ` Grant Grundler
2005-04-28 22:47 ` Benjamin Herrenschmidt
2005-04-28 23:38 ` Grant Grundler [this message]
2005-04-29 15:42 ` David S. Miller
2005-04-29 22:16 ` Jesse Barnes
2005-04-28 6:35 ` Benjamin Herrenschmidt
2005-05-03 5:37 ` pci-sysfs resource mmap broken PATCH#2 Benjamin Herrenschmidt
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=20050428233828.GI10171@colo.lackof.org \
--to=grundler@parisc-linux.org \
--cc=benh@kernel.crashing.org \
--cc=bjorn.helgaas@hp.com \
--cc=davem@davemloft.net \
--cc=davem@redhat.com \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@atrey.karlin.mff.cuni.cz \
/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.