* [LTP] [PATCH] route{4,6}-rmmod: Filter out ":" and fix typos
@ 2020-10-22 10:12 Feiyu Zhu
2020-10-22 13:52 ` Alexey Kodanev
0 siblings, 1 reply; 3+ messages in thread
From: Feiyu Zhu @ 2020-10-22 10:12 UTC (permalink / raw)
To: ltp
Reported-by: Jin Cao <caoj.fnst@cn.fujitsu.com>
Signed-off-by: Feiyu Zhu <zhufy.jy@cn.fujitsu.com>
---
testcases/network/stress/route/route4-rmmod | 4 ++--
testcases/network/stress/route/route6-rmmod | 4 ++--
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/testcases/network/stress/route/route4-rmmod b/testcases/network/stress/route/route4-rmmod
index 7aa1958..1ba255e 100644
--- a/testcases/network/stress/route/route4-rmmod
+++ b/testcases/network/stress/route/route4-rmmod
@@ -164,14 +164,14 @@ do_setup()
lhost_module=`ethtool -i $lhost_ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
# Chack the other active interface uses the same driver
- for ifname in `ifconfig | grep ^eth | awk '{ print $1}'`; do
+ for ifname in `ifconfig | grep ^eth | awk '{print $1}' | sed "s/://"`; do
if [ $lhost_ifname = $ifname ]; then
continue
fi
module=`ethtool -i $ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
if [ $lhost_module = $module ]; then
- tst_resm TBROK "An active interface $ifname uses the same network deriver $module with the test intreface."
+ tst_resm TBROK "An active interface $ifname uses the same network driver $module with the test interface."
exit $TST_TOTAL
fi
done
diff --git a/testcases/network/stress/route/route6-rmmod b/testcases/network/stress/route/route6-rmmod
index 765a57a..146fa77 100644
--- a/testcases/network/stress/route/route6-rmmod
+++ b/testcases/network/stress/route/route6-rmmod
@@ -160,14 +160,14 @@ do_setup()
lhost_module=`ethtool -i $lhost_ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
# Chack the other active interface uses the same driver
- for ifname in `ifconfig | grep ^eth | awk '{ print $1}'`; do
+ for ifname in `ifconfig | grep ^eth | awk '{print $1}' | sed "s/://"`; do
if [ $lhost_ifname = $ifname ]; then
continue
fi
module=`ethtool -i $ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
if [ $lhost_module = $module ]; then
- tst_resm TBROK "An active interface $ifname uses the same network deriver $module with the test intreface."
+ tst_resm TBROK "An active interface $ifname uses the same network driver $module with the test interface."
exit $TST_TOTAL
fi
done
--
1.8.3.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [LTP] [PATCH] route{4,6}-rmmod: Filter out ":" and fix typos
2020-10-22 10:12 [LTP] [PATCH] route{4,6}-rmmod: Filter out ":" and fix typos Feiyu Zhu
@ 2020-10-22 13:52 ` Alexey Kodanev
2020-10-26 7:01 ` Petr Vorel
0 siblings, 1 reply; 3+ messages in thread
From: Alexey Kodanev @ 2020-10-22 13:52 UTC (permalink / raw)
To: ltp
On 22.10.2020 13:12, Feiyu Zhu wrote:
> Reported-by: Jin Cao <caoj.fnst@cn.fujitsu.com>
> Signed-off-by: Feiyu Zhu <zhufy.jy@cn.fujitsu.com>
> ---
> testcases/network/stress/route/route4-rmmod | 4 ++--
> testcases/network/stress/route/route6-rmmod | 4 ++--
> 2 files changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/testcases/network/stress/route/route4-rmmod b/testcases/network/stress/route/route4-rmmod
> index 7aa1958..1ba255e 100644
> --- a/testcases/network/stress/route/route4-rmmod
> +++ b/testcases/network/stress/route/route4-rmmod
> @@ -164,14 +164,14 @@ do_setup()
> lhost_module=`ethtool -i $lhost_ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
>
> # Chack the other active interface uses the same driver
> - for ifname in `ifconfig | grep ^eth | awk '{ print $1}'`; do
> + for ifname in `ifconfig | grep ^eth | awk '{print $1}' | sed "s/://"`; do
Thanks for fixing it, but it is not enough here. This command is wrong,
not only does it use obsolete tool, but also assumes that interface
names start with 'eth'. So it skips the check for active connections
that already in use with the driver (it is going to remove) for all
modern systems.
I think we should at least remove this test from the runtest file
until the proper fix. Also this test and route4{6}-redirect are
really need to be rewritten with new test API, please have a look
at route-change-*.sh.
> if [ $lhost_ifname = $ifname ]; then
> continue
> fi
>
> module=`ethtool -i $ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
> if [ $lhost_module = $module ]; then
> - tst_resm TBROK "An active interface $ifname uses the same network deriver $module with the test intreface."
> + tst_resm TBROK "An active interface $ifname uses the same network driver $module with the test interface."
> exit $TST_TOTAL
> fi
> done
> diff --git a/testcases/network/stress/route/route6-rmmod b/testcases/network/stress/route/route6-rmmod
> index 765a57a..146fa77 100644
> --- a/testcases/network/stress/route/route6-rmmod
> +++ b/testcases/network/stress/route/route6-rmmod
> @@ -160,14 +160,14 @@ do_setup()
> lhost_module=`ethtool -i $lhost_ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
>
> # Chack the other active interface uses the same driver
> - for ifname in `ifconfig | grep ^eth | awk '{ print $1}'`; do
> + for ifname in `ifconfig | grep ^eth | awk '{print $1}' | sed "s/://"`; do
> if [ $lhost_ifname = $ifname ]; then
> continue
> fi
>
> module=`ethtool -i $ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
> if [ $lhost_module = $module ]; then
> - tst_resm TBROK "An active interface $ifname uses the same network deriver $module with the test intreface."
> + tst_resm TBROK "An active interface $ifname uses the same network driver $module with the test interface."
> exit $TST_TOTAL
> fi
> done
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* [LTP] [PATCH] route{4,6}-rmmod: Filter out ":" and fix typos
2020-10-22 13:52 ` Alexey Kodanev
@ 2020-10-26 7:01 ` Petr Vorel
0 siblings, 0 replies; 3+ messages in thread
From: Petr Vorel @ 2020-10-26 7:01 UTC (permalink / raw)
To: ltp
Hi Alexey, Feiyu,
> On 22.10.2020 13:12, Feiyu Zhu wrote:
> > Reported-by: Jin Cao <caoj.fnst@cn.fujitsu.com>
> > Signed-off-by: Feiyu Zhu <zhufy.jy@cn.fujitsu.com>
> > ---
> > testcases/network/stress/route/route4-rmmod | 4 ++--
> > testcases/network/stress/route/route6-rmmod | 4 ++--
> > 2 files changed, 4 insertions(+), 4 deletions(-)
> > diff --git a/testcases/network/stress/route/route4-rmmod b/testcases/network/stress/route/route4-rmmod
> > index 7aa1958..1ba255e 100644
> > --- a/testcases/network/stress/route/route4-rmmod
> > +++ b/testcases/network/stress/route/route4-rmmod
> > @@ -164,14 +164,14 @@ do_setup()
> > lhost_module=`ethtool -i $lhost_ifname | grep driver | sed "s/driver:[[:blank:]]*//"`
> > # Chack the other active interface uses the same driver
> > - for ifname in `ifconfig | grep ^eth | awk '{ print $1}'`; do
> > + for ifname in `ifconfig | grep ^eth | awk '{print $1}' | sed "s/://"`; do
> Thanks for fixing it, but it is not enough here. This command is wrong,
> not only does it use obsolete tool, but also assumes that interface
> names start with 'eth'. So it skips the check for active connections
> that already in use with the driver (it is going to remove) for all
> modern systems.
+1
> I think we should at least remove this test from the runtest file
> until the proper fix.
Good idea, I've sent a patch for this.
https://patchwork.ozlabs.org/project/ltp/patch/20201026070006.25298-1-pvorel@suse.cz/
> Also this test and route4{6}-redirect are
> really need to be rewritten with new test API, please have a look
> at route-change-*.sh.
Kind regards,
Petr
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2020-10-26 7:01 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-22 10:12 [LTP] [PATCH] route{4,6}-rmmod: Filter out ":" and fix typos Feiyu Zhu
2020-10-22 13:52 ` Alexey Kodanev
2020-10-26 7:01 ` Petr Vorel
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.