From: Gerd Hoffmann <kraxel@redhat.com>
To: Pat Campbell <plc@novell.com>
Cc: Derek Murray <Derek.Murray@cl.cam.ac.uk>,
"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
"Daniel P. Berrange" <berrange@redhat.com>,
Markus Armbruster <armbru@redhat.com>
Subject: Re: [RFC] Dynamic modes support for PV xenfb (included)
Date: Thu, 31 Jan 2008 09:22:40 +0100 [thread overview]
Message-ID: <47A18550.9020204@redhat.com> (raw)
In-Reply-To: <47A0DDB4.20103@novell.com>
Pat Campbell wrote:
> Markus Armbruster wrote:
>> I can't see how using grants for the page directory is an improvement
>> over your previous iteration, which used pd[]. Can you explain?
>>
> I will try, I am not the greatest communicator lately.
The real question is why stop half-way with the grant tables conversion?
You should put grant references instead of mfns into the page list
pages, i.e. map the framebuffer itself using grants too. That will kill
even more 32/64 bit differences btw.
> While I was working on this I "briefly" looked at the grant APIs to see
> how hard it would be to create a new API that allocates and claims a
> continuous block of references.
You don't need that. xc_gnttab_map_grant_refs() takes a list of grant
refs and creates a linear mapping of those for you.
Only problem is that gntdev has a 128 grants limit, this needs fixing.
Derek?
cheers,
Gerd
--
http://kraxel.fedorapeople.org/xenner/
next prev parent reply other threads:[~2008-01-31 8:22 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-01-28 0:48 [RFC] Dynamic modes support for PV xenfb (included) Pat Campbell
2008-01-28 8:07 ` Keir Fraser
2008-01-30 9:43 ` Markus Armbruster
2008-01-30 13:41 ` Pat Campbell
2008-01-30 17:45 ` Markus Armbruster
2008-01-30 20:27 ` Pat Campbell
2008-01-31 8:22 ` Gerd Hoffmann [this message]
2008-01-31 8:50 ` Markus Armbruster
2008-01-31 9:06 ` Markus Armbruster
2008-01-30 20:43 ` Daniel P. Berrange
2008-02-28 10:36 ` Keir Fraser
2008-02-28 11:36 ` Markus Armbruster
-- strict thread matches above, loose matches on Subject: below --
2008-03-09 21:19 Pat Campbell
2008-03-09 21:29 ` Samuel Thibault
2008-03-10 12:36 ` Samuel Thibault
2008-03-10 16:16 ` Samuel Thibault
2008-03-10 16:36 ` Samuel Thibault
2008-03-12 17:04 ` Markus Armbruster
2008-03-13 13:05 ` Pat Campbell
2008-03-13 19:53 ` Pat Campbell
2008-03-14 16:39 ` Markus Armbruster
2008-02-04 18:49 [RFC]Dynamic " Pat Campbell
2008-02-05 9:52 ` Markus Armbruster
2008-02-05 23:38 ` Pat Campbell
2008-02-06 15:49 ` Markus Armbruster
2007-12-31 12:27 [RFC] Dynamic " Pat Campbell
2008-01-02 10:51 ` Gerd Hoffmann
[not found] <476133BC0200001800606E1D@sinclair.provo.novell.com>
[not found] ` <476999300200001800607A16@sinclair.provo.novell.com>
2007-12-20 5:20 ` Pat Campbell
2007-12-23 21:58 ` Mark Williamson
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=47A18550.9020204@redhat.com \
--to=kraxel@redhat.com \
--cc=Derek.Murray@cl.cam.ac.uk \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=plc@novell.com \
--cc=xen-devel@lists.xensource.com \
/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.