From: Dan Carpenter <dan.carpenter@oracle.com>
To: jedrzej.jagielski@intel.com
Cc: kernel-janitors@vger.kernel.org
Subject: [bug report] i40e: Fix race condition while adding/deleting MAC/VLAN filters
Date: Tue, 8 Feb 2022 10:20:04 +0300 [thread overview]
Message-ID: <20220208072004.GA19479@kili> (raw)
Hello Jedrzej Jagielski,
The patch 53a9e346e159: "i40e: Fix race condition while
adding/deleting MAC/VLAN filters" from Jan 14, 2022, leads to the
following Smatch static checker warning:
drivers/net/ethernet/intel/i40e/i40e_main.c:2153 i40e_aqc_del_filters()
error: uninitialized symbol 'aq_status'.
drivers/net/ethernet/intel/i40e/i40e_main.c
2140 static
2141 void i40e_aqc_del_filters(struct i40e_vsi *vsi, const char *vsi_name,
2142 struct i40e_aqc_remove_macvlan_element_data *list,
2143 int num_del, int *retval)
2144 {
2145 struct i40e_hw *hw = &vsi->back->hw;
2146 enum i40e_admin_queue_err aq_status;
2147 i40e_status aq_ret;
2148
2149 aq_ret = i40e_aq_remove_macvlan_v2(hw, vsi->seid, list, num_del, NULL,
2150 &aq_status);
2151
2152 /* Explicitly ignore and do not report when firmware returns ENOENT */
--> 2153 if (aq_ret && !(aq_status == I40E_AQ_RC_ENOENT)) {
The error pathes doesn't set aq_status.
2154 *retval = -EIO;
2155 dev_info(&vsi->back->pdev->dev,
2156 "ignoring delete macvlan error on %s, err %s, aq_err %s\n",
2157 vsi_name, i40e_stat_str(hw, aq_ret),
2158 i40e_aq_str(hw, aq_status));
2159 }
2160 }
regards,
dan carpenter
next reply other threads:[~2022-02-08 7:20 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-02-08 7:20 Dan Carpenter [this message]
2022-02-08 10:57 ` [bug report] i40e: Fix race condition while adding/deleting MAC/VLAN filters Dan Carpenter
2022-02-10 9:29 ` Jagielski, Jedrzej
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=20220208072004.GA19479@kili \
--to=dan.carpenter@oracle.com \
--cc=jedrzej.jagielski@intel.com \
--cc=kernel-janitors@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.