Quoting Tvrtko Ursulin (2019-12-16 12:07:01) > +static void i915_gem_remove_client(struct drm_i915_file_private *file_priv) > +{ > + struct i915_drm_clients *clients = &file_priv->dev_priv->clients; > + struct i915_drm_client *client = &file_priv->client; > + > + if (!client->name) > + return; /* intel_fbdev_init registers a client before sysfs */ > + > + sysfs_remove_file(client->root, (struct attribute *)&client->attr.pid); > + sysfs_remove_file(client->root, (struct attribute *)&client->attr.name); > + kobject_put(client->root); Do we need to remove individual files if we unplug the root? sysfs_remove_dir(client->root) ? -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx