Searched hist:"11 f3f500ec8a75c96087f3bed87aa2b1c5de7498" (Results 1 – 2 of 2) sorted by relevance
/openbmc/linux/include/linux/sched/ |
H A D | task.h | diff 11f3f500ec8a75c96087f3bed87aa2b1c5de7498 Fri Mar 10 16:03:27 CST 2023 Mike Christie <michael.christie@oracle.com> fork: add kernel_clone_args flag to not dup/clone files
Each vhost device gets a thread that is used to perform IO and management operations. Instead of a thread that is accessing a device, the thread is part of the device, so when it creates a thread using a helper based on copy_process we can't dup or clone the parent's files/FDS because it would do an extra increment on ourself.
Later, when we do:
Qemu process exits: do_exit -> exit_files -> put_files_struct -> close_files
we would leak the device's resources because of that extra refcount on the fd or file_struct.
This patch adds a no_files option so these worker threads can prevent taking an extra refcount on themselves.
Signed-off-by: Mike Christie <michael.christie@oracle.com> Acked-by: Christian Brauner <brauner@kernel.org> Acked-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Christian Brauner (Microsoft) <brauner@kernel.org>
|
/openbmc/linux/kernel/ |
H A D | fork.c | diff 11f3f500ec8a75c96087f3bed87aa2b1c5de7498 Fri Mar 10 16:03:27 CST 2023 Mike Christie <michael.christie@oracle.com> fork: add kernel_clone_args flag to not dup/clone files
Each vhost device gets a thread that is used to perform IO and management operations. Instead of a thread that is accessing a device, the thread is part of the device, so when it creates a thread using a helper based on copy_process we can't dup or clone the parent's files/FDS because it would do an extra increment on ourself.
Later, when we do:
Qemu process exits: do_exit -> exit_files -> put_files_struct -> close_files
we would leak the device's resources because of that extra refcount on the fd or file_struct.
This patch adds a no_files option so these worker threads can prevent taking an extra refcount on themselves.
Signed-off-by: Mike Christie <michael.christie@oracle.com> Acked-by: Christian Brauner <brauner@kernel.org> Acked-by: Michael S. Tsirkin <mst@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Christian Brauner (Microsoft) <brauner@kernel.org>
|