From: "Randy.Dunlap" <rddunlap@osdl.org>
To: Ian Pratt <Ian.Pratt@cl.cam.ac.uk>
Cc: Christoph Hellwig <hch@infradead.org>,
linux-kernel@vger.kernel.org, Steven.Hand@cl.cam.ac.uk,
Christian.Limpach@cl.cam.ac.uk, Keir.Fraser@cl.cam.ac.uk,
akpm@osdl.org
Subject: Re: [4/7] Xen VMM patch set : /dev/mem io_remap_page_range for CONFIG_XEN
Date: Sat, 27 Nov 2004 08:51:59 -0800 [thread overview]
Message-ID: <41A8B0AF.8000906@osdl.org> (raw)
In-Reply-To: <E1CY1rC-0007Y2-00@mta1.cl.cam.ac.uk>
Ian Pratt wrote:
>>>>@@ -42,7 +42,12 @@ extern void tapechar_init(void);
>>>> */
>>>> static inline int uncached_access(struct file *file, unsigned long addr)
>>>
>>>Any chance you could just move uncached_access() to some asm/ header for
>>>all arches instead of making the ifdef mess even worse?
>>
>>I suppose a generic definition could go in asm-generic/iomap.h
>>with per-architecture definitions in asm/io.h. However, I think
>>it would make sense to wait until PAT support gets added and then
>>think through exactly what needs doing rather than reorganising
>>things now.
>
>
> What do people think about this? Should I stick with the current
> patch that adds another #ifdef to uncached_access, or should I
> try pulling it out into asm-generic/iomap.h with per-arch
> definitions in asm/io.h ?
>
> Is there anyone working on PAT support? It would be good to have
> their input.
Someone from NVidia has posted some PAT patches a few times.
(Terence Ripperda) There's a thread beginning here:
http://marc.theaimsgroup.com/?l=linux-kernel&m=108180930118848&w=2
--
~Randy
next prev parent reply other threads:[~2004-11-27 16:55 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-19 23:16 Xen VMM patch set - take 2 Ian Pratt
2004-11-19 23:20 ` [1/7] Xen VMM patch set : add ptep_establish_new to make va available Ian Pratt
2004-11-19 23:20 ` [2/7] Xen VMM patch set : return code for arch_free_page Ian Pratt
2004-11-30 2:28 ` Andrea Arcangeli
2004-11-19 23:21 ` [3/7] Xen VMM patch set : runtime disable of VT console Ian Pratt
2004-11-19 23:22 ` [4/7] Xen VMM patch set : /dev/mem io_remap_page_range for CONFIG_XEN Ian Pratt
2004-11-20 2:03 ` William Lee Irwin III
2004-11-20 16:31 ` Christoph Hellwig
2004-11-20 18:03 ` Ian Pratt
2004-11-27 12:39 ` Ian Pratt
2004-11-27 16:51 ` Randy.Dunlap [this message]
2004-11-30 3:08 ` Andrea Arcangeli
2004-11-30 3:14 ` David S. Miller
2004-11-30 3:38 ` William Lee Irwin III
2004-11-30 8:56 ` Ian Pratt
2004-11-30 9:04 ` Arjan van de Ven
2004-11-30 12:09 ` Ian Pratt
2004-11-30 12:14 ` Arjan van de Ven
2004-11-30 12:18 ` Arjan van de Ven
2004-11-30 13:16 ` Ian Pratt
2004-11-30 18:03 ` Andrea Arcangeli
2004-12-04 23:49 ` Ian Pratt
2004-12-05 0:40 ` Andrea Arcangeli
2004-12-05 2:47 ` William Lee Irwin III
2004-11-19 23:23 ` [5/7] Xen VMM patch set : split free_irq into teardown_irq Ian Pratt
2004-11-20 16:32 ` Christoph Hellwig
2004-11-20 19:20 ` Ian Pratt
2004-11-27 12:45 ` Ian Pratt
2004-11-19 23:25 ` [6/7] Xen VMM patch set : add alloc_skb_from_cache Ian Pratt
2004-11-20 2:11 ` James Morris
2004-11-20 6:03 ` Chris Wedgwood
2004-11-20 6:47 ` David S. Miller
2004-11-20 10:28 ` Keir Fraser
2004-11-20 10:33 ` Chris Wedgwood
2004-11-19 23:28 ` [7/7] Xen VMM patch set : handle fragemented skbs correctly in icmp_filter Ian Pratt
2004-11-25 15:46 ` Xen VMM patch set - take 2 Ian Pratt
2004-11-28 17:07 ` Rik van Riel
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=41A8B0AF.8000906@osdl.org \
--to=rddunlap@osdl.org \
--cc=Christian.Limpach@cl.cam.ac.uk \
--cc=Ian.Pratt@cl.cam.ac.uk \
--cc=Keir.Fraser@cl.cam.ac.uk \
--cc=Steven.Hand@cl.cam.ac.uk \
--cc=akpm@osdl.org \
--cc=hch@infradead.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 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.