Skip to content

Commit 152d84f

Browse files
committed
idpf: remove dealloc vector msg err in idpf_intr_rel
JIRA: https://issues.redhat.com/browse/RHEL-29553 commit 6009e63 Author: Alan Brady <alan.brady@intel.com> Date: Thu Feb 22 11:04:41 2024 -0800 idpf: remove dealloc vector msg err in idpf_intr_rel This error message is at best not really helpful and at worst misleading. If we're here in idpf_intr_rel we're likely trying to do remove or reset. If we're in reset, this message will fail because we lose the virtchnl on reset and HW is going to clean up those resources regardless in that case. If we're in remove and we get an error here, we're going to reset the device at the end of remove anyway so not a big deal. Just remove this message it's not useful. Tested-by: Alexander Lobakin <aleksander.lobakin@intel.com> Signed-off-by: Alan Brady <alan.brady@intel.com> Tested-by: Krishneil Singh <krishneil.k.singh@intel.com> Signed-off-by: Tony Nguyen <anthony.l.nguyen@intel.com> Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
1 parent 3ad641e commit 152d84f

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

drivers/net/ethernet/intel/idpf/idpf_lib.c

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -79,19 +79,12 @@ static void idpf_mb_intr_rel_irq(struct idpf_adapter *adapter)
7979
*/
8080
void idpf_intr_rel(struct idpf_adapter *adapter)
8181
{
82-
int err;
83-
8482
if (!adapter->msix_entries)
8583
return;
8684

8785
idpf_mb_intr_rel_irq(adapter);
8886
pci_free_irq_vectors(adapter->pdev);
89-
90-
err = idpf_send_dealloc_vectors_msg(adapter);
91-
if (err)
92-
dev_err(&adapter->pdev->dev,
93-
"Failed to deallocate vectors: %d\n", err);
94-
87+
idpf_send_dealloc_vectors_msg(adapter);
9588
idpf_deinit_vector_stack(adapter);
9689
kfree(adapter->msix_entries);
9790
adapter->msix_entries = NULL;

0 commit comments

Comments
 (0)