On Fri, 11 Aug 2023 12:05:35 +0200 Simon Horman wrote:

quoted

 	if (status) {
 		pr_err("failed to register PCI driver, err %d\n", status);
 		goto err_dest_lag_wq;
+		destroy_workqueue(ice_wq);
+		ice_debugfs_exit();  

Hi Paul and Tony,

this new code seems to be unreachable.
Should it go before the goto statement?

:o catch of the week! :)
-- 
pw-bot: cr