All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Nicolin Chen <nicolinc@nvidia.com>
Cc: llvm@lists.linux.dev, kbuild-all@lists.01.org
Subject: Re: [RFC][PATCH v2 12/13] iommu/arm-smmu-v3: Add support for NVIDIA CMDQ-Virtualization hw
Date: Tue, 31 Aug 2021 17:13:16 +0800	[thread overview]
Message-ID: <202108311739.L7o60B6U-lkp@intel.com> (raw)
In-Reply-To: <20210831025923.15812-13-nicolinc@nvidia.com>

[-- Attachment #1: Type: text/plain, Size: 3718 bytes --]

Hi Nicolin,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on iommu/next]
[also build test WARNING on next-20210830]
[cannot apply to vfio/next linus/master v5.14]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nicolin-Chen/iommu-arm-smmu-v3-Add-NVIDIA-implementation/20210831-111013
base:   https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next
config: arm64-randconfig-r015-20210831 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 4b1fde8a2b681dad2ce0c082a5d6422caa06b0bc)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        # https://github.com/0day-ci/linux/commit/9a22c788ed0320df4a0fb8689c3113d0bd0a91da
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nicolin-Chen/iommu-arm-smmu-v3-Add-NVIDIA-implementation/20210831-111013
        git checkout 9a22c788ed0320df4a0fb8689c3113d0bd0a91da
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c:410:21: warning: no previous prototype for function 'nvidia_smmu_create' [-Wmissing-prototypes]
   struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
                       ^
   drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c:410:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   ^
   static 
   1 warning generated.


vim +/nvidia_smmu_create +410 drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c

   373	
   374	#ifdef CONFIG_ACPI
   375	struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   376	{
   377		struct nvidia_smmu *nsmmu = NULL;
   378		struct acpi_iort_node *node;
   379		struct acpi_device *adev;
   380		struct device *cmdqv_dev;
   381		const char *match_uid;
   382	
   383		if (acpi_disabled)
   384			return NULL;
   385	
   386		/* Look for a device in the DSDT whose _UID matches the SMMU's iort_node identifier */
   387		node = *(struct acpi_iort_node **)dev_get_platdata(smmu->dev);
   388		match_uid = kasprintf(GFP_KERNEL, "%u", node->identifier);
   389		adev = acpi_dev_get_first_match_dev(NVIDIA_SMMU_CMDQV_HID, match_uid, -1);
   390		kfree(match_uid);
   391	
   392		if (!adev)
   393			return NULL;
   394	
   395		cmdqv_dev = bus_find_device_by_acpi_dev(&platform_bus_type, adev);
   396		if (!cmdqv_dev)
   397			return NULL;
   398	
   399		dev_info(smmu->dev, "found companion CMDQV device, %s", dev_name(cmdqv_dev));
   400	
   401		nsmmu = devm_krealloc(smmu->dev, smmu, sizeof(*nsmmu), GFP_KERNEL);
   402		if (!nsmmu)
   403			return ERR_PTR(-ENOMEM);
   404	
   405		nsmmu->cmdqv_dev = cmdqv_dev;
   406	
   407		return nsmmu;
   408	}
   409	#else
 > 410	struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   411	{
   412		return NULL;
   413	}
   414	#endif
   415	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 39823 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [RFC][PATCH v2 12/13] iommu/arm-smmu-v3: Add support for NVIDIA CMDQ-Virtualization hw
Date: Tue, 31 Aug 2021 17:13:16 +0800	[thread overview]
Message-ID: <202108311739.L7o60B6U-lkp@intel.com> (raw)
In-Reply-To: <20210831025923.15812-13-nicolinc@nvidia.com>

[-- Attachment #1: Type: text/plain, Size: 3810 bytes --]

Hi Nicolin,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on iommu/next]
[also build test WARNING on next-20210830]
[cannot apply to vfio/next linus/master v5.14]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    https://github.com/0day-ci/linux/commits/Nicolin-Chen/iommu-arm-smmu-v3-Add-NVIDIA-implementation/20210831-111013
base:   https://git.kernel.org/pub/scm/linux/kernel/git/joro/iommu.git next
config: arm64-randconfig-r015-20210831 (attached as .config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 4b1fde8a2b681dad2ce0c082a5d6422caa06b0bc)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        # https://github.com/0day-ci/linux/commit/9a22c788ed0320df4a0fb8689c3113d0bd0a91da
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Nicolin-Chen/iommu-arm-smmu-v3-Add-NVIDIA-implementation/20210831-111013
        git checkout 9a22c788ed0320df4a0fb8689c3113d0bd0a91da
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c:410:21: warning: no previous prototype for function 'nvidia_smmu_create' [-Wmissing-prototypes]
   struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
                       ^
   drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c:410:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   ^
   static 
   1 warning generated.


