On 11.10.21 15:32, Christian Brauner wrote:
From: Christian Brauner <christian.brauner@xxxxxxxxxx> Instead of duplicating the same code in two places use the newly added pidfd_get_task() helper. This fixes an (unimportant for now) bug where PIDTYPE_PID is used whereas PIDTYPE_TGID should have been used.
What would have been the effect of the BUG? Is it worth Fixes: or better even separating out the fix?
Link: https://lore.kernel.org/r/20211004125050.1153693-3-christian.brauner@xxxxxxxxxx Cc: Vlastimil Babka <vbabka@xxxxxxx> Cc: Suren Baghdasaryan <surenb@xxxxxxxxxx> Cc: Matthew Bobrowski <repnop@xxxxxxxxxx> Cc: Alexander Duyck <alexander.h.duyck@xxxxxxxxxxxxxxx> Cc: David Hildenbrand <david@xxxxxxxxxx> Cc: Jan Kara <jack@xxxxxxx> Cc: Minchan Kim <minchan@xxxxxxxxxx> Reviewed-by: Matthew Bobrowski <repnop@xxxxxxxxxx> Signed-off-by: Christian Brauner <christian.brauner@xxxxxxxxxx> --- /* v2 */ unchanged
Acked-by: David Hildenbrand <david@xxxxxxxxxx> -- Thanks, David / dhildenb