[PATCH][next] ice: remove redundant assignment to pointer vsi

Subsystems: intel ethernet drivers, networking drivers, the rest

STALE2407d

3 messages, 3 authors, 2020-02-26 · open the first message on its own page

[PATCH][next] ice: remove redundant assignment to pointer vsi

From: Colin King <hidden>
Date: 2020-02-22 17:11:04

From: Colin Ian King <redacted>

Pointer vsi is being re-assigned a value that is never read,
the assignment is redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <redacted>
---
 drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
index 169b72a94b9d..2c8e334c47a0 100644
--- a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
@@ -2191,7 +2191,6 @@ static int ice_vc_ena_qs_msg(struct ice_vf *vf, u8 *msg)
 		set_bit(vf_q_id, vf->rxq_ena);
 	}
 
-	vsi = pf->vsi[vf->lan_vsi_idx];
 	q_map = vqs->tx_queues;
 	for_each_set_bit(vf_q_id, &q_map, ICE_MAX_BASE_QS_PER_VF) {
 		if (!ice_vc_isvalid_q_id(vf, vqs->vsi_id, vf_q_id)) {
-- 
2.25.0

RE: [Intel-wired-lan] [PATCH][next] ice: remove redundant assignment to pointer vsi

From: Allan, Bruce W <hidden>
Date: 2020-02-25 19:16:40

-----Original Message-----
From: Intel-wired-lan <redacted> On Behalf Of
Colin King
Sent: Saturday, February 22, 2020 9:11 AM
To: Kirsher, Jeffrey T <redacted>; David S . Miller
[off-list ref]; intel-wired-lan@lists.osuosl.org;
netdev@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: [Intel-wired-lan] [PATCH][next] ice: remove redundant assignment to
pointer vsi

From: Colin Ian King <redacted>

Pointer vsi is being re-assigned a value that is never read,
the assignment is redundant and can be removed.

Addresses-Coverity: ("Unused value")
What version of coverity and what options were used?
quoted hunk
Signed-off-by: Colin Ian King <redacted>
---
 drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
index 169b72a94b9d..2c8e334c47a0 100644
--- a/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
+++ b/drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c
@@ -2191,7 +2191,6 @@ static int ice_vc_ena_qs_msg(struct ice_vf *vf, u8
*msg)
 		set_bit(vf_q_id, vf->rxq_ena);
 	}

-	vsi = pf->vsi[vf->lan_vsi_idx];
 	q_map = vqs->tx_queues;
 	for_each_set_bit(vf_q_id, &q_map, ICE_MAX_BASE_QS_PER_VF) {
 		if (!ice_vc_isvalid_q_id(vf, vqs->vsi_id, vf_q_id)) {
--
2.25.0

_______________________________________________
Intel-wired-lan mailing list
Intel-wired-lan@osuosl.org
https://lists.osuosl.org/mailman/listinfo/intel-wired-lan

RE: [Intel-wired-lan] [PATCH][next] ice: remove redundant assignment to pointer vsi

From: Bowers, AndrewX <hidden>
Date: 2020-02-26 22:06:33

-----Original Message-----
From: Intel-wired-lan [mailto:intel-wired-lan-bounces@osuosl.org] On
Behalf Of Colin King
Sent: Saturday, February 22, 2020 9:11 AM
To: Kirsher, Jeffrey T <redacted>; David S . Miller
[off-list ref]; intel-wired-lan@lists.osuosl.org;
netdev@vger.kernel.org
Cc: kernel-janitors@vger.kernel.org; linux-kernel@vger.kernel.org
Subject: [Intel-wired-lan] [PATCH][next] ice: remove redundant assignment
to pointer vsi

From: Colin Ian King <redacted>

Pointer vsi is being re-assigned a value that is never read, the assignment is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <redacted>
---
 drivers/net/ethernet/intel/ice/ice_virtchnl_pf.c | 1 -
 1 file changed, 1 deletion(-)
Tested-by: Andrew Bowers <redacted>
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help