All of lore.kernel.org
 help / color / mirror / Atom feed
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

  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.