On 12/2/20 7:21 PM, Toms Atteka wrote: > If system is missing ethtool, test fails silently. > > Signed-off-by: Toms Atteka <cpp.code...@gmail.com> > --- > Documentation/intro/install/afxdp.rst | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/Documentation/intro/install/afxdp.rst > b/Documentation/intro/install/afxdp.rst > index 3c8f78825..a597658e2 100644 > --- a/Documentation/intro/install/afxdp.rst > +++ b/Documentation/intro/install/afxdp.rst > @@ -166,7 +166,7 @@ Finally, build and install OVS:: > > To kick start end-to-end autotesting:: > > - uname -a # make sure having 5.0+ kernel > + uname -a # make sure having 5.0+ kernel and ethtool installed
This comment is for 'uname -a', so ethtool related information doesn't belong there. It's better to write an extra sentence before actual commands to describe requirements. Something like this: --- To kick start end-to-end autotesting (make sure having 5.0+ kernel and ethtool installed):: make check-afxdp TESTSUITEFLAGS='1' --- What do you think? Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev