diff mbox

[RFC,v2,net-next,02/16] bpf: update MAINTAINERS entry

Message ID 1405657206-12060-3-git-send-email-ast@plumgrid.com
State RFC, archived
Delegated to: David Miller
Headers show

Commit Message

Alexei Starovoitov July 18, 2014, 4:19 a.m. UTC
Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
---
 MAINTAINERS |    7 +++++++
 1 file changed, 7 insertions(+)

Comments

Kees Cook July 23, 2014, 5:37 p.m. UTC | #1
On Thu, Jul 17, 2014 at 9:19 PM, Alexei Starovoitov <ast@plumgrid.com> wrote:
> Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
> ---
>  MAINTAINERS |    7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index ae8cd00215b2..32e24ff46da3 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -1912,6 +1912,13 @@ S:       Supported
>  F:     drivers/net/bonding/
>  F:     include/uapi/linux/if_bonding.h
>
> +BPF (Safe dynamic programs and tools)

bikeshed: I feel like this shouldn't be an acronym. Maybe instead:

BERKELEY PACKET FILTER (BPF: Safe dynamic programs and tools)

-Kees

> +M:     Alexei Starovoitov <ast@kernel.org>
> +L:     netdev@vger.kernel.org
> +L:     linux-kernel@vger.kernel.org
> +S:     Supported
> +F:     kernel/bpf/
> +
>  BROADCOM B44 10/100 ETHERNET DRIVER
>  M:     Gary Zambrano <zambrano@broadcom.com>
>  L:     netdev@vger.kernel.org
> --
> 1.7.9.5
>
Alexei Starovoitov July 23, 2014, 5:48 p.m. UTC | #2
On Wed, Jul 23, 2014 at 10:37 AM, Kees Cook <keescook@chromium.org> wrote:
> On Thu, Jul 17, 2014 at 9:19 PM, Alexei Starovoitov <ast@plumgrid.com> wrote:
>> Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
>> ---
>>  MAINTAINERS |    7 +++++++
>>  1 file changed, 7 insertions(+)
>>
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index ae8cd00215b2..32e24ff46da3 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -1912,6 +1912,13 @@ S:       Supported
>>  F:     drivers/net/bonding/
>>  F:     include/uapi/linux/if_bonding.h
>>
>> +BPF (Safe dynamic programs and tools)
>
> bikeshed: I feel like this shouldn't be an acronym. Maybe instead:
>
> BERKELEY PACKET FILTER (BPF: Safe dynamic programs and tools)

pile on :)

I think eBPF is no longer acronym. 'e' stands for 'extended',
but BPF is no longer 'packet filter' only and definitely not 'berkeley'.
So I'd rather keep BPF as a magic abbreviation without spelling it out,
since full name is historic and no longer meaningful.
I've considered coming up with brand new abbreviation and full name
for this instruction set, but none looked good and all lose in comparison
to 'eBPF' name, which is concise and carries enough historical
references to explain the idea behind new ISA.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Kees Cook July 23, 2014, 6:39 p.m. UTC | #3
On Wed, Jul 23, 2014 at 10:48 AM, Alexei Starovoitov <ast@plumgrid.com> wrote:
> On Wed, Jul 23, 2014 at 10:37 AM, Kees Cook <keescook@chromium.org> wrote:
>> On Thu, Jul 17, 2014 at 9:19 PM, Alexei Starovoitov <ast@plumgrid.com> wrote:
>>> Signed-off-by: Alexei Starovoitov <ast@plumgrid.com>
>>> ---
>>>  MAINTAINERS |    7 +++++++
>>>  1 file changed, 7 insertions(+)
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index ae8cd00215b2..32e24ff46da3 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -1912,6 +1912,13 @@ S:       Supported
>>>  F:     drivers/net/bonding/
>>>  F:     include/uapi/linux/if_bonding.h
>>>
>>> +BPF (Safe dynamic programs and tools)
>>
>> bikeshed: I feel like this shouldn't be an acronym. Maybe instead:
>>
>> BERKELEY PACKET FILTER (BPF: Safe dynamic programs and tools)
>
> pile on :)
>
> I think eBPF is no longer acronym. 'e' stands for 'extended',
> but BPF is no longer 'packet filter' only and definitely not 'berkeley'.
> So I'd rather keep BPF as a magic abbreviation without spelling it out,
> since full name is historic and no longer meaningful.
> I've considered coming up with brand new abbreviation and full name
> for this instruction set, but none looked good and all lose in comparison
> to 'eBPF' name, which is concise and carries enough historical
> references to explain the idea behind new ISA.

Yeah, that's a fair point. No sense in using "BEE PEE EFF" :)

-Kees
diff mbox

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index ae8cd00215b2..32e24ff46da3 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -1912,6 +1912,13 @@  S:	Supported
 F:	drivers/net/bonding/
 F:	include/uapi/linux/if_bonding.h
 
+BPF (Safe dynamic programs and tools)
+M:	Alexei Starovoitov <ast@kernel.org>
+L:	netdev@vger.kernel.org
+L:	linux-kernel@vger.kernel.org
+S:	Supported
+F:	kernel/bpf/
+
 BROADCOM B44 10/100 ETHERNET DRIVER
 M:	Gary Zambrano <zambrano@broadcom.com>
 L:	netdev@vger.kernel.org