From: kernel test robot <lkp@intel.com>
To: James Smart <jsmart2021@gmail.com>
Cc: oe-kbuild-all@lists.linux.dev, linux-kernel@vger.kernel.org,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Hannes Reinecke <hare@suse.de>, Daniel Wagner <dwagner@suse.de>,
Ram Vegesna <ram.vegesna@broadcom.com>
Subject: drivers/scsi/elx/efct/efct_hw.c:227:1: warning: the frame size of 1136 bytes is larger than 1024 bytes
Date: Tue, 6 Aug 2024 17:42:13 +0800 [thread overview]
Message-ID: <202408061721.gfnwpv1W-lkp@intel.com> (raw)
Hi James,
First bad commit (maybe != root cause):
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: b446a2dae984fa5bd56dd7c3a02a426f87e05813
commit: ebc076b3eddc807729bd81f7bc48e798a3ddc477 scsi: elx: efct: Tie into kernel Kconfig and build process
date: 3 years, 2 months ago
config: arm-allmodconfig (https://download.01.org/0day-ci/archive/20240806/202408061721.gfnwpv1W-lkp@intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240806/202408061721.gfnwpv1W-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408061721.gfnwpv1W-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/scsi/elx/efct/efct_hw.c: In function 'efct_hw_cb_link':
>> drivers/scsi/elx/efct/efct_hw.c:227:1: warning: the frame size of 1136 bytes is larger than 1024 bytes [-Wframe-larger-than=]
227 | }
| ^
vim +227 drivers/scsi/elx/efct/efct_hw.c
4df84e8466242d James Smart 2021-06-01 162
4df84e8466242d James Smart 2021-06-01 163 static int
4df84e8466242d James Smart 2021-06-01 164 efct_hw_cb_link(void *ctx, void *e)
4df84e8466242d James Smart 2021-06-01 165 {
4df84e8466242d James Smart 2021-06-01 166 struct efct_hw *hw = ctx;
4df84e8466242d James Smart 2021-06-01 167 struct sli4_link_event *event = e;
4df84e8466242d James Smart 2021-06-01 168 struct efc_domain *d = NULL;
4df84e8466242d James Smart 2021-06-01 169 int rc = 0;
4df84e8466242d James Smart 2021-06-01 170 struct efct *efct = hw->os;
4df84e8466242d James Smart 2021-06-01 171
4df84e8466242d James Smart 2021-06-01 172 efct_hw_link_event_init(hw);
4df84e8466242d James Smart 2021-06-01 173
4df84e8466242d James Smart 2021-06-01 174 switch (event->status) {
4df84e8466242d James Smart 2021-06-01 175 case SLI4_LINK_STATUS_UP:
4df84e8466242d James Smart 2021-06-01 176
4df84e8466242d James Smart 2021-06-01 177 hw->link = *event;
4df84e8466242d James Smart 2021-06-01 178 efct->efcport->link_status = EFC_LINK_STATUS_UP;
4df84e8466242d James Smart 2021-06-01 179
4df84e8466242d James Smart 2021-06-01 180 if (event->topology == SLI4_LINK_TOPO_NON_FC_AL) {
4df84e8466242d James Smart 2021-06-01 181 struct efc_domain_record drec = {0};
4df84e8466242d James Smart 2021-06-01 182
4df84e8466242d James Smart 2021-06-01 183 efc_log_info(hw->os, "Link Up, NPORT, speed is %d\n",
4df84e8466242d James Smart 2021-06-01 184 event->speed);
4df84e8466242d James Smart 2021-06-01 185 drec.speed = event->speed;
4df84e8466242d James Smart 2021-06-01 186 drec.fc_id = event->fc_id;
4df84e8466242d James Smart 2021-06-01 187 drec.is_nport = true;
4df84e8466242d James Smart 2021-06-01 188 efc_domain_cb(efct->efcport, EFC_HW_DOMAIN_FOUND,
4df84e8466242d James Smart 2021-06-01 189 &drec);
4df84e8466242d James Smart 2021-06-01 190 } else if (event->topology == SLI4_LINK_TOPO_FC_AL) {
4df84e8466242d James Smart 2021-06-01 191 u8 buf[SLI4_BMBX_SIZE];
4df84e8466242d James Smart 2021-06-01 192
4df84e8466242d James Smart 2021-06-01 193 efc_log_info(hw->os, "Link Up, LOOP, speed is %d\n",
4df84e8466242d James Smart 2021-06-01 194 event->speed);
4df84e8466242d James Smart 2021-06-01 195
4df84e8466242d James Smart 2021-06-01 196 if (!sli_cmd_read_topology(&hw->sli, buf,
4df84e8466242d James Smart 2021-06-01 197 &hw->loop_map)) {
4df84e8466242d James Smart 2021-06-01 198 rc = efct_hw_command(hw, buf, EFCT_CMD_NOWAIT,
4df84e8466242d James Smart 2021-06-01 199 __efct_read_topology_cb, NULL);
4df84e8466242d James Smart 2021-06-01 200 }
4df84e8466242d James Smart 2021-06-01 201
4df84e8466242d James Smart 2021-06-01 202 if (rc)
4df84e8466242d James Smart 2021-06-01 203 efc_log_debug(hw->os, "READ_TOPOLOGY failed\n");
4df84e8466242d James Smart 2021-06-01 204 } else {
4df84e8466242d James Smart 2021-06-01 205 efc_log_info(hw->os, "%s(%#x), speed is %d\n",
4df84e8466242d James Smart 2021-06-01 206 "Link Up, unsupported topology ",
4df84e8466242d James Smart 2021-06-01 207 event->topology, event->speed);
4df84e8466242d James Smart 2021-06-01 208 }
4df84e8466242d James Smart 2021-06-01 209 break;
4df84e8466242d James Smart 2021-06-01 210 case SLI4_LINK_STATUS_DOWN:
4df84e8466242d James Smart 2021-06-01 211 efc_log_info(hw->os, "Link down\n");
4df84e8466242d James Smart 2021-06-01 212
4df84e8466242d James Smart 2021-06-01 213 hw->link.status = event->status;
4df84e8466242d James Smart 2021-06-01 214 efct->efcport->link_status = EFC_LINK_STATUS_DOWN;
4df84e8466242d James Smart 2021-06-01 215
4df84e8466242d James Smart 2021-06-01 216 d = efct->efcport->domain;
4df84e8466242d James Smart 2021-06-01 217 if (d)
4df84e8466242d James Smart 2021-06-01 218 efc_domain_cb(efct->efcport, EFC_HW_DOMAIN_LOST, d);
4df84e8466242d James Smart 2021-06-01 219 break;
4df84e8466242d James Smart 2021-06-01 220 default:
4df84e8466242d James Smart 2021-06-01 221 efc_log_debug(hw->os, "unhandled link status %#x\n",
4df84e8466242d James Smart 2021-06-01 222 event->status);
4df84e8466242d James Smart 2021-06-01 223 break;
4df84e8466242d James Smart 2021-06-01 224 }
4df84e8466242d James Smart 2021-06-01 225
4df84e8466242d James Smart 2021-06-01 226 return 0;
4df84e8466242d James Smart 2021-06-01 @227 }
4df84e8466242d James Smart 2021-06-01 228
:::::: The code at line 227 was first introduced by commit
:::::: 4df84e8466242de835416a4ec0c856c0e2ed26eb scsi: elx: efct: Driver initialization routines
:::::: TO: James Smart <jsmart2021@gmail.com>
:::::: CC: Martin K. Petersen <martin.petersen@oracle.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2024-08-06 9:42 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-06 9:42 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2026-03-15 13:44 drivers/scsi/elx/efct/efct_hw.c:227:1: warning: the frame size of 1136 bytes is larger than 1024 bytes kernel test robot
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=202408061721.gfnwpv1W-lkp@intel.com \
--to=lkp@intel.com \
--cc=dwagner@suse.de \
--cc=hare@suse.de \
--cc=jsmart2021@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=oe-kbuild-all@lists.linux.dev \
--cc=ram.vegesna@broadcom.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.