On Wed, Jun 17, 2020 at 08:30:27PM +0200, Daniel Glöckner wrote:
> Hello Sascha,
>
> Am 17.06.20 um 10:11 schrieb Sascha Hauer:
> > +static inline void wq_cancel_work(struct work_queue *wq)
> > +{
> > + struct work_struct *work, *tmp;
> > +
> > + list_for_each_entry_safe(work, tmp, &wq->work, list) {
> > + list_del(&work->list);
> > + wq->cancel(work);
> > + }
> > +}
>
> Why is this function in the header?
> So far it is called only from wq_unregister.
Right, we can move it there. I think my original intention was that
the user calls wq_cancel_work() before calling wq_unregister().
Regards,
Sascha
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox
