Re: [PATCH v2] bpf: fix: Race condition in bpf_trampoline_unlink_cgroup_shim

[Martin KaFai Lau <martin.lau@linux.dev>]

> 
>> +			/* Reusing existing shim attached by the other program. */
>> +			mutex_unlock(&tr->mutex);
>> +			bpf_trampoline_put(tr); /* bpf_trampoline_get above */
>> +			return 0;
>> +		}
>>   	}
> 
> This commit fixes a use-after-free introduced by commit 69fd337a975c
> ("bpf: per-cgroup lsm flavor"), which added the bpf_link_inc() call
> that could resurrect a zero-refcount link.
> 
> Should this include a Fixes: tag?
> 
> Suggested:
> Fixes: 69fd337a975c ("bpf: per-cgroup lsm flavor")

On top of the Fixes tag, please add bpf to the subject, e.g. "[PATCH bpf 
v3] bpf: Fix race condition...". Please also use the 
FirstName[[:space:]]LastName <email> format in the "Signed-off-by". For 
example: Lang Xu <xulang@uniontech.com>. Understood that there are 
different name formatting but the reviewers can't possibly know all of 
them, so it is useful to write it in a way that most people in the list 
can understand.