Message ID | 20230420094332.1507900-1-korantwork@gmail.com |
---|---|
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp205736vqo; Thu, 20 Apr 2023 02:47:47 -0700 (PDT) X-Google-Smtp-Source: AKy350YEvo6JIHT03HOXj9pnEX+HrOM2oZWJ5iu11wJbxJ6mVElEAdHN2Kcgs6RIxXuBZshTlx8U X-Received: by 2002:a17:90b:3e84:b0:246:fd0f:3811 with SMTP id rj4-20020a17090b3e8400b00246fd0f3811mr962321pjb.40.1681984066953; Thu, 20 Apr 2023 02:47:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1681984066; cv=none; d=google.com; s=arc-20160816; b=uPu3Wtp78DsAzkoY0pCFDwK7mRboMdMvMzcci7ZRixMYbnZxJLiUsw/P1CZziQ0h4I N/ff5JNEr6sj2gPYgdfTQJ+fJlAfV1P2pJBe4mqxvvzom+fefO74Y//o3bZAOb0jdQvU /17clWpt+o6NaEgsAr5aBYQozoOIYcDFKjPPu5MGaYL8hkzkKkXefySoxFHOg5By/z0C 0+jxIJXHlbGO07WWl1DczR0J5diGKvDA3CdDw+yK0eHc2VlDFVi5t8oCbpAoK/N/9N5w Kthd84a4DnkpGQ57Ry2492tkXnZUkrRFdSetJgqnPaD1OREi5t1N6h2B2x3LAkszGkON NrsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from:dkim-signature; bh=GL/FkQKGL/P5lWSjTq7c3Pg0SaQ1eNQckaMYnyh+DUk=; b=hoWOshjgS3bEYXZQXaHLbdj2qsGGJe1eB7rcUQuhotGcHhVfU+fcoDihdHgGpLqBLi s0njFc8ZOSji/Ad/zCw2tvo2iKehtwDczBg4ZkDWofsT2yqwBo5MlGJKLJLCSPrq1QxO 7AXgjvNoNOoW0VgoPt4mku7S/49eQ+L0UtiGonoxfgaxyJ+oU0h19PFJoa996JLftkvP liIFNNGHbapJWppYr6Opl643u5v2xxGURWVbqjH6gkUW2/jJsYwMgSUgjkhtitBPtr/G 8ZmbX4v/IxxO0Nn30JGMMwZU3Q5pc7S6hJWnQpc7QvNwr/dXL11mrk3Boi2nnKIMW0kn Vw0Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=RhhzFDMK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id 19-20020a631553000000b0050f79f0651fsi1286409pgv.95.2023.04.20.02.47.31; Thu, 20 Apr 2023 02:47:46 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20221208 header.b=RhhzFDMK; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234115AbjDTJpU (ORCPT <rfc822;cjcooper78@gmail.com> + 99 others); Thu, 20 Apr 2023 05:45:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57546 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234566AbjDTJou (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Thu, 20 Apr 2023 05:44:50 -0400 Received: from mail-pf1-x443.google.com (mail-pf1-x443.google.com [IPv6:2607:f8b0:4864:20::443]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 590E972B7; Thu, 20 Apr 2023 02:43:37 -0700 (PDT) Received: by mail-pf1-x443.google.com with SMTP id d2e1a72fcca58-63b7b54642cso668970b3a.0; Thu, 20 Apr 2023 02:43:37 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1681983817; x=1684575817; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=GL/FkQKGL/P5lWSjTq7c3Pg0SaQ1eNQckaMYnyh+DUk=; b=RhhzFDMKZ1kIqLGtozulZoedaUTHVms6wb7ySjjZL/ubfFgxvE0Gx0jC0mJUqdVEBt VWot2xAfxFc2dHWxMlcwBWGk38ZAb0/RmnJSJInmJeV5nVgo+2DqCuZjio543R/OFDLo 1o0lyU+jTpPauvg2a22g928ZVIItgitUyJ1AKWdtaxMUY1lbHa0JsPsisdmEtHTFUV9L WVjvuanX6Syivyj1VKvYFwf16MlYl2w9SaigCfuQ9xQ/NxrYuc/tEMjVkpSfTMSK/pEE CRabIEBsaFWUt/w+RI6I36yNrxaqrr7xG3JFU8uxfsAW/EqWMM9z7+flucJv0yPWtcr4 z4fQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1681983817; x=1684575817; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=GL/FkQKGL/P5lWSjTq7c3Pg0SaQ1eNQckaMYnyh+DUk=; b=QkWkkBetIIkD9U1f1uExW3Uu1rsJ0GJXuspsQq8QCs7V5Mnjbzhgb2xiJo2wUY5GzG hUASMZgUutiQxBNfE1GMKbc+NhblZVu1Idu+NwuaBw3k4WuIZpRaugaS/4cELltpqExX n358F/E9wVLzy7aXtEyrteGdQqn8feOvHMLhvHCZWK96QeNSVBC8Q27Pu9Wqy+xmtql9 aSLW+O3gE0sDFRekLe6wWakIGYMzDKYV3jAdL9wLasEsNDAcBhUCJ9AekEFDQQWNHPPs 6e17WuyVCF10mpG4D9JqKc/cXeX4QuVpikoF/BFffhKYYpg7LNy1bmc0hyR4i+dyJ5yh EJTQ== X-Gm-Message-State: AAQBX9dMOH+g0t/AE3mNqhOxFdqo23wAcDVl0dhhRGWcGEr2adXwio7D zBXLHTCEjLFwluWWrJyf74r0cj4NyYV6Y7Fu X-Received: by 2002:a05:6a20:440f:b0:de:13c4:5529 with SMTP id ce15-20020a056a20440f00b000de13c45529mr1428465pzb.62.1681983816675; Thu, 20 Apr 2023 02:43:36 -0700 (PDT) Received: from localhost.localdomain ([43.132.141.8]) by smtp.gmail.com with ESMTPSA id t1-20020a62d141000000b0063b8ce0e860sm874336pfl.21.2023.04.20.02.43.34 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Apr 2023 02:43:36 -0700 (PDT) From: korantwork@gmail.com To: dlemoal@kernel.org, helgaas@kernel.org, nirmal.patel@linux.intel.com, kbusch@kernel.org, jonathan.derrick@linux.dev, lpieralisi@kernel.org Cc: linux-pci@vger.kernel.org, linux-kernel@vger.kernel.org, Xinghui Li <korantli@tencent.com> Subject: [PATCH v2 0/2] PCI: vmd: Fix two issues in VMD reported by Smatch Date: Thu, 20 Apr 2023 17:43:30 +0800 Message-Id: <20230420094332.1507900-1-korantwork@gmail.com> X-Mailer: git-send-email 2.31.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1763688124592998517?= X-GMAIL-MSGID: =?utf-8?q?1763688124592998517?= |
Series |
PCI: vmd: Fix two issues in VMD reported by Smatch
|
|
Message
Xinghui Li
April 20, 2023, 9:43 a.m. UTC
From: Xinghui Li <korantli@tencent.com>
Fix two issues when building kernel with Smatch check.
v1->v2:
According to Damien's suggestion, I split it from 1 patch to 2 different
patches. Change 'inconsistent indenting' patch's title from 'fix' to
'clean up'.
Xinghui Li (2):
PCI: vmd: Fix one uninitialized symbol error reported by Smatch
PCI: vmd: Clean up one inconsistent indenting warn reported by Smatch
drivers/pci/controller/vmd.c | 11 ++++++-----
1 file changed, 6 insertions(+), 5 deletions(-)
Comments
[+CC Christoph] Hello, > Fix two issues when building kernel with Smatch check. > > v1->v2: > According to Damien's suggestion, I split it from 1 patch to 2 different > patches. Change 'inconsistent indenting' patch's title from 'fix' to > 'clean up'. > > Xinghui Li (2): > PCI: vmd: Fix one uninitialized symbol error reported by Smatch Applied to controller/vmd, thank you! [1/1] PCI: vmd: Fix uninitialized variable usage in vmd_enable_domain() https://git.kernel.org/pci/pci/c/0c0206dc4f5b > PCI: vmd: Clean up one inconsistent indenting warn reported by Smatch Even though this is a very nice clean-up, I did not take this patch at this time, as there has been a similar patch posted in the past, and Christoph Hellwig suggested, as part of his review, an alternative approach worth considering. Have a look at the following and let me know what you think: https://patchwork.kernel.org/project/linux-pci/patch/20221115054847.77829-1-jiapeng.chong@linux.alibaba.com/ Krzysztof
On Sun, Jun 25, 2023 at 12:33 AM Krzysztof Wilczyński <kw@linux.com> wrote: > > [1/1] PCI: vmd: Fix uninitialized variable usage in vmd_enable_domain() > https://git.kernel.org/pci/pci/c/0c0206dc4f5b > > > PCI: vmd: Clean up one inconsistent indenting warn reported by Smatch > > Even though this is a very nice clean-up, I did not take this patch at this > time, as there has been a similar patch posted in the past, and Christoph > Hellwig suggested, as part of his review, an alternative approach worth > considering. > > Have a look at the following and let me know what you think: > > https://patchwork.kernel.org/project/linux-pci/patch/20221115054847.77829-1-jiapeng.chong@linux.alibaba.com/ I think Christoph Hellwig's suggestion is indeed better. So, should I submit a new patch to address this issue? Thanks~
Hello, > > > PCI: vmd: Clean up one inconsistent indenting warn reported by Smatch > > > > Even though this is a very nice clean-up, I did not take this patch at this > > time, as there has been a similar patch posted in the past, and Christoph > > Hellwig suggested, as part of his review, an alternative approach worth > > considering. > > > > Have a look at the following and let me know what you think: > > > > https://patchwork.kernel.org/project/linux-pci/patch/20221115054847.77829-1-jiapeng.chong@linux.alibaba.com/ > > I think Christoph Hellwig's suggestion is indeed better. > So, should I submit a new patch to address this issue? Please. A single new patch would suffice. Make sure to explain as part of the commit log why this specific change that started as a Smatch complaint about an incorrect indentation. Thank you! Krzysztof