From: Olaf Hering <olaf@aepfle.de>
To: Keir Fraser <keir.xen@gmail.com>
Cc: xen-devel@lists.xensource.com, Ian Jackson <Ian.Jackson@eu.citrix.com>
Subject: Re: [PATCH 00 of 12] xenpaging fixes for xen-unstable
Date: Tue, 7 Jun 2011 15:38:47 +0200 [thread overview]
Message-ID: <20110607133847.GA15418@aepfle.de> (raw)
In-Reply-To: <CA13D7CD.1BB88%keir.xen@gmail.com>
On Tue, Jun 07, Keir Fraser wrote:
> Once they're in, what is left to do? Do you need help with the in-Xen
> waitqueue stuff? Do you have test cases that fail, a private patch queue,
> etc?
Keir,
I have no new patches, just returned to xenpaging end of last week.
A few months ago I collected my TODO list below.
One thing that I would need help with is the machine_to_phys_mapping[]
handling, an attempt to fix it was reverted before the 4.1 release. See
the thread in the URL below.
My testcase was a SLES11SP1 pv-on-hvm guest, rebooted in an endless
loop. That catched all of the issues. With the
waitqueue/HVMCOPY_gfn_paged_out issue its best to customize the
xenpaging policy to page the range of gfns were the guest pagetables are
stored.
I will send a series of patches to try later.
Olaf
Todo:
- implement xl support
- implement stopping of xenpaging
- implement live migration
- implement config option for XENPAGING_DEBUG and XENPAGING_POLICY_MRU_SIZE
- implement config option for xenpaging_dir
- implement better starting gfn in xenpaging policy
an initial gfn number in the middle of the gfn range may avoid page-ins
during BIOS startup
- fix machine_to_phys_mapping[] array handling during page deallocation
the gfn of a released page must be maintained properly in the array
http://lists.xensource.com/archives/html/xen-devel/2011-01/msg00824.html
- fix HVMCOPY_gfn_paged_out handling
some callers of __hvm_copy() do not handle HVMCOPY_gfn_paged_out, such
as hypercalls and the MMIO emulation
the recently added waitqueue feature in Xen 4.1 should be used
- remove all retry code from gfn_to_mfn() calls
use the waitqueue feature to hide page-in from the caller and cover
all cases where a retry is currently missing
- do not bounce p2mt to xenpaging
p2m_mem_paging_populate/p2m_mem_paging_resume dont make use of p2mt
- cleanup typeof gfn which is passed around in xenpaging
unsigned long, uint64_t, xen_pfn_t
- cleanup return types of functions
some functions return 0 unconditionally, could be void
- remove srand function
next prev parent reply other threads:[~2011-06-07 13:38 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-06-07 9:53 [PATCH 00 of 12] xenpaging fixes for xen-unstable Olaf Hering
2011-06-07 9:53 ` [PATCH 01 of 12] xenpaging: correct dropping of pages to avoid full ring buffer Olaf Hering
2011-06-07 9:53 ` [PATCH 02 of 12] xenpaging: do not bounce p2mt to xenpaging Olaf Hering
2011-06-07 9:53 ` [PATCH 03 of 12] xenpaging: remove srand call Olaf Hering
2011-06-07 9:53 ` [PATCH 04 of 12] xenpaging: remove return values from functions that can not fail Olaf Hering
2011-06-07 9:53 ` [PATCH 05 of 12] xenpaging: catch xc_mem_paging_resume errors Olaf Hering
2011-06-07 9:53 ` [PATCH 06 of 12] xenpaging: remove local domain_id variable Olaf Hering
2011-06-07 9:53 ` [PATCH 07 of 12] xenpaging: move num_pages into xenpaging struct Olaf Hering
2011-06-07 9:53 ` [PATCH 08 of 12] xenpaging: start paging in the middle of gfn range Olaf Hering
2011-06-07 9:53 ` [PATCH 09 of 12] xenpaging: pass integer to xenpaging_populate_page Olaf Hering
2011-06-07 9:53 ` [PATCH 10 of 12] xenpaging: add helper function for unlinking pagefile Olaf Hering
2011-06-07 9:53 ` [PATCH 11 of 12] xenpaging: add watch thread to catch guest shutdown Olaf Hering
2011-06-07 10:23 ` Ian Campbell
2011-06-07 19:07 ` Olaf Hering
2011-06-07 9:53 ` [PATCH 12 of 12] xenpaging: implement stopping of pager by sending SIGTERM/SIGINT Olaf Hering
2011-06-07 12:27 ` [PATCH 00 of 12] xenpaging fixes for xen-unstable Keir Fraser
2011-06-07 13:38 ` Olaf Hering [this message]
2011-06-08 16:01 ` Keir Fraser
2011-06-10 7:28 ` Keir Fraser
2011-06-10 11:27 ` Olaf Hering
2011-06-10 12:52 ` Keir Fraser
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=20110607133847.GA15418@aepfle.de \
--to=olaf@aepfle.de \
--cc=Ian.Jackson@eu.citrix.com \
--cc=keir.xen@gmail.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.