| Message ID | 20230729002403.4278-1-skhan@linuxfoundation.org (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 04786c0659dbacf1f626f70c2b19c1bc334f1098 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [RESEND,v2] selftests:connector: Fix input argument error paths to skip | expand |
| Context | Check | Description |
|---|---|---|
| netdev/tree_selection | success | Not a local patch |
On 7/28/23 18:24, Shuah Khan wrote: > Fix input argument parsing paths to skip from their error legs. > This fix helps to avoid false test failure reports without running > the test. > > Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> > --- > v2: Removed root check based on Anjali's review comments. > Add netdev to RESEND > > tools/testing/selftests/connector/proc_filter.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/connector/proc_filter.c b/tools/testing/selftests/connector/proc_filter.c > index 4fe8c6763fd8..4a825b997666 100644 > --- a/tools/testing/selftests/connector/proc_filter.c > +++ b/tools/testing/selftests/connector/proc_filter.c > @@ -248,7 +248,7 @@ int main(int argc, char *argv[]) > > if (argc > 2) { > printf("Expected 0(assume no-filter) or 1 argument(-f)\n"); > - exit(1); > + exit(KSFT_SKIP); > } > > if (argc == 2) { > @@ -256,7 +256,7 @@ int main(int argc, char *argv[]) > filter = 1; > } else { > printf("Valid option : -f (for filter feature)\n"); > - exit(1); > + exit(KSFT_SKIP); > } > } > Hi Jakub, I sent v2 for patch 3 in the series. Do you want me to send the entire series again with this revised 3rd patch. thanks, -- Shuah
On Mon, 31 Jul 2023 14:23:45 -0600 Shuah Khan wrote: > I sent v2 for patch 3 in the series. Do you want me to send the > entire series again with this revised 3rd patch. I think it's all good. The build bot couldn't parse the partial series but the patches are pretty trivial, so low risk of breakage. Anjali, it would be good to get your Review / Ack tag on this patch, since you're the author.
> On Jul 28, 2023, at 5:24 PM, Shuah Khan <skhan@linuxfoundation.org> wrote: > > Fix input argument parsing paths to skip from their error legs. > This fix helps to avoid false test failure reports without running > the test. > > Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> Reviewed-by: Anjali Kulkarni <anjali.k.kulkarni@oracle.com> > --- > v2: Removed root check based on Anjali's review comments. > Add netdev to RESEND > > tools/testing/selftests/connector/proc_filter.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/tools/testing/selftests/connector/proc_filter.c b/tools/testing/selftests/connector/proc_filter.c > index 4fe8c6763fd8..4a825b997666 100644 > --- a/tools/testing/selftests/connector/proc_filter.c > +++ b/tools/testing/selftests/connector/proc_filter.c > @@ -248,7 +248,7 @@ int main(int argc, char *argv[]) > > if (argc > 2) { > printf("Expected 0(assume no-filter) or 1 argument(-f)\n"); > - exit(1); > + exit(KSFT_SKIP); > } > > if (argc == 2) { > @@ -256,7 +256,7 @@ int main(int argc, char *argv[]) > filter = 1; > } else { > printf("Valid option : -f (for filter feature)\n"); > - exit(1); > + exit(KSFT_SKIP); > } > } > > -- > 2.39.2 >
> On Jul 31, 2023, at 2:07 PM, Jakub Kicinski <kuba@kernel.org> wrote: > > On Mon, 31 Jul 2023 14:23:45 -0600 Shuah Khan wrote: >> I sent v2 for patch 3 in the series. Do you want me to send the >> entire series again with this revised 3rd patch. > > I think it's all good. The build bot couldn't parse the partial series > but the patches are pretty trivial, so low risk of breakage. > > Anjali, it would be good to get your Review / Ack tag on this patch, > since you're the author. Yes done. Anjali
Hello: This patch was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Fri, 28 Jul 2023 18:24:03 -0600 you wrote: > Fix input argument parsing paths to skip from their error legs. > This fix helps to avoid false test failure reports without running > the test. > > Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> > --- > v2: Removed root check based on Anjali's review comments. > Add netdev to RESEND > > [...] Here is the summary with links: - [RESEND,v2] selftests:connector: Fix input argument error paths to skip https://git.kernel.org/netdev/net-next/c/04786c0659db You are awesome, thank you!
diff --git a/tools/testing/selftests/connector/proc_filter.c b/tools/testing/selftests/connector/proc_filter.c index 4fe8c6763fd8..4a825b997666 100644 --- a/tools/testing/selftests/connector/proc_filter.c +++ b/tools/testing/selftests/connector/proc_filter.c @@ -248,7 +248,7 @@ int main(int argc, char *argv[]) if (argc > 2) { printf("Expected 0(assume no-filter) or 1 argument(-f)\n"); - exit(1); + exit(KSFT_SKIP); } if (argc == 2) { @@ -256,7 +256,7 @@ int main(int argc, char *argv[]) filter = 1; } else { printf("Valid option : -f (for filter feature)\n"); - exit(1); + exit(KSFT_SKIP); } }
Fix input argument parsing paths to skip from their error legs. This fix helps to avoid false test failure reports without running the test. Signed-off-by: Shuah Khan <skhan@linuxfoundation.org> --- v2: Removed root check based on Anjali's review comments. Add netdev to RESEND tools/testing/selftests/connector/proc_filter.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)