On 6/16/2023 11:50 PM, Jakub Kicinski wrote:

On Wed, 14 Jun 2023 10:14:17 -0700 Tony Nguyen wrote:

quoted

+ * @IDPF_REL_RES_IN_PROG: Resources release in progress

quoted

+ * @IDPF_CANCEL_SERVICE_TASK: Do not schedule service task if bit is set
+ * @IDPF_REMOVE_IN_PROG: Driver remove in progress

Why all the X-in-progress flags, again?

IDPF_REMOVE_IN_PROG is used for the conditional checks to indicate that 
the rmmod is in progress and there is no scope to process any hardware 
resets if the bit is set.

IDPF_HR_RESET_IN_PROG is used on hardware reset to not send any 
statistics message to the device Control Plane and to avoid releasing 
the configured netdevs if the bit is set.

After taking a deeper look, IDPF_REL_RES_IN_PROG does not serve the 
purpose as IDPF_REMOVE_IN_PROG already takes care of it. Will cleanup 
this flag.

quoted

+	set_bit(IDPF_CANCEL_SERVICE_TASK, adapter->flags);
+	cancel_delayed_work_sync(&adapter->serv_task);
+	clear_bit(IDPF_CANCEL_SERVICE_TASK, adapter->flags);

Pretty sure workqueue protects from self-requeueing.

Agree with you. Will remove both CANCEL_STATS_TASK and 
CANCEL_SERVICE_TASK flags.

Thanks,
Pavan