* [PATCH] bpf: update the comment about the length of analysis
@ 2017-03-01 8:25 Gary Lin
2017-03-01 17:15 ` Alexei Starovoitov
2017-03-01 23:03 ` David Miller
0 siblings, 2 replies; 3+ messages in thread
From: Gary Lin @ 2017-03-01 8:25 UTC (permalink / raw)
To: netdev, linux-kernel; +Cc: Alexei Starovoitov, Daniel Borkmann
Commit 07016151a446 ("bpf, verifier: further improve search
pruning") increased the limit of processed instructions from
32k to 64k, but the comment still mentioned the 32k limit.
This commit updates the comment to reflect the change.
Cc: Alexei Starovoitov <ast@kernel.org>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Signed-off-by: Gary Lin <glin@suse.com>
---
kernel/bpf/verifier.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/bpf/verifier.c b/kernel/bpf/verifier.c
index cdc43b899f28..0960f65c6da7 100644
--- a/kernel/bpf/verifier.c
+++ b/kernel/bpf/verifier.c
@@ -33,7 +33,7 @@
* - out of bounds or malformed jumps
* The second pass is all possible path descent from the 1st insn.
* Since it's analyzing all pathes through the program, the length of the
- * analysis is limited to 32k insn, which may be hit even if total number of
+ * analysis is limited to 64k insn, which may be hit even if total number of
* insn is less then 4K, but there are too many branches that change stack/regs.
* Number of 'branches to be analyzed' is limited to 1k
*
--
2.12.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] bpf: update the comment about the length of analysis
2017-03-01 8:25 [PATCH] bpf: update the comment about the length of analysis Gary Lin
@ 2017-03-01 17:15 ` Alexei Starovoitov
2017-03-01 23:03 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: Alexei Starovoitov @ 2017-03-01 17:15 UTC (permalink / raw)
To: Gary Lin; +Cc: netdev, linux-kernel, Alexei Starovoitov, Daniel Borkmann
On Wed, Mar 01, 2017 at 04:25:51PM +0800, Gary Lin wrote:
> Commit 07016151a446 ("bpf, verifier: further improve search
> pruning") increased the limit of processed instructions from
> 32k to 64k, but the comment still mentioned the 32k limit.
> This commit updates the comment to reflect the change.
>
> Cc: Alexei Starovoitov <ast@kernel.org>
> Cc: Daniel Borkmann <daniel@iogearbox.net>
> Signed-off-by: Gary Lin <glin@suse.com>
Acked-by: Alexei Starovoitov <ast@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] bpf: update the comment about the length of analysis
2017-03-01 8:25 [PATCH] bpf: update the comment about the length of analysis Gary Lin
2017-03-01 17:15 ` Alexei Starovoitov
@ 2017-03-01 23:03 ` David Miller
1 sibling, 0 replies; 3+ messages in thread
From: David Miller @ 2017-03-01 23:03 UTC (permalink / raw)
To: glin; +Cc: netdev, linux-kernel, ast, daniel
From: Gary Lin <glin@suse.com>
Date: Wed, 1 Mar 2017 16:25:51 +0800
> Commit 07016151a446 ("bpf, verifier: further improve search
> pruning") increased the limit of processed instructions from
> 32k to 64k, but the comment still mentioned the 32k limit.
> This commit updates the comment to reflect the change.
>
> Cc: Alexei Starovoitov <ast@kernel.org>
> Cc: Daniel Borkmann <daniel@iogearbox.net>
> Signed-off-by: Gary Lin <glin@suse.com>
Applied, thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-03-02 0:39 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-01 8:25 [PATCH] bpf: update the comment about the length of analysis Gary Lin
2017-03-01 17:15 ` Alexei Starovoitov
2017-03-01 23:03 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).