On Tue, 17 Nov 2020 12:32:36 +0100 Antonio Cardace wrote: > On Mon, Nov 16, 2020 at 04:45:03PM -0800, Jakub Kicinski wrote: > > On Sat, 14 Nov 2020 00:16:55 +0100 Antonio Cardace wrote: > > > Add scripts to test ring and coalesce settings > > > of netdevsim. > > > > > > Signed-off-by: Antonio Cardace <acard...@redhat.com> > > > > > @@ -0,0 +1,68 @@ > > > +#!/bin/bash > > > +# SPDX-License-Identifier: GPL-2.0-only > > > + > > > +source ethtool-common.sh > > > + > > > +function get_value { > > > + local key=$1 > > > + > > > + echo $(ethtool -c $NSIM_NETDEV | \ > > > + awk -F':' -v pattern="$key:" '$0 ~ pattern {gsub(/[ \t]/, "", > > > $2); print $2}') > > > +} > > > + > > > +if ! ethtool -h | grep -q coalesce; then > > > + echo "SKIP: No --coalesce support in ethtool" > > > + exit 4 > > > > I think the skip exit code for selftests is 2 > In the ethtool-pause.sh selftest the exit code is 4 (I copied it from > there), should I change that too?
Sorry I misremembered it's 4. We can leave that as is.