From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753977AbdLDLL0 (ORCPT ); Mon, 4 Dec 2017 06:11:26 -0500 Received: from mail-it0-f67.google.com ([209.85.214.67]:42922 "EHLO mail-it0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751630AbdLDLLX (ORCPT ); Mon, 4 Dec 2017 06:11:23 -0500 X-Google-Smtp-Source: AGs4zMYgAQhh22zxUSwGTK29sKJ56YkNTkQWRJjl0Tg6EeV29DiTEpH9u0S9yuqfX/lH+cZ6/5K7ekH/J2498vLBmS8= MIME-Version: 1.0 In-Reply-To: <20171201080049.GB4218@cbox> References: <1511909297-8585-1-git-send-email-gomonovych@gmail.com> <20171201080049.GB4218@cbox> From: "Gomonovych, Vasyl" Date: Mon, 4 Dec 2017 12:11:22 +0100 Message-ID: Subject: Re: [PATCH] KVM: arm: Use PTR_ERR_OR_ZERO() To: Christoffer Dall Cc: Christoffer Dall , marc.zyngier@arm.com, eric.auger@redhat.com, andre.przywara@arm.com, wanghaibin.wang@huawei.com, linux-arm-kernel@lists.infradead.org, kvmarm@lists.cs.columbia.edu, linux-kernel@vger.kernel.org Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id vB4BBUjn006896 Hi Christoffer It is just syntax sugar of course and in mentioned function context it looks harmonically because it is in the end of function return statement. But in context of around source files it is looks not so harmonically because existing code uses old approach. And this old approach is only in one place here. So it is just a try to fix the warning and may be force to use it in a future changes. Regards Vasyl On Fri, Dec 1, 2017 at 9:00 AM, Christoffer Dall wrote: > Hi Vasyl, > > On Tue, Nov 28, 2017 at 11:48:17PM +0100, Vasyl Gomonovych wrote: >> Fix ptr_ret.cocci warnings: >> virt/kvm/arm/vgic/vgic-its.c:971:1-3: WARNING: PTR_ERR_OR_ZERO can be used >> >> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR >> >> Generated by: scripts/coccinelle/api/ptr_ret.cocci > > For new code, sure it would make sense to use this. > > But why is it worth changing existing code to use PTR_ERR_OR_ZERO? > > Thanks, > -Christoffer > >> >> Signed-off-by: Vasyl Gomonovych >> --- >> virt/kvm/arm/vgic/vgic-its.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/virt/kvm/arm/vgic/vgic-its.c b/virt/kvm/arm/vgic/vgic-its.c >> index 1f761a9991e7..28d85754320b 100644 >> --- a/virt/kvm/arm/vgic/vgic-its.c >> +++ b/virt/kvm/arm/vgic/vgic-its.c >> @@ -1032,10 +1032,8 @@ static int vgic_its_cmd_handle_mapd(struct kvm *kvm, struct vgic_its *its, >> >> device = vgic_its_alloc_device(its, device_id, itt_addr, >> num_eventid_bits); >> - if (IS_ERR(device)) >> - return PTR_ERR(device); >> >> - return 0; >> + return PTR_ERR_OR_ZERO(device); >> } >> >> /* >> -- >> 1.9.1 >> -- Доброї вам пори дня. From mboxrd@z Thu Jan 1 00:00:00 1970 From: gomonovych@gmail.com (Gomonovych, Vasyl) Date: Mon, 4 Dec 2017 12:11:22 +0100 Subject: [PATCH] KVM: arm: Use PTR_ERR_OR_ZERO() In-Reply-To: <20171201080049.GB4218@cbox> References: <1511909297-8585-1-git-send-email-gomonovych@gmail.com> <20171201080049.GB4218@cbox> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Christoffer It is just syntax sugar of course and in mentioned function context it looks harmonically because it is in the end of function return statement. But in context of around source files it is looks not so harmonically because existing code uses old approach. And this old approach is only in one place here. So it is just a try to fix the warning and may be force to use it in a future changes. Regards Vasyl On Fri, Dec 1, 2017 at 9:00 AM, Christoffer Dall wrote: > Hi Vasyl, > > On Tue, Nov 28, 2017 at 11:48:17PM +0100, Vasyl Gomonovych wrote: >> Fix ptr_ret.cocci warnings: >> virt/kvm/arm/vgic/vgic-its.c:971:1-3: WARNING: PTR_ERR_OR_ZERO can be used >> >> Use PTR_ERR_OR_ZERO rather than if(IS_ERR(...)) + PTR_ERR >> >> Generated by: scripts/coccinelle/api/ptr_ret.cocci > > For new code, sure it would make sense to use this. > > But why is it worth changing existing code to use PTR_ERR_OR_ZERO? > > Thanks, > -Christoffer > >> >> Signed-off-by: Vasyl Gomonovych >> --- >> virt/kvm/arm/vgic/vgic-its.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/virt/kvm/arm/vgic/vgic-its.c b/virt/kvm/arm/vgic/vgic-its.c >> index 1f761a9991e7..28d85754320b 100644 >> --- a/virt/kvm/arm/vgic/vgic-its.c >> +++ b/virt/kvm/arm/vgic/vgic-its.c >> @@ -1032,10 +1032,8 @@ static int vgic_its_cmd_handle_mapd(struct kvm *kvm, struct vgic_its *its, >> >> device = vgic_its_alloc_device(its, device_id, itt_addr, >> num_eventid_bits); >> - if (IS_ERR(device)) >> - return PTR_ERR(device); >> >> - return 0; >> + return PTR_ERR_OR_ZERO(device); >> } >> >> /* >> -- >> 1.9.1 >> -- ?????? ??? ???? ???.