From: Yoshisato YANAGISAWA <yanagisawa.yoshisato@lab.ntt.co.jp>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: xen-devel <xen-devel@lists.xensource.com>,
Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
Subject: Re: Design document of Kemari
Date: Mon, 09 Mar 2009 11:05:41 +0900 [thread overview]
Message-ID: <49B47975.8000601@lab.ntt.co.jp> (raw)
In-Reply-To: <C5D5D16C.43DE%keir.fraser@eu.citrix.com>
Hi Keir,
On 2009/03/06 3:58, Keir Fraser wrote:
> On 02/03/2009 13:30, "Yoshiaki Tamura" <tamura.yoshiaki@lab.ntt.co.jp>
> wrote:
>
>> You can also get the undivided patches from the following links.
>> http://www.osrg.net/kemari/download/kemari-v1-xen-20081120.patch
>> http://www.osrg.net/kemari/download/kemari-v1-linux-20081120.patch
>
> The two obvious things I note are the duplicated save/restore functions,
> which will simply bitrot, and the additions to evtchn.c. In particular the
Since I have implemented the save/restore functions, will you let me ask
some?
I could not understand the meaning of "the duplicated save/restore
functions". Do you mean there are some duplicated code inside
xc_dom_kemari_save.c and xc_dom_kemari_restore.c? Or, do you mean there
are some duplicated code, e.g. test_bit and clear_bit and so on, between
xc_dom_kemari_save and xc_domain_save, and between xc_dom_kemari_restore
and xc_domain_restore?
> suspicious. Apart from that there are various spurious white-space and
> debug-tracing changes which I assume you would clean up anyway. Those are
> just the things that jump out from a quick read through.
I will clean up those spurious white-space.
Thank you in advance,
--
Yoshisato YANAGISAWA, Ph.D.
Open Source Software Computing Project
NTT Cyber Space Laboratories
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
1-1 Hikarinooka, Yokosuka-Shi,
Kanagawa 239-0847, Japan
next prev parent reply other threads:[~2009-03-09 2:05 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-11-25 5:04 [RFC][PATCH] Kemari v1.0 released Yoshiaki Tamura
2009-03-02 13:30 ` Design document of Kemari Yoshiaki Tamura
2009-03-05 18:58 ` Keir Fraser
2009-03-06 0:52 ` Yoshiaki Tamura
2009-03-06 16:28 ` John Levon
2009-03-06 16:48 ` Yoshiaki Tamura
2009-03-06 17:02 ` John Levon
2009-03-06 17:26 ` Yoshiaki Tamura
2009-03-06 17:36 ` John Levon
2009-03-06 23:03 ` Brendan Cully
2009-03-09 2:05 ` Yoshisato YANAGISAWA [this message]
2009-03-09 8:13 ` Keir Fraser
2009-03-09 22:28 ` Yoshiaki Tamura
2009-03-10 8:21 ` Keir Fraser
2009-03-10 10:16 ` Yoshisato YANAGISAWA
2009-03-10 10:38 ` Keir Fraser
2009-03-10 16:51 ` Brendan Cully
2009-03-04 22:43 ` Yoshiaki Tamura
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=49B47975.8000601@lab.ntt.co.jp \
--to=yanagisawa.yoshisato@lab.ntt.co.jp \
--cc=keir.fraser@eu.citrix.com \
--cc=tamura.yoshiaki@lab.ntt.co.jp \
--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.