Re: [PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[SeongJae Park]

On Sat, Nov 4, 2017 at 2:59 AM, Paul E. McKenney
 wrote:
> On Fri, Nov 03, 2017 at 07:17:19PM +0900, SeongJae Park wrote:
>> This patchset contains trivial fixup and enhancements for KVM-based 
>> rcutorture
>> test framework.
>
> Nice updates!  These survived some rcutorture testing, so I have queued
> them for further testing and review.  I did update the commit logs,
> and had to hand-apply one patch, so could you please double-check my
> -rcu tree to make sure that I didn't mess anything up?
>
> Thanx, Paul

Hello Paul,


I checked out the 'rcu/dev' branch of your '-rcu' tree.  It looks like the
commit logs are improved very well (thank you!) and the changes are applied
completely as I intended.


Thanks,
SeongJae Park

>
>> SeongJae Park (9):
>>   rcutorture/configinit: Fix build directory error message
>>   rcutorture: Remove unused script, config2frag.sh
>>   rcutorture/kvm.sh: Remove unused variable, `alldone`
>>   rcutorture/kvm.sh: Use consistent usage for --qemu-args
>>   rcutorture/kvm.sh: Support execution from any directory
>>   rcutorture/kvm-recheck-*: Improve result directory readability check
>>   rcutorture/kvm-build.sh: Skip build directory check
>>   rcutorture: Simplify logging
>>   rcutorture: Simplify functions.sh include path
>>
>>  .../selftests/rcutorture/bin/config2frag.sh| 25 -
>>  .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
>>  .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
>>  .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
>>  .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
>>  .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
>>  .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
>>  .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
>>  .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
>>  tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 
>> +-
>>  10 files changed, 26 insertions(+), 68 deletions(-)
>>  delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh
>>
>> --
>> 2.13.0
>>
>

Re: [PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[SeongJae Park]

On Sat, Nov 4, 2017 at 2:59 AM, Paul E. McKenney
 wrote:
> On Fri, Nov 03, 2017 at 07:17:19PM +0900, SeongJae Park wrote:
>> This patchset contains trivial fixup and enhancements for KVM-based 
>> rcutorture
>> test framework.
>
> Nice updates!  These survived some rcutorture testing, so I have queued
> them for further testing and review.  I did update the commit logs,
> and had to hand-apply one patch, so could you please double-check my
> -rcu tree to make sure that I didn't mess anything up?
>
> Thanx, Paul

Hello Paul,


I checked out the 'rcu/dev' branch of your '-rcu' tree.  It looks like the
commit logs are improved very well (thank you!) and the changes are applied
completely as I intended.


Thanks,
SeongJae Park

>
>> SeongJae Park (9):
>>   rcutorture/configinit: Fix build directory error message
>>   rcutorture: Remove unused script, config2frag.sh
>>   rcutorture/kvm.sh: Remove unused variable, `alldone`
>>   rcutorture/kvm.sh: Use consistent usage for --qemu-args
>>   rcutorture/kvm.sh: Support execution from any directory
>>   rcutorture/kvm-recheck-*: Improve result directory readability check
>>   rcutorture/kvm-build.sh: Skip build directory check
>>   rcutorture: Simplify logging
>>   rcutorture: Simplify functions.sh include path
>>
>>  .../selftests/rcutorture/bin/config2frag.sh| 25 -
>>  .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
>>  .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
>>  .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
>>  .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
>>  .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
>>  .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
>>  .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
>>  .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
>>  tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 
>> +-
>>  10 files changed, 26 insertions(+), 68 deletions(-)
>>  delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh
>>
>> --
>> 2.13.0
>>
>

Re: [PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[Paul E. McKenney]

On Fri, Nov 03, 2017 at 07:17:19PM +0900, SeongJae Park wrote:
> This patchset contains trivial fixup and enhancements for KVM-based rcutorture
> test framework.

Nice updates!  These survived some rcutorture testing, so I have queued
them for further testing and review.  I did update the commit logs,
and had to hand-apply one patch, so could you please double-check my
-rcu tree to make sure that I didn't mess anything up?

Thanx, Paul

> SeongJae Park (9):
>   rcutorture/configinit: Fix build directory error message
>   rcutorture: Remove unused script, config2frag.sh
>   rcutorture/kvm.sh: Remove unused variable, `alldone`
>   rcutorture/kvm.sh: Use consistent usage for --qemu-args
>   rcutorture/kvm.sh: Support execution from any directory
>   rcutorture/kvm-recheck-*: Improve result directory readability check
>   rcutorture/kvm-build.sh: Skip build directory check
>   rcutorture: Simplify logging
>   rcutorture: Simplify functions.sh include path
> 
>  .../selftests/rcutorture/bin/config2frag.sh| 25 -
>  .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
>  .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
>  .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
>  .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
>  .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
>  .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
>  .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
>  .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
>  tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 
> +-
>  10 files changed, 26 insertions(+), 68 deletions(-)
>  delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh
> 
> -- 
> 2.13.0
> 

Re: [PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[Paul E. McKenney]

On Fri, Nov 03, 2017 at 07:17:19PM +0900, SeongJae Park wrote:
> This patchset contains trivial fixup and enhancements for KVM-based rcutorture
> test framework.

Nice updates!  These survived some rcutorture testing, so I have queued
them for further testing and review.  I did update the commit logs,
and had to hand-apply one patch, so could you please double-check my
-rcu tree to make sure that I didn't mess anything up?

Thanx, Paul

> SeongJae Park (9):
>   rcutorture/configinit: Fix build directory error message
>   rcutorture: Remove unused script, config2frag.sh
>   rcutorture/kvm.sh: Remove unused variable, `alldone`
>   rcutorture/kvm.sh: Use consistent usage for --qemu-args
>   rcutorture/kvm.sh: Support execution from any directory
>   rcutorture/kvm-recheck-*: Improve result directory readability check
>   rcutorture/kvm-build.sh: Skip build directory check
>   rcutorture: Simplify logging
>   rcutorture: Simplify functions.sh include path
> 
>  .../selftests/rcutorture/bin/config2frag.sh| 25 -
>  .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
>  .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
>  .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
>  .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
>  .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
>  .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
>  .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
>  .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
>  tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 
> +-
>  10 files changed, 26 insertions(+), 68 deletions(-)
>  delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh
> 
> -- 
> 2.13.0
> 

[PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[SeongJae Park]

This patchset contains trivial fixup and enhancements for KVM-based rcutorture
test framework.

SeongJae Park (9):
  rcutorture/configinit: Fix build directory error message
  rcutorture: Remove unused script, config2frag.sh
  rcutorture/kvm.sh: Remove unused variable, `alldone`
  rcutorture/kvm.sh: Use consistent usage for --qemu-args
  rcutorture/kvm.sh: Support execution from any directory
  rcutorture/kvm-recheck-*: Improve result directory readability check
  rcutorture/kvm-build.sh: Skip build directory check
  rcutorture: Simplify logging
  rcutorture: Simplify functions.sh include path

 .../selftests/rcutorture/bin/config2frag.sh| 25 -
 .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
 .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
 .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
 .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
 .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
 .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
 .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
 .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
 tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 +-
 10 files changed, 26 insertions(+), 68 deletions(-)
 delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh

-- 
2.13.0

[PATCH 0/9] Trivial fixup for KVM-based rcutorture test framework

[SeongJae Park]

This patchset contains trivial fixup and enhancements for KVM-based rcutorture
test framework.

SeongJae Park (9):
  rcutorture/configinit: Fix build directory error message
  rcutorture: Remove unused script, config2frag.sh
  rcutorture/kvm.sh: Remove unused variable, `alldone`
  rcutorture/kvm.sh: Use consistent usage for --qemu-args
  rcutorture/kvm.sh: Support execution from any directory
  rcutorture/kvm-recheck-*: Improve result directory readability check
  rcutorture/kvm-build.sh: Skip build directory check
  rcutorture: Simplify logging
  rcutorture: Simplify functions.sh include path

 .../selftests/rcutorture/bin/config2frag.sh| 25 -
 .../testing/selftests/rcutorture/bin/configinit.sh |  2 +-
 .../testing/selftests/rcutorture/bin/kvm-build.sh  |  5 ---
 .../selftests/rcutorture/bin/kvm-recheck-lock.sh   |  2 +-
 .../selftests/rcutorture/bin/kvm-recheck-rcu.sh|  4 +--
 .../rcutorture/bin/kvm-recheck-rcuperf-ftrace.sh   |  2 +-
 .../rcutorture/bin/kvm-recheck-rcuperf.sh  |  4 +--
 .../selftests/rcutorture/bin/kvm-recheck.sh|  2 +-
 .../selftests/rcutorture/bin/kvm-test-1-run.sh |  6 ++--
 tools/testing/selftests/rcutorture/bin/kvm.sh  | 42 +-
 10 files changed, 26 insertions(+), 68 deletions(-)
 delete mode 100755 tools/testing/selftests/rcutorture/bin/config2frag.sh

-- 
2.13.0