[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f90bc4fc-d9e3-468c-8b94-73bea4b2d764@intel.com>
Date: Tue, 21 Nov 2023 15:05:11 -0800
From: Tony Nguyen <anthony.l.nguyen@...el.com>
To: Ivan Vecera <ivecera@...hat.com>,
<intel-wired-lan@...ts.osuosl.org>
CC: Jesse Brandeburg <jesse.brandeburg@...el.com>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Jakub Kicinski <kuba@...nel.org>,
Paolo Abeni <pabeni@...hat.com>,
<linux-kernel@...r.kernel.org>,
Jacob Keller <jacob.e.keller@...el.com>,
Wojciech Drewek <wojciech.drewek@...el.com>,
Simon Horman <horms@...nel.org>, <mschmidt@...hat.com>,
<netdev@...r.kernel.org>
Subject: Re: [PATCH iwl-next v3 3/5] i40e: Add helpers to find VSI and VEB by
SEID and use them
On 11/16/2023 7:21 AM, Ivan Vecera wrote:
> @@ -13197,17 +13193,14 @@ static int i40e_ndo_bridge_getlink(struct sk_buff *skb, u32 pid, u32 seq,
> struct i40e_vsi *vsi = np->vsi;
> struct i40e_pf *pf = vsi->back;
> struct i40e_veb *veb = NULL;
> - int i;
>
> /* Only for PF VSI for now */
> if (vsi->seid != pf->vsi[pf->lan_vsi]->seid)
> return -EOPNOTSUPP;
>
> /* Find the HW bridge for the PF VSI */
> - i40e_pf_for_each_veb(pf, i, veb)
> - if (veb->seid == vsi->uplink_seid)
> - break;
> - if (i == I40E_MAX_VEB)
> + veb = i40e_pf_get_veb_by_seid(pf, vsi->uplink_seid);
> + if (!vsi)
> return 0;
In addition to the kdoc stuff that Simon pointed out. Should this null
check be against veb?
> return ndo_dflt_bridge_getlink(skb, pid, seq, dev, veb->bridge_mode,
Powered by blists - more mailing lists