vim +/nvidia_smmu_create +410 drivers/iommu/arm/arm-smmu-v3/nvidia-smmu-v3.c

   373	
   374	#ifdef CONFIG_ACPI
   375	struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   376	{
   377		struct nvidia_smmu *nsmmu = NULL;
   378		struct acpi_iort_node *node;
   379		struct acpi_device *adev;
   380		struct device *cmdqv_dev;
   381		const char *match_uid;
   382	
   383		if (acpi_disabled)
   384			return NULL;
   385	
   386		/* Look for a device in the DSDT whose _UID matches the SMMU's iort_node identifier */
   387		node = *(struct acpi_iort_node **)dev_get_platdata(smmu->dev);
   388		match_uid = kasprintf(GFP_KERNEL, "%u", node->identifier);
   389		adev = acpi_dev_get_first_match_dev(NVIDIA_SMMU_CMDQV_HID, match_uid, -1);
   390		kfree(match_uid);
   391	
   392		if (!adev)
   393			return NULL;
   394	
   395		cmdqv_dev = bus_find_device_by_acpi_dev(&platform_bus_type, adev);
   396		if (!cmdqv_dev)
   397			return NULL;
   398	
   399		dev_info(smmu->dev, "found companion CMDQV device, %s", dev_name(cmdqv_dev));
   400	
   401		nsmmu = devm_krealloc(smmu->dev, smmu, sizeof(*nsmmu), GFP_KERNEL);
   402		if (!nsmmu)
   403			return ERR_PTR(-ENOMEM);
   404	
   405		nsmmu->cmdqv_dev = cmdqv_dev;
   406	
   407		return nsmmu;
   408	}
   409	#else
 > 410	struct nvidia_smmu *nvidia_smmu_create(struct arm_smmu_device *smmu)
   411	{
   412		return NULL;
   413	}
   414	#endif
   415	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 39823 bytes --]

  parent reply	other threads:[~2021-08-31  9:13 UTC|newest]

Thread overview: 63+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-31  2:59 [RFC][PATCH v2 00/13] iommu/arm-smmu-v3: Add NVIDIA implementation Nicolin Chen
2021-08-31  2:59 ` Nicolin Chen
2021-08-31  2:59 ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 01/13] iommu: Add set_nesting_vmid/get_nesting_vmid functions Nicolin Chen via iommu
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59 ` [RFC][PATCH v2 02/13] vfio: add VFIO_IOMMU_GET_VMID and VFIO_IOMMU_SET_VMID Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 03/13] vfio: Document VMID control for IOMMU Virtualization Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 04/13] vfio: add set_vmid and get_vmid for vfio_iommu_type1 Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 05/13] vfio/type1: Implement set_vmid and get_vmid Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 06/13] vfio/type1: Set/get VMID to/from iommu driver Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 07/13] iommu/arm-smmu-v3: Add shared VMID support for NESTING Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 08/13] iommu/arm-smmu-v3: Add VMID alloc/free helpers Nicolin Chen via iommu
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59 ` [RFC][PATCH v2 09/13] iommu/arm-smmu-v3: Pass dev pointer to arm_smmu_detach_dev Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 10/13] iommu/arm-smmu-v3: Pass cmdq pointer in arm_smmu_cmdq_issue_cmdlist() Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 11/13] iommu/arm-smmu-v3: Add implementation infrastructure Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  2:59 ` [RFC][PATCH v2 12/13] iommu/arm-smmu-v3: Add support for NVIDIA CMDQ-Virtualization hw Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  8:03   ` kernel test robot
2021-08-31  9:13   ` kernel test robot [this message]
2021-08-31  9:13     ` kernel test robot
2021-08-31  2:59 ` [RFC][PATCH v2 13/13] iommu/nvidia-smmu-v3: Add mdev interface support Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen
2021-08-31  2:59   ` Nicolin Chen via iommu
2021-08-31  8:58   ` kernel test robot
2021-08-31 10:26   ` kernel test robot
2021-08-31 10:26     ` kernel test robot
2021-08-31 16:15 ` [RFC][PATCH v2 00/13] iommu/arm-smmu-v3: Add NVIDIA implementation Alex Williamson
2021-08-31 16:15   ` Alex Williamson
2021-08-31 16:15   ` Alex Williamson
2021-09-01  6:55   ` Tian, Kevin
2021-09-01  6:55     ` Tian, Kevin
2021-09-01  6:55     ` Tian, Kevin
2021-09-02 14:45     ` Jason Gunthorpe
2021-09-02 14:45       ` Jason Gunthorpe
2021-09-02 14:45       ` Jason Gunthorpe via iommu
2021-09-02 22:27       ` Tian, Kevin
2021-09-02 22:27         ` Tian, Kevin
2021-09-02 22:27         ` Tian, Kevin
2021-09-16 18:21         ` Nicolin Chen
2021-09-16 18:21           ` Nicolin Chen
2021-09-16 18:21           ` Nicolin Chen via iommu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=202108311739.L7o60B6U-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=kbuild-all@lists.01.org \
    --cc=llvm@lists.linux.dev \
    --cc=nicolinc@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.