All of lore.kernel.org
 help / color / mirror / Atom feed
From: kernel test robot <lkp@intel.com>
To: Cheng Xu <chengyou@linux.alibaba.com>, jgg@ziepe.ca, dledford@redhat.com
Cc: kbuild-all@lists.01.org, leon@kernel.org,
	linux-rdma@vger.kernel.org, KaiShen@linux.alibaba.com,
	chengyou@linux.alibaba.com, tonylu@linux.alibaba.com,
	BMT@zurich.ibm.com
Subject: Re: [PATCH for-next v4 12/12] RDMA/erdma: Add driver to kernel build environment
Date: Thu, 17 Mar 2022 11:14:51 +0800	[thread overview]
Message-ID: <202203171051.nVmoVnU3-lkp@intel.com> (raw)
In-Reply-To: <20220314064739.81647-13-chengyou@linux.alibaba.com>

Hi Cheng,

I love your patch! Yet something to improve:

[auto build test ERROR on rdma/for-next]
[also build test ERROR on linus/master v5.17-rc8 next-20220316]
[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/Cheng-Xu/Elastic-RDMA-Adapter-ERDMA-driver/20220314-144952
base:   https://git.kernel.org/pub/scm/linux/kernel/git/rdma/rdma.git for-next
config: um-allyesconfig (https://download.01.org/0day-ci/archive/20220317/202203171051.nVmoVnU3-lkp@intel.com/config)
compiler: gcc-9 (Ubuntu 9.4.0-1ubuntu1~20.04) 9.4.0
reproduce (this is a W=1 build):
        # https://github.com/0day-ci/linux/commit/ad7293bc27eaf3a5e28fffcc325fb9814e6a170c
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Cheng-Xu/Elastic-RDMA-Adapter-ERDMA-driver/20220314-144952
        git checkout ad7293bc27eaf3a5e28fffcc325fb9814e6a170c
        # save the config file to linux build tree
        mkdir build_dir
        make W=1 O=build_dir ARCH=um SHELL=/bin/bash

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

All errors (new ones prefixed by >>):

   drivers/infiniband/hw/erdma/erdma_main.c: In function 'erdma_probe_dev':
>> drivers/infiniband/hw/erdma/erdma_main.c:266:34: error: 'struct device' has no member named 'numa_node'
     266 |  dev->attrs.numa_node = pdev->dev.numa_node;
         |                                  ^


vim +266 drivers/infiniband/hw/erdma/erdma_main.c

766cdafe9cd7d4 Cheng Xu 2022-03-14  241  
766cdafe9cd7d4 Cheng Xu 2022-03-14  242  static int erdma_probe_dev(struct pci_dev *pdev)
766cdafe9cd7d4 Cheng Xu 2022-03-14  243  {
766cdafe9cd7d4 Cheng Xu 2022-03-14  244  	int err;
766cdafe9cd7d4 Cheng Xu 2022-03-14  245  	struct erdma_dev *dev;
766cdafe9cd7d4 Cheng Xu 2022-03-14  246  	u32 version;
766cdafe9cd7d4 Cheng Xu 2022-03-14  247  	int bars;
766cdafe9cd7d4 Cheng Xu 2022-03-14  248  
766cdafe9cd7d4 Cheng Xu 2022-03-14  249  	err = pci_enable_device(pdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  250  	if (err) {
766cdafe9cd7d4 Cheng Xu 2022-03-14  251  		dev_err(&pdev->dev, "pci_enable_device failed(%d)\n", err);
766cdafe9cd7d4 Cheng Xu 2022-03-14  252  		return err;
766cdafe9cd7d4 Cheng Xu 2022-03-14  253  	}
766cdafe9cd7d4 Cheng Xu 2022-03-14  254  
766cdafe9cd7d4 Cheng Xu 2022-03-14  255  	pci_set_master(pdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  256  
766cdafe9cd7d4 Cheng Xu 2022-03-14  257  	dev = ib_alloc_device(erdma_dev, ibdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  258  	if (!dev) {
766cdafe9cd7d4 Cheng Xu 2022-03-14  259  		dev_err(&pdev->dev, "ib_alloc_device failed\n");
766cdafe9cd7d4 Cheng Xu 2022-03-14  260  		err = -ENOMEM;
766cdafe9cd7d4 Cheng Xu 2022-03-14  261  		goto err_disable_device;
766cdafe9cd7d4 Cheng Xu 2022-03-14  262  	}
766cdafe9cd7d4 Cheng Xu 2022-03-14  263  
766cdafe9cd7d4 Cheng Xu 2022-03-14  264  	pci_set_drvdata(pdev, dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  265  	dev->pdev = pdev;
766cdafe9cd7d4 Cheng Xu 2022-03-14 @266  	dev->attrs.numa_node = pdev->dev.numa_node;
766cdafe9cd7d4 Cheng Xu 2022-03-14  267  
766cdafe9cd7d4 Cheng Xu 2022-03-14  268  	bars = pci_select_bars(pdev, IORESOURCE_MEM);
766cdafe9cd7d4 Cheng Xu 2022-03-14  269  	err = pci_request_selected_regions(pdev, bars, DRV_MODULE_NAME);
766cdafe9cd7d4 Cheng Xu 2022-03-14  270  	if (bars != ERDMA_BAR_MASK || err) {
766cdafe9cd7d4 Cheng Xu 2022-03-14  271  		err = err == 0 ? -EINVAL : err;
766cdafe9cd7d4 Cheng Xu 2022-03-14  272  		goto err_ib_device_release;
766cdafe9cd7d4 Cheng Xu 2022-03-14  273  	}
766cdafe9cd7d4 Cheng Xu 2022-03-14  274  
766cdafe9cd7d4 Cheng Xu 2022-03-14  275  	dev->func_bar_addr = pci_resource_start(pdev, ERDMA_FUNC_BAR);
766cdafe9cd7d4 Cheng Xu 2022-03-14  276  	dev->func_bar_len = pci_resource_len(pdev, ERDMA_FUNC_BAR);
766cdafe9cd7d4 Cheng Xu 2022-03-14  277  
766cdafe9cd7d4 Cheng Xu 2022-03-14  278  	dev->func_bar = devm_ioremap(&pdev->dev,
766cdafe9cd7d4 Cheng Xu 2022-03-14  279  				     dev->func_bar_addr,
766cdafe9cd7d4 Cheng Xu 2022-03-14  280  				     dev->func_bar_len);
766cdafe9cd7d4 Cheng Xu 2022-03-14  281  	if (!dev->func_bar) {
766cdafe9cd7d4 Cheng Xu 2022-03-14  282  		dev_err(&pdev->dev, "devm_ioremap failed.\n");
766cdafe9cd7d4 Cheng Xu 2022-03-14  283  		err = -EFAULT;
766cdafe9cd7d4 Cheng Xu 2022-03-14  284  		goto err_release_bars;
766cdafe9cd7d4 Cheng Xu 2022-03-14  285  	}
766cdafe9cd7d4 Cheng Xu 2022-03-14  286  
766cdafe9cd7d4 Cheng Xu 2022-03-14  287  	version = erdma_reg_read32(dev, ERDMA_REGS_VERSION_REG);
766cdafe9cd7d4 Cheng Xu 2022-03-14  288  	if (version == 0) {
766cdafe9cd7d4 Cheng Xu 2022-03-14  289  		/* we knows that it is a non-functional function. */
766cdafe9cd7d4 Cheng Xu 2022-03-14  290  		err = -ENODEV;
766cdafe9cd7d4 Cheng Xu 2022-03-14  291  		goto err_iounmap_func_bar;
766cdafe9cd7d4 Cheng Xu 2022-03-14  292  	}
766cdafe9cd7d4 Cheng Xu 2022-03-14  293  
766cdafe9cd7d4 Cheng Xu 2022-03-14  294  	err = erdma_device_init(dev, pdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  295  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  296  		goto err_iounmap_func_bar;
766cdafe9cd7d4 Cheng Xu 2022-03-14  297  
766cdafe9cd7d4 Cheng Xu 2022-03-14  298  	err = erdma_request_vectors(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  299  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  300  		goto err_iounmap_func_bar;
766cdafe9cd7d4 Cheng Xu 2022-03-14  301  
766cdafe9cd7d4 Cheng Xu 2022-03-14  302  	err = erdma_comm_irq_init(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  303  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  304  		goto err_free_vectors;
766cdafe9cd7d4 Cheng Xu 2022-03-14  305  
766cdafe9cd7d4 Cheng Xu 2022-03-14  306  	err = erdma_aeq_init(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  307  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  308  		goto err_uninit_comm_irq;
766cdafe9cd7d4 Cheng Xu 2022-03-14  309  
766cdafe9cd7d4 Cheng Xu 2022-03-14  310  	err = erdma_cmdq_init(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  311  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  312  		goto err_uninit_aeq;
766cdafe9cd7d4 Cheng Xu 2022-03-14  313  
766cdafe9cd7d4 Cheng Xu 2022-03-14  314  	err = erdma_ceqs_init(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  315  	if (err)
766cdafe9cd7d4 Cheng Xu 2022-03-14  316  		goto err_uninit_cmdq;
766cdafe9cd7d4 Cheng Xu 2022-03-14  317  
766cdafe9cd7d4 Cheng Xu 2022-03-14  318  	erdma_finish_cmdq_init(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  319  
766cdafe9cd7d4 Cheng Xu 2022-03-14  320  	return 0;
766cdafe9cd7d4 Cheng Xu 2022-03-14  321  
766cdafe9cd7d4 Cheng Xu 2022-03-14  322  err_uninit_cmdq:
766cdafe9cd7d4 Cheng Xu 2022-03-14  323  	erdma_device_uninit(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  324  	erdma_cmdq_destroy(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  325  
766cdafe9cd7d4 Cheng Xu 2022-03-14  326  err_uninit_aeq:
766cdafe9cd7d4 Cheng Xu 2022-03-14  327  	erdma_aeq_destroy(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  328  
766cdafe9cd7d4 Cheng Xu 2022-03-14  329  err_uninit_comm_irq:
766cdafe9cd7d4 Cheng Xu 2022-03-14  330  	erdma_comm_irq_uninit(dev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  331  
766cdafe9cd7d4 Cheng Xu 2022-03-14  332  err_free_vectors:
766cdafe9cd7d4 Cheng Xu 2022-03-14  333  	pci_free_irq_vectors(dev->pdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  334  
766cdafe9cd7d4 Cheng Xu 2022-03-14  335  err_iounmap_func_bar:
766cdafe9cd7d4 Cheng Xu 2022-03-14  336  	devm_iounmap(&pdev->dev, dev->func_bar);
766cdafe9cd7d4 Cheng Xu 2022-03-14  337  
766cdafe9cd7d4 Cheng Xu 2022-03-14  338  err_release_bars:
766cdafe9cd7d4 Cheng Xu 2022-03-14  339  	pci_release_selected_regions(pdev, bars);
766cdafe9cd7d4 Cheng Xu 2022-03-14  340  
766cdafe9cd7d4 Cheng Xu 2022-03-14  341  err_ib_device_release:
766cdafe9cd7d4 Cheng Xu 2022-03-14  342  	ib_dealloc_device(&dev->ibdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  343  
766cdafe9cd7d4 Cheng Xu 2022-03-14  344  err_disable_device:
766cdafe9cd7d4 Cheng Xu 2022-03-14  345  	pci_disable_device(pdev);
766cdafe9cd7d4 Cheng Xu 2022-03-14  346  
766cdafe9cd7d4 Cheng Xu 2022-03-14  347  	return err;
766cdafe9cd7d4 Cheng Xu 2022-03-14  348  }
766cdafe9cd7d4 Cheng Xu 2022-03-14  349  

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

      reply	other threads:[~2022-03-17  3:15 UTC|newest]

Thread overview: 39+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-14  6:47 [PATCH for-next v4 00/12] Elastic RDMA Adapter (ERDMA) driver Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 01/12] RDMA: Add ERDMA to rdma_driver_id definition Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 02/12] RDMA/core: Allow calling query_port when netdev isn't attached in iWarp Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 03/12] RDMA/erdma: Add the hardware related definitions Cheng Xu
2022-03-18 10:27   ` Wenpeng Liang
2022-03-19  7:53     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 04/12] RDMA/erdma: Add main include file Cheng Xu
2022-03-18 10:35   ` Wenpeng Liang
2022-03-19  8:11     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 05/12] RDMA/erdma: Add cmdq implementation Cheng Xu
2022-03-18 11:13   ` Wenpeng Liang
2022-03-19  8:38     ` Cheng Xu
2022-03-18 11:16   ` Wenpeng Liang
2022-03-18 18:17     ` Jason Gunthorpe
2022-03-19  1:26       ` Wenpeng Liang
2022-03-18 12:57   ` Wenpeng Liang
2022-03-19  9:18     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 06/12] RDMA/erdma: Add event queue implementation Cheng Xu
2022-03-18 11:43   ` Wenpeng Liang
2022-03-18 18:18     ` Jason Gunthorpe
2022-03-19  9:43       ` Cheng Xu
2022-03-21 22:23         ` Jason Gunthorpe
2022-03-22  3:06           ` Cheng Xu
2022-03-19  8:54     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 07/12] RDMA/erdma: Add verbs header file Cheng Xu
2022-03-18 11:46   ` Wenpeng Liang
2022-03-19  8:55     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 08/12] RDMA/erdma: Add verbs implementation Cheng Xu
2022-03-18 12:24   ` Wenpeng Liang
2022-03-19  9:06     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 09/12] RDMA/erdma: Add connection management (CM) support Cheng Xu
2022-03-18 12:38   ` Wenpeng Liang
2022-03-19  9:10     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 10/12] RDMA/erdma: Add the erdma module Cheng Xu
2022-03-18 12:46   ` Wenpeng Liang
2022-03-19  9:13     ` Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 11/12] RDMA/erdma: Add the ABI definitions Cheng Xu
2022-03-14  6:47 ` [PATCH for-next v4 12/12] RDMA/erdma: Add driver to kernel build environment Cheng Xu
2022-03-17  3:14   ` kernel test robot [this message]

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=202203171051.nVmoVnU3-lkp@intel.com \
    --to=lkp@intel.com \
    --cc=BMT@zurich.ibm.com \
    --cc=KaiShen@linux.alibaba.com \
    --cc=chengyou@linux.alibaba.com \
    --cc=dledford@redhat.com \
    --cc=jgg@ziepe.ca \
    --cc=kbuild-all@lists.01.org \
    --cc=leon@kernel.org \
    --cc=linux-rdma@vger.kernel.org \
    --cc=tonylu@linux.alibaba.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.