On Mon, Nov 25, 2024 at 11:22 PM Mike Christie <michael.christie@xxxxxxxxxx> wrote: > > On 11/5/24 1:25 AM, Cindy Lu wrote: > > +static int vhost_attach_cgroups(struct vhost_dev *dev) > > +{ > > + struct vhost_worker *worker; > > + unsigned long i; > > + int ret; > > + > > + /* > > + * Free the default worker we created and cleanup workers userspace > > + * created but couldn't clean up (it forgot or crashed). > > + */ > > I think this comment got added here by accident. > will remove this Thanks Cindy > > + > > + xa_for_each(&dev->worker_xa, i, worker) { > > + ret = vhost_worker_cgroups_kthread(worker); > > + if (ret) > > + return ret; > > + } > > + return ret; > > +} > > + > > /* Caller should have device mutex */ > > bool vhost_dev_has_owner(struct vhost_dev *dev) > > { >