On Mon, Nov 25, 2024 at 11:22 PM Mike Christie
[off-list ref] wrote:

On 11/5/24 1:25 AM, Cindy Lu wrote:

quoted

+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

quoted

+
+     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)
 {