On Fri, Apr 12, 2019 at 4:32 PM Alexei Starovoitov <alexei.starovoitov@xxxxxxxxx> wrote: > > On Fri, Apr 12, 2019 at 04:24:51PM -0700, Song Liu wrote: > > On Fri, Apr 12, 2019 at 2:41 PM Alexei Starovoitov <ast@xxxxxxxxxx> wrote: > > > > > > Add two tests to check that sequence of 1024 jumps is verifiable. > > > > > > Signed-off-by: Alexei Starovoitov <ast@xxxxxxxxxx> > > > > Acked-by: Song Liu <songliubraving@xxxxxx> > > > > Shall we add a test that go beyond the 1M limit? > > 1m is not uapi limit. I'm working on the doc patch to stress that point. > Adding a test to check that it fails at 1m would kinda imply > that it is uapi and I very much want to avoid that. > > The purpose of these tests is to stress the verifier to its > internal limits, but not more. > In particular in these two tests 1024, 8, 512, and another 1M > are limits too. > Yeah, this makes sense. Thanks for the explanation. Song
