Received: by 10.192.165.156 with SMTP id m28csp134856imm; Wed, 18 Apr 2018 18:53:38 -0700 (PDT) X-Google-Smtp-Source: AIpwx48Q6lwDkfSpnhpoHc+7MEle4xo+6TpLD6IjhoFCixtAZ5irha4YGe9wlqXmscCbUNfRCdnB X-Received: by 10.99.124.1 with SMTP id x1mr2941718pgc.286.1524102818862; Wed, 18 Apr 2018 18:53:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1524102818; cv=none; d=google.com; s=arc-20160816; b=f0LQ5Zxsnm4/h5dBMyTeXB4wMQoC5ZzDvhyXFIfufs3phkLSACIz6sROuvgnjtewir 5wLgkw16QOvetUfiM+oaQwg8TeMJhgqF0RIfb9z/scjLSA+AmhXpkwwSFFZcjd1Cws7P wUpwS6uzeR7SyWvKVC1wKbaXr9sjiocwL/mRCnIi8UjBUDv7tfs518Io+ASc11xY3J4K fxIZA6PrxTfFsZBwy8Lb1s76Gbjh/PuBmqlRBnDjeAAOjVaVKo6DKvYhqR4whvxkhPjE roolWLoM9vyZbDTclZNBp6JhefVG4eTgMD8WupXAQ5pN9WODBXWCqRKGtAqI60NGfrqc 6Z1A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:content-transfer-encoding :content-id:content-language:accept-language:in-reply-to:references :message-id:date:thread-index:thread-topic:subject:cc:to:from :arc-authentication-results; bh=XJV7dlKdVEjZLVc5s6mxKCYKlQEHCc9T13VGYT4+iYI=; b=tbxvLCKZl4gSh9zqF1BwbQFvrRux7RVD7TLD63j5/Okphub5ue26yUMXit15Ovo19v rTMHARAuNFVj5Y9aH3ORO+NE70ZRvWN1f8d6zJTGSTiafPcUC8+XTruslaDJgNWMSGj+ GsifjnX3McYKRTbmahRMrnLblCXemFchQKu4R5MjuR1vbqyh/lo6WmOE7+L9UcFvcK4w u0w/QyiKkGtXx1MLeExRa6DtQfPUSpZftO4msjDvu8cnZtEZGGZ2ZZmcrRHJZBII7UWE 01ux1TJ3XfwSbAGysBzoBDpA+3WiyYtOEOOCnmUXMVgfdimdDtMO6VC4JC257/wNphmt zS5w== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id c1si2033977pgp.557.2018.04.18.18.53.24; Wed, 18 Apr 2018 18:53:38 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752907AbeDSBwB convert rfc822-to-8bit (ORCPT + 99 others); Wed, 18 Apr 2018 21:52:01 -0400 Received: from mx01.hxt-semitech.com.96.203.223.in-addr.arpa ([223.203.96.7]:55809 "EHLO barracuda.hxt-semitech.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752700AbeDSBwA (ORCPT ); Wed, 18 Apr 2018 21:52:00 -0400 X-ASG-Debug-ID: 1524102717-093b7e10b3c3990001-xx1T2L Received: from HXTBJIDCEMVIW02.hxtcorp.net (localhost [10.128.0.15]) by barracuda.hxt-semitech.com with ESMTP id gWxDOCCx67nLUQ1B (version=TLSv1 cipher=ECDHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 19 Apr 2018 09:51:57 +0800 (CST) X-Barracuda-Envelope-From: shunyong.yang@hxt-semitech.com Received: from HXTBJIDCEMVIW01.hxtcorp.net (10.128.0.14) by HXTBJIDCEMVIW02.hxtcorp.net (10.128.0.15) with Microsoft SMTP Server (TLS) id 15.0.847.32; Thu, 19 Apr 2018 09:52:02 +0800 Received: from HXTBJIDCEMVIW01.hxtcorp.net ([fe80::f451:a443:c0b5:87d1]) by HXTBJIDCEMVIW01.hxtcorp.net ([fe80::f451:a443:c0b5:87d1%12]) with mapi id 15.00.0847.030; Thu, 19 Apr 2018 09:52:02 +0800 From: "Yang, Shunyong" To: "sohil.mehta@intel.com" , "gary.hook@amd.com" , "iommu@lists.linux-foundation.org" , "ghook@amd.com" CC: "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v3 2/2] iommu/amd: Add basic debugfs infrastructure for AMD IOMMU Thread-Topic: [PATCH v3 2/2] iommu/amd: Add basic debugfs infrastructure for AMD IOMMU X-ASG-Orig-Subj: Re: [PATCH v3 2/2] iommu/amd: Add basic debugfs infrastructure for AMD IOMMU Thread-Index: AQHT1tpRJAZW59A4ZEqPfHlYD/OogKQFmZcAgAAR5ACAAMT2gIAACcYAgABT/QA= Date: Thu, 19 Apr 2018 01:52:02 +0000 Message-ID: <1524102722.10902.1.camel@hxt-semitech.com> References: <152302042701.47565.17954813724758433858.stgit@sosxen2.amd.com> <152302067362.47565.628887796165116844.stgit@sosxen2.amd.com> <1523664612.2747.13.camel@intel.com> <1524031122.4031.5.camel@hxt-semitech.com> <1524036520.2747.31.camel@intel.com> <1524040290.4031.25.camel@hxt-semitech.com> <1524082659.2747.37.camel@intel.com> <151f3396-959b-86f6-e056-58d38682d858@amd.com> In-Reply-To: <151f3396-959b-86f6-e056-58d38682d858@amd.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.64.6.5] Content-Type: text/plain; charset="iso-8859-15" Content-ID: <156E9AA99371004CA43B33CE2A76ADFD@hxt-semitech.com> Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-Barracuda-Connect: localhost[10.128.0.15] X-Barracuda-Start-Time: 1524102717 X-Barracuda-Encrypted: ECDHE-RSA-AES256-SHA X-Barracuda-URL: https://192.168.50.101:443/cgi-mod/mark.cgi X-Virus-Scanned: by bsmtpd at hxt-semitech.com X-Barracuda-BRTS-Status: 1 X-Barracuda-Bayes: INNOCENT GLOBAL 0.6345 1.0000 0.9281 X-Barracuda-Spam-Score: 1.43 X-Barracuda-Spam-Status: No, SCORE=1.43 using global scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=9.0 tests=BSF_RULE7568M X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.50042 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- 0.50 BSF_RULE7568M Custom Rule 7568M Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, Gary, On Wed, 2018-04-18 at 16:51 -0400, Hook, Gary wrote: > On 4/18/2018 4:16 PM, Mehta, Sohil wrote: > > > > On Wed, 2018-04-18 at 08:31 +0000, Yang, Shunyong wrote: > > > > > > Maybe the original design is to call?debugfs_initialized() before > > > calling?debugfs_create_xxx()? > > I am unaware of the original design. Someone else would probably > > have > > more context. However, looking at other places in the kernel where > > debugfs_create_xx() is used, the common convention seems to be to > > avoid > > calling?debugfs_initialized(). > > > > ??Sohil > > > debugfs_initialized() was introduced in commit c0f92ba99 back in? > 2.6.30-rc1. It was intended as a helper, not as a gatekeeper, which > is? > why one doesn't see it used. Given that my use in this proposed patch > is? > straightforward, I'm not seeing the need here. I had just seen some? > other code that used it, and copied the model. > > Unless someone comes along to say, yes, use it, I'll not. > I agree with you and Sohil on removing the unnecessary function calling. Thanks. Shunyong. > Gary