All of lore.kernel.org
 help / color / mirror / Atom feed
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

             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.