From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751819AbdAVSRg (ORCPT ); Sun, 22 Jan 2017 13:17:36 -0500 Received: from mout.web.de ([212.227.15.3]:55499 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751649AbdAVSRa (ORCPT ); Sun, 22 Jan 2017 13:17:30 -0500 Subject: [PATCH 7/9] KVM: Return an error code only as a constant in kvm_get_dirty_log() To: kvm@vger.kernel.org, Paolo Bonzini , =?UTF-8?B?UmFkaW0gS3LEjW3DocWZ?= References: Cc: LKML , kernel-janitors@vger.kernel.org From: SF Markus Elfring Message-ID: Date: Sun, 22 Jan 2017 19:17:22 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.6.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Provags-ID: V03:K0:JkBMFcdaSNAXimcD7h4Ou+oAFHZPgjLKmyYhwMvhoXGWVq5stxe cCTm2zUiDl/1HVRhh7SVAPBZNcQiYajopVRo/c+ZxCl0QFgkB2C0Km293WEjZBYrjhHavLC BVNip6seSRhyOfvBa+7MsVzq6aH2SnD8W0AeaVMyUM8+3ZwHK40h1inaWOPiXJbfOBd85d0 bCGLMolVvA19u9TVRs5iQ== X-UI-Out-Filterresults: notjunk:1;V01:K0:2/Q09pZae9A=:+KgYiE5+MmHx6H3C7CZBQK SJ/Eo+gRyWyAnWmkPo/2sV9DbCUfMoqRG+nfN7nIz2nG2MncZNCtKwwwWk/VE78Df0hifKFqD phK+XVZR2CMmF9rQaoxYL12CDXu07EQZNYfa+uxzDLPWjbLWtSErXhuzWd+ZyPnqcsoTPZJfV kAhgNjLmOP/xj4NDIp2A17G+zIicV8/Nd9rpIKuMF3IOD9GHckKtE7+ZhPbuW+pciDG27RRzY Vxvi0HAMVwZ6+n6BsXOjKZZiXoTYmAVl7Yro1NRZpWzdhxIxeldUBCCDJQQZNrz/HMobBkBYI UX3OxdLn/LwqD9c9A4qp3nJYQQzfcEKb8CBeDcnaipaIWZhYHlc6dMI8gVpWNYQE9Feb3v0Gf +8xGTesxOBBQcarZU8Ci7j1K04tCbXOHWFsIbnwRCxdVIRQRHfDXDiaJm5G2VXsHk2ezDVXCT JfG5vJs2Ib/pcxxfgZSWR4jlPORic7e0aAleqk0qB/Rm810VHs/HoDIOLTnJP56GiAF00uJkH MrO9UUOUfjunC/cDWPbCk/2rLurLGjyPdt8279Zc39L6PE3/c0Fwq3USUtJuKrHQIHTIx09hQ pAPtT4fSLNi5FF8u3H+nC1RAMoU1o+ljyfjrdKO492TYTO/pjY8F2SEpRHJMcS42Mby70XbWc BRcsE3enbX/jgizA90qixsRzN7gFJ8k/Q64Eg0rx25zMEXSgkurDPq6i8DOleMTD9Kjz2QFBC SpedM9AJ2EkI5FlZOFLxuy25pk8es+WjmgYcO6FVkKnzN+8OIa+2tCHn0mxDZ58+2DQKFmRiF L0E5Pju Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sun, 22 Jan 2017 17:41:07 +0100 * Return an error code without storing it in an intermediate variable. * Delete the local variable "r" and the jump label "out" which became unnecessary with this refactoring. Signed-off-by: Markus Elfring --- virt/kvm/kvm_main.c | 16 +++++----------- 1 file changed, 5 insertions(+), 11 deletions(-) diff --git a/virt/kvm/kvm_main.c b/virt/kvm/kvm_main.c index a651a137ef21..ca88f3620ee3 100644 --- a/virt/kvm/kvm_main.c +++ b/virt/kvm/kvm_main.c @@ -1094,37 +1094,31 @@ int kvm_get_dirty_log(struct kvm *kvm, { struct kvm_memslots *slots; struct kvm_memory_slot *memslot; - int r, i, as_id, id; + int i, as_id, id; unsigned long n; unsigned long any = 0; - r = -EINVAL; as_id = log->slot >> 16; id = (u16)log->slot; if (as_id >= KVM_ADDRESS_SPACE_NUM || id >= KVM_USER_MEM_SLOTS) - goto out; + return -EINVAL; slots = __kvm_memslots(kvm, as_id); memslot = id_to_memslot(slots, id); - r = -ENOENT; if (!memslot->dirty_bitmap) - goto out; + return -ENOENT; n = kvm_dirty_bitmap_bytes(memslot); for (i = 0; !any && i < n/sizeof(long); ++i) any = memslot->dirty_bitmap[i]; - r = -EFAULT; if (copy_to_user(log->dirty_bitmap, memslot->dirty_bitmap, n)) - goto out; + return -EFAULT; if (any) *is_dirty = 1; - - r = 0; -out: - return r; + return 0; } EXPORT_SYMBOL_GPL(kvm_get_dirty_log); -- 2.11.0