All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Cooper <andrew.cooper3@citrix.com>
To: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
	yunfang tai <fang8802@gmail.com>
Cc: xen-devel@lists.xenproject.org
Subject: Re: save & restore failed when tmem enabled in Xen 4.1 & Xen 4.3
Date: Fri, 5 Jun 2015 20:00:53 +0100	[thread overview]
Message-ID: <5571F1E5.2070407@citrix.com> (raw)
In-Reply-To: <20150605184501.GA21113@l.oracle.com>

On 05/06/15 19:45, Konrad Rzeszutek Wilk wrote:
> On Thu, Jun 04, 2015 at 10:27:06PM +0800, yunfang tai wrote:
>> Hi all,
> Hey!
>>     Recently, I am testing the TMEM support on Xen. I discovered that when
>> enabled TMEM in ubuntu 14.10 as guest on Xen 4.1 & Xen 4.3, "xm save" & "xm
>> restore“ failed after there are more than 1000 pages put in persistent pool
>> of TMEM in Xen. My operations are list as follows:
> Is it exactly 1000 or just about? I presume it does not matter how much but
> that you discovered it by having 1000 of them?
>
>> In ubuntu guest (8 cores , 8GB):
>> sudo modprobe tmem
>> (than wait for the selfballoon to finish)
>> dd if=/dev/zero of=/tmp/test.img bs=10M count=1000
>> dd if=/tmp/test.img of=/dev/null bs=10M
>> dd if=/tmp/test.img of=/dev/null bs=10M
>> .........
>> (until more than 1000 pages put in persistent pool)
>> In Domain 0:
>> (add tmem in grub.cfg)
>> xm save ubuntu test.save
>> xm restore ubuntu test.save
>>
>> When TMEM is not enabled, save & restore success after these operations.
>> But if TMEM is enabled, save & restore fail.
> Are there any errors from the logs? Anything?
>> Does anyone test about save & restore when enabled TMEM in Xen?? Is there
>> anything I do wrong?
> Well lets see what broke. But I think Andrew discovered that the
> migration protocol when it came to 'tmem' was not up to snuff. CC-ing him
> just to confirm.
>
> (Andrew, for the persistent part of this - it conceptually should
> get all of the tmem memory that pushed to the hypervisor back in the
> image. When you were looking at migrationv2 did you just skim through
> that or mostly ignored it?)

Took a look at the code, attempted to figure out what was going on, then
decided to ignore it for the time being.

As a baseline, there is no error checking of hypercalls or their
returned data putting the data into the stream.

Migration v2 currently has no TMEM support, and I would suggest
re-implementing it from scratch over attempting to port what currently
exists for legacy.

~Andrew

_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

  reply	other threads:[~2015-06-05 19:00 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-06-04 14:27 save & restore failed when tmem enabled in Xen 4.1 & Xen 4.3 yunfang tai
2015-06-05 18:45 ` Konrad Rzeszutek Wilk
2015-06-05 19:00   ` Andrew Cooper [this message]
2015-06-08  6:30     ` yunfang tai
2015-06-09  7:06       ` Yang Hongyang
2015-06-08  6:22   ` yunfang tai

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=5571F1E5.2070407@citrix.com \
    --to=andrew.cooper3@citrix.com \
    --cc=fang8802@gmail.com \
    --cc=konrad.wilk@oracle.com \
    --cc=xen-devel@lists.xenproject.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.