From: Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
To: Keir Fraser <keir.fraser@eu.citrix.com>
Cc: xen-devel <xen-devel@lists.xensource.com>
Subject: Re: Design document of Kemari
Date: Fri, 06 Mar 2009 09:52:21 +0900 [thread overview]
Message-ID: <49B073C5.6080702@lab.ntt.co.jp> (raw)
In-Reply-To: <C5D5D16C.43DE%keir.fraser@eu.citrix.com>
Keir,
Thanks for your quick review.
I'm glad my messages have finally reached to the list.
Although the issues you pointed out here aren't fixed yet, let me post the
divided patches which are updated a little. We'll keep up cleaning and updating
the code.
Yoshi
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
> hack to relax a validity check in evtchn_bind_interdomain() looks pretty
> 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.
>
> -- Keir
>
>
>
>
>
next prev parent reply other threads:[~2009-03-06 0:52 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 [this message]
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
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=49B073C5.6080702@lab.ntt.co.jp \
--to=tamura.yoshiaki@lab.ntt.co.jp \
--cc=keir.fraser@eu.citrix.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.