From: Cyrill Gorcunov <gorcunov@openvz.org>
To: Pekka Enberg <penberg@kernel.org>
Cc: Yang Bai <hamo.by@gmail.com>,
kvm@vger.kernel.org, Sasha Levin <levinsasha928@gmail.com>,
Asias He <asias.hejun@gmail.com>, Ingo Molnar <mingo@elte.hu>
Subject: Re: [PATCH] kvm tool: rewrite kvm__init
Date: Thu, 9 Feb 2012 17:07:41 +0400 [thread overview]
Message-ID: <20120209130741.GG5709@moon> (raw)
In-Reply-To: <CAOJsxLFmMRoDhGk7-9H5wUYgHqf9TXQa+gVxEh1zqSVKnmF4Uw@mail.gmail.com>
On Thu, Feb 09, 2012 at 03:01:26PM +0200, Pekka Enberg wrote:
> On Thu, Feb 9, 2012 at 7:40 AM, Yang Bai <hamo.by@gmail.com> wrote:
> > Since the different issues have been handled in the
> > internal of kvm__init, it can only return NULL if error
> > happened.
> >
> > Signed-off-by: Yang Bai <hamo.by@gmail.com>
>
> Sorry, I don't understand what this patch is attempting to fix? Why do
> you think it's better to drop the explicit error codes and always
> return NULL upon error?
>
Yeah, I somehow don't get it as well.
Cyrill
next prev parent reply other threads:[~2012-02-09 13:07 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-02-09 5:40 [PATCH] kvm tool: rewrite kvm__init Yang Bai
2012-02-09 13:01 ` Pekka Enberg
2012-02-09 13:07 ` Cyrill Gorcunov [this message]
2012-02-10 2:34 ` Yang Bai
2012-02-10 4:53 ` Sasha Levin
2012-02-10 3:03 ` Yang Bai
2012-02-10 4:17 ` Sasha Levin
2012-02-10 5:04 ` Yang Bai
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=20120209130741.GG5709@moon \
--to=gorcunov@openvz.org \
--cc=asias.hejun@gmail.com \
--cc=hamo.by@gmail.com \
--cc=kvm@vger.kernel.org \
--cc=levinsasha928@gmail.com \
--cc=mingo@elte.hu \
--cc=penberg@kernel.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.