From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jiri Pirko Subject: Re: [RFC PATCH net-next 02/12] selftests: forwarding: Add a test for FDB learning Date: Thu, 18 Jan 2018 00:31:24 +0100 Message-ID: <20180117233124.GA26967@nanopsycho> References: <20180115191853.26129-1-idosch@mellanox.com> <20180115191853.26129-3-idosch@mellanox.com> <33d3f4a4-8b92-60da-3e1f-327effb7eda0@gmail.com> <20180117210137.GJ2114@nanopsycho> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: David Ahern , Ido Schimmel , netdev@vger.kernel.org, linux-kselftest@vger.kernel.org, David Miller , shuah@kernel.org, Nikolay Aleksandrov , Andy Gospodarek , Jiri Pirko , mlxsw , Saeed Mahameed , tariqt@mellanox.com, Jamal Hadi Salim , Lucas Bates , Florian Fainelli , Vivien Didelot , Andrew Lunn , jakub.kicinski@netronome.com, Simon Horman To: Roopa Prabhu Return-path: Received: from mail-wm0-f66.google.com ([74.125.82.66]:42799 "EHLO mail-wm0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752901AbeAQXb1 (ORCPT ); Wed, 17 Jan 2018 18:31:27 -0500 Received: by mail-wm0-f66.google.com with SMTP id b141so18559807wme.1 for ; Wed, 17 Jan 2018 15:31:26 -0800 (PST) Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: Wed, Jan 17, 2018 at 11:59:10PM CET, roopa@cumulusnetworks.com wrote: >On Wed, Jan 17, 2018 at 2:46 PM, Roopa Prabhu wrote: >> On Wed, Jan 17, 2018 at 1:01 PM, Jiri Pirko wrote: >>> Wed, Jan 17, 2018 at 09:48:54PM CET, dsahern@gmail.com wrote: >>>>On 1/15/18 11:18 AM, Ido Schimmel wrote: >>>>> diff --git a/tools/testing/selftests/forwarding/lib.sh b/tools/testing/selftests/forwarding/lib.sh >>>>> index bb423371f4de..264bf0af4c4d 100644 >>>>> --- a/tools/testing/selftests/forwarding/lib.sh >>>>> +++ b/tools/testing/selftests/forwarding/lib.sh >>>>> @@ -22,6 +22,11 @@ if [[ ! -x "$(command -v jq)" ]]; then >>>>> exit 0 >>>>> fi >>>>> >>>>> +if [[ ! -x "$(command -v mausezahn)" ]]; then >>>>> + echo "SKIP: mausezahn not installed" >>>>> + exit 0 >>>>> +fi >>>>> + >>>> >>>>The checks are good, but hitting a collision with this one: >>>>$ ./router.sh >>>>SKIP: mausezahn not installed >>>> >>>>For debian, it is mz not mausezahn. >>> >>> That is weird. If you look at the sources, the binary name is >>> "mausezahn". Looks like debian is doing some renaming :/ >>> >> >> I have seen/used both versions. Debian packages from some old upstream >> source which calls it mz. >> https://packages.debian.org/sid/main/mz >> >> I have used latest mz (with ipv6 support etc) from netsniff-ng which >> builds it as mausezahn. > > >actually Debian also builds >https://packages.debian.org/sid/netsniff-ng and installs 'mausezahn' > >so you might have to check and use which exists... (mausezahn overriding mz). I believe we can stick with mausezahn, since it is apparently available on Debian too.