NAK [OEM-5.14] Re: [F/H/I/J/Unstable][OEM-5.10/OEM-5.13/OEM-5.14][SRU][PATCH 0/1] selftests: icmp_redirect: pass xfail=0 to log_test()

[Timo Aaltonen]

On 28.12.2021 11.43, Po-Hsu Lin wrote:
> [Impact]
> If any sub-test in this icmp_redirect.sh is failing but not expected
> to fail. The script will complain:
>      ./icmp_redirect.sh: line 72: [: 1: unary operator expected
> 
> This is because when the sub-test is not expected to fail, we won't
> pass any value for the xfail local variable in log_test() and thus
> it's empty. Fix this by passing 0 as the 4th variable to log_test()
> for non-xfail cases.
> 
> This issue can be found on our KVM kernel with SRU tests, they're
> failing because of missing some kernel configs.
> 
> [Fix]
> * 39d8622c72 "selftests: icmp_redirect: pass xfail=0 to log_test()"
> 
> This patch can be cherry-picked into all of the affected kernels.
> 
> [Test Plan]
> Run the patched icmp_redirect.sh script with a KVM kernel (e.g. F/kvm),
> and this "unary operator expected" error message should no longer exist.
> 
> [Where problems could occur]
> Change limited to test script, it won't have any impact to real kernel
> function. And this issue is most likely specific to KVM kernels.
> 
> Po-Hsu Lin (1):
>    selftests: icmp_redirect: pass xfail=0 to log_test()
> 
>   tools/testing/selftests/net/icmp_redirect.sh | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 

already applied via stable@



-- 
t