All of lore.kernel.org
 help / color / mirror / Atom feed
From: kbuild test robot <lkp@intel.com>
To: Manish Rangankar <manish.rangankar@cavium.com>
Cc: kbuild-all@01.org, martin.petersen@oracle.com, lduncan@suse.com,
	cleech@redhat.com, linux-scsi@vger.kernel.org,
	netdev@vger.kernel.org, QLogic-Storage-Upstream@cavium.com,
	Yuval.Mintz@cavium.com
Subject: Re: [PATCH v2 3/6] qedi: Add QLogic FastLinQ offload iSCSI driver framework.
Date: Tue, 8 Nov 2016 18:52:13 +0800	[thread overview]
Message-ID: <201611081805.KzpHTNxg%fengguang.wu@intel.com> (raw)
In-Reply-To: <1478588223-16183-4-git-send-email-manish.rangankar@cavium.com>

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

Hi Manish,

[auto build test ERROR on net-next/master]
[also build test ERROR on v4.9-rc4]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Manish-Rangankar/qed-Add-support-for-hardware-offloaded-iSCSI/20161108-180027
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=ia64 

Note: the linux-review/Manish-Rangankar/qed-Add-support-for-hardware-offloaded-iSCSI/20161108-180027 HEAD dd4d1d0e0785d20cdcfdf9b2c792c564a79b2de2 builds fine.
      It only hurts bisectibility.

All error/warnings (new ones prefixed by >>):

   drivers/scsi/qedi/qedi_main.c: In function 'qedi_iscsi_event_cb':
>> drivers/scsi/qedi/qedi_main.c:87:14: error: dereferencing pointer to incomplete type 'struct qedi_endpoint'
      if (qedi_ep->state == EP_STATE_OFLDCONN_START)
                 ^~
>> drivers/scsi/qedi/qedi_main.c:87:25: error: 'EP_STATE_OFLDCONN_START' undeclared (first use in this function)
      if (qedi_ep->state == EP_STATE_OFLDCONN_START)
                            ^~~~~~~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:87:25: note: each undeclared identifier is reported only once for each function it appears in
>> drivers/scsi/qedi/qedi_main.c:88:21: error: 'EP_STATE_OFLDCONN_COMPL' undeclared (first use in this function)
       qedi_ep->state = EP_STATE_OFLDCONN_COMPL;
                        ^~~~~~~~~~~~~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:93:20: error: 'EP_STATE_DISCONN_COMPL' undeclared (first use in this function)
      qedi_ep->state = EP_STATE_DISCONN_COMPL;
                       ^~~~~~~~~~~~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:97:3: error: implicit declaration of function 'qedi_process_iscsi_error' [-Werror=implicit-function-declaration]
      qedi_process_iscsi_error(qedi_ep, data);
      ^~~~~~~~~~~~~~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:106:3: error: implicit declaration of function 'qedi_process_tcp_error' [-Werror=implicit-function-declaration]
      qedi_process_tcp_error(qedi_ep, data);
      ^~~~~~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_host_alloc':
