Skip to content

test_sysctl: fix up merge conflicts

Luis R. Rodriguez linux-next lkml 2 versions – Applied

Status from this message: [merged] test_sysctl-add-simple-proc_dointvec-case.patch removed from -mm tree

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest6 patches 0 repliesApplied 6/6
  2. v16 patches 0 repliesSuperseded

Patches in v2 All 6 applied

#Subject and filesSizeRepliesApplied
1/6test_sysctl: add dedicated proc sysctl test driverlib/Kconfig.debug lib/Makefile lib/test_sysctl.c +3 more+130 -40 repliesApplied
2/6test_sysctl: add generic script to expand on teststools/testing/selftests/sysctl/Makefile tools/testing/selftests/sysctl/common_tests tools/testing/selftests/sysctl/run_numerictests +2 more+495 -2200 repliesApplied
3/6test_sysctl: test against PAGE_SIZE for inttools/testing/selftests/sysctl/sysctl.sh+66 -00 repliesApplied
4/6test_sysctl: add simple proc_dointvec() caselib/test_sysctl.c tools/testing/selftests/sysctl/sysctl.sh+73 -00 repliesApplied
5/6test_sysctl: add simple proc_douintvec() caselib/test_sysctl.c tools/testing/selftests/sysctl/sysctl.sh+74 -00 repliesApplied
6/6test_sysctl: test against int proc_dointvec() array supportlib/test_sysctl.c tools/testing/selftests/sysctl/sysctl.sh+102 -00 repliesApplied

Reviews trailers per patch

1 of 6 patches has a Reviewed-by or Acked-by.

PatchKees Cook
1/6A-b
2/6
3/6
4/6
5/6
6/6

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/6] test_sysctl: fix up merge conflicts

Andrew,

as requested, this addresses the merge conflict present on kselftest/next on linux-next. The merge conflict was caused by Orson Zhai's patch titled, "tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict" and my changes which shuffled the script around. I've made Orson's check a function, and integrated his exit strategy to restore the production sysctl writes_strict setting. To do this I've also added a PROD_SYCTL, which now users can also override through an environment variable.

These changes are also present on my linux-next 20170630-test-sysctl branch based on linux-next next20170630 [0]. If there are any questionms please let me know.

[0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-next.git/log/?h=20170630-test-sysctl

Luis

Luis R. Rodriguez (6):
test_sysctl: add dedicated proc sysctl test driver
test_sysctl: add generic script to expand on tests
test_sysctl: test against PAGE_SIZE for int
test_sysctl: add simple proc_dointvec() case
test_sysctl: add simple proc_douintvec() case
test_sysctl: test against int proc_dointvec() array support

 lib/Kconfig.debug                               |  11 +
 lib/Makefile                                    |   1 +
 lib/test_sysctl.c                               | 148 +++++
 tools/testing/selftests/sysctl/Makefile         |   3 +-
 tools/testing/selftests/sysctl/common_tests     | 131 ----
 tools/testing/selftests/sysctl/config           |   1 +
 tools/testing/selftests/sysctl/run_numerictests |  10 -
 tools/testing/selftests/sysctl/run_stringtests  |  77 ---
 tools/testing/selftests/sysctl/sysctl.sh        | 774 ++++++++++++++++++++++++
 9 files changed, 936 insertions(+), 220 deletions(-)
 create mode 100644 lib/test_sysctl.c
 delete mode 100644 tools/testing/selftests/sysctl/common_tests
 create mode 100644 tools/testing/selftests/sysctl/config
 delete mode 100755 tools/testing/selftests/sysctl/run_numerictests
 delete mode 100755 tools/testing/selftests/sysctl/run_stringtests
 create mode 100755 tools/testing/selftests/sysctl/sysctl.sh
Signature · 1 line
2.11.0