From: kbuild test robot <lkp@intel.com>
To: Michal Kalderon <michal.kalderon@marvell.com>
Cc: kbuild-all@lists.01.org, michal.kalderon@marvell.com,
ariel.elior@marvell.com, davem@davemloft.net,
netdev@vger.kernel.org, linux-rdma@vger.kernel.org,
linux-scsi@vger.kernel.org
Subject: Re: [PATCH net-next 03/14] qed: FW 8.42.2.0 Queue Manager changes
Date: Sat, 25 Jan 2020 03:09:23 +0800 [thread overview]
Message-ID: <202001250351.id5rPYki%lkp@intel.com> (raw)
In-Reply-To: <20200122152627.14903-4-michal.kalderon@marvell.com>
[-- Attachment #1: Type: text/plain, Size: 7674 bytes --]
Hi Michal,
I love your patch! Perhaps something to improve:
[auto build test WARNING on net-next/master]
[also build test WARNING on linus/master v5.5-rc7 next-20200121]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system. BTW, we also suggest to use '--base' option to specify the
base tree in git format-patch, please see https://stackoverflow.com/a/37406982]
url: https://github.com/0day-ci/linux/commits/Michal-Kalderon/qed-Utilize-FW-8-42-2-0/20200124-181924
base: https://git.kernel.org/pub/scm/linux/kernel/git/davem/net-next.git 9bbc8be29d66cc34b650510f2c67b5c55235fe5d
config: i386-randconfig-d002-20200124 (attached as .config)
compiler: gcc-7 (Debian 7.5.0-3) 7.5.0
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
drivers/net/ethernet/qlogic/qed/qed_dev.c: In function 'qed_hw_init_common':
>> drivers/net/ethernet/qlogic/qed/qed_dev.c:2666:1: warning: the frame size of 1064 bytes is larger than 1024 bytes [-Wframe-larger-than=]
}
^
vim +2666 drivers/net/ethernet/qlogic/qed/qed_dev.c
60afed72f51c74 Tomer Tayar 2017-04-06 2596
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2597 static int qed_hw_init_common(struct qed_hwfn *p_hwfn,
1a635e488ecf6f Yuval Mintz 2016-08-15 2598 struct qed_ptt *p_ptt, int hw_mode)
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2599 {
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2600 struct qed_qm_info *qm_info = &p_hwfn->qm_info;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2601 struct qed_qm_common_rt_init_params params;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2602 struct qed_dev *cdev = p_hwfn->cdev;
9c79ddaa0f962d Mintz, Yuval 2017-03-14 2603 u8 vf_id, max_num_vfs;
dbb799c39717e7 Yuval Mintz 2016-06-03 2604 u16 num_pfs, pf_id;
1408cc1fa48c54 Yuval Mintz 2016-05-11 2605 u32 concrete_fid;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2606 int rc = 0;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2607
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2608 qed_init_cau_rt_data(cdev);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2609
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2610 /* Program GTT windows */
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2611 qed_gtt_init(p_hwfn);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2612
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2613 if (p_hwfn->mcp_info) {
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2614 if (p_hwfn->mcp_info->func_info.bandwidth_max)
c7281d591332b9 Gustavo A. R. Silva 2018-03-22 2615 qm_info->pf_rl_en = true;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2616 if (p_hwfn->mcp_info->func_info.bandwidth_min)
c7281d591332b9 Gustavo A. R. Silva 2018-03-22 2617 qm_info->pf_wfq_en = true;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2618 }
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2619
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2620 memset(¶ms, 0, sizeof(params));
78cea9ffaa34d2 Tomer Tayar 2017-05-23 2621 params.max_ports_per_engine = p_hwfn->cdev->num_ports_in_engine;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2622 params.max_phys_tcs_per_port = qm_info->max_phys_tcs_per_port;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2623 params.pf_rl_en = qm_info->pf_rl_en;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2624 params.pf_wfq_en = qm_info->pf_wfq_en;
1a5a5e8c1f7312 Michal Kalderon 2020-01-22 2625 params.global_rl_en = qm_info->vport_rl_en;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2626 params.vport_wfq_en = qm_info->vport_wfq_en;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2627 params.port_params = qm_info->qm_port_params;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2628
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2629 qed_qm_common_rt_init(p_hwfn, ¶ms);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2630
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2631 qed_cxt_hw_init_common(p_hwfn);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2632
60afed72f51c74 Tomer Tayar 2017-04-06 2633 qed_init_cache_line_size(p_hwfn, p_ptt);
60afed72f51c74 Tomer Tayar 2017-04-06 2634
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2635 rc = qed_init_run(p_hwfn, p_ptt, PHASE_ENGINE, ANY_PHASE_ID, hw_mode);
1a635e488ecf6f Yuval Mintz 2016-08-15 2636 if (rc)
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2637 return rc;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2638
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2639 qed_wr(p_hwfn, p_ptt, PSWRQ2_REG_L2P_VALIDATE_VFID, 0);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2640 qed_wr(p_hwfn, p_ptt, PGLUE_B_REG_USE_CLIENTID_IN_TAG, 1);
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2641
dbb799c39717e7 Yuval Mintz 2016-06-03 2642 if (QED_IS_BB(p_hwfn->cdev)) {
dbb799c39717e7 Yuval Mintz 2016-06-03 2643 num_pfs = NUM_OF_ENG_PFS(p_hwfn->cdev);
dbb799c39717e7 Yuval Mintz 2016-06-03 2644 for (pf_id = 0; pf_id < num_pfs; pf_id++) {
dbb799c39717e7 Yuval Mintz 2016-06-03 2645 qed_fid_pretend(p_hwfn, p_ptt, pf_id);
dbb799c39717e7 Yuval Mintz 2016-06-03 2646 qed_wr(p_hwfn, p_ptt, PRS_REG_SEARCH_ROCE, 0x0);
dbb799c39717e7 Yuval Mintz 2016-06-03 2647 qed_wr(p_hwfn, p_ptt, PRS_REG_SEARCH_TCP, 0x0);
dbb799c39717e7 Yuval Mintz 2016-06-03 2648 }
dbb799c39717e7 Yuval Mintz 2016-06-03 2649 /* pretend to original PF */
dbb799c39717e7 Yuval Mintz 2016-06-03 2650 qed_fid_pretend(p_hwfn, p_ptt, p_hwfn->rel_pf_id);
dbb799c39717e7 Yuval Mintz 2016-06-03 2651 }
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2652
9c79ddaa0f962d Mintz, Yuval 2017-03-14 2653 max_num_vfs = QED_IS_AH(cdev) ? MAX_NUM_VFS_K2 : MAX_NUM_VFS_BB;
9c79ddaa0f962d Mintz, Yuval 2017-03-14 2654 for (vf_id = 0; vf_id < max_num_vfs; vf_id++) {
1408cc1fa48c54 Yuval Mintz 2016-05-11 2655 concrete_fid = qed_vfid_to_concrete(p_hwfn, vf_id);
1408cc1fa48c54 Yuval Mintz 2016-05-11 2656 qed_fid_pretend(p_hwfn, p_ptt, (u16) concrete_fid);
1408cc1fa48c54 Yuval Mintz 2016-05-11 2657 qed_wr(p_hwfn, p_ptt, CCFC_REG_STRONG_ENABLE_VF, 0x1);
05fafbfb3d77f4 Yuval Mintz 2016-08-19 2658 qed_wr(p_hwfn, p_ptt, CCFC_REG_WEAK_ENABLE_VF, 0x0);
05fafbfb3d77f4 Yuval Mintz 2016-08-19 2659 qed_wr(p_hwfn, p_ptt, TCFC_REG_STRONG_ENABLE_VF, 0x1);
05fafbfb3d77f4 Yuval Mintz 2016-08-19 2660 qed_wr(p_hwfn, p_ptt, TCFC_REG_WEAK_ENABLE_VF, 0x0);
1408cc1fa48c54 Yuval Mintz 2016-05-11 2661 }
1408cc1fa48c54 Yuval Mintz 2016-05-11 2662 /* pretend to original PF */
1408cc1fa48c54 Yuval Mintz 2016-05-11 2663 qed_fid_pretend(p_hwfn, p_ptt, p_hwfn->rel_pf_id);
1408cc1fa48c54 Yuval Mintz 2016-05-11 2664
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2665 return rc;
fe56b9e6a8d957 Yuval Mintz 2015-10-26 @2666 }
fe56b9e6a8d957 Yuval Mintz 2015-10-26 2667
:::::: The code at line 2666 was first introduced by commit
:::::: fe56b9e6a8d957d6a20729d626027f800c17a2da qed: Add module with basic common support
:::::: TO: Yuval Mintz <Yuval.Mintz@qlogic.com>
:::::: CC: David S. Miller <davem@davemloft.net>
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org Intel Corporation
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 35258 bytes --]
next prev parent reply other threads:[~2020-01-24 19:10 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-22 15:26 [PATCH net-next 00/14] qed*: Utilize FW 8.42.2.0 Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 01/14] qed: FW 8.42.2.0 Internal ram offsets modifications Michal Kalderon
2020-01-22 15:45 ` Jakub Kicinski
2020-01-22 16:02 ` Michal Kalderon
2020-01-23 14:26 ` Jakub Kicinski
2020-01-22 15:26 ` [PATCH net-next 02/14] qed: FW 8.42.2.0 Expose new registers and change windows Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 03/14] qed: FW 8.42.2.0 Queue Manager changes Michal Kalderon
2020-01-24 19:09 ` kbuild test robot [this message]
2020-01-22 15:26 ` [PATCH net-next 04/14] qed: FW 8.42.2.0 Parser offsets modified Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 05/14] qed: Use dmae to write to widebus registers in fw_funcs Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 06/14] qed: FW 8.42.2.0 Additional ll2 type Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 07/14] qed: Add abstraction for different hsi values per chip Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 08/14] qed: FW 8.42.2.0 iscsi/fcoe changes Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 09/14] qed: FW 8.42.2.0 HSI changes Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 10/14] qed: FW 8.42.2.0 Add fw overlay feature Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 11/14] qed: Debug feature: ilt and mdump Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 12/14] qed: rt init valid initialization changed Michal Kalderon
2020-01-22 15:26 ` [PATCH net-next 13/14] qed: FW 8.42.2.0 debug features Michal Kalderon
2020-01-22 15:54 ` Jakub Kicinski
2020-01-22 16:03 ` Michal Kalderon
2020-01-22 16:14 ` Leon Romanovsky
2020-01-22 16:41 ` [EXT] " Michal Kalderon
2020-01-22 18:22 ` Leon Romanovsky
2020-01-22 19:12 ` David Miller
2020-01-22 15:26 ` [PATCH net-next 14/14] qed: bump driver version Michal Kalderon
2020-01-22 16:13 ` Leon Romanovsky
2020-01-22 16:39 ` [EXT] " Michal Kalderon
2020-01-22 18:21 ` Leon Romanovsky
2020-01-23 8:18 ` Michal Kalderon
2020-01-23 12:12 ` Leon Romanovsky
2020-01-23 13:10 ` Leon Romanovsky
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=202001250351.id5rPYki%lkp@intel.com \
--to=lkp@intel.com \
--cc=ariel.elior@marvell.com \
--cc=davem@davemloft.net \
--cc=kbuild-all@lists.01.org \
--cc=linux-rdma@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=michal.kalderon@marvell.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).