Re: [net-next v3 05/15] net: cavium/liquidio: Convert tasklet API to new bottom half workqueue mechanism

From: Jakub Kicinski
Date: Wed Jul 31 2024 - 22:24:27 EST


On Tue, 30 Jul 2024 11:33:53 -0700 Allen Pais wrote:
> - tasklet_enable(&oct_priv->droq_tasklet);
> + enable_and_queue_work(system_bh_wq, &oct_priv->droq_bh_work);
>
> if (atomic_read(&lio->ifstate) & LIO_IFSTATE_REGISTERED)
> unregister_netdev(netdev);

> if (OCTEON_CN23XX_PF(oct))
> oct->droq[0]->ops.poll_mode = 0;
>
> - tasklet_enable(&oct_priv->droq_tasklet);
> + enable_and_queue_work(system_bh_wq, &oct_priv->droq_bh_work);

Could you shed some light in the cover letter or this patch why
tasklet_enable() is converted to enable_and_queue_work() at
the face of it those two do not appear to do the same thing?

I'll apply patches 1-4 already.