Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1260282 > unrolled thread

[PATCH] bpf: fix trivial comment typo

Started byMatthew Fernandez <matthew.fernandez@gmail.com>
First post2015-11-02 02:10 +0100
Last post2015-11-05 01:10 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] bpf: fix trivial comment typo Matthew Fernandez <matthew.fernandez@gmail.com> - 2015-11-02 02:10 +0100
    Re: [PATCH] bpf: fix trivial comment typo David Miller <davem@davemloft.net> - 2015-11-02 22:40 +0100
      Re: [PATCH] bpf: fix trivial comment typo Matthew Fernandez <matthew.fernandez@gmail.com> - 2015-11-02 22:50 +0100
        Re: [PATCH] bpf: fix trivial comment typo Daniel Borkmann <daniel@iogearbox.net> - 2015-11-03 10:50 +0100
          Re: [PATCH] bpf: fix trivial comment typo Matthew Fernandez <matthew.fernandez@gmail.com> - 2015-11-05 01:10 +0100

#1260282 — [PATCH] bpf: fix trivial comment typo

FromMatthew Fernandez <matthew.fernandez@gmail.com>
Date2015-11-02 02:10 +0100
Subject[PATCH] bpf: fix trivial comment typo
Message-ID<qqbNf-7jC-1@gated-at.bofh.it>
bpf: fix trivial comment typo

Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>

diff --git a/kernel/bpf/core.c b/kernel/bpf/core.c
index 67c380c..996db5c 100644
--- a/kernel/bpf/core.c
+++ b/kernel/bpf/core.c
@@ -456,7 +456,7 @@ select_insn:

  		/* ARG1 at this point is guaranteed to point to CTX from
  		 * the verifier side due to the fact that the tail call is
-		 * handeled like a helper, that is, bpf_tail_call_proto,
+		 * handled like a helper, that is, bpf_tail_call_proto,
  		 * where arg1_type is ARG_PTR_TO_CTX.
  		 */
  		insn = prog->insnsi;
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]


#1260996

FromDavid Miller <davem@davemloft.net>
Date2015-11-02 22:40 +0100
Message-ID<qquZB-250-25@gated-at.bofh.it>
In reply to#1260282
From: Matthew Fernandez <matthew.fernandez@gmail.com>
Date: Mon, 2 Nov 2015 11:59:03 +1100

> bpf: fix trivial comment typo
> 
> Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>

This doesn't apply to any tree.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1261007

FromMatthew Fernandez <matthew.fernandez@gmail.com>
Date2015-11-02 22:50 +0100
Message-ID<qqv9f-28h-1@gated-at.bofh.it>
In reply to#1260996
On 03/11/15 08:31, David Miller wrote:
> From: Matthew Fernandez <matthew.fernandez@gmail.com>
> Date: Mon, 2 Nov 2015 11:59:03 +1100
>
>> bpf: fix trivial comment typo
>>
>> Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>
>
> This doesn't apply to any tree.

I'm sorry, I think I must be missing something. This seems to apply cleanly to the current tip of mainline 
(e86328c489d7) to me. Was this not in the expected format? It wasn't my intention to waste your time, so I apologise for 
any newbie errors.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1261339

FromDaniel Borkmann <daniel@iogearbox.net>
Date2015-11-03 10:50 +0100
Message-ID<qqGo2-WT-11@gated-at.bofh.it>
In reply to#1261007
On 11/02/2015 10:48 PM, Matthew Fernandez wrote:
> On 03/11/15 08:31, David Miller wrote:
>> From: Matthew Fernandez <matthew.fernandez@gmail.com>
>> Date: Mon, 2 Nov 2015 11:59:03 +1100
>>
>>> bpf: fix trivial comment typo
>>>
>>> Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>
>>
>> This doesn't apply to any tree.
>
> I'm sorry, I think I must be missing something. This seems to apply cleanly to the current tip of mainline (e86328c489d7) to me. Was this not in the expected format? It wasn't my intention to waste your time, so I apologise for any newbie errors.

You might want to check Documentation/networking/netdev-FAQ.txt ;),
and rebase your spelling fix f.e. to the latest net-next tree.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]


#1262758

FromMatthew Fernandez <matthew.fernandez@gmail.com>
Date2015-11-05 01:10 +0100
Message-ID<qrghP-7yp-7@gated-at.bofh.it>
In reply to#1261339
On 03/11/15 20:48, Daniel Borkmann wrote:
> On 11/02/2015 10:48 PM, Matthew Fernandez wrote:
>> On 03/11/15 08:31, David Miller wrote:
>>> From: Matthew Fernandez <matthew.fernandez@gmail.com>
>>> Date: Mon, 2 Nov 2015 11:59:03 +1100
>>>
>>>> bpf: fix trivial comment typo
>>>>
>>>> Signed-off-by: Matthew Fernandez <matthew.fernandez@gmail.com>
>>>
>>> This doesn't apply to any tree.
>>
>> I'm sorry, I think I must be missing something. This seems to apply cleanly to the current tip of
>> mainline (e86328c489d7) to me. Was this not in the expected format? It wasn't my intention to
>> waste your time, so I apologise for any newbie errors.
>
> You might want to check Documentation/networking/netdev-FAQ.txt ;),
> and rebase your spelling fix f.e. to the latest net-next tree.

Ah right, sorry about that. Thanks for the info, Daniel. From netdev-FAQ it sounds like this patch 
would be more appropriate for the net tree, but as the target file is identical in both right now 
I'll leave it to someone else's discretion to decide which is more suitable. Will rebase and resubmit.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web