From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.3 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8124EC3F68F for ; Wed, 11 Dec 2019 16:26:58 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 561B22073D for ; Wed, 11 Dec 2019 16:26:58 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=ziepe.ca header.i=@ziepe.ca header.b="CaW8ujBX" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1730217AbfLKQ05 (ORCPT ); Wed, 11 Dec 2019 11:26:57 -0500 Received: from mail-ot1-f68.google.com ([209.85.210.68]:35502 "EHLO mail-ot1-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729522AbfLKQ04 (ORCPT ); Wed, 11 Dec 2019 11:26:56 -0500 Received: by mail-ot1-f68.google.com with SMTP id o9so388696ote.2 for ; Wed, 11 Dec 2019 08:26:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ziepe.ca; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to:user-agent; bh=Z6zdU/xrkKE1DcOraB+HKtpGPsk3EYPT+vSqo90xQK8=; b=CaW8ujBXnOth7gzj1/yTbWXBB0zw/xzfLeR+oez1pY6varRdlaJda+WLsMdpouROKO rHf5xGhjp0BQxAAxsK+Qz53wPMUkC8ixRAf9epqgy0o1d5yuk8aF8ZfWVDQMA6lCvisk k7IpYRD57FfHHzRhbKGGZT3thtXY9Sr5E1FnNW9Xx62TK0MB2XqENh93qMRbHHrFno2i k/BgUy24YVt+8MmsveGZlpU/SgGTIVhL+/5IVp6EWoH+yzARD8aITWpcJdaet/wZQlY/ PtwERhu+kS18Efh2dzwQy0lN/gF3AkkoHiT6UeqNPfs0XiGoiRKosmagKRP1ElvnXYUo fPSw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to:user-agent; bh=Z6zdU/xrkKE1DcOraB+HKtpGPsk3EYPT+vSqo90xQK8=; b=mP9R+JmCKk7twLVrbkOns+rfICdtglWhGIGfOeT1klsq+dmk+qAW1OvNvI/yJjQFC4 vLQyMDdi4aY0shxiFkNhQQCVbfqgS2xRgAnGBsg/eUs2Wjifqjn4s5Qt6NLndMC/UuhJ I6r3WtCR44bx1LAkkhdbZa6YTmqMuO5Eck6UAq8vlRR0DFTRSJoK6k22psq9uDkuAuyW hLkx1ccngyVhO14/czgZCe4NBfEK3A8qfO8XlJs+HAhmAWlc04vy1twlqO1k3x3UzGii Z1Vc5KUT+TRyMBb/Cdh7FutN3NcGSciU5k/qJDyvF56nQJdgoqWmhnzzi0ee/V2juVCn uoTg== X-Gm-Message-State: APjAAAXp90busyDRtdNekXyHmrobKwR6sVJRuNwo7dRwmJx/guJfZYxl 7qhkB1oye2jr9ufJaCAU7NxVjg== X-Google-Smtp-Source: APXvYqzDPcIfKmYiNqIZudbnrsEtajQSOgxKei4/X6mu9+jpsjBeyvf0gsPeDXOQteLqPsfBJeHOUw== X-Received: by 2002:a9d:32e5:: with SMTP id u92mr2974798otb.85.1576081615921; Wed, 11 Dec 2019 08:26:55 -0800 (PST) Received: from ziepe.ca ([217.140.111.136]) by smtp.gmail.com with ESMTPSA id t71sm949802oif.45.2019.12.11.08.26.55 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Wed, 11 Dec 2019 08:26:55 -0800 (PST) Received: from jgg by jggl.ziepe.ca with local (Exim 4.90_1) (envelope-from ) id 1if4pC-0001vW-Tt; Wed, 11 Dec 2019 12:26:54 -0400 Date: Wed, 11 Dec 2019 12:26:54 -0400 From: Jason Gunthorpe To: Max Hirsch Cc: Doug Ledford , Parav Pandit , Leon Romanovsky , Steve Wise , Bart Van Assche , Danit Goldberg , Matthew Wilcox , Dag Moxnes , Myungho Jung , linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] RDMA/cma: Fix checkpatch error Message-ID: <20191211162654.GD6622@ziepe.ca> References: <20191211111628.2955-1-max.hirsch@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191211111628.2955-1-max.hirsch@gmail.com> User-Agent: Mutt/1.9.4 (2018-02-28) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 11, 2019 at 11:16:26AM +0000, Max Hirsch wrote: > When running checkpatch on cma.c the following error was found: I think checkpatch will complain about your patch, did you run it? > ERROR: do not use assignment in if condition > #413: FILE: drivers/infiniband/tmp.c:413: > + if ((ret = (id_priv->state == comp))) > > This patch moves the assignment of ret to the previous line. The if statement then checks the value of ret assigned on the previous line. The assigned value of ret is not changed. Testing involved recompiling and loading the kernel. After the changes checkpatch does not report this the error in cma.c. > > Signed-off-by: Max Hirsch > drivers/infiniband/core/cma.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/infiniband/core/cma.c b/drivers/infiniband/core/cma.c > index 25f2b70fd8ef..bdb7a8493517 100644 > +++ b/drivers/infiniband/core/cma.c > @@ -410,7 +410,8 @@ static int cma_comp_exch(struct rdma_id_private *id_priv, > int ret; > > spin_lock_irqsave(&id_priv->lock, flags); > - if ((ret = (id_priv->state == comp))) > + ret = (id_priv->state == comp); Brackets are not needed Ret and the return result should be changed to a bool Jason