| Message ID | 20220517035258.43945-1-yangx.jy@fujitsu.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [blktests] common/multipath-over-rdma: Remove unused debug operation | expand |
On Tue, May 17, 2022 at 11:52:58AM +0800, Xiao Yang wrote: > The loop ("for m in ;") will never be entered and it seems > unnecessary to debug sereval modules during test. So I try > to remove the debug operation. > > Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com> > --- > common/multipath-over-rdma | 6 ------ > 1 file changed, 6 deletions(-) > > diff --git a/common/multipath-over-rdma b/common/multipath-over-rdma > index cef05ec..8b285e6 100644 > --- a/common/multipath-over-rdma > +++ b/common/multipath-over-rdma > @@ -655,12 +655,6 @@ setup_test() { > fi > ) > > - if [ -d /sys/kernel/debug/dynamic_debug ]; then > - for m in ; do > - echo "module $m +pmf" >/sys/kernel/debug/dynamic_debug/control > - done > - fi > - > setup_rdma || return $? > start_target || return $? > echo "Test setup finished" >>"$FULL" > -- > 2.34.1 Thanks, applied.
diff --git a/common/multipath-over-rdma b/common/multipath-over-rdma index cef05ec..8b285e6 100644 --- a/common/multipath-over-rdma +++ b/common/multipath-over-rdma @@ -655,12 +655,6 @@ setup_test() { fi ) - if [ -d /sys/kernel/debug/dynamic_debug ]; then - for m in ; do - echo "module $m +pmf" >/sys/kernel/debug/dynamic_debug/control - done - fi - setup_rdma || return $? start_target || return $? echo "Test setup finished" >>"$FULL"
The loop ("for m in ;") will never be entered and it seems unnecessary to debug sereval modules during test. So I try to remove the debug operation. Signed-off-by: Xiao Yang <yangx.jy@fujitsu.com> --- common/multipath-over-rdma | 6 ------ 1 file changed, 6 deletions(-)