On 2023/8/10 6:10, Martin KaFai Lau wrote:
> On 8/9/23 7:05 AM, Yue Haibing wrote:
>> Commit a3b80e107894 ("bpf: Allocate ID for bpf_link")
>> removed the implementation but not the declaration.
>
> It is good to remove unimplemented function. However, how many more of this you have already discovered? Unless it is like hundred line changes, can you please batch them all together and send them in one patch?
Ok, glad to follow this rule. For bpf I only found this one left now.
>
>>
>> Signed-off-by: Yue Haibing <yuehaibing@xxxxxxxxxx>
>> ---
>> include/linux/bpf.h | 1 -
>> 1 file changed, 1 deletion(-)
>>
>> diff --git a/include/linux/bpf.h b/include/linux/bpf.h
>> index db3fe5a61b05..cfabbcf47bdb 100644
>> --- a/include/linux/bpf.h
>> +++ b/include/linux/bpf.h
>> @@ -2120,7 +2120,6 @@ void bpf_link_cleanup(struct bpf_link_primer *primer);
>> void bpf_link_inc(struct bpf_link *link);
>> void bpf_link_put(struct bpf_link *link);
>> int bpf_link_new_fd(struct bpf_link *link);
>> -struct file *bpf_link_new_file(struct bpf_link *link, int *reserved_fd);
>> struct bpf_link *bpf_link_get_from_fd(u32 ufd);
>> struct bpf_link *bpf_link_get_curr_or_next(u32 *id);
>>
>
> .
