Re: [PATCH v4 1/6] netlink: Reverse the patch which removed filtering

From: Anjali Kulkarni
Date: Thu Jun 01 2023 - 12:50:22 EST




> On Jun 1, 2023, at 9:43 AM, Jakub Kicinski <kuba@xxxxxxxxxx> wrote:
>
> On Thu, 1 Jun 2023 16:34:08 +0000 Anjali Kulkarni wrote:
>>> The code may have security implications, I really don't feel like I can
>>> be the sole reviewer. There's a bunch of experts working at Oracle,
>>> maybe you could get one of them to put their name on it? I can apply
>>> the patches, I just want to be sure I'm not the _only_ reviewer.
>>
>> Thanks so much for your response. There is someone at Oracle who
>> looked at this some time ago and is familiar enough with this to
>> review the code - but he is not a kernel committer - he sends
>> occasional patches upstream which get committed - would it be ok if
>> he reviewed it along with you and then you could commit it? If you
>> know of someone from Oracle who could also potentially review it,
>> please let me know.
>
> I meant someone seasoned. IMHO one of the benefits of employing
> upstream experts for corporation like Oracle should be that you
> can lean on them for reviews:
>
> $ git log --format='%ae' --author='Oracle' --since='2 years ago' | sort | uniq -c | sort -rn
> 811 willy@xxxxxxxxxxxxx
> 312 rmk+kernel@xxxxxxxxxxxxxxx
> 91 Liam.Howlett@xxxxxxxxxx
> 60 vishal.moola@xxxxxxxxx
>
> $ git log --format='%ae' --author='@oracle.com' --since='2 years ago' | sort | uniq -c | sort -rn | head -10
> 451 chuck.lever@xxxxxxxxxx
> 154 michael.christie@xxxxxxxxxx
> 118 nick.alcock@xxxxxxxxxx
> 71 martin.petersen@xxxxxxxxxx
> 59 mike.kravetz@xxxxxxxxxx
> 58 sidhartha.kumar@xxxxxxxxxx
> 55 liam.howlett@xxxxxxxxxx
> 53 anand.jain@xxxxxxxxxx
> 32 dai.ngo@xxxxxxxxxx
> 32 allison.henderson@xxxxxxxxxx

Thanks, let me check.
Anjali