netdev - Re: [RFC PATCH net-next] selftests: forwarding: add a test for MAC Merge layer

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 17 Mar 2023 16:35:58 +0100
From:   Petr Machata <petrm@...dia.com>
To:     Petr Machata <petrm@...dia.com>
CC:     Vladimir Oltean <vladimir.oltean@....com>, <netdev@...r.kernel.org>
Subject: Re: [RFC PATCH net-next] selftests: forwarding: add a test for MAC
 Merge layer


Petr Machata <petrm@...dia.com> writes:

> Petr Machata <petrm@...dia.com> writes:
>
>> Petr Machata <petrm@...dia.com> writes:
>>
>>> Vladimir Oltean <vladimir.oltean@....com> writes:
>>>
>>>> diff --git a/tools/testing/selftests/drivers/net/mlxsw/sch_tbf_ets.sh b/tools/testing/selftests/drivers/net/mlxsw/sch_tbf_ets.sh
>>>> index c6ce0b448bf3..bf57400e14ee 100755
>>>> --- a/tools/testing/selftests/drivers/net/mlxsw/sch_tbf_ets.sh
>>>> +++ b/tools/testing/selftests/drivers/net/mlxsw/sch_tbf_ets.sh
>>>> @@ -2,7 +2,7 @@
>>>>  # SPDX-License-Identifier: GPL-2.0
>>>>  
>>>>  source qos_lib.sh
>>>> -bail_on_lldpad
>>>> +bail_on_lldpad "configure DCB" "configure Qdiscs"
>>>
>>> ... lib.sh isn't sourced at this point yet. `source
>>> $lib_dir/sch_tbf_ets.sh' brings that in later in the file, so the bail
>>> would need to be below that. But if it is, it won't run until after the
>>> test, which is useless.
>
> I added a shim as shown below. Comments welcome. Your patch then needs a
> bit of adaptation, plus I've dropped all the now-useless imports of
> qos_lib.sh. I'll pass this through our regression, and if nothing
> explodes, I'll point you at a branch tomorrow, and you can make the two
> patches a part of your larger patchset?

(I only pushed this to our regression today. The patches are the top two
ones here:

    https://github.com/pmachata/linux_mlxsw/commits/petrm_selftests_bail_on_lldpad_move

I'll let you know on Monday whether anything exploded in regression.)