[PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[Dave Thaler]

The proposed intro paragraph text is derived from the first paragraph
of the IETF BPF WG charter at https://datatracker.ietf.org/wg/bpf/about/

Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
---
 Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Documentation/bpf/standardization/instruction-set.rst b/Documentation/bpf/standardization/instruction-set.rst
index d03d90afb..b44bdacd0 100644
--- a/Documentation/bpf/standardization/instruction-set.rst
+++ b/Documentation/bpf/standardization/instruction-set.rst
@@ -5,7 +5,11 @@
 BPF Instruction Set Architecture (ISA)
 ======================================
 
-This document specifies the BPF instruction set architecture (ISA).
+eBPF (which is no longer an acronym for anything), also commonly
+referred to as BPF, is a technology with origins in the Linux kernel
+that can run untrusted programs in a privileged context such as an
+operating system kernel. This document specifies the BPF instruction
+set architecture (ISA).
 
 Documentation conventions
 =========================
-- 
2.40.1

[Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[Dave Thaler]

The proposed intro paragraph text is derived from the first paragraph
of the IETF BPF WG charter at https://datatracker.ietf.org/wg/bpf/about/

Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
---
 Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Documentation/bpf/standardization/instruction-set.rst b/Documentation/bpf/standardization/instruction-set.rst
index d03d90afb..b44bdacd0 100644
--- a/Documentation/bpf/standardization/instruction-set.rst
+++ b/Documentation/bpf/standardization/instruction-set.rst
@@ -5,7 +5,11 @@
 BPF Instruction Set Architecture (ISA)
 ======================================
 
-This document specifies the BPF instruction set architecture (ISA).
+eBPF (which is no longer an acronym for anything), also commonly
+referred to as BPF, is a technology with origins in the Linux kernel
+that can run untrusted programs in a privileged context such as an
+operating system kernel. This document specifies the BPF instruction
+set architecture (ISA).
 
 Documentation conventions
 =========================
-- 
2.40.1

-- 
Bpf mailing list
Bpf@ietf.org
https://www.ietf.org/mailman/listinfo/bpf

Re: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[David Vernet]

[-- Attachment #1: Type: text/plain, Size: 1659 bytes --]

On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> The proposed intro paragraph text is derived from the first paragraph
> of the IETF BPF WG charter at https://datatracker.ietf.org/wg/bpf/about/
> 
> Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> ---
>  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/bpf/standardization/instruction-set.rst b/Documentation/bpf/standardization/instruction-set.rst
> index d03d90afb..b44bdacd0 100644
> --- a/Documentation/bpf/standardization/instruction-set.rst
> +++ b/Documentation/bpf/standardization/instruction-set.rst
> @@ -5,7 +5,11 @@
>  BPF Instruction Set Architecture (ISA)
>  ======================================
>  
> -This document specifies the BPF instruction set architecture (ISA).
> +eBPF (which is no longer an acronym for anything), also commonly
> +referred to as BPF, is a technology with origins in the Linux kernel
> +that can run untrusted programs in a privileged context such as an

Perhaps this should be phrased as:

...that can run untrusted programs in privileged contexts such as the
operating system kernel.

Not sure if that's actually a grammar correction but it sounds more
correct in my head. Wdyt?

Regardless:

Acked-by: David Vernet <void@manifault.com>

> +operating system kernel. This document specifies the BPF instruction
> +set architecture (ISA).
>  
>  Documentation conventions
>  =========================
> -- 
> 2.40.1
> 
> -- 
> Bpf mailing list
> Bpf@ietf.org
> https://www.ietf.org/mailman/listinfo/bpf

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[David Vernet]

[-- Attachment #1.1: Type: text/plain, Size: 1659 bytes --]

On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> The proposed intro paragraph text is derived from the first paragraph
> of the IETF BPF WG charter at https://datatracker.ietf.org/wg/bpf/about/
> 
> Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> ---
>  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/bpf/standardization/instruction-set.rst b/Documentation/bpf/standardization/instruction-set.rst
> index d03d90afb..b44bdacd0 100644
> --- a/Documentation/bpf/standardization/instruction-set.rst
> +++ b/Documentation/bpf/standardization/instruction-set.rst
> @@ -5,7 +5,11 @@
>  BPF Instruction Set Architecture (ISA)
>  ======================================
>  
> -This document specifies the BPF instruction set architecture (ISA).
> +eBPF (which is no longer an acronym for anything), also commonly
> +referred to as BPF, is a technology with origins in the Linux kernel
> +that can run untrusted programs in a privileged context such as an

Perhaps this should be phrased as:

...that can run untrusted programs in privileged contexts such as the
operating system kernel.

Not sure if that's actually a grammar correction but it sounds more
correct in my head. Wdyt?

Regardless:

Acked-by: David Vernet <void@manifault.com>

> +operating system kernel. This document specifies the BPF instruction
> +set architecture (ISA).
>  
>  Documentation conventions
>  =========================
> -- 
> 2.40.1
> 
> -- 
> Bpf mailing list
> Bpf@ietf.org
> https://www.ietf.org/mailman/listinfo/bpf

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 76 bytes --]

-- 
Bpf mailing list
Bpf@ietf.org
https://www.ietf.org/mailman/listinfo/bpf

RE: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[dthaler1968]

David Vernet <void@manifault.com> writes: 
> On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> > The proposed intro paragraph text is derived from the first paragraph
> > of the IETF BPF WG charter at
> > https://datatracker.ietf.org/wg/bpf/about/
> >
> > Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> > ---
> >  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/bpf/standardization/instruction-set.rst
> > b/Documentation/bpf/standardization/instruction-set.rst
> > index d03d90afb..b44bdacd0 100644
> > --- a/Documentation/bpf/standardization/instruction-set.rst
> > +++ b/Documentation/bpf/standardization/instruction-set.rst
> > @@ -5,7 +5,11 @@
> >  BPF Instruction Set Architecture (ISA)
> > ======================================
> >
> > -This document specifies the BPF instruction set architecture (ISA).
> > +eBPF (which is no longer an acronym for anything), also commonly
> > +referred to as BPF, is a technology with origins in the Linux kernel
> > +that can run untrusted programs in a privileged context such as an
> 
> Perhaps this should be phrased as:
> 
> ...that can run untrusted programs in privileged contexts such as the
operating
> system kernel.
> 
> Not sure if that's actually a grammar correction but it sounds more
correct in my
> head. Wdyt?

That sounds less grammatically correct to my reading, since "contexts" would
be plural but "kernel" is singular.   The intent of the original sentence
was that
multiple programs (plural) can run in the same privileged context (singular)
such
as a kernel (singular).

Dave

Re: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[dthaler1968=40googlemail.com]

David Vernet <void@manifault.com> writes: 
> On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> > The proposed intro paragraph text is derived from the first paragraph
> > of the IETF BPF WG charter at
> > https://datatracker.ietf.org/wg/bpf/about/
> >
> > Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> > ---
> >  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
> >  1 file changed, 5 insertions(+), 1 deletion(-)
> >
> > diff --git a/Documentation/bpf/standardization/instruction-set.rst
> > b/Documentation/bpf/standardization/instruction-set.rst
> > index d03d90afb..b44bdacd0 100644
> > --- a/Documentation/bpf/standardization/instruction-set.rst
> > +++ b/Documentation/bpf/standardization/instruction-set.rst
> > @@ -5,7 +5,11 @@
> >  BPF Instruction Set Architecture (ISA)
> > ======================================
> >
> > -This document specifies the BPF instruction set architecture (ISA).
> > +eBPF (which is no longer an acronym for anything), also commonly
> > +referred to as BPF, is a technology with origins in the Linux kernel
> > +that can run untrusted programs in a privileged context such as an
> 
> Perhaps this should be phrased as:
> 
> ...that can run untrusted programs in privileged contexts such as the
operating
> system kernel.
> 
> Not sure if that's actually a grammar correction but it sounds more
correct in my
> head. Wdyt?

That sounds less grammatically correct to my reading, since "contexts" would
be plural but "kernel" is singular.   The intent of the original sentence
was that
multiple programs (plural) can run in the same privileged context (singular)
such
as a kernel (singular).

Dave

-- 
Bpf mailing list
Bpf@ietf.org
https://www.ietf.org/mailman/listinfo/bpf

Re: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[David Vernet]

[-- Attachment #1: Type: text/plain, Size: 1905 bytes --]

On Mon, Apr 22, 2024 at 01:18:05PM -0700, dthaler1968@googlemail.com wrote:
> David Vernet <void@manifault.com> writes: 
> > On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> > > The proposed intro paragraph text is derived from the first paragraph
> > > of the IETF BPF WG charter at
> > > https://datatracker.ietf.org/wg/bpf/about/
> > >
> > > Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> > > ---
> > >  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
> > >  1 file changed, 5 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/bpf/standardization/instruction-set.rst
> > > b/Documentation/bpf/standardization/instruction-set.rst
> > > index d03d90afb..b44bdacd0 100644
> > > --- a/Documentation/bpf/standardization/instruction-set.rst
> > > +++ b/Documentation/bpf/standardization/instruction-set.rst
> > > @@ -5,7 +5,11 @@
> > >  BPF Instruction Set Architecture (ISA)
> > > ======================================
> > >
> > > -This document specifies the BPF instruction set architecture (ISA).
> > > +eBPF (which is no longer an acronym for anything), also commonly
> > > +referred to as BPF, is a technology with origins in the Linux kernel
> > > +that can run untrusted programs in a privileged context such as an
> > 
> > Perhaps this should be phrased as:
> > 
> > ...that can run untrusted programs in privileged contexts such as the
> > operating system kernel.
> > 
> > Not sure if that's actually a grammar correction but it sounds more
> > correct in my head. Wdyt?
> 
> That sounds less grammatically correct to my reading, since "contexts"
> would be plural but "kernel" is singular.   The intent of the original
> sentence was that multiple programs (plural) can run in the same
> privileged context (singular) such as a kernel (singular).

Fair enough. My ack above stands.

Thanks,
David

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

Re: [Bpf] [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[David Vernet]

[-- Attachment #1.1: Type: text/plain, Size: 1905 bytes --]

On Mon, Apr 22, 2024 at 01:18:05PM -0700, dthaler1968@googlemail.com wrote:
> David Vernet <void@manifault.com> writes: 
> > On Mon, Apr 22, 2024 at 12:09:42PM -0700, Dave Thaler wrote:
> > > The proposed intro paragraph text is derived from the first paragraph
> > > of the IETF BPF WG charter at
> > > https://datatracker.ietf.org/wg/bpf/about/
> > >
> > > Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> > > ---
> > >  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
> > >  1 file changed, 5 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/Documentation/bpf/standardization/instruction-set.rst
> > > b/Documentation/bpf/standardization/instruction-set.rst
> > > index d03d90afb..b44bdacd0 100644
> > > --- a/Documentation/bpf/standardization/instruction-set.rst
> > > +++ b/Documentation/bpf/standardization/instruction-set.rst
> > > @@ -5,7 +5,11 @@
> > >  BPF Instruction Set Architecture (ISA)
> > > ======================================
> > >
> > > -This document specifies the BPF instruction set architecture (ISA).
> > > +eBPF (which is no longer an acronym for anything), also commonly
> > > +referred to as BPF, is a technology with origins in the Linux kernel
> > > +that can run untrusted programs in a privileged context such as an
> > 
> > Perhaps this should be phrased as:
> > 
> > ...that can run untrusted programs in privileged contexts such as the
> > operating system kernel.
> > 
> > Not sure if that's actually a grammar correction but it sounds more
> > correct in my head. Wdyt?
> 
> That sounds less grammatically correct to my reading, since "contexts"
> would be plural but "kernel" is singular.   The intent of the original
> sentence was that multiple programs (plural) can run in the same
> privileged context (singular) such as a kernel (singular).

Fair enough. My ack above stands.

Thanks,
David

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 76 bytes --]

-- 
Bpf mailing list
Bpf@ietf.org
https://www.ietf.org/mailman/listinfo/bpf

Re: [PATCH bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft

[patchwork-bot+netdevbpf]

Hello:

This patch was applied to bpf/bpf-next.git (master)
by Alexei Starovoitov <ast@kernel.org>:

On Mon, 22 Apr 2024 12:09:42 -0700 you wrote:
> The proposed intro paragraph text is derived from the first paragraph
> of the IETF BPF WG charter at https://datatracker.ietf.org/wg/bpf/about/
> 
> Signed-off-by: Dave Thaler <dthaler1968@gmail.com>
> ---
>  Documentation/bpf/standardization/instruction-set.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)

Here is the summary with links:
  - [bpf-next] bpf, docs: Add introduction for use in the ISA Internet Draft
    https://git.kernel.org/bpf/bpf-next/c/e51b907d4032

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html