From patchwork Thu Jun 3 07:23:23 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Xiao Guangrong X-Patchwork-Id: 104041 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by demeter.kernel.org (8.14.3/8.14.3) with ESMTP id o537QrHC009519 for ; Thu, 3 Jun 2010 07:26:55 GMT Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753175Ab0FCH0v (ORCPT ); Thu, 3 Jun 2010 03:26:51 -0400 Received: from cn.fujitsu.com ([222.73.24.84]:56576 "EHLO song.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-OK) by vger.kernel.org with ESMTP id S1752675Ab0FCH0v convert rfc822-to-8bit (ORCPT ); Thu, 3 Jun 2010 03:26:51 -0400 Received: from tang.cn.fujitsu.com (tang.cn.fujitsu.com [10.167.250.3]) by song.cn.fujitsu.com (Postfix) with ESMTP id 2A3AD170209; Thu, 3 Jun 2010 15:26:50 +0800 (CST) Received: from fnst.cn.fujitsu.com (tang.cn.fujitsu.com [127.0.0.1]) by tang.cn.fujitsu.com (8.14.3/8.13.1) with ESMTP id o537OamH018045; Thu, 3 Jun 2010 15:24:36 +0800 Received: from [10.167.141.99] (unknown [10.167.141.99]) by fnst.cn.fujitsu.com (Postfix) with ESMTPA id B695C10C10D; Thu, 3 Jun 2010 15:30:42 +0800 (CST) Message-ID: <4C07586B.4070709@cn.fujitsu.com> Date: Thu, 03 Jun 2010 15:23:23 +0800 From: Xiao Guangrong User-Agent: Thunderbird 2.0.0.24 (Windows/20100228) MIME-Version: 1.0 To: Jan Kiszka CC: Avi Kivity , Marcelo Tosatti , kvm Subject: Re: [PATCH] KVM: mmu: Remove unused local variable References: <4C0754DD.3030305@web.de> In-Reply-To: <4C0754DD.3030305@web.de> Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org X-Greylist: IP, sender and recipient auto-whitelisted, not delayed by milter-greylist-4.2.3 (demeter.kernel.org [140.211.167.41]); Thu, 03 Jun 2010 07:26:56 +0000 (UTC) diff --git a/arch/x86/kvm/mmu.c b/arch/x86/kvm/mmu.c index c16c4ca..a62e3ba 100644 --- a/arch/x86/kvm/mmu.c +++ b/arch/x86/kvm/mmu.c @@ -2049,7 +2049,7 @@ static void kvm_send_hwpoison_signal(struct kvm *kvm, gfn_t gfn) /* Touch the page, so send SIGBUS */ hva = (void __user *)gfn_to_hva(kvm, gfn); - (void)copy_from_user(buf, hva, 1); + r = copy_from_user(buf, hva, 1); } static int kvm_handle_bad_page(struct kvm *kvm, gfn_t gfn, pfn_t pfn)