On Wed, Feb 01, 2012 at 09:26:00AM +0200, Pekka Enberg wrote: > On Wed, 1 Feb 2012, Cyrill Gorcunov wrote: > >I suspect we need something like > >--- > >tools/kvm/builtin-run.c | 5 +++++ > >tools/kvm/kvm.c | 2 +- > >2 files changed, 6 insertions(+), 1 deletion(-) > > > >Index: linux-2.6.git/tools/kvm/builtin-run.c > >=================================================================== > >--- linux-2.6.git.orig/tools/kvm/builtin-run.c > >+++ linux-2.6.git/tools/kvm/builtin-run.c > >@@ -997,6 +997,11 @@ static int kvm_cmd_run_init(int argc, co > > } > > > > kvm = kvm__init(dev, hugetlbfs_path, ram_size, guest_name); > >+ if (IS_ERR(kvm)) { > >+ r = (int)PTR_ERR(kvm); > > The cast is not needed. > Yeah, it's leftover from draft patch ;) Sasha, would you make some new version or I should create new patch? Cyrill -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html