On 5/28/20 7:18 PM, Yannick Cote wrote: > The test-klp-callbacks change implement a synchronization replacement of > initial code to use completion variables instead of delays. The > completion variable interlocks the busy module with the concurrent > loading of the target livepatch patches which works with the execution > flow instead of estimated time delays. > > The test-klp-shadow-vars changes first refactors the code to be more of > a readable example as well as continuing to verify the component code. > The patch is broken in two to display the renaming and restructuring in > part 1 and the addition and change of logic in part 2. The last change > frees memory before bailing in case of errors. > > Patchset to be merged via the livepatching tree is against: livepatching/for-next > > Joe Lawrence (1): > selftests/livepatch: rework test-klp-callbacks to use completion > variables > > Yannick Cote (3): > selftests/livepatch: rework test-klp-shadow-vars > selftests/livepatch: more verification in test-klp-shadow-vars > selftests/livepatch: fix mem leaks in test-klp-shadow-vars > > lib/livepatch/test_klp_callbacks_busy.c | 42 +++- > lib/livepatch/test_klp_shadow_vars.c | 222 +++++++++--------- > .../selftests/livepatch/test-callbacks.sh | 29 ++- > .../selftests/livepatch/test-shadow-vars.sh | 85 ++++--- > 4 files changed, 214 insertions(+), 164 deletions(-) > Series looks good to me, with one minor typo in patch 3 (s/kpatch-patch//), which Miroslav as already mentioned. Reviewed-by: Kamalesh Babulal <kamalesh@xxxxxxxxxxxxxxxxxx> -- Kamalesh