>> drivers/scsi/qedi/qedi_main.c:414:28: error: 'qedi_host_template' undeclared (first use in this function)
     shost = iscsi_host_alloc(&qedi_host_template,
                               ^~~~~~~~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:433:27: error: 'ISCSI_MAX_SESS_PER_HBA' undeclared (first use in this function)
     qedi->max_active_conns = ISCSI_MAX_SESS_PER_HBA;
                              ^~~~~~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_queue_cqe':
>> drivers/scsi/qedi/qedi_main.c:571:15: error: dereferencing pointer to incomplete type 'struct qedi_conn'
     conn = q_conn->cls_conn->dd_data;
                  ^~
>> drivers/scsi/qedi/qedi_main.c:581:27: error: dereferencing pointer to incomplete type 'struct qedi_cmd'
      INIT_LIST_HEAD(&qedi_cmd->cqe_work.list);
                              ^~
   drivers/scsi/qedi/qedi_main.c: At top level:
   drivers/scsi/qedi/qedi_main.c:1095:15: error: variable 'qedi_ll2_cb_ops' has initializer but incomplete type
    static struct qed_ll2_cb_ops qedi_ll2_cb_ops = {
                  ^~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1096:2: error: unknown field 'rx_cb' specified in initializer
     .rx_cb = qedi_ll2_rx,
     ^
   drivers/scsi/qedi/qedi_main.c:1096:11: error: 'qedi_ll2_rx' undeclared here (not in a function)
     .rx_cb = qedi_ll2_rx,
              ^~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1096:11: warning: excess elements in struct initializer
   drivers/scsi/qedi/qedi_main.c:1096:11: note: (near initialization for 'qedi_ll2_cb_ops')
   drivers/scsi/qedi/qedi_main.c:1097:2: error: unknown field 'tx_cb' specified in initializer
     .tx_cb = NULL,
     ^
   In file included from include/uapi/linux/posix_types.h:4:0,
                    from include/uapi/linux/types.h:13,
                    from include/linux/types.h:5,
                    from include/linux/list.h:4,
                    from include/linux/module.h:9,
                    from drivers/scsi/qedi/qedi_main.c:10:
   include/linux/stddef.h:7:14: warning: excess elements in struct initializer
    #define NULL ((void *)0)
                 ^
   drivers/scsi/qedi/qedi_main.c:1097:11: note: in expansion of macro 'NULL'
     .tx_cb = NULL,
              ^~~~
   include/linux/stddef.h:7:14: note: (near initialization for 'qedi_ll2_cb_ops')
    #define NULL ((void *)0)
                 ^
   drivers/scsi/qedi/qedi_main.c:1097:11: note: in expansion of macro 'NULL'
     .tx_cb = NULL,
              ^~~~
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_percpu_io_thread':
>> drivers/scsi/qedi/qedi_main.c:1117:5: error: implicit declaration of function 'qedi_fp_process_cqes' [-Werror=implicit-function-declaration]
        qedi_fp_process_cqes(work);
        ^~~~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c: In function '__qedi_remove':
   drivers/scsi/qedi/qedi_main.c:1226:16: error: dereferencing pointer to incomplete type 'const struct qed_ll2_ops'
      qedi_ops->ll2->stop(qedi->cdev);
                   ^~
   drivers/scsi/qedi/qedi_main.c:1242:3: error: implicit declaration of function 'qedi_free_uio' [-Werror=implicit-function-declaration]
      qedi_free_uio(qedi->udev);
      ^~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1252:3: error: implicit declaration of function 'qedi_ll2_free_skbs' [-Werror=implicit-function-declaration]
      qedi_ll2_free_skbs(qedi);
      ^~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c: In function '__qedi_probe':
   drivers/scsi/qedi/qedi_main.c:1259:24: error: storage size of 'params' isn't known
     struct qed_ll2_params params;
                           ^~~~~~
>> drivers/scsi/qedi/qedi_main.c:1376:15: error: 'DEF_PATH_MTU' undeclared (first use in this function)
     params.mtu = DEF_PATH_MTU + IPV6_HDR_LEN + TCP_HDR_LEN;
                  ^~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:1376:30: error: 'IPV6_HDR_LEN' undeclared (first use in this function)
     params.mtu = DEF_PATH_MTU + IPV6_HDR_LEN + TCP_HDR_LEN;
                                 ^~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:1376:45: error: 'TCP_HDR_LEN' undeclared (first use in this function)
     params.mtu = DEF_PATH_MTU + IPV6_HDR_LEN + TCP_HDR_LEN;
                                                ^~~~~~~~~~~
   In file included from drivers/scsi/qedi/qedi_main.c:18:0:
   drivers/scsi/qedi/qedi_main.c:1394:39: error: 'qedi_ll2_recv_thread' undeclared (first use in this function)
      qedi->ll2_recv_thread = kthread_run(qedi_ll2_recv_thread,
                                          ^
   include/linux/kthread.h:25:25: note: in definition of macro 'kthread_create'
     kthread_create_on_node(threadfn, data, NUMA_NO_NODE, namefmt, ##arg)
                            ^~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1394:27: note: in expansion of macro 'kthread_run'
      qedi->ll2_recv_thread = kthread_run(qedi_ll2_recv_thread,
                              ^~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:1422:38: error: 'qedi_debugfs_ops' undeclared (first use in this function)
     qedi_dbg_host_init(&qedi->dbg_ctx, &qedi_debugfs_ops,
                                         ^~~~~~~~~~~~~~~~
>> drivers/scsi/qedi/qedi_main.c:1423:8: error: 'qedi_dbg_fops' undeclared (first use in this function)
          &qedi_dbg_fops);
           ^~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1439:8: error: implicit declaration of function 'qedi_alloc_uio_rings' [-Werror=implicit-function-declaration]
      rc = qedi_alloc_uio_rings(qedi);
           ^~~~~~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1446:8: error: implicit declaration of function 'qedi_init_uio' [-Werror=implicit-function-declaration]
      rc = qedi_init_uio(qedi);
           ^~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c:1259:24: warning: unused variable 'params' [-Wunused-variable]
     struct qed_ll2_params params;
                           ^~~~~~
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_init':
>> drivers/scsi/qedi/qedi_main.c:1586:1: warning: label 'exit_qedi_init_1' defined but not used [-Wunused-label]
    exit_qedi_init_1:
    ^~~~~~~~~~~~~~~~
   drivers/scsi/qedi/qedi_main.c: At top level:
   drivers/scsi/qedi/qedi_main.c:1095:30: error: storage size of 'qedi_ll2_cb_ops' isn't known
    static struct qed_ll2_cb_ops qedi_ll2_cb_ops = {
                                 ^~~~~~~~~~~~~~~
   cc1: some warnings being treated as errors

vim +87 drivers/scsi/qedi/qedi_main.c

    81			WARN_ON(1);
    82			return -ENODEV;
    83		}
    84	
    85		switch (fw_event_code) {
    86		case ISCSI_EVENT_TYPE_ASYN_CONNECT_COMPLETE:
  > 87			if (qedi_ep->state == EP_STATE_OFLDCONN_START)
  > 88				qedi_ep->state = EP_STATE_OFLDCONN_COMPL;
    89	
    90			wake_up_interruptible(&qedi_ep->tcp_ofld_wait);
    91			break;
    92		case ISCSI_EVENT_TYPE_ASYN_TERMINATE_DONE:
  > 93			qedi_ep->state = EP_STATE_DISCONN_COMPL;
    94			wake_up_interruptible(&qedi_ep->tcp_ofld_wait);
    95			break;
    96		case ISCSI_EVENT_TYPE_ISCSI_CONN_ERROR:
  > 97			qedi_process_iscsi_error(qedi_ep, data);
    98			break;
    99		case ISCSI_EVENT_TYPE_ASYN_ABORT_RCVD:
   100		case ISCSI_EVENT_TYPE_ASYN_SYN_RCVD:
   101		case ISCSI_EVENT_TYPE_ASYN_MAX_RT_TIME:
   102		case ISCSI_EVENT_TYPE_ASYN_MAX_RT_CNT:
   103		case ISCSI_EVENT_TYPE_ASYN_MAX_KA_PROBES_CNT:
   104		case ISCSI_EVENT_TYPE_ASYN_FIN_WAIT2:
   105		case ISCSI_EVENT_TYPE_TCP_CONN_ERROR:
 > 106			qedi_process_tcp_error(qedi_ep, data);
   107			break;
   108		default:
   109			QEDI_ERR(&qedi->dbg_ctx, "Recv Unknown Event %u\n",

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

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

  reply	other threads:[~2016-11-08 10:55 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-11-08  6:56 [PATCH v2 0/6] Add QLogic FastLinQ iSCSI (qedi) driver Manish Rangankar
2016-11-08  6:56 ` Manish Rangankar
2016-11-08  6:56 ` [PATCH v2 1/6] qed: Add support for hardware offloaded iSCSI Manish Rangankar
2016-11-08  6:56   ` Manish Rangankar
2016-11-11 15:57   ` Hannes Reinecke
2016-11-11 18:12     ` Arun Easi
2016-11-11 18:12       ` Arun Easi
2016-11-08  6:56 ` [PATCH v2 2/6] qed: Add iSCSI out of order packet handling Manish Rangankar
2016-11-08  6:56   ` Manish Rangankar
2016-11-11 15:59   ` Hannes Reinecke
2016-11-08  6:57 ` [PATCH v2 3/6] qedi: Add QLogic FastLinQ offload iSCSI driver framework Manish Rangankar
2016-11-08  6:57   ` Manish Rangankar
2016-11-08 10:52   ` kbuild test robot [this message]
2016-11-08 20:04     ` Arun Easi
2016-11-08 20:04       ` Arun Easi
2016-11-08 23:49       ` Martin K. Petersen
2016-11-08 23:49         ` Martin K. Petersen
2016-11-09  0:00         ` Arun Easi
2016-11-09  0:00           ` Arun Easi
2016-11-09  0:07           ` Martin K. Petersen
2016-11-09  0:07             ` Martin K. Petersen
2016-11-08 11:13   ` kbuild test robot
2016-11-11 16:43   ` Hannes Reinecke
2016-11-15  6:14     ` Rangankar, Manish
2016-11-15  7:21       ` Hannes Reinecke
2017-01-01 12:28   ` Or Gerlitz
2016-11-08  6:57 ` [PATCH v2 4/6] qedi: Add LL2 iSCSI interface for offload iSCSI Manish Rangankar
2016-11-08  6:57   ` Manish Rangankar
2016-11-11 16:44   ` Hannes Reinecke
2016-11-08  6:57 ` [PATCH v2 5/6] qedi: Add support for iSCSI session management Manish Rangankar
2016-11-08  6:57   ` Manish Rangankar
2016-11-11 16:47   ` Hannes Reinecke
2016-11-08  6:57 ` [PATCH v2 6/6] qedi: Add support for data path Manish Rangankar
2016-11-08  6:57   ` Manish Rangankar
2016-11-08 11:09   ` kbuild test robot
2016-11-11 16:49   ` Hannes Reinecke
2016-11-29 17:21 ` [PATCH v2 0/6] Add QLogic FastLinQ iSCSI (qedi) driver Martin K. Petersen
2016-11-29 17:21   ` Martin K. Petersen

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=201611081805.KzpHTNxg%fengguang.wu@intel.com \
    --to=lkp@intel.com \
    --cc=QLogic-Storage-Upstream@cavium.com \
    --cc=Yuval.Mintz@cavium.com \
    --cc=cleech@redhat.com \
    --cc=kbuild-all@01.org \
    --cc=lduncan@suse.com \
    --cc=linux-scsi@vger.kernel.org \
    --cc=manish.rangankar@cavium.com \
    --cc=martin.petersen@oracle.com \
    --cc=netdev@vger.kernel.org \
    /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.