On Mon, Nov 1, 2021 at 4:03 PM Yonghong Song <yhs@xxxxxx> wrote: > > > > On 11/1/21 3:21 PM, Alexei Starovoitov wrote: > > From: Alexei Starovoitov <ast@xxxxxxxxxx> > > > > Similar to unsigned bounds propagation fix signed bounds. > > The 'Fixes' tag is a hint. There is no security bug here. > > The verifier was too conservative. > > > > Fixes: 3f50f132d840 ("bpf: Verifier, do explicit ALU32 bounds tracking") > > Signed-off-by: Alexei Starovoitov <ast@xxxxxxxxxx> > > The change looks good. Should a new test_verifier test be added > to exercise the new change? I think manually string comparing output the way VERBOSE_ACCEPT is doing is an overkill here. The real test case in .c will take some time to craft.