All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Jan Beulich" <jbeulich@novell.com>
To: Yunhong Jiang <yunhong.jiang@intel.com>
Cc: Tim Deegan <Tim.Deegan@eu.citrix.com>,
	"xen-devel@lists.xensource.com" <xen-devel@lists.xensource.com>,
	Keir Fraser <keir.fraser@eu.citrix.com>
Subject: RE: Re: [PATCH] Support swap a page from user spacetools	-- Was RE: [RFC][PATCH] Basic support for page offline
Date: Fri, 20 Mar 2009 09:37:58 +0000	[thread overview]
Message-ID: <49C37206.76E4.0078.0@novell.com> (raw)
In-Reply-To: <E2263E4A5B2284449EEBD0AAB751098401CE2D1007@PDSMSX501.ccr.corp.intel.com>

>>> "Jiang, Yunhong" <yunhong.jiang@intel.com> 20.03.09 03:52 >>>
>The method to exchange the offline pending page for PV domain is:
>1) Suspend the guest.
>2) Allocate a new page for the guest
>3) Get a copy for the content
>4) User space tools will scan all page table page to see if any reference to the offending page, if yes, then it will hypercall to Xen
>to replace the entry to point to the new one. (Through the mmu_*ops)
>5) After update all page tables, user space tools will try to exchange the old page with the new page. If the new mfn has no
>reference anymore (i.e. count_info & count_mask = 1), the exchange will update the m2p and return success, otherwise it will
>return fail. (the page may be referenced by other domain, like grant table or foreign mapped).

Hmm, if you consider the possibility of this case, then you should also consider the possibility of a page still being accessible by another domain at the point where you copy its content, but no longer in use when you do the exchange (which means that the content may have changed between the two points in time).

>6) If step 5 is success, user space tools will update the content of the new page and the p2m table, else it will try to undo step 4
>to revert page table changes.
>7) Resume the guest.

Jan

  parent reply	other threads:[~2009-03-20  9:37 UTC|newest]

Thread overview: 61+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-09  8:54 [RFC][PATCH] Basic support for page offline Jiang, Yunhong
2009-02-10  9:15 ` Tim Deegan
2009-02-10  9:29   ` Jiang, Yunhong
2009-02-10  9:42     ` Tim Deegan
2009-02-10 10:29     ` Keir Fraser
2009-02-10 21:09 ` Frank van der Linden
2009-02-11  0:16   ` Jiang, Yunhong
2009-02-11  0:39     ` Frank van der Linden
2009-02-11  1:08       ` Jiang, Yunhong
2009-02-11  4:08         ` Frank Van Der Linden
2009-02-13 17:03 ` Tim Deegan
2009-02-13 17:36   ` Keir Fraser
2009-02-15  9:39     ` Jiang, Yunhong
2009-02-15  9:48   ` Jiang, Yunhong
2009-02-16 14:31     ` Tim Deegan
2009-02-16 15:25       ` Jiang, Yunhong
2009-02-18 14:51       ` Jiang, Yunhong
2009-02-18 15:20         ` Tim Deegan
2009-02-19  8:44           ` Jiang, Yunhong
2009-02-19 14:37             ` Jiang, Yunhong
2009-03-02 11:56               ` Tim Deegan
2009-03-04  8:23                 ` Jiang, Yunhong
2009-03-18 10:24                 ` [PATCH] Support swap a page from user space tools -- Was " Jiang, Yunhong
2009-03-18 10:32                   ` Jiang, Yunhong
2009-03-18 10:42                     ` Keir Fraser
2009-03-18 17:34                   ` Tim Deegan
2009-03-19  5:12                     ` Jiang, Yunhong
2009-03-19  9:32                       ` Tim Deegan
2009-03-19  9:45                         ` Keir Fraser
2009-03-19  9:57                           ` Jiang, Yunhong
2009-03-19 10:13                             ` Keir Fraser
2009-03-19 13:01                               ` Jiang, Yunhong
2009-03-19 13:22                                 ` Keir Fraser
2009-03-19 14:26                                   ` Jiang, Yunhong
2009-03-19 14:36                                     ` Keir Fraser
2009-03-19 14:42                                       ` Jiang, Yunhong
2009-03-19 14:48                                         ` Jiang, Yunhong
2009-03-19 16:45                                         ` Keir Fraser
2009-03-20  2:52                                           ` Jiang, Yunhong
2009-03-20  9:05                                             ` Keir Fraser
2009-03-20  9:16                                               ` Jiang, Yunhong
2009-03-20  9:28                                                 ` Keir Fraser
2009-03-20  9:42                                                   ` Re: [PATCH] Support swap a page from user space tools-- " Jan Beulich
2009-03-20  9:48                                                     ` Keir Fraser
2009-03-20  9:44                                                   ` Re: [PATCH] Support swap a page from user space tools -- " Jiang, Yunhong
2009-03-20  9:52                                                     ` Keir Fraser
2009-03-20  9:37                                             ` Jan Beulich [this message]
2009-03-20  9:41                                               ` Re: [PATCH] Support swap a page from user spacetools " Jiang, Yunhong
2009-03-20  9:42                                               ` Keir Fraser
2009-03-20  9:52                                                 ` Jiang, Yunhong
2009-03-20  9:58                                                   ` Keir Fraser
2009-03-20  9:59                                                     ` Jiang, Yunhong
2009-03-20 10:03                                                       ` Keir Fraser
2009-03-20 10:05                                                         ` Jiang, Yunhong
2009-03-20 10:07                                                         ` Keir Fraser
2009-03-20 10:13                                                           ` Jiang, Yunhong
2009-03-20 10:21                                                             ` Keir Fraser
2009-03-20 10:36                                                               ` Jiang, Yunhong
2009-03-20 10:40                                                                 ` Keir Fraser
2009-03-20 10:19                                                           ` Keir Fraser
2009-03-19  9:48                         ` [PATCH] Support swap a page from user space tools " Jiang, Yunhong

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=49C37206.76E4.0078.0@novell.com \
    --to=jbeulich@novell.com \
    --cc=Tim.Deegan@eu.citrix.com \
    --cc=keir.fraser@eu.citrix.com \
    --cc=xen-devel@lists.xensource.com \
    --cc=yunhong.jiang@intel.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.