1109aba47SJacob Keller // SPDX-License-Identifier: GPL-2.0
2109aba47SJacob Keller /* Copyright (C) 2022, Intel Corporation. */
3109aba47SJacob Keller 
4109aba47SJacob Keller #include "ice_vf_lib_private.h"
5109aba47SJacob Keller #include "ice.h"
6109aba47SJacob Keller #include "ice_lib.h"
7109aba47SJacob Keller #include "ice_fltr.h"
8109aba47SJacob Keller #include "ice_virtchnl_allowlist.h"
9109aba47SJacob Keller 
10109aba47SJacob Keller /* Public functions which may be accessed by all driver files */
11109aba47SJacob Keller 
12109aba47SJacob Keller /**
13109aba47SJacob Keller  * ice_get_vf_by_id - Get pointer to VF by ID
14109aba47SJacob Keller  * @pf: the PF private structure
15109aba47SJacob Keller  * @vf_id: the VF ID to locate
16109aba47SJacob Keller  *
17109aba47SJacob Keller  * Locate and return a pointer to the VF structure associated with a given ID.
18109aba47SJacob Keller  * Returns NULL if the ID does not have a valid VF structure associated with
19109aba47SJacob Keller  * it.
20109aba47SJacob Keller  *
21109aba47SJacob Keller  * This function takes a reference to the VF, which must be released by
22109aba47SJacob Keller  * calling ice_put_vf() once the caller is finished accessing the VF structure
23109aba47SJacob Keller  * returned.
24109aba47SJacob Keller  */
ice_get_vf_by_id(struct ice_pf * pf,u16 vf_id)25109aba47SJacob Keller struct ice_vf *ice_get_vf_by_id(struct ice_pf *pf, u16 vf_id)
26109aba47SJacob Keller {
27109aba47SJacob Keller 	struct ice_vf *vf;
28109aba47SJacob Keller 
29109aba47SJacob Keller 	rcu_read_lock();
30109aba47SJacob Keller 	hash_for_each_possible_rcu(pf->vfs.table, vf, entry, vf_id) {
31109aba47SJacob Keller 		if (vf->vf_id == vf_id) {
32109aba47SJacob Keller 			struct ice_vf *found;
33109aba47SJacob Keller 
34109aba47SJacob Keller 			if (kref_get_unless_zero(&vf->refcnt))
35109aba47SJacob Keller 				found = vf;
36109aba47SJacob Keller 			else
37109aba47SJacob Keller 				found = NULL;
38109aba47SJacob Keller 
39109aba47SJacob Keller 			rcu_read_unlock();
40109aba47SJacob Keller 			return found;
41109aba47SJacob Keller 		}
42109aba47SJacob Keller 	}
43109aba47SJacob Keller 	rcu_read_unlock();
44109aba47SJacob Keller 
45109aba47SJacob Keller 	return NULL;
46109aba47SJacob Keller }
47109aba47SJacob Keller 
48109aba47SJacob Keller /**
49109aba47SJacob Keller  * ice_release_vf - Release VF associated with a refcount
50109aba47SJacob Keller  * @ref: the kref decremented to zero
51109aba47SJacob Keller  *
52109aba47SJacob Keller  * Callback function for kref_put to release a VF once its reference count has
53109aba47SJacob Keller  * hit zero.
54109aba47SJacob Keller  */
ice_release_vf(struct kref * ref)55109aba47SJacob Keller static void ice_release_vf(struct kref *ref)
56109aba47SJacob Keller {
57109aba47SJacob Keller 	struct ice_vf *vf = container_of(ref, struct ice_vf, refcnt);
58109aba47SJacob Keller 
599c6f7878SJacob Keller 	vf->vf_ops->free(vf);
60109aba47SJacob Keller }
61109aba47SJacob Keller 
62109aba47SJacob Keller /**
63109aba47SJacob Keller  * ice_put_vf - Release a reference to a VF
64109aba47SJacob Keller  * @vf: the VF structure to decrease reference count on
65109aba47SJacob Keller  *
66109aba47SJacob Keller  * Decrease the reference count for a VF, and free the entry if it is no
67109aba47SJacob Keller  * longer in use.
68109aba47SJacob Keller  *
69109aba47SJacob Keller  * This must be called after ice_get_vf_by_id() once the reference to the VF
70109aba47SJacob Keller  * structure is no longer used. Otherwise, the VF structure will never be
71109aba47SJacob Keller  * freed.
72109aba47SJacob Keller  */
ice_put_vf(struct ice_vf * vf)73109aba47SJacob Keller void ice_put_vf(struct ice_vf *vf)
74109aba47SJacob Keller {
75109aba47SJacob Keller 	kref_put(&vf->refcnt, ice_release_vf);
76109aba47SJacob Keller }
77109aba47SJacob Keller 
78109aba47SJacob Keller /**
79109aba47SJacob Keller  * ice_has_vfs - Return true if the PF has any associated VFs
80109aba47SJacob Keller  * @pf: the PF private structure
81109aba47SJacob Keller  *
82109aba47SJacob Keller  * Return whether or not the PF has any allocated VFs.
83109aba47SJacob Keller  *
84109aba47SJacob Keller  * Note that this function only guarantees that there are no VFs at the point
85109aba47SJacob Keller  * of calling it. It does not guarantee that no more VFs will be added.
86109aba47SJacob Keller  */
ice_has_vfs(struct ice_pf * pf)87109aba47SJacob Keller bool ice_has_vfs(struct ice_pf *pf)
88109aba47SJacob Keller {
89109aba47SJacob Keller 	/* A simple check that the hash table is not empty does not require
90109aba47SJacob Keller 	 * the mutex or rcu_read_lock.
91109aba47SJacob Keller 	 */
92109aba47SJacob Keller 	return !hash_empty(pf->vfs.table);
93109aba47SJacob Keller }
94109aba47SJacob Keller 
95109aba47SJacob Keller /**
96109aba47SJacob Keller  * ice_get_num_vfs - Get number of allocated VFs
97109aba47SJacob Keller  * @pf: the PF private structure
98109aba47SJacob Keller  *
99109aba47SJacob Keller  * Return the total number of allocated VFs. NOTE: VF IDs are not guaranteed
100109aba47SJacob Keller  * to be contiguous. Do not assume that a VF ID is guaranteed to be less than
101109aba47SJacob Keller  * the output of this function.
102109aba47SJacob Keller  */
ice_get_num_vfs(struct ice_pf * pf)103109aba47SJacob Keller u16 ice_get_num_vfs(struct ice_pf *pf)
104109aba47SJacob Keller {
105109aba47SJacob Keller 	struct ice_vf *vf;
106109aba47SJacob Keller 	unsigned int bkt;
107109aba47SJacob Keller 	u16 num_vfs = 0;
108109aba47SJacob Keller 
109109aba47SJacob Keller 	rcu_read_lock();
110109aba47SJacob Keller 	ice_for_each_vf_rcu(pf, bkt, vf)
111109aba47SJacob Keller 		num_vfs++;
112109aba47SJacob Keller 	rcu_read_unlock();
113109aba47SJacob Keller 
114109aba47SJacob Keller 	return num_vfs;
115109aba47SJacob Keller }
116109aba47SJacob Keller 
117109aba47SJacob Keller /**
118109aba47SJacob Keller  * ice_get_vf_vsi - get VF's VSI based on the stored index
119109aba47SJacob Keller  * @vf: VF used to get VSI
120109aba47SJacob Keller  */
ice_get_vf_vsi(struct ice_vf * vf)121109aba47SJacob Keller struct ice_vsi *ice_get_vf_vsi(struct ice_vf *vf)
122109aba47SJacob Keller {
123109aba47SJacob Keller 	if (vf->lan_vsi_idx == ICE_NO_VSI)
124109aba47SJacob Keller 		return NULL;
125109aba47SJacob Keller 
126109aba47SJacob Keller 	return vf->pf->vsi[vf->lan_vsi_idx];
127109aba47SJacob Keller }
128109aba47SJacob Keller 
129109aba47SJacob Keller /**
130109aba47SJacob Keller  * ice_is_vf_disabled
131109aba47SJacob Keller  * @vf: pointer to the VF info
132109aba47SJacob Keller  *
133109aba47SJacob Keller  * If the PF has been disabled, there is no need resetting VF until PF is
134109aba47SJacob Keller  * active again. Similarly, if the VF has been disabled, this means something
135109aba47SJacob Keller  * else is resetting the VF, so we shouldn't continue.
136109aba47SJacob Keller  *
137109aba47SJacob Keller  * Returns true if the caller should consider the VF as disabled whether
138109aba47SJacob Keller  * because that single VF is explicitly disabled or because the PF is
139109aba47SJacob Keller  * currently disabled.
140109aba47SJacob Keller  */
ice_is_vf_disabled(struct ice_vf * vf)141109aba47SJacob Keller bool ice_is_vf_disabled(struct ice_vf *vf)
142109aba47SJacob Keller {
143109aba47SJacob Keller 	struct ice_pf *pf = vf->pf;
144109aba47SJacob Keller 
145109aba47SJacob Keller 	return (test_bit(ICE_VF_DIS, pf->state) ||
146109aba47SJacob Keller 		test_bit(ICE_VF_STATE_DIS, vf->vf_states));
147109aba47SJacob Keller }
148109aba47SJacob Keller 
149109aba47SJacob Keller /**
150109aba47SJacob Keller  * ice_wait_on_vf_reset - poll to make sure a given VF is ready after reset
151109aba47SJacob Keller  * @vf: The VF being resseting
152109aba47SJacob Keller  *
153109aba47SJacob Keller  * The max poll time is about ~800ms, which is about the maximum time it takes
154109aba47SJacob Keller  * for a VF to be reset and/or a VF driver to be removed.
155109aba47SJacob Keller  */
ice_wait_on_vf_reset(struct ice_vf * vf)156109aba47SJacob Keller static void ice_wait_on_vf_reset(struct ice_vf *vf)
157109aba47SJacob Keller {
158109aba47SJacob Keller 	int i;
159109aba47SJacob Keller 
160109aba47SJacob Keller 	for (i = 0; i < ICE_MAX_VF_RESET_TRIES; i++) {
161109aba47SJacob Keller 		if (test_bit(ICE_VF_STATE_INIT, vf->vf_states))
162109aba47SJacob Keller 			break;
163109aba47SJacob Keller 		msleep(ICE_MAX_VF_RESET_SLEEP_MS);
164109aba47SJacob Keller 	}
165109aba47SJacob Keller }
166109aba47SJacob Keller 
167109aba47SJacob Keller /**
168109aba47SJacob Keller  * ice_check_vf_ready_for_cfg - check if VF is ready to be configured/queried
169109aba47SJacob Keller  * @vf: VF to check if it's ready to be configured/queried
170109aba47SJacob Keller  *
171109aba47SJacob Keller  * The purpose of this function is to make sure the VF is not in reset, not
172109aba47SJacob Keller  * disabled, and initialized so it can be configured and/or queried by a host
173109aba47SJacob Keller  * administrator.
174109aba47SJacob Keller  */
ice_check_vf_ready_for_cfg(struct ice_vf * vf)175109aba47SJacob Keller int ice_check_vf_ready_for_cfg(struct ice_vf *vf)
176109aba47SJacob Keller {
177109aba47SJacob Keller 	ice_wait_on_vf_reset(vf);
178109aba47SJacob Keller 
179109aba47SJacob Keller 	if (ice_is_vf_disabled(vf))
180109aba47SJacob Keller 		return -EINVAL;
181109aba47SJacob Keller 
1825a57ee83SJacob Keller 	if (ice_check_vf_init(vf))
183109aba47SJacob Keller 		return -EBUSY;
184109aba47SJacob Keller 
185109aba47SJacob Keller 	return 0;
186109aba47SJacob Keller }
187109aba47SJacob Keller 
188109aba47SJacob Keller /**
18916686d7fSJacob Keller  * ice_trigger_vf_reset - Reset a VF on HW
19016686d7fSJacob Keller  * @vf: pointer to the VF structure
19116686d7fSJacob Keller  * @is_vflr: true if VFLR was issued, false if not
19216686d7fSJacob Keller  * @is_pfr: true if the reset was triggered due to a previous PFR
19316686d7fSJacob Keller  *
19416686d7fSJacob Keller  * Trigger hardware to start a reset for a particular VF. Expects the caller
19516686d7fSJacob Keller  * to wait the proper amount of time to allow hardware to reset the VF before
19616686d7fSJacob Keller  * it cleans up and restores VF functionality.
19716686d7fSJacob Keller  */
ice_trigger_vf_reset(struct ice_vf * vf,bool is_vflr,bool is_pfr)19816686d7fSJacob Keller static void ice_trigger_vf_reset(struct ice_vf *vf, bool is_vflr, bool is_pfr)
19916686d7fSJacob Keller {
20016686d7fSJacob Keller 	/* Inform VF that it is no longer active, as a warning */
20116686d7fSJacob Keller 	clear_bit(ICE_VF_STATE_ACTIVE, vf->vf_states);
20216686d7fSJacob Keller 
20316686d7fSJacob Keller 	/* Disable VF's configuration API during reset. The flag is re-enabled
20416686d7fSJacob Keller 	 * when it's safe again to access VF's VSI.
20516686d7fSJacob Keller 	 */
20616686d7fSJacob Keller 	clear_bit(ICE_VF_STATE_INIT, vf->vf_states);
20716686d7fSJacob Keller 
20816686d7fSJacob Keller 	/* VF_MBX_ARQLEN and VF_MBX_ATQLEN are cleared by PFR, so the driver
20916686d7fSJacob Keller 	 * needs to clear them in the case of VFR/VFLR. If this is done for
21016686d7fSJacob Keller 	 * PFR, it can mess up VF resets because the VF driver may already
21116686d7fSJacob Keller 	 * have started cleanup by the time we get here.
21216686d7fSJacob Keller 	 */
21316686d7fSJacob Keller 	if (!is_pfr)
21416686d7fSJacob Keller 		vf->vf_ops->clear_mbx_register(vf);
21516686d7fSJacob Keller 
21616686d7fSJacob Keller 	vf->vf_ops->trigger_reset_register(vf, is_vflr);
21716686d7fSJacob Keller }
21816686d7fSJacob Keller 
ice_vf_clear_counters(struct ice_vf * vf)21916686d7fSJacob Keller static void ice_vf_clear_counters(struct ice_vf *vf)
22016686d7fSJacob Keller {
22116686d7fSJacob Keller 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
22216686d7fSJacob Keller 
223baeb705fSJacob Keller 	if (vsi)
22416686d7fSJacob Keller 		vsi->num_vlan = 0;
225baeb705fSJacob Keller 
226baeb705fSJacob Keller 	vf->num_mac = 0;
22716686d7fSJacob Keller 	memset(&vf->mdd_tx_events, 0, sizeof(vf->mdd_tx_events));
22816686d7fSJacob Keller 	memset(&vf->mdd_rx_events, 0, sizeof(vf->mdd_rx_events));
22916686d7fSJacob Keller }
23016686d7fSJacob Keller 
23116686d7fSJacob Keller /**
23216686d7fSJacob Keller  * ice_vf_pre_vsi_rebuild - tasks to be done prior to VSI rebuild
23316686d7fSJacob Keller  * @vf: VF to perform pre VSI rebuild tasks
23416686d7fSJacob Keller  *
23516686d7fSJacob Keller  * These tasks are items that don't need to be amortized since they are most
23616686d7fSJacob Keller  * likely called in a for loop with all VF(s) in the reset_all_vfs() case.
23716686d7fSJacob Keller  */
ice_vf_pre_vsi_rebuild(struct ice_vf * vf)23816686d7fSJacob Keller static void ice_vf_pre_vsi_rebuild(struct ice_vf *vf)
23916686d7fSJacob Keller {
240537dfe06SJacob Keller 	/* Close any IRQ mapping now */
241537dfe06SJacob Keller 	if (vf->vf_ops->irq_close)
242537dfe06SJacob Keller 		vf->vf_ops->irq_close(vf);
243537dfe06SJacob Keller 
24416686d7fSJacob Keller 	ice_vf_clear_counters(vf);
24516686d7fSJacob Keller 	vf->vf_ops->clear_reset_trigger(vf);
24616686d7fSJacob Keller }
24716686d7fSJacob Keller 
24816686d7fSJacob Keller /**
2495531bb85SJacob Keller  * ice_vf_recreate_vsi - Release and re-create the VF's VSI
2505531bb85SJacob Keller  * @vf: VF to recreate the VSI for
2515531bb85SJacob Keller  *
2525531bb85SJacob Keller  * This is only called when a single VF is being reset (i.e. VVF, VFLR, host
2535531bb85SJacob Keller  * VF configuration change, etc)
2545531bb85SJacob Keller  *
2555531bb85SJacob Keller  * It releases and then re-creates a new VSI.
2565531bb85SJacob Keller  */
ice_vf_recreate_vsi(struct ice_vf * vf)2575531bb85SJacob Keller static int ice_vf_recreate_vsi(struct ice_vf *vf)
2585531bb85SJacob Keller {
2595531bb85SJacob Keller 	struct ice_pf *pf = vf->pf;
2605531bb85SJacob Keller 	int err;
2615531bb85SJacob Keller 
2625531bb85SJacob Keller 	ice_vf_vsi_release(vf);
2635531bb85SJacob Keller 
2645531bb85SJacob Keller 	err = vf->vf_ops->create_vsi(vf);
2655531bb85SJacob Keller 	if (err) {
2665531bb85SJacob Keller 		dev_err(ice_pf_to_dev(pf),
2675531bb85SJacob Keller 			"Failed to recreate the VF%u's VSI, error %d\n",
2685531bb85SJacob Keller 			vf->vf_id, err);
2695531bb85SJacob Keller 		return err;
2705531bb85SJacob Keller 	}
2715531bb85SJacob Keller 
2725531bb85SJacob Keller 	return 0;
2735531bb85SJacob Keller }
2745531bb85SJacob Keller 
2755531bb85SJacob Keller /**
27616686d7fSJacob Keller  * ice_vf_rebuild_vsi - rebuild the VF's VSI
27716686d7fSJacob Keller  * @vf: VF to rebuild the VSI for
27816686d7fSJacob Keller  *
27916686d7fSJacob Keller  * This is only called when all VF(s) are being reset (i.e. PCIe Reset on the
28016686d7fSJacob Keller  * host, PFR, CORER, etc.).
2815531bb85SJacob Keller  *
2825531bb85SJacob Keller  * It reprograms the VSI configuration back into hardware.
28316686d7fSJacob Keller  */
ice_vf_rebuild_vsi(struct ice_vf * vf)28416686d7fSJacob Keller static int ice_vf_rebuild_vsi(struct ice_vf *vf)
28516686d7fSJacob Keller {
28616686d7fSJacob Keller 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
28716686d7fSJacob Keller 	struct ice_pf *pf = vf->pf;
28816686d7fSJacob Keller 
289baeb705fSJacob Keller 	if (WARN_ON(!vsi))
290baeb705fSJacob Keller 		return -EINVAL;
291baeb705fSJacob Keller 
2926624e780SMichal Swiatkowski 	if (ice_vsi_rebuild(vsi, ICE_VSI_FLAG_INIT)) {
29316686d7fSJacob Keller 		dev_err(ice_pf_to_dev(pf), "failed to rebuild VF %d VSI\n",
29416686d7fSJacob Keller 			vf->vf_id);
29516686d7fSJacob Keller 		return -EIO;
29616686d7fSJacob Keller 	}
29716686d7fSJacob Keller 	/* vsi->idx will remain the same in this case so don't update
29816686d7fSJacob Keller 	 * vf->lan_vsi_idx
29916686d7fSJacob Keller 	 */
30016686d7fSJacob Keller 	vsi->vsi_num = ice_get_hw_vsi_num(&pf->hw, vsi->idx);
30116686d7fSJacob Keller 	vf->lan_vsi_num = vsi->vsi_num;
30216686d7fSJacob Keller 
30316686d7fSJacob Keller 	return 0;
30416686d7fSJacob Keller }
30516686d7fSJacob Keller 
30616686d7fSJacob Keller /**
307cc9c60c9SJan Sokolowski  * ice_vf_rebuild_host_vlan_cfg - add VLAN 0 filter or rebuild the Port VLAN
308cc9c60c9SJan Sokolowski  * @vf: VF to add MAC filters for
309cc9c60c9SJan Sokolowski  * @vsi: Pointer to VSI
310cc9c60c9SJan Sokolowski  *
311cc9c60c9SJan Sokolowski  * Called after a VF VSI has been re-added/rebuilt during reset. The PF driver
312cc9c60c9SJan Sokolowski  * always re-adds either a VLAN 0 or port VLAN based filter after reset.
313cc9c60c9SJan Sokolowski  */
ice_vf_rebuild_host_vlan_cfg(struct ice_vf * vf,struct ice_vsi * vsi)314cc9c60c9SJan Sokolowski static int ice_vf_rebuild_host_vlan_cfg(struct ice_vf *vf, struct ice_vsi *vsi)
315cc9c60c9SJan Sokolowski {
316cc9c60c9SJan Sokolowski 	struct ice_vsi_vlan_ops *vlan_ops = ice_get_compat_vsi_vlan_ops(vsi);
317cc9c60c9SJan Sokolowski 	struct device *dev = ice_pf_to_dev(vf->pf);
318cc9c60c9SJan Sokolowski 	int err;
319cc9c60c9SJan Sokolowski 
320cc9c60c9SJan Sokolowski 	if (ice_vf_is_port_vlan_ena(vf)) {
321cc9c60c9SJan Sokolowski 		err = vlan_ops->set_port_vlan(vsi, &vf->port_vlan_info);
322cc9c60c9SJan Sokolowski 		if (err) {
323cc9c60c9SJan Sokolowski 			dev_err(dev, "failed to configure port VLAN via VSI parameters for VF %u, error %d\n",
324cc9c60c9SJan Sokolowski 				vf->vf_id, err);
325cc9c60c9SJan Sokolowski 			return err;
326cc9c60c9SJan Sokolowski 		}
327cc9c60c9SJan Sokolowski 
328cc9c60c9SJan Sokolowski 		err = vlan_ops->add_vlan(vsi, &vf->port_vlan_info);
329cc9c60c9SJan Sokolowski 	} else {
330cc9c60c9SJan Sokolowski 		err = ice_vsi_add_vlan_zero(vsi);
331cc9c60c9SJan Sokolowski 	}
332cc9c60c9SJan Sokolowski 
333cc9c60c9SJan Sokolowski 	if (err) {
334cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to add VLAN %u filter for VF %u during VF rebuild, error %d\n",
335cc9c60c9SJan Sokolowski 			ice_vf_is_port_vlan_ena(vf) ?
336cc9c60c9SJan Sokolowski 			ice_vf_get_port_vlan_id(vf) : 0, vf->vf_id, err);
337cc9c60c9SJan Sokolowski 		return err;
338cc9c60c9SJan Sokolowski 	}
339cc9c60c9SJan Sokolowski 
340cc9c60c9SJan Sokolowski 	err = vlan_ops->ena_rx_filtering(vsi);
341cc9c60c9SJan Sokolowski 	if (err)
342cc9c60c9SJan Sokolowski 		dev_warn(dev, "failed to enable Rx VLAN filtering for VF %d VSI %d during VF rebuild, error %d\n",
343cc9c60c9SJan Sokolowski 			 vf->vf_id, vsi->idx, err);
344cc9c60c9SJan Sokolowski 
345cc9c60c9SJan Sokolowski 	return 0;
346cc9c60c9SJan Sokolowski }
347cc9c60c9SJan Sokolowski 
348cc9c60c9SJan Sokolowski /**
349cc9c60c9SJan Sokolowski  * ice_vf_rebuild_host_tx_rate_cfg - re-apply the Tx rate limiting configuration
350cc9c60c9SJan Sokolowski  * @vf: VF to re-apply the configuration for
351cc9c60c9SJan Sokolowski  *
352cc9c60c9SJan Sokolowski  * Called after a VF VSI has been re-added/rebuild during reset. The PF driver
353cc9c60c9SJan Sokolowski  * needs to re-apply the host configured Tx rate limiting configuration.
354cc9c60c9SJan Sokolowski  */
ice_vf_rebuild_host_tx_rate_cfg(struct ice_vf * vf)355cc9c60c9SJan Sokolowski static int ice_vf_rebuild_host_tx_rate_cfg(struct ice_vf *vf)
356cc9c60c9SJan Sokolowski {
357cc9c60c9SJan Sokolowski 	struct device *dev = ice_pf_to_dev(vf->pf);
358cc9c60c9SJan Sokolowski 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
359cc9c60c9SJan Sokolowski 	int err;
360cc9c60c9SJan Sokolowski 
361cc9c60c9SJan Sokolowski 	if (WARN_ON(!vsi))
362cc9c60c9SJan Sokolowski 		return -EINVAL;
363cc9c60c9SJan Sokolowski 
364cc9c60c9SJan Sokolowski 	if (vf->min_tx_rate) {
365cc9c60c9SJan Sokolowski 		err = ice_set_min_bw_limit(vsi, (u64)vf->min_tx_rate * 1000);
366cc9c60c9SJan Sokolowski 		if (err) {
367cc9c60c9SJan Sokolowski 			dev_err(dev, "failed to set min Tx rate to %d Mbps for VF %u, error %d\n",
368cc9c60c9SJan Sokolowski 				vf->min_tx_rate, vf->vf_id, err);
369cc9c60c9SJan Sokolowski 			return err;
370cc9c60c9SJan Sokolowski 		}
371cc9c60c9SJan Sokolowski 	}
372cc9c60c9SJan Sokolowski 
373cc9c60c9SJan Sokolowski 	if (vf->max_tx_rate) {
374cc9c60c9SJan Sokolowski 		err = ice_set_max_bw_limit(vsi, (u64)vf->max_tx_rate * 1000);
375cc9c60c9SJan Sokolowski 		if (err) {
376cc9c60c9SJan Sokolowski 			dev_err(dev, "failed to set max Tx rate to %d Mbps for VF %u, error %d\n",
377cc9c60c9SJan Sokolowski 				vf->max_tx_rate, vf->vf_id, err);
378cc9c60c9SJan Sokolowski 			return err;
379cc9c60c9SJan Sokolowski 		}
380cc9c60c9SJan Sokolowski 	}
381cc9c60c9SJan Sokolowski 
382cc9c60c9SJan Sokolowski 	return 0;
383cc9c60c9SJan Sokolowski }
384cc9c60c9SJan Sokolowski 
385cc9c60c9SJan Sokolowski /**
386cc9c60c9SJan Sokolowski  * ice_vf_set_host_trust_cfg - set trust setting based on pre-reset value
387cc9c60c9SJan Sokolowski  * @vf: VF to configure trust setting for
388cc9c60c9SJan Sokolowski  */
ice_vf_set_host_trust_cfg(struct ice_vf * vf)389cc9c60c9SJan Sokolowski static void ice_vf_set_host_trust_cfg(struct ice_vf *vf)
390cc9c60c9SJan Sokolowski {
39154e852daSTony Nguyen 	assign_bit(ICE_VIRTCHNL_VF_CAP_PRIVILEGE, &vf->vf_caps, vf->trusted);
392cc9c60c9SJan Sokolowski }
393cc9c60c9SJan Sokolowski 
394cc9c60c9SJan Sokolowski /**
395cc9c60c9SJan Sokolowski  * ice_vf_rebuild_host_mac_cfg - add broadcast and the VF's perm_addr/LAA
396cc9c60c9SJan Sokolowski  * @vf: VF to add MAC filters for
397cc9c60c9SJan Sokolowski  *
398cc9c60c9SJan Sokolowski  * Called after a VF VSI has been re-added/rebuilt during reset. The PF driver
399cc9c60c9SJan Sokolowski  * always re-adds a broadcast filter and the VF's perm_addr/LAA after reset.
400cc9c60c9SJan Sokolowski  */
ice_vf_rebuild_host_mac_cfg(struct ice_vf * vf)401cc9c60c9SJan Sokolowski static int ice_vf_rebuild_host_mac_cfg(struct ice_vf *vf)
402cc9c60c9SJan Sokolowski {
403cc9c60c9SJan Sokolowski 	struct device *dev = ice_pf_to_dev(vf->pf);
404cc9c60c9SJan Sokolowski 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
405cc9c60c9SJan Sokolowski 	u8 broadcast[ETH_ALEN];
406cc9c60c9SJan Sokolowski 	int status;
407cc9c60c9SJan Sokolowski 
408cc9c60c9SJan Sokolowski 	if (WARN_ON(!vsi))
409cc9c60c9SJan Sokolowski 		return -EINVAL;
410cc9c60c9SJan Sokolowski 
411cc9c60c9SJan Sokolowski 	if (ice_is_eswitch_mode_switchdev(vf->pf))
412cc9c60c9SJan Sokolowski 		return 0;
413cc9c60c9SJan Sokolowski 
414cc9c60c9SJan Sokolowski 	eth_broadcast_addr(broadcast);
415cc9c60c9SJan Sokolowski 	status = ice_fltr_add_mac(vsi, broadcast, ICE_FWD_TO_VSI);
416cc9c60c9SJan Sokolowski 	if (status) {
417cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to add broadcast MAC filter for VF %u, error %d\n",
418cc9c60c9SJan Sokolowski 			vf->vf_id, status);
419cc9c60c9SJan Sokolowski 		return status;
420cc9c60c9SJan Sokolowski 	}
421cc9c60c9SJan Sokolowski 
422cc9c60c9SJan Sokolowski 	vf->num_mac++;
423cc9c60c9SJan Sokolowski 
424cc9c60c9SJan Sokolowski 	if (is_valid_ether_addr(vf->hw_lan_addr)) {
425cc9c60c9SJan Sokolowski 		status = ice_fltr_add_mac(vsi, vf->hw_lan_addr,
426cc9c60c9SJan Sokolowski 					  ICE_FWD_TO_VSI);
427cc9c60c9SJan Sokolowski 		if (status) {
428cc9c60c9SJan Sokolowski 			dev_err(dev, "failed to add default unicast MAC filter %pM for VF %u, error %d\n",
429cc9c60c9SJan Sokolowski 				&vf->hw_lan_addr[0], vf->vf_id,
430cc9c60c9SJan Sokolowski 				status);
431cc9c60c9SJan Sokolowski 			return status;
432cc9c60c9SJan Sokolowski 		}
433cc9c60c9SJan Sokolowski 		vf->num_mac++;
434cc9c60c9SJan Sokolowski 
435cc9c60c9SJan Sokolowski 		ether_addr_copy(vf->dev_lan_addr, vf->hw_lan_addr);
436cc9c60c9SJan Sokolowski 	}
437cc9c60c9SJan Sokolowski 
438cc9c60c9SJan Sokolowski 	return 0;
439cc9c60c9SJan Sokolowski }
440cc9c60c9SJan Sokolowski 
441cc9c60c9SJan Sokolowski /**
442cc9c60c9SJan Sokolowski  * ice_vf_rebuild_aggregator_node_cfg - rebuild aggregator node config
443cc9c60c9SJan Sokolowski  * @vsi: Pointer to VSI
444cc9c60c9SJan Sokolowski  *
445cc9c60c9SJan Sokolowski  * This function moves VSI into corresponding scheduler aggregator node
446cc9c60c9SJan Sokolowski  * based on cached value of "aggregator node info" per VSI
447cc9c60c9SJan Sokolowski  */
ice_vf_rebuild_aggregator_node_cfg(struct ice_vsi * vsi)448cc9c60c9SJan Sokolowski static void ice_vf_rebuild_aggregator_node_cfg(struct ice_vsi *vsi)
449cc9c60c9SJan Sokolowski {
450cc9c60c9SJan Sokolowski 	struct ice_pf *pf = vsi->back;
451cc9c60c9SJan Sokolowski 	struct device *dev;
452cc9c60c9SJan Sokolowski 	int status;
453cc9c60c9SJan Sokolowski 
454cc9c60c9SJan Sokolowski 	if (!vsi->agg_node)
455cc9c60c9SJan Sokolowski 		return;
456cc9c60c9SJan Sokolowski 
457cc9c60c9SJan Sokolowski 	dev = ice_pf_to_dev(pf);
458cc9c60c9SJan Sokolowski 	if (vsi->agg_node->num_vsis == ICE_MAX_VSIS_IN_AGG_NODE) {
459cc9c60c9SJan Sokolowski 		dev_dbg(dev,
460cc9c60c9SJan Sokolowski 			"agg_id %u already has reached max_num_vsis %u\n",
461cc9c60c9SJan Sokolowski 			vsi->agg_node->agg_id, vsi->agg_node->num_vsis);
462cc9c60c9SJan Sokolowski 		return;
463cc9c60c9SJan Sokolowski 	}
464cc9c60c9SJan Sokolowski 
465cc9c60c9SJan Sokolowski 	status = ice_move_vsi_to_agg(pf->hw.port_info, vsi->agg_node->agg_id,
466cc9c60c9SJan Sokolowski 				     vsi->idx, vsi->tc_cfg.ena_tc);
467cc9c60c9SJan Sokolowski 	if (status)
468cc9c60c9SJan Sokolowski 		dev_dbg(dev, "unable to move VSI idx %u into aggregator %u node",
469cc9c60c9SJan Sokolowski 			vsi->idx, vsi->agg_node->agg_id);
470cc9c60c9SJan Sokolowski 	else
471cc9c60c9SJan Sokolowski 		vsi->agg_node->num_vsis++;
472cc9c60c9SJan Sokolowski }
473cc9c60c9SJan Sokolowski 
474cc9c60c9SJan Sokolowski /**
475cc9c60c9SJan Sokolowski  * ice_vf_rebuild_host_cfg - host admin configuration is persistent across reset
476cc9c60c9SJan Sokolowski  * @vf: VF to rebuild host configuration on
477cc9c60c9SJan Sokolowski  */
ice_vf_rebuild_host_cfg(struct ice_vf * vf)478cc9c60c9SJan Sokolowski static void ice_vf_rebuild_host_cfg(struct ice_vf *vf)
479cc9c60c9SJan Sokolowski {
480cc9c60c9SJan Sokolowski 	struct device *dev = ice_pf_to_dev(vf->pf);
481cc9c60c9SJan Sokolowski 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
482cc9c60c9SJan Sokolowski 
483cc9c60c9SJan Sokolowski 	if (WARN_ON(!vsi))
484cc9c60c9SJan Sokolowski 		return;
485cc9c60c9SJan Sokolowski 
486cc9c60c9SJan Sokolowski 	ice_vf_set_host_trust_cfg(vf);
487cc9c60c9SJan Sokolowski 
488cc9c60c9SJan Sokolowski 	if (ice_vf_rebuild_host_mac_cfg(vf))
489cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to rebuild default MAC configuration for VF %d\n",
490cc9c60c9SJan Sokolowski 			vf->vf_id);
491cc9c60c9SJan Sokolowski 
492cc9c60c9SJan Sokolowski 	if (ice_vf_rebuild_host_vlan_cfg(vf, vsi))
493cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to rebuild VLAN configuration for VF %u\n",
494cc9c60c9SJan Sokolowski 			vf->vf_id);
495cc9c60c9SJan Sokolowski 
496cc9c60c9SJan Sokolowski 	if (ice_vf_rebuild_host_tx_rate_cfg(vf))
497cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to rebuild Tx rate limiting configuration for VF %u\n",
498cc9c60c9SJan Sokolowski 			vf->vf_id);
499cc9c60c9SJan Sokolowski 
500cc9c60c9SJan Sokolowski 	if (ice_vsi_apply_spoofchk(vsi, vf->spoofchk))
501cc9c60c9SJan Sokolowski 		dev_err(dev, "failed to rebuild spoofchk configuration for VF %d\n",
502cc9c60c9SJan Sokolowski 			vf->vf_id);
503cc9c60c9SJan Sokolowski 
504cc9c60c9SJan Sokolowski 	/* rebuild aggregator node config for main VF VSI */
505cc9c60c9SJan Sokolowski 	ice_vf_rebuild_aggregator_node_cfg(vsi);
506cc9c60c9SJan Sokolowski }
507cc9c60c9SJan Sokolowski 
508cc9c60c9SJan Sokolowski /**
509cc9c60c9SJan Sokolowski  * ice_set_vf_state_qs_dis - Set VF queues state to disabled
510cc9c60c9SJan Sokolowski  * @vf: pointer to the VF structure
511cc9c60c9SJan Sokolowski  */
ice_set_vf_state_qs_dis(struct ice_vf * vf)512cc9c60c9SJan Sokolowski static void ice_set_vf_state_qs_dis(struct ice_vf *vf)
513cc9c60c9SJan Sokolowski {
514cc9c60c9SJan Sokolowski 	/* Clear Rx/Tx enabled queues flag */
515cc9c60c9SJan Sokolowski 	bitmap_zero(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF);
516cc9c60c9SJan Sokolowski 	bitmap_zero(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF);
517cc9c60c9SJan Sokolowski 	clear_bit(ICE_VF_STATE_QS_ENA, vf->vf_states);
518cc9c60c9SJan Sokolowski }
519cc9c60c9SJan Sokolowski 
520cc9c60c9SJan Sokolowski /**
521cc9c60c9SJan Sokolowski  * ice_vf_set_initialized - VF is ready for VIRTCHNL communication
522cc9c60c9SJan Sokolowski  * @vf: VF to set in initialized state
523cc9c60c9SJan Sokolowski  *
524cc9c60c9SJan Sokolowski  * After this function the VF will be ready to receive/handle the
525cc9c60c9SJan Sokolowski  * VIRTCHNL_OP_GET_VF_RESOURCES message
526cc9c60c9SJan Sokolowski  */
ice_vf_set_initialized(struct ice_vf * vf)527cc9c60c9SJan Sokolowski static void ice_vf_set_initialized(struct ice_vf *vf)
528cc9c60c9SJan Sokolowski {
529cc9c60c9SJan Sokolowski 	ice_set_vf_state_qs_dis(vf);
530cc9c60c9SJan Sokolowski 	clear_bit(ICE_VF_STATE_MC_PROMISC, vf->vf_states);
531cc9c60c9SJan Sokolowski 	clear_bit(ICE_VF_STATE_UC_PROMISC, vf->vf_states);
532cc9c60c9SJan Sokolowski 	clear_bit(ICE_VF_STATE_DIS, vf->vf_states);
533cc9c60c9SJan Sokolowski 	set_bit(ICE_VF_STATE_INIT, vf->vf_states);
534cc9c60c9SJan Sokolowski 	memset(&vf->vlan_v2_caps, 0, sizeof(vf->vlan_v2_caps));
535cc9c60c9SJan Sokolowski }
536cc9c60c9SJan Sokolowski 
537cc9c60c9SJan Sokolowski /**
538aeead3d0SJacob Keller  * ice_vf_post_vsi_rebuild - Reset tasks that occur after VSI rebuild
539aeead3d0SJacob Keller  * @vf: the VF being reset
540aeead3d0SJacob Keller  *
541aeead3d0SJacob Keller  * Perform reset tasks which must occur after the VSI has been re-created or
542aeead3d0SJacob Keller  * rebuilt during a VF reset.
543aeead3d0SJacob Keller  */
ice_vf_post_vsi_rebuild(struct ice_vf * vf)544aeead3d0SJacob Keller static void ice_vf_post_vsi_rebuild(struct ice_vf *vf)
545aeead3d0SJacob Keller {
546aeead3d0SJacob Keller 	ice_vf_rebuild_host_cfg(vf);
547aeead3d0SJacob Keller 	ice_vf_set_initialized(vf);
548aeead3d0SJacob Keller 
549aeead3d0SJacob Keller 	vf->vf_ops->post_vsi_rebuild(vf);
550aeead3d0SJacob Keller }
551aeead3d0SJacob Keller 
552aeead3d0SJacob Keller /**
553d7393425SMichal Wilczynski  * ice_is_any_vf_in_unicast_promisc - check if any VF(s)
554d7393425SMichal Wilczynski  * are in unicast promiscuous mode
555109aba47SJacob Keller  * @pf: PF structure for accessing VF(s)
556109aba47SJacob Keller  *
557d7393425SMichal Wilczynski  * Return false if no VF(s) are in unicast promiscuous mode,
558109aba47SJacob Keller  * else return true
559109aba47SJacob Keller  */
ice_is_any_vf_in_unicast_promisc(struct ice_pf * pf)560d7393425SMichal Wilczynski bool ice_is_any_vf_in_unicast_promisc(struct ice_pf *pf)
561109aba47SJacob Keller {
562109aba47SJacob Keller 	bool is_vf_promisc = false;
563109aba47SJacob Keller 	struct ice_vf *vf;
564109aba47SJacob Keller 	unsigned int bkt;
565109aba47SJacob Keller 
566109aba47SJacob Keller 	rcu_read_lock();
567109aba47SJacob Keller 	ice_for_each_vf_rcu(pf, bkt, vf) {
568109aba47SJacob Keller 		/* found a VF that has promiscuous mode configured */
569d7393425SMichal Wilczynski 		if (test_bit(ICE_VF_STATE_UC_PROMISC, vf->vf_states)) {
570109aba47SJacob Keller 			is_vf_promisc = true;
571109aba47SJacob Keller 			break;
572109aba47SJacob Keller 		}
573109aba47SJacob Keller 	}
574109aba47SJacob Keller 	rcu_read_unlock();
575109aba47SJacob Keller 
576109aba47SJacob Keller 	return is_vf_promisc;
577109aba47SJacob Keller }
578109aba47SJacob Keller 
579109aba47SJacob Keller /**
580a419526dSMichal Wilczynski  * ice_vf_get_promisc_masks - Calculate masks for promiscuous modes
581a419526dSMichal Wilczynski  * @vf: the VF pointer
582a419526dSMichal Wilczynski  * @vsi: the VSI to configure
583a419526dSMichal Wilczynski  * @ucast_m: promiscuous mask to apply to unicast
584a419526dSMichal Wilczynski  * @mcast_m: promiscuous mask to apply to multicast
585a419526dSMichal Wilczynski  *
586a419526dSMichal Wilczynski  * Decide which mask should be used for unicast and multicast filter,
587a419526dSMichal Wilczynski  * based on presence of VLANs
588a419526dSMichal Wilczynski  */
589a419526dSMichal Wilczynski void
ice_vf_get_promisc_masks(struct ice_vf * vf,struct ice_vsi * vsi,u8 * ucast_m,u8 * mcast_m)590a419526dSMichal Wilczynski ice_vf_get_promisc_masks(struct ice_vf *vf, struct ice_vsi *vsi,
591a419526dSMichal Wilczynski 			 u8 *ucast_m, u8 *mcast_m)
592a419526dSMichal Wilczynski {
593a419526dSMichal Wilczynski 	if (ice_vf_is_port_vlan_ena(vf) ||
594a419526dSMichal Wilczynski 	    ice_vsi_has_non_zero_vlans(vsi)) {
595a419526dSMichal Wilczynski 		*mcast_m = ICE_MCAST_VLAN_PROMISC_BITS;
596a419526dSMichal Wilczynski 		*ucast_m = ICE_UCAST_VLAN_PROMISC_BITS;
597a419526dSMichal Wilczynski 	} else {
598a419526dSMichal Wilczynski 		*mcast_m = ICE_MCAST_PROMISC_BITS;
599a419526dSMichal Wilczynski 		*ucast_m = ICE_UCAST_PROMISC_BITS;
600a419526dSMichal Wilczynski 	}
601a419526dSMichal Wilczynski }
602a419526dSMichal Wilczynski 
603a419526dSMichal Wilczynski /**
604a419526dSMichal Wilczynski  * ice_vf_clear_all_promisc_modes - Clear promisc/allmulticast on VF VSI
605a419526dSMichal Wilczynski  * @vf: the VF pointer
606a419526dSMichal Wilczynski  * @vsi: the VSI to configure
607a419526dSMichal Wilczynski  *
608a419526dSMichal Wilczynski  * Clear all promiscuous/allmulticast filters for a VF
609a419526dSMichal Wilczynski  */
610a419526dSMichal Wilczynski static int
ice_vf_clear_all_promisc_modes(struct ice_vf * vf,struct ice_vsi * vsi)611a419526dSMichal Wilczynski ice_vf_clear_all_promisc_modes(struct ice_vf *vf, struct ice_vsi *vsi)
612a419526dSMichal Wilczynski {
613a419526dSMichal Wilczynski 	struct ice_pf *pf = vf->pf;
614a419526dSMichal Wilczynski 	u8 ucast_m, mcast_m;
615a419526dSMichal Wilczynski 	int ret = 0;
616a419526dSMichal Wilczynski 
617a419526dSMichal Wilczynski 	ice_vf_get_promisc_masks(vf, vsi, &ucast_m, &mcast_m);
618a419526dSMichal Wilczynski 	if (test_bit(ICE_VF_STATE_UC_PROMISC, vf->vf_states)) {
619a419526dSMichal Wilczynski 		if (!test_bit(ICE_FLAG_VF_TRUE_PROMISC_ENA, pf->flags)) {
620a419526dSMichal Wilczynski 			if (ice_is_dflt_vsi_in_use(vsi->port_info))
621a419526dSMichal Wilczynski 				ret = ice_clear_dflt_vsi(vsi);
622a419526dSMichal Wilczynski 		} else {
623a419526dSMichal Wilczynski 			ret = ice_vf_clear_vsi_promisc(vf, vsi, ucast_m);
624a419526dSMichal Wilczynski 		}
625a419526dSMichal Wilczynski 
626a419526dSMichal Wilczynski 		if (ret) {
627a419526dSMichal Wilczynski 			dev_err(ice_pf_to_dev(vf->pf), "Disabling promiscuous mode failed\n");
628a419526dSMichal Wilczynski 		} else {
629a419526dSMichal Wilczynski 			clear_bit(ICE_VF_STATE_UC_PROMISC, vf->vf_states);
630a419526dSMichal Wilczynski 			dev_info(ice_pf_to_dev(vf->pf), "Disabling promiscuous mode succeeded\n");
631a419526dSMichal Wilczynski 		}
632a419526dSMichal Wilczynski 	}
633a419526dSMichal Wilczynski 
634a419526dSMichal Wilczynski 	if (test_bit(ICE_VF_STATE_MC_PROMISC, vf->vf_states)) {
635a419526dSMichal Wilczynski 		ret = ice_vf_clear_vsi_promisc(vf, vsi, mcast_m);
636a419526dSMichal Wilczynski 		if (ret) {
637a419526dSMichal Wilczynski 			dev_err(ice_pf_to_dev(vf->pf), "Disabling allmulticast mode failed\n");
638a419526dSMichal Wilczynski 		} else {
639a419526dSMichal Wilczynski 			clear_bit(ICE_VF_STATE_MC_PROMISC, vf->vf_states);
640a419526dSMichal Wilczynski 			dev_info(ice_pf_to_dev(vf->pf), "Disabling allmulticast mode succeeded\n");
641a419526dSMichal Wilczynski 		}
642a419526dSMichal Wilczynski 	}
643a419526dSMichal Wilczynski 	return ret;
644a419526dSMichal Wilczynski }
645a419526dSMichal Wilczynski 
646a419526dSMichal Wilczynski /**
647109aba47SJacob Keller  * ice_vf_set_vsi_promisc - Enable promiscuous mode for a VF VSI
648109aba47SJacob Keller  * @vf: the VF to configure
649109aba47SJacob Keller  * @vsi: the VF's VSI
650109aba47SJacob Keller  * @promisc_m: the promiscuous mode to enable
651109aba47SJacob Keller  */
652109aba47SJacob Keller int
ice_vf_set_vsi_promisc(struct ice_vf * vf,struct ice_vsi * vsi,u8 promisc_m)653109aba47SJacob Keller ice_vf_set_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m)
654109aba47SJacob Keller {
655109aba47SJacob Keller 	struct ice_hw *hw = &vsi->back->hw;
656109aba47SJacob Keller 	int status;
657109aba47SJacob Keller 
658109aba47SJacob Keller 	if (ice_vf_is_port_vlan_ena(vf))
659109aba47SJacob Keller 		status = ice_fltr_set_vsi_promisc(hw, vsi->idx, promisc_m,
660109aba47SJacob Keller 						  ice_vf_get_port_vlan_id(vf));
661109aba47SJacob Keller 	else if (ice_vsi_has_non_zero_vlans(vsi))
662109aba47SJacob Keller 		status = ice_fltr_set_vlan_vsi_promisc(hw, vsi, promisc_m);
663109aba47SJacob Keller 	else
664109aba47SJacob Keller 		status = ice_fltr_set_vsi_promisc(hw, vsi->idx, promisc_m, 0);
665109aba47SJacob Keller 
666109aba47SJacob Keller 	if (status && status != -EEXIST) {
667109aba47SJacob Keller 		dev_err(ice_pf_to_dev(vsi->back), "enable Tx/Rx filter promiscuous mode on VF-%u failed, error: %d\n",
668109aba47SJacob Keller 			vf->vf_id, status);
669109aba47SJacob Keller 		return status;
670109aba47SJacob Keller 	}
671109aba47SJacob Keller 
672109aba47SJacob Keller 	return 0;
673109aba47SJacob Keller }
674109aba47SJacob Keller 
675109aba47SJacob Keller /**
676109aba47SJacob Keller  * ice_vf_clear_vsi_promisc - Disable promiscuous mode for a VF VSI
677109aba47SJacob Keller  * @vf: the VF to configure
678109aba47SJacob Keller  * @vsi: the VF's VSI
679109aba47SJacob Keller  * @promisc_m: the promiscuous mode to disable
680109aba47SJacob Keller  */
681109aba47SJacob Keller int
ice_vf_clear_vsi_promisc(struct ice_vf * vf,struct ice_vsi * vsi,u8 promisc_m)682109aba47SJacob Keller ice_vf_clear_vsi_promisc(struct ice_vf *vf, struct ice_vsi *vsi, u8 promisc_m)
683109aba47SJacob Keller {
684109aba47SJacob Keller 	struct ice_hw *hw = &vsi->back->hw;
685109aba47SJacob Keller 	int status;
686109aba47SJacob Keller 
687109aba47SJacob Keller 	if (ice_vf_is_port_vlan_ena(vf))
688109aba47SJacob Keller 		status = ice_fltr_clear_vsi_promisc(hw, vsi->idx, promisc_m,
689109aba47SJacob Keller 						    ice_vf_get_port_vlan_id(vf));
690109aba47SJacob Keller 	else if (ice_vsi_has_non_zero_vlans(vsi))
691109aba47SJacob Keller 		status = ice_fltr_clear_vlan_vsi_promisc(hw, vsi, promisc_m);
692109aba47SJacob Keller 	else
693109aba47SJacob Keller 		status = ice_fltr_clear_vsi_promisc(hw, vsi->idx, promisc_m, 0);
694109aba47SJacob Keller 
695109aba47SJacob Keller 	if (status && status != -ENOENT) {
696109aba47SJacob Keller 		dev_err(ice_pf_to_dev(vsi->back), "disable Tx/Rx filter promiscuous mode on VF-%u failed, error: %d\n",
697109aba47SJacob Keller 			vf->vf_id, status);
698109aba47SJacob Keller 		return status;
699109aba47SJacob Keller 	}
700109aba47SJacob Keller 
701109aba47SJacob Keller 	return 0;
702109aba47SJacob Keller }
703109aba47SJacob Keller 
704109aba47SJacob Keller /**
70516686d7fSJacob Keller  * ice_reset_all_vfs - reset all allocated VFs in one go
70616686d7fSJacob Keller  * @pf: pointer to the PF structure
70716686d7fSJacob Keller  *
70800be8197SJacob Keller  * Reset all VFs at once, in response to a PF or other device reset.
70900be8197SJacob Keller  *
71016686d7fSJacob Keller  * First, tell the hardware to reset each VF, then do all the waiting in one
71116686d7fSJacob Keller  * chunk, and finally finish restoring each VF after the wait. This is useful
71216686d7fSJacob Keller  * during PF routines which need to reset all VFs, as otherwise it must perform
71316686d7fSJacob Keller  * these resets in a serialized fashion.
71416686d7fSJacob Keller  */
ice_reset_all_vfs(struct ice_pf * pf)715fe99d1c0SJacob Keller void ice_reset_all_vfs(struct ice_pf *pf)
71616686d7fSJacob Keller {
71716686d7fSJacob Keller 	struct device *dev = ice_pf_to_dev(pf);
71816686d7fSJacob Keller 	struct ice_hw *hw = &pf->hw;
71916686d7fSJacob Keller 	struct ice_vf *vf;
72016686d7fSJacob Keller 	unsigned int bkt;
72116686d7fSJacob Keller 
72216686d7fSJacob Keller 	/* If we don't have any VFs, then there is nothing to reset */
72316686d7fSJacob Keller 	if (!ice_has_vfs(pf))
724fe99d1c0SJacob Keller 		return;
72516686d7fSJacob Keller 
72616686d7fSJacob Keller 	mutex_lock(&pf->vfs.table_lock);
72716686d7fSJacob Keller 
72816686d7fSJacob Keller 	/* clear all malicious info if the VFs are getting reset */
72916686d7fSJacob Keller 	ice_for_each_vf(pf, bkt, vf)
7308cd8a6b1SJacob Keller 		ice_mbx_clear_malvf(&vf->mbx_info);
73116686d7fSJacob Keller 
73216686d7fSJacob Keller 	/* If VFs have been disabled, there is no need to reset */
73316686d7fSJacob Keller 	if (test_and_set_bit(ICE_VF_DIS, pf->state)) {
73416686d7fSJacob Keller 		mutex_unlock(&pf->vfs.table_lock);
735fe99d1c0SJacob Keller 		return;
73616686d7fSJacob Keller 	}
73716686d7fSJacob Keller 
73816686d7fSJacob Keller 	/* Begin reset on all VFs at once */
73916686d7fSJacob Keller 	ice_for_each_vf(pf, bkt, vf)
740dac57288SJacob Keller 		ice_trigger_vf_reset(vf, true, true);
74116686d7fSJacob Keller 
74216686d7fSJacob Keller 	/* HW requires some time to make sure it can flush the FIFO for a VF
74316686d7fSJacob Keller 	 * when it resets it. Now that we've triggered all of the VFs, iterate
74416686d7fSJacob Keller 	 * the table again and wait for each VF to complete.
74516686d7fSJacob Keller 	 */
74616686d7fSJacob Keller 	ice_for_each_vf(pf, bkt, vf) {
74716686d7fSJacob Keller 		if (!vf->vf_ops->poll_reset_status(vf)) {
74816686d7fSJacob Keller 			/* Display a warning if at least one VF didn't manage
74916686d7fSJacob Keller 			 * to reset in time, but continue on with the
75016686d7fSJacob Keller 			 * operation.
75116686d7fSJacob Keller 			 */
75216686d7fSJacob Keller 			dev_warn(dev, "VF %u reset check timeout\n", vf->vf_id);
75316686d7fSJacob Keller 			break;
75416686d7fSJacob Keller 		}
75516686d7fSJacob Keller 	}
75616686d7fSJacob Keller 
75716686d7fSJacob Keller 	/* free VF resources to begin resetting the VSI state */
75816686d7fSJacob Keller 	ice_for_each_vf(pf, bkt, vf) {
75916686d7fSJacob Keller 		mutex_lock(&vf->cfg_lock);
76016686d7fSJacob Keller 
76116686d7fSJacob Keller 		vf->driver_caps = 0;
76216686d7fSJacob Keller 		ice_vc_set_default_allowlist(vf);
76316686d7fSJacob Keller 
76416686d7fSJacob Keller 		ice_vf_fdir_exit(vf);
76516686d7fSJacob Keller 		ice_vf_fdir_init(vf);
76616686d7fSJacob Keller 		/* clean VF control VSI when resetting VFs since it should be
76716686d7fSJacob Keller 		 * setup only when VF creates its first FDIR rule.
76816686d7fSJacob Keller 		 */
76916686d7fSJacob Keller 		if (vf->ctrl_vsi_idx != ICE_NO_VSI)
77016686d7fSJacob Keller 			ice_vf_ctrl_invalidate_vsi(vf);
77116686d7fSJacob Keller 
77216686d7fSJacob Keller 		ice_vf_pre_vsi_rebuild(vf);
77316686d7fSJacob Keller 		ice_vf_rebuild_vsi(vf);
774aeead3d0SJacob Keller 		ice_vf_post_vsi_rebuild(vf);
77516686d7fSJacob Keller 
77616686d7fSJacob Keller 		mutex_unlock(&vf->cfg_lock);
77716686d7fSJacob Keller 	}
77816686d7fSJacob Keller 
77916686d7fSJacob Keller 	if (ice_is_eswitch_mode_switchdev(pf))
78016686d7fSJacob Keller 		if (ice_eswitch_rebuild(pf))
78116686d7fSJacob Keller 			dev_warn(dev, "eswitch rebuild failed\n");
78216686d7fSJacob Keller 
78316686d7fSJacob Keller 	ice_flush(hw);
78416686d7fSJacob Keller 	clear_bit(ICE_VF_DIS, pf->state);
78516686d7fSJacob Keller 
78616686d7fSJacob Keller 	mutex_unlock(&pf->vfs.table_lock);
78716686d7fSJacob Keller }
78816686d7fSJacob Keller 
78916686d7fSJacob Keller /**
7909dbb33daSJacob Keller  * ice_notify_vf_reset - Notify VF of a reset event
7919dbb33daSJacob Keller  * @vf: pointer to the VF structure
7929dbb33daSJacob Keller  */
ice_notify_vf_reset(struct ice_vf * vf)7939dbb33daSJacob Keller static void ice_notify_vf_reset(struct ice_vf *vf)
7949dbb33daSJacob Keller {
7959dbb33daSJacob Keller 	struct ice_hw *hw = &vf->pf->hw;
7969dbb33daSJacob Keller 	struct virtchnl_pf_event pfe;
7979dbb33daSJacob Keller 
7989dbb33daSJacob Keller 	/* Bail out if VF is in disabled state, neither initialized, nor active
7999dbb33daSJacob Keller 	 * state - otherwise proceed with notifications
8009dbb33daSJacob Keller 	 */
8019dbb33daSJacob Keller 	if ((!test_bit(ICE_VF_STATE_INIT, vf->vf_states) &&
8029dbb33daSJacob Keller 	     !test_bit(ICE_VF_STATE_ACTIVE, vf->vf_states)) ||
8039dbb33daSJacob Keller 	    test_bit(ICE_VF_STATE_DIS, vf->vf_states))
8049dbb33daSJacob Keller 		return;
8059dbb33daSJacob Keller 
8069dbb33daSJacob Keller 	pfe.event = VIRTCHNL_EVENT_RESET_IMPENDING;
8079dbb33daSJacob Keller 	pfe.severity = PF_EVENT_SEVERITY_CERTAIN_DOOM;
8089dbb33daSJacob Keller 	ice_aq_send_msg_to_vf(hw, vf->vf_id, VIRTCHNL_OP_EVENT,
8099dbb33daSJacob Keller 			      VIRTCHNL_STATUS_SUCCESS, (u8 *)&pfe, sizeof(pfe),
8109dbb33daSJacob Keller 			      NULL);
8119dbb33daSJacob Keller }
8129dbb33daSJacob Keller 
8139dbb33daSJacob Keller /**
81416686d7fSJacob Keller  * ice_reset_vf - Reset a particular VF
81516686d7fSJacob Keller  * @vf: pointer to the VF structure
8167eb517e4SJacob Keller  * @flags: flags controlling behavior of the reset
8177eb517e4SJacob Keller  *
8187eb517e4SJacob Keller  * Flags:
8197eb517e4SJacob Keller  *   ICE_VF_RESET_VFLR - Indicates a reset is due to VFLR event
8209dbb33daSJacob Keller  *   ICE_VF_RESET_NOTIFY - Send VF a notification prior to reset
821f5f085c0SJacob Keller  *   ICE_VF_RESET_LOCK - Acquire VF cfg_lock before resetting
82216686d7fSJacob Keller  *
82319c3e1edSJacob Keller  * Returns 0 if the VF is currently in reset, if resets are disabled, or if
82419c3e1edSJacob Keller  * the VF resets successfully. Returns an error code if the VF fails to
8254fe193ccSJacob Keller  * rebuild.
82616686d7fSJacob Keller  */
ice_reset_vf(struct ice_vf * vf,u32 flags)8277eb517e4SJacob Keller int ice_reset_vf(struct ice_vf *vf, u32 flags)
82816686d7fSJacob Keller {
82916686d7fSJacob Keller 	struct ice_pf *pf = vf->pf;
830fd7f7a8aSDave Ertman 	struct ice_lag *lag;
83116686d7fSJacob Keller 	struct ice_vsi *vsi;
832fd7f7a8aSDave Ertman 	u8 act_prt, pri_prt;
83316686d7fSJacob Keller 	struct device *dev;
834f5f085c0SJacob Keller 	int err = 0;
83516686d7fSJacob Keller 	bool rsd;
83616686d7fSJacob Keller 
83716686d7fSJacob Keller 	dev = ice_pf_to_dev(pf);
838fd7f7a8aSDave Ertman 	act_prt = ICE_LAG_INVALID_PORT;
839fd7f7a8aSDave Ertman 	pri_prt = pf->hw.port_info->lport;
84016686d7fSJacob Keller 
8419dbb33daSJacob Keller 	if (flags & ICE_VF_RESET_NOTIFY)
8429dbb33daSJacob Keller 		ice_notify_vf_reset(vf);
8439dbb33daSJacob Keller 
84416686d7fSJacob Keller 	if (test_bit(ICE_VF_RESETS_DISABLED, pf->state)) {
84516686d7fSJacob Keller 		dev_dbg(dev, "Trying to reset VF %d, but all VF resets are disabled\n",
84616686d7fSJacob Keller 			vf->vf_id);
8474fe193ccSJacob Keller 		return 0;
84816686d7fSJacob Keller 	}
84916686d7fSJacob Keller 
85074071777SJacob Keller 	if (flags & ICE_VF_RESET_LOCK)
85174071777SJacob Keller 		mutex_lock(&vf->cfg_lock);
85274071777SJacob Keller 	else
85374071777SJacob Keller 		lockdep_assert_held(&vf->cfg_lock);
85474071777SJacob Keller 
855fd7f7a8aSDave Ertman 	lag = pf->lag;
856fd7f7a8aSDave Ertman 	mutex_lock(&pf->lag_mutex);
857fd7f7a8aSDave Ertman 	if (lag && lag->bonded && lag->primary) {
858fd7f7a8aSDave Ertman 		act_prt = lag->active_port;
859fd7f7a8aSDave Ertman 		if (act_prt != pri_prt && act_prt != ICE_LAG_INVALID_PORT &&
860fd7f7a8aSDave Ertman 		    lag->upper_netdev)
861fd7f7a8aSDave Ertman 			ice_lag_move_vf_nodes_cfg(lag, act_prt, pri_prt);
862fd7f7a8aSDave Ertman 		else
863fd7f7a8aSDave Ertman 			act_prt = ICE_LAG_INVALID_PORT;
864fd7f7a8aSDave Ertman 	}
865fd7f7a8aSDave Ertman 
86616686d7fSJacob Keller 	if (ice_is_vf_disabled(vf)) {
867efe41860SPrzemyslaw Patynowski 		vsi = ice_get_vf_vsi(vf);
868cf90b743SMichal Jaron 		if (!vsi) {
869cf90b743SMichal Jaron 			dev_dbg(dev, "VF is already removed\n");
87067f6317dSPetr Oros 			err = -EINVAL;
87167f6317dSPetr Oros 			goto out_unlock;
872cf90b743SMichal Jaron 		}
873efe41860SPrzemyslaw Patynowski 		ice_vsi_stop_lan_tx_rings(vsi, ICE_NO_RESET, vf->vf_id);
874f23df522SNorbert Zulinski 
875f23df522SNorbert Zulinski 		if (ice_vsi_is_rx_queue_active(vsi))
876efe41860SPrzemyslaw Patynowski 			ice_vsi_stop_all_rx_rings(vsi);
877f23df522SNorbert Zulinski 
87816686d7fSJacob Keller 		dev_dbg(dev, "VF is already disabled, there is no need for resetting it, telling VM, all is fine %d\n",
87916686d7fSJacob Keller 			vf->vf_id);
88067f6317dSPetr Oros 		goto out_unlock;
88116686d7fSJacob Keller 	}
88216686d7fSJacob Keller 
88316686d7fSJacob Keller 	/* Set VF disable bit state here, before triggering reset */
88416686d7fSJacob Keller 	set_bit(ICE_VF_STATE_DIS, vf->vf_states);
8857eb517e4SJacob Keller 	ice_trigger_vf_reset(vf, flags & ICE_VF_RESET_VFLR, false);
88616686d7fSJacob Keller 
88716686d7fSJacob Keller 	vsi = ice_get_vf_vsi(vf);
888baeb705fSJacob Keller 	if (WARN_ON(!vsi)) {
889baeb705fSJacob Keller 		err = -EIO;
890baeb705fSJacob Keller 		goto out_unlock;
891baeb705fSJacob Keller 	}
89216686d7fSJacob Keller 
89316686d7fSJacob Keller 	ice_dis_vf_qs(vf);
89416686d7fSJacob Keller 
89516686d7fSJacob Keller 	/* Call Disable LAN Tx queue AQ whether or not queues are
89616686d7fSJacob Keller 	 * enabled. This is needed for successful completion of VFR.
89716686d7fSJacob Keller 	 */
89816686d7fSJacob Keller 	ice_dis_vsi_txq(vsi->port_info, vsi->idx, 0, 0, NULL, NULL,
89916686d7fSJacob Keller 			NULL, vf->vf_ops->reset_type, vf->vf_id, NULL);
90016686d7fSJacob Keller 
90116686d7fSJacob Keller 	/* poll VPGEN_VFRSTAT reg to make sure
90216686d7fSJacob Keller 	 * that reset is complete
90316686d7fSJacob Keller 	 */
90416686d7fSJacob Keller 	rsd = vf->vf_ops->poll_reset_status(vf);
90516686d7fSJacob Keller 
90616686d7fSJacob Keller 	/* Display a warning if VF didn't manage to reset in time, but need to
90716686d7fSJacob Keller 	 * continue on with the operation.
90816686d7fSJacob Keller 	 */
90916686d7fSJacob Keller 	if (!rsd)
91016686d7fSJacob Keller 		dev_warn(dev, "VF reset check timeout on VF %d\n", vf->vf_id);
91116686d7fSJacob Keller 
91216686d7fSJacob Keller 	vf->driver_caps = 0;
91316686d7fSJacob Keller 	ice_vc_set_default_allowlist(vf);
91416686d7fSJacob Keller 
91516686d7fSJacob Keller 	/* disable promiscuous modes in case they were enabled
91616686d7fSJacob Keller 	 * ignore any error if disabling process failed
91716686d7fSJacob Keller 	 */
918a419526dSMichal Wilczynski 	ice_vf_clear_all_promisc_modes(vf, vsi);
91916686d7fSJacob Keller 
92016686d7fSJacob Keller 	ice_vf_fdir_exit(vf);
92116686d7fSJacob Keller 	ice_vf_fdir_init(vf);
92216686d7fSJacob Keller 	/* clean VF control VSI when resetting VF since it should be setup
92316686d7fSJacob Keller 	 * only when VF creates its first FDIR rule.
92416686d7fSJacob Keller 	 */
92516686d7fSJacob Keller 	if (vf->ctrl_vsi_idx != ICE_NO_VSI)
92616686d7fSJacob Keller 		ice_vf_ctrl_vsi_release(vf);
92716686d7fSJacob Keller 
92816686d7fSJacob Keller 	ice_vf_pre_vsi_rebuild(vf);
92916686d7fSJacob Keller 
9305531bb85SJacob Keller 	if (ice_vf_recreate_vsi(vf)) {
93116686d7fSJacob Keller 		dev_err(dev, "Failed to release and setup the VF%u's VSI\n",
93216686d7fSJacob Keller 			vf->vf_id);
933f5f085c0SJacob Keller 		err = -EFAULT;
934f5f085c0SJacob Keller 		goto out_unlock;
93516686d7fSJacob Keller 	}
93616686d7fSJacob Keller 
937aeead3d0SJacob Keller 	ice_vf_post_vsi_rebuild(vf);
93816686d7fSJacob Keller 	vsi = ice_get_vf_vsi(vf);
939baeb705fSJacob Keller 	if (WARN_ON(!vsi)) {
940baeb705fSJacob Keller 		err = -EINVAL;
941baeb705fSJacob Keller 		goto out_unlock;
942baeb705fSJacob Keller 	}
943baeb705fSJacob Keller 
94416686d7fSJacob Keller 	ice_eswitch_update_repr(vsi);
94516686d7fSJacob Keller 
94616686d7fSJacob Keller 	/* if the VF has been reset allow it to come up again */
9478cd8a6b1SJacob Keller 	ice_mbx_clear_malvf(&vf->mbx_info);
94816686d7fSJacob Keller 
949f5f085c0SJacob Keller out_unlock:
950fd7f7a8aSDave Ertman 	if (lag && lag->bonded && lag->primary &&
951fd7f7a8aSDave Ertman 	    act_prt != ICE_LAG_INVALID_PORT)
952fd7f7a8aSDave Ertman 		ice_lag_move_vf_nodes_cfg(lag, pri_prt, act_prt);
953fd7f7a8aSDave Ertman 	mutex_unlock(&pf->lag_mutex);
954fd7f7a8aSDave Ertman 
95574071777SJacob Keller 	if (flags & ICE_VF_RESET_LOCK)
95674071777SJacob Keller 		mutex_unlock(&vf->cfg_lock);
95774071777SJacob Keller 
958f5f085c0SJacob Keller 	return err;
95916686d7fSJacob Keller }
96016686d7fSJacob Keller 
96116686d7fSJacob Keller /**
962fa4a15c8SJacob Keller  * ice_set_vf_state_dis - Set VF state to disabled
963fa4a15c8SJacob Keller  * @vf: pointer to the VF structure
964fa4a15c8SJacob Keller  */
ice_set_vf_state_dis(struct ice_vf * vf)965fa4a15c8SJacob Keller void ice_set_vf_state_dis(struct ice_vf *vf)
966fa4a15c8SJacob Keller {
967fa4a15c8SJacob Keller 	ice_set_vf_state_qs_dis(vf);
968fa4a15c8SJacob Keller 	vf->vf_ops->clear_reset_state(vf);
969fa4a15c8SJacob Keller }
970fa4a15c8SJacob Keller 
971109aba47SJacob Keller /* Private functions only accessed from other virtualization files */
972109aba47SJacob Keller 
973109aba47SJacob Keller /**
974b5dcff1fSJacob Keller  * ice_initialize_vf_entry - Initialize a VF entry
975b5dcff1fSJacob Keller  * @vf: pointer to the VF structure
976b5dcff1fSJacob Keller  */
ice_initialize_vf_entry(struct ice_vf * vf)977b5dcff1fSJacob Keller void ice_initialize_vf_entry(struct ice_vf *vf)
978b5dcff1fSJacob Keller {
979b5dcff1fSJacob Keller 	struct ice_pf *pf = vf->pf;
980b5dcff1fSJacob Keller 	struct ice_vfs *vfs;
981b5dcff1fSJacob Keller 
982b5dcff1fSJacob Keller 	vfs = &pf->vfs;
983b5dcff1fSJacob Keller 
984b5dcff1fSJacob Keller 	/* assign default capabilities */
985b5dcff1fSJacob Keller 	vf->spoofchk = true;
986b5dcff1fSJacob Keller 	vf->num_vf_qs = vfs->num_qps_per;
987b5dcff1fSJacob Keller 	ice_vc_set_default_allowlist(vf);
988b5dcff1fSJacob Keller 	ice_virtchnl_set_dflt_ops(vf);
989b5dcff1fSJacob Keller 
990b5dcff1fSJacob Keller 	/* ctrl_vsi_idx will be set to a valid value only when iAVF
991b5dcff1fSJacob Keller 	 * creates its first fdir rule.
992b5dcff1fSJacob Keller 	 */
993b5dcff1fSJacob Keller 	ice_vf_ctrl_invalidate_vsi(vf);
994b5dcff1fSJacob Keller 	ice_vf_fdir_init(vf);
995b5dcff1fSJacob Keller 
996e4eaf893SJacob Keller 	/* Initialize mailbox info for this VF */
997e4eaf893SJacob Keller 	ice_mbx_init_vf_info(&pf->hw, &vf->mbx_info);
998e4eaf893SJacob Keller 
999b5dcff1fSJacob Keller 	mutex_init(&vf->cfg_lock);
1000b5dcff1fSJacob Keller }
1001b5dcff1fSJacob Keller 
1002b5dcff1fSJacob Keller /**
1003109aba47SJacob Keller  * ice_dis_vf_qs - Disable the VF queues
1004109aba47SJacob Keller  * @vf: pointer to the VF structure
1005109aba47SJacob Keller  */
ice_dis_vf_qs(struct ice_vf * vf)1006109aba47SJacob Keller void ice_dis_vf_qs(struct ice_vf *vf)
1007109aba47SJacob Keller {
1008109aba47SJacob Keller 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
1009109aba47SJacob Keller 
1010baeb705fSJacob Keller 	if (WARN_ON(!vsi))
1011baeb705fSJacob Keller 		return;
1012baeb705fSJacob Keller 
1013109aba47SJacob Keller 	ice_vsi_stop_lan_tx_rings(vsi, ICE_NO_RESET, vf->vf_id);
1014109aba47SJacob Keller 	ice_vsi_stop_all_rx_rings(vsi);
1015109aba47SJacob Keller 	ice_set_vf_state_qs_dis(vf);
1016109aba47SJacob Keller }
1017109aba47SJacob Keller 
1018109aba47SJacob Keller /**
1019e384cf35SMd Fahad Iqbal Polash  * ice_err_to_virt_err - translate errors for VF return code
1020e384cf35SMd Fahad Iqbal Polash  * @err: error return code
1021e384cf35SMd Fahad Iqbal Polash  */
ice_err_to_virt_err(int err)1022e384cf35SMd Fahad Iqbal Polash enum virtchnl_status_code ice_err_to_virt_err(int err)
1023e384cf35SMd Fahad Iqbal Polash {
1024e384cf35SMd Fahad Iqbal Polash 	switch (err) {
1025e384cf35SMd Fahad Iqbal Polash 	case 0:
1026e384cf35SMd Fahad Iqbal Polash 		return VIRTCHNL_STATUS_SUCCESS;
1027e384cf35SMd Fahad Iqbal Polash 	case -EINVAL:
1028e384cf35SMd Fahad Iqbal Polash 	case -ENODEV:
1029e384cf35SMd Fahad Iqbal Polash 		return VIRTCHNL_STATUS_ERR_PARAM;
1030e384cf35SMd Fahad Iqbal Polash 	case -ENOMEM:
1031e384cf35SMd Fahad Iqbal Polash 		return VIRTCHNL_STATUS_ERR_NO_MEMORY;
1032e384cf35SMd Fahad Iqbal Polash 	case -EALREADY:
1033e384cf35SMd Fahad Iqbal Polash 	case -EBUSY:
1034e384cf35SMd Fahad Iqbal Polash 	case -EIO:
1035e384cf35SMd Fahad Iqbal Polash 	case -ENOSPC:
1036e384cf35SMd Fahad Iqbal Polash 		return VIRTCHNL_STATUS_ERR_ADMIN_QUEUE_ERROR;
1037e384cf35SMd Fahad Iqbal Polash 	default:
1038e384cf35SMd Fahad Iqbal Polash 		return VIRTCHNL_STATUS_ERR_NOT_SUPPORTED;
1039e384cf35SMd Fahad Iqbal Polash 	}
1040e384cf35SMd Fahad Iqbal Polash }
1041e384cf35SMd Fahad Iqbal Polash 
1042e384cf35SMd Fahad Iqbal Polash /**
1043109aba47SJacob Keller  * ice_check_vf_init - helper to check if VF init complete
1044109aba47SJacob Keller  * @vf: the pointer to the VF to check
1045109aba47SJacob Keller  */
ice_check_vf_init(struct ice_vf * vf)10465a57ee83SJacob Keller int ice_check_vf_init(struct ice_vf *vf)
1047109aba47SJacob Keller {
10485a57ee83SJacob Keller 	struct ice_pf *pf = vf->pf;
10495a57ee83SJacob Keller 
1050109aba47SJacob Keller 	if (!test_bit(ICE_VF_STATE_INIT, vf->vf_states)) {
1051109aba47SJacob Keller 		dev_err(ice_pf_to_dev(pf), "VF ID: %u in reset. Try again.\n",
1052109aba47SJacob Keller 			vf->vf_id);
1053109aba47SJacob Keller 		return -EBUSY;
1054109aba47SJacob Keller 	}
1055109aba47SJacob Keller 	return 0;
1056109aba47SJacob Keller }
1057109aba47SJacob Keller 
1058109aba47SJacob Keller /**
1059109aba47SJacob Keller  * ice_vf_get_port_info - Get the VF's port info structure
1060109aba47SJacob Keller  * @vf: VF used to get the port info structure for
1061109aba47SJacob Keller  */
ice_vf_get_port_info(struct ice_vf * vf)1062109aba47SJacob Keller struct ice_port_info *ice_vf_get_port_info(struct ice_vf *vf)
1063109aba47SJacob Keller {
1064109aba47SJacob Keller 	return vf->pf->hw.port_info;
1065109aba47SJacob Keller }
1066109aba47SJacob Keller 
106771c114e8SJacob Keller /**
106871c114e8SJacob Keller  * ice_cfg_mac_antispoof - Configure MAC antispoof checking behavior
106971c114e8SJacob Keller  * @vsi: the VSI to configure
107071c114e8SJacob Keller  * @enable: whether to enable or disable the spoof checking
107171c114e8SJacob Keller  *
107271c114e8SJacob Keller  * Configure a VSI to enable (or disable) spoof checking behavior.
107371c114e8SJacob Keller  */
ice_cfg_mac_antispoof(struct ice_vsi * vsi,bool enable)1074109aba47SJacob Keller static int ice_cfg_mac_antispoof(struct ice_vsi *vsi, bool enable)
1075109aba47SJacob Keller {
1076109aba47SJacob Keller 	struct ice_vsi_ctx *ctx;
1077109aba47SJacob Keller 	int err;
1078109aba47SJacob Keller 
1079109aba47SJacob Keller 	ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
1080109aba47SJacob Keller 	if (!ctx)
1081109aba47SJacob Keller 		return -ENOMEM;
1082109aba47SJacob Keller 
1083109aba47SJacob Keller 	ctx->info.sec_flags = vsi->info.sec_flags;
1084109aba47SJacob Keller 	ctx->info.valid_sections = cpu_to_le16(ICE_AQ_VSI_PROP_SECURITY_VALID);
1085109aba47SJacob Keller 
1086109aba47SJacob Keller 	if (enable)
1087109aba47SJacob Keller 		ctx->info.sec_flags |= ICE_AQ_VSI_SEC_FLAG_ENA_MAC_ANTI_SPOOF;
1088109aba47SJacob Keller 	else
1089109aba47SJacob Keller 		ctx->info.sec_flags &= ~ICE_AQ_VSI_SEC_FLAG_ENA_MAC_ANTI_SPOOF;
1090109aba47SJacob Keller 
1091109aba47SJacob Keller 	err = ice_update_vsi(&vsi->back->hw, vsi->idx, ctx, NULL);
1092109aba47SJacob Keller 	if (err)
1093109aba47SJacob Keller 		dev_err(ice_pf_to_dev(vsi->back), "Failed to configure Tx MAC anti-spoof %s for VSI %d, error %d\n",
1094109aba47SJacob Keller 			enable ? "ON" : "OFF", vsi->vsi_num, err);
1095109aba47SJacob Keller 	else
1096109aba47SJacob Keller 		vsi->info.sec_flags = ctx->info.sec_flags;
1097109aba47SJacob Keller 
1098109aba47SJacob Keller 	kfree(ctx);
1099109aba47SJacob Keller 
1100109aba47SJacob Keller 	return err;
1101109aba47SJacob Keller }
1102109aba47SJacob Keller 
1103109aba47SJacob Keller /**
1104109aba47SJacob Keller  * ice_vsi_ena_spoofchk - enable Tx spoof checking for this VSI
1105109aba47SJacob Keller  * @vsi: VSI to enable Tx spoof checking for
1106109aba47SJacob Keller  */
ice_vsi_ena_spoofchk(struct ice_vsi * vsi)1107109aba47SJacob Keller static int ice_vsi_ena_spoofchk(struct ice_vsi *vsi)
1108109aba47SJacob Keller {
1109109aba47SJacob Keller 	struct ice_vsi_vlan_ops *vlan_ops;
1110664d4646SSylwester Dziedziuch 	int err = 0;
1111109aba47SJacob Keller 
1112109aba47SJacob Keller 	vlan_ops = ice_get_compat_vsi_vlan_ops(vsi);
1113109aba47SJacob Keller 
1114664d4646SSylwester Dziedziuch 	/* Allow VF with VLAN 0 only to send all tagged traffic */
1115664d4646SSylwester Dziedziuch 	if (vsi->type != ICE_VSI_VF || ice_vsi_has_non_zero_vlans(vsi)) {
1116109aba47SJacob Keller 		err = vlan_ops->ena_tx_filtering(vsi);
1117109aba47SJacob Keller 		if (err)
1118109aba47SJacob Keller 			return err;
1119664d4646SSylwester Dziedziuch 	}
1120109aba47SJacob Keller 
1121109aba47SJacob Keller 	return ice_cfg_mac_antispoof(vsi, true);
1122109aba47SJacob Keller }
1123109aba47SJacob Keller 
1124109aba47SJacob Keller /**
1125109aba47SJacob Keller  * ice_vsi_dis_spoofchk - disable Tx spoof checking for this VSI
1126109aba47SJacob Keller  * @vsi: VSI to disable Tx spoof checking for
1127109aba47SJacob Keller  */
ice_vsi_dis_spoofchk(struct ice_vsi * vsi)1128109aba47SJacob Keller static int ice_vsi_dis_spoofchk(struct ice_vsi *vsi)
1129109aba47SJacob Keller {
1130109aba47SJacob Keller 	struct ice_vsi_vlan_ops *vlan_ops;
1131109aba47SJacob Keller 	int err;
1132109aba47SJacob Keller 
1133109aba47SJacob Keller 	vlan_ops = ice_get_compat_vsi_vlan_ops(vsi);
1134109aba47SJacob Keller 
1135109aba47SJacob Keller 	err = vlan_ops->dis_tx_filtering(vsi);
1136109aba47SJacob Keller 	if (err)
1137109aba47SJacob Keller 		return err;
1138109aba47SJacob Keller 
1139109aba47SJacob Keller 	return ice_cfg_mac_antispoof(vsi, false);
1140109aba47SJacob Keller }
1141109aba47SJacob Keller 
1142109aba47SJacob Keller /**
1143109aba47SJacob Keller  * ice_vsi_apply_spoofchk - Apply Tx spoof checking setting to a VSI
1144109aba47SJacob Keller  * @vsi: VSI associated to the VF
1145109aba47SJacob Keller  * @enable: whether to enable or disable the spoof checking
1146109aba47SJacob Keller  */
ice_vsi_apply_spoofchk(struct ice_vsi * vsi,bool enable)1147109aba47SJacob Keller int ice_vsi_apply_spoofchk(struct ice_vsi *vsi, bool enable)
1148109aba47SJacob Keller {
1149109aba47SJacob Keller 	int err;
1150109aba47SJacob Keller 
1151109aba47SJacob Keller 	if (enable)
1152109aba47SJacob Keller 		err = ice_vsi_ena_spoofchk(vsi);
1153109aba47SJacob Keller 	else
1154109aba47SJacob Keller 		err = ice_vsi_dis_spoofchk(vsi);
1155109aba47SJacob Keller 
1156109aba47SJacob Keller 	return err;
1157109aba47SJacob Keller }
1158109aba47SJacob Keller 
1159109aba47SJacob Keller /**
1160109aba47SJacob Keller  * ice_is_vf_trusted
1161109aba47SJacob Keller  * @vf: pointer to the VF info
1162109aba47SJacob Keller  */
ice_is_vf_trusted(struct ice_vf * vf)1163109aba47SJacob Keller bool ice_is_vf_trusted(struct ice_vf *vf)
1164109aba47SJacob Keller {
1165109aba47SJacob Keller 	return test_bit(ICE_VIRTCHNL_VF_CAP_PRIVILEGE, &vf->vf_caps);
1166109aba47SJacob Keller }
1167109aba47SJacob Keller 
1168109aba47SJacob Keller /**
1169109aba47SJacob Keller  * ice_vf_has_no_qs_ena - check if the VF has any Rx or Tx queues enabled
1170109aba47SJacob Keller  * @vf: the VF to check
1171109aba47SJacob Keller  *
1172109aba47SJacob Keller  * Returns true if the VF has no Rx and no Tx queues enabled and returns false
1173109aba47SJacob Keller  * otherwise
1174109aba47SJacob Keller  */
ice_vf_has_no_qs_ena(struct ice_vf * vf)1175109aba47SJacob Keller bool ice_vf_has_no_qs_ena(struct ice_vf *vf)
1176109aba47SJacob Keller {
1177109aba47SJacob Keller 	return (!bitmap_weight(vf->rxq_ena, ICE_MAX_RSS_QS_PER_VF) &&
1178109aba47SJacob Keller 		!bitmap_weight(vf->txq_ena, ICE_MAX_RSS_QS_PER_VF));
1179109aba47SJacob Keller }
1180109aba47SJacob Keller 
1181109aba47SJacob Keller /**
1182109aba47SJacob Keller  * ice_is_vf_link_up - check if the VF's link is up
1183109aba47SJacob Keller  * @vf: VF to check if link is up
1184109aba47SJacob Keller  */
ice_is_vf_link_up(struct ice_vf * vf)1185109aba47SJacob Keller bool ice_is_vf_link_up(struct ice_vf *vf)
1186109aba47SJacob Keller {
11875a57ee83SJacob Keller 	struct ice_port_info *pi = ice_vf_get_port_info(vf);
1188109aba47SJacob Keller 
11895a57ee83SJacob Keller 	if (ice_check_vf_init(vf))
1190109aba47SJacob Keller 		return false;
1191109aba47SJacob Keller 
1192109aba47SJacob Keller 	if (ice_vf_has_no_qs_ena(vf))
1193109aba47SJacob Keller 		return false;
1194109aba47SJacob Keller 	else if (vf->link_forced)
1195109aba47SJacob Keller 		return vf->link_up;
1196109aba47SJacob Keller 	else
11975a57ee83SJacob Keller 		return pi->phy.link_info.link_info &
1198109aba47SJacob Keller 			ICE_AQ_LINK_UP;
1199109aba47SJacob Keller }
1200109aba47SJacob Keller 
1201109aba47SJacob Keller /**
1202109aba47SJacob Keller  * ice_vf_ctrl_invalidate_vsi - invalidate ctrl_vsi_idx to remove VSI access
1203109aba47SJacob Keller  * @vf: VF that control VSI is being invalidated on
1204109aba47SJacob Keller  */
ice_vf_ctrl_invalidate_vsi(struct ice_vf * vf)1205109aba47SJacob Keller void ice_vf_ctrl_invalidate_vsi(struct ice_vf *vf)
1206109aba47SJacob Keller {
1207109aba47SJacob Keller 	vf->ctrl_vsi_idx = ICE_NO_VSI;
1208109aba47SJacob Keller }
1209109aba47SJacob Keller 
1210109aba47SJacob Keller /**
1211109aba47SJacob Keller  * ice_vf_ctrl_vsi_release - invalidate the VF's control VSI after freeing it
1212109aba47SJacob Keller  * @vf: VF that control VSI is being released on
1213109aba47SJacob Keller  */
ice_vf_ctrl_vsi_release(struct ice_vf * vf)1214109aba47SJacob Keller void ice_vf_ctrl_vsi_release(struct ice_vf *vf)
1215109aba47SJacob Keller {
1216109aba47SJacob Keller 	ice_vsi_release(vf->pf->vsi[vf->ctrl_vsi_idx]);
1217109aba47SJacob Keller 	ice_vf_ctrl_invalidate_vsi(vf);
1218109aba47SJacob Keller }
1219109aba47SJacob Keller 
1220109aba47SJacob Keller /**
1221109aba47SJacob Keller  * ice_vf_ctrl_vsi_setup - Set up a VF control VSI
1222109aba47SJacob Keller  * @vf: VF to setup control VSI for
1223109aba47SJacob Keller  *
1224109aba47SJacob Keller  * Returns pointer to the successfully allocated VSI struct on success,
1225109aba47SJacob Keller  * otherwise returns NULL on failure.
1226109aba47SJacob Keller  */
ice_vf_ctrl_vsi_setup(struct ice_vf * vf)1227109aba47SJacob Keller struct ice_vsi *ice_vf_ctrl_vsi_setup(struct ice_vf *vf)
1228109aba47SJacob Keller {
12295e509ab2SJacob Keller 	struct ice_vsi_cfg_params params = {};
1230109aba47SJacob Keller 	struct ice_pf *pf = vf->pf;
1231109aba47SJacob Keller 	struct ice_vsi *vsi;
1232109aba47SJacob Keller 
12335e509ab2SJacob Keller 	params.type = ICE_VSI_CTRL;
12345e509ab2SJacob Keller 	params.pi = ice_vf_get_port_info(vf);
12355e509ab2SJacob Keller 	params.vf = vf;
12365e509ab2SJacob Keller 	params.flags = ICE_VSI_FLAG_INIT;
12375e509ab2SJacob Keller 
12385e509ab2SJacob Keller 	vsi = ice_vsi_setup(pf, &params);
1239109aba47SJacob Keller 	if (!vsi) {
1240109aba47SJacob Keller 		dev_err(ice_pf_to_dev(pf), "Failed to create VF control VSI\n");
1241109aba47SJacob Keller 		ice_vf_ctrl_invalidate_vsi(vf);
1242109aba47SJacob Keller 	}
1243109aba47SJacob Keller 
1244109aba47SJacob Keller 	return vsi;
1245109aba47SJacob Keller }
1246109aba47SJacob Keller 
1247109aba47SJacob Keller /**
1248b1b56942SJacob Keller  * ice_vf_init_host_cfg - Initialize host admin configuration
1249b1b56942SJacob Keller  * @vf: VF to initialize
1250b1b56942SJacob Keller  * @vsi: the VSI created at initialization
1251b1b56942SJacob Keller  *
1252b1b56942SJacob Keller  * Initialize the VF host configuration. Called during VF creation to setup
1253b1b56942SJacob Keller  * VLAN 0, add the VF VSI broadcast filter, and setup spoof checking. It
1254b1b56942SJacob Keller  * should only be called during VF creation.
1255b1b56942SJacob Keller  */
ice_vf_init_host_cfg(struct ice_vf * vf,struct ice_vsi * vsi)1256b1b56942SJacob Keller int ice_vf_init_host_cfg(struct ice_vf *vf, struct ice_vsi *vsi)
1257b1b56942SJacob Keller {
1258b1b56942SJacob Keller 	struct ice_vsi_vlan_ops *vlan_ops;
1259b1b56942SJacob Keller 	struct ice_pf *pf = vf->pf;
1260b1b56942SJacob Keller 	u8 broadcast[ETH_ALEN];
1261b1b56942SJacob Keller 	struct device *dev;
1262b1b56942SJacob Keller 	int err;
1263b1b56942SJacob Keller 
1264b1b56942SJacob Keller 	dev = ice_pf_to_dev(pf);
1265b1b56942SJacob Keller 
1266b1b56942SJacob Keller 	err = ice_vsi_add_vlan_zero(vsi);
1267b1b56942SJacob Keller 	if (err) {
1268b1b56942SJacob Keller 		dev_warn(dev, "Failed to add VLAN 0 filter for VF %d\n",
1269b1b56942SJacob Keller 			 vf->vf_id);
1270b1b56942SJacob Keller 		return err;
1271b1b56942SJacob Keller 	}
1272b1b56942SJacob Keller 
1273b1b56942SJacob Keller 	vlan_ops = ice_get_compat_vsi_vlan_ops(vsi);
1274b1b56942SJacob Keller 	err = vlan_ops->ena_rx_filtering(vsi);
1275b1b56942SJacob Keller 	if (err) {
1276b1b56942SJacob Keller 		dev_warn(dev, "Failed to enable Rx VLAN filtering for VF %d\n",
1277b1b56942SJacob Keller 			 vf->vf_id);
1278b1b56942SJacob Keller 		return err;
1279b1b56942SJacob Keller 	}
1280b1b56942SJacob Keller 
1281b1b56942SJacob Keller 	eth_broadcast_addr(broadcast);
1282b1b56942SJacob Keller 	err = ice_fltr_add_mac(vsi, broadcast, ICE_FWD_TO_VSI);
1283b1b56942SJacob Keller 	if (err) {
1284b1b56942SJacob Keller 		dev_err(dev, "Failed to add broadcast MAC filter for VF %d, status %d\n",
1285b1b56942SJacob Keller 			vf->vf_id, err);
1286b1b56942SJacob Keller 		return err;
1287b1b56942SJacob Keller 	}
1288b1b56942SJacob Keller 
1289b1b56942SJacob Keller 	vf->num_mac = 1;
1290b1b56942SJacob Keller 
1291b1b56942SJacob Keller 	err = ice_vsi_apply_spoofchk(vsi, vf->spoofchk);
1292b1b56942SJacob Keller 	if (err) {
1293b1b56942SJacob Keller 		dev_warn(dev, "Failed to initialize spoofchk setting for VF %d\n",
1294b1b56942SJacob Keller 			 vf->vf_id);
1295b1b56942SJacob Keller 		return err;
1296b1b56942SJacob Keller 	}
1297b1b56942SJacob Keller 
1298b1b56942SJacob Keller 	return 0;
1299b1b56942SJacob Keller }
1300b1b56942SJacob Keller 
1301b1b56942SJacob Keller /**
1302109aba47SJacob Keller  * ice_vf_invalidate_vsi - invalidate vsi_idx/vsi_num to remove VSI access
1303109aba47SJacob Keller  * @vf: VF to remove access to VSI for
1304109aba47SJacob Keller  */
ice_vf_invalidate_vsi(struct ice_vf * vf)1305109aba47SJacob Keller void ice_vf_invalidate_vsi(struct ice_vf *vf)
1306109aba47SJacob Keller {
1307109aba47SJacob Keller 	vf->lan_vsi_idx = ICE_NO_VSI;
1308109aba47SJacob Keller 	vf->lan_vsi_num = ICE_NO_VSI;
1309109aba47SJacob Keller }
1310109aba47SJacob Keller 
1311109aba47SJacob Keller /**
13121efee073SJacob Keller  * ice_vf_vsi_release - Release the VF VSI and invalidate indexes
13131efee073SJacob Keller  * @vf: pointer to the VF structure
13141efee073SJacob Keller  *
13151efee073SJacob Keller  * Release the VF associated with this VSI and then invalidate the VSI
13161efee073SJacob Keller  * indexes.
13171efee073SJacob Keller  */
ice_vf_vsi_release(struct ice_vf * vf)13181efee073SJacob Keller void ice_vf_vsi_release(struct ice_vf *vf)
13191efee073SJacob Keller {
13201efee073SJacob Keller 	struct ice_vsi *vsi = ice_get_vf_vsi(vf);
13211efee073SJacob Keller 
13221efee073SJacob Keller 	if (WARN_ON(!vsi))
13231efee073SJacob Keller 		return;
13241efee073SJacob Keller 
13251efee073SJacob Keller 	ice_vsi_release(vsi);
13261efee073SJacob Keller 	ice_vf_invalidate_vsi(vf);
13271efee073SJacob Keller }
13281efee073SJacob Keller 
13291efee073SJacob Keller /**
1330369bb5a2SPiotr Raczynski  * ice_get_vf_ctrl_vsi - Get first VF control VSI pointer
1331369bb5a2SPiotr Raczynski  * @pf: the PF private structure
1332369bb5a2SPiotr Raczynski  * @vsi: pointer to the VSI
1333369bb5a2SPiotr Raczynski  *
1334369bb5a2SPiotr Raczynski  * Return first found VF control VSI other than the vsi
1335369bb5a2SPiotr Raczynski  * passed by parameter. This function is used to determine
1336369bb5a2SPiotr Raczynski  * whether new resources have to be allocated for control VSI
1337369bb5a2SPiotr Raczynski  * or they can be shared with existing one.
1338369bb5a2SPiotr Raczynski  *
1339369bb5a2SPiotr Raczynski  * Return found VF control VSI pointer other itself. Return
1340369bb5a2SPiotr Raczynski  * NULL Otherwise.
1341369bb5a2SPiotr Raczynski  *
1342369bb5a2SPiotr Raczynski  */
ice_get_vf_ctrl_vsi(struct ice_pf * pf,struct ice_vsi * vsi)1343369bb5a2SPiotr Raczynski struct ice_vsi *ice_get_vf_ctrl_vsi(struct ice_pf *pf, struct ice_vsi *vsi)
1344369bb5a2SPiotr Raczynski {
1345369bb5a2SPiotr Raczynski 	struct ice_vsi *ctrl_vsi = NULL;
1346369bb5a2SPiotr Raczynski 	struct ice_vf *vf;
1347369bb5a2SPiotr Raczynski 	unsigned int bkt;
1348369bb5a2SPiotr Raczynski 
1349369bb5a2SPiotr Raczynski 	rcu_read_lock();
1350369bb5a2SPiotr Raczynski 	ice_for_each_vf_rcu(pf, bkt, vf) {
1351369bb5a2SPiotr Raczynski 		if (vf != vsi->vf && vf->ctrl_vsi_idx != ICE_NO_VSI) {
1352369bb5a2SPiotr Raczynski 			ctrl_vsi = pf->vsi[vf->ctrl_vsi_idx];
1353369bb5a2SPiotr Raczynski 			break;
1354369bb5a2SPiotr Raczynski 		}
1355369bb5a2SPiotr Raczynski 	}
1356369bb5a2SPiotr Raczynski 
1357369bb5a2SPiotr Raczynski 	rcu_read_unlock();
1358369bb5a2SPiotr Raczynski 	return ctrl_vsi;
1359369bb5a2SPiotr Raczynski }
1360