Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <778cbcdc-24b2-12f4-aebc-d62d5bdaf2de@alu.unizg.hr>
Date:   Tue, 1 Aug 2023 22:41:06 +0200
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.13.1
Subject: Re: [PATCH v1 01/11] selftests: forwarding: custom_multipath_hash.sh:
 add cleanup for SIGTERM sent by timeout
Content-Language: en-US
To:     Ido Schimmel <idosch@idosch.org>
Cc:     petrm@nvidia.com, razor@blackwall.org,
        Ido Schimmel <idosch@nvidia.com>, netdev@vger.kernel.org,
        linux-kselftest@vger.kernel.org, linux-kernel@vger.kernel.org,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>, Shuah Khan <shuah@kernel.org>
References: <ZL6OljQubhVtQjcD@shredder>
 <cab8ea8a-98f4-ef9b-4215-e2a93cccaab1@alu.unizg.hr>
 <ZMEQGIOQXv6so30x@shredder>
 <a9b6d9f5-14ae-a931-ab7b-d31b5e40f5df@alu.unizg.hr>
 <ZMYXABUN9OzfN5D3@shredder>
 <da3f4f4e-47a7-25be-fa61-aebeba1d8d0c@alu.unizg.hr>
 <ZMdouQRypZCGZhV0@shredder>
 <2f203995-5ae0-13bc-d1a6-997c2b36a2b8@alu.unizg.hr>
 <ZMei0VMIH/l1GzVM@shredder>
 <cadad022-b241-398d-c79d-187596356a72@alu.unizg.hr>
 <ZMfXsVAfpizMKH/U@shredder>
From:   Mirsad Todorovac <mirsad.todorovac@alu.unizg.hr>
In-Reply-To: <ZMfXsVAfpizMKH/U@shredder>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 7/31/23 17:48, Ido Schimmel wrote:
> On Mon, Jul 31, 2023 at 05:13:37PM +0200, Mirsad Todorovac wrote:
>> You can add:
>>
>> Tested-by: Mirsad Todorovac <mirsad.todorovac@alu.unizg.hr>
> 
> Added your tags to all 17 patches. Available here:
> https://github.com/idosch/linux/tree/submit/selftests_fix_v2

Yes, thanks!

It is good to be known that I am available for testing patches on the available
equipment.

> Will submit later this week (most likely on Wednesday) after I verify
> they don't cause other regressions.
> 
> Thanks for testing and reporting.

Not at all.

I am most pleased that we nailed the errors and fails.

It was a great exercise for my little grey cells in a great environment of quality
professionals.

Much obliged.

Still, maybe you should add to README about the forwarding.config.sample opportunity?

I think what we all want as the community is greater test coverage, to discover the most
bugs on the most platforms ...

Kind regards,
Mirsad Todorovac