netdev - Re: [PATCH net-next 0/2] selftests: pmtu: List/flush IPv4 cached routes, improve IPv6 test

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:   Mon, 17 Jun 2019 12:20:13 +0200
From:   Stefano Brivio <sbrivio@...hat.com>
To:     David Miller <davem@...emloft.net>
Cc:     jishi@...hat.com, weiwan@...gle.com, dsahern@...il.com,
        kafai@...com, edumazet@...gle.com,
        matti.vaittinen@...rohmeurope.com, netdev@...r.kernel.org
Subject: Re: [PATCH net-next 0/2] selftests: pmtu: List/flush IPv4 cached
 routes, improve IPv6 test

On Sun, 16 Jun 2019 20:45:52 -0700 (PDT)
David Miller <davem@...emloft.net> wrote:

> From: Stefano Brivio <sbrivio@...hat.com>
> Date: Sat, 15 Jun 2019 03:38:16 +0200
> 
> > This series introduce a new test, list_flush_ipv4_exception, and improves
> > the existing list_flush_ipv6_exception test by making it as demanding as
> > the IPv4 one.  
> 
> I suspect this will need a respin because semantics are still being discussed

Maybe not a respin, because we're discussing netlink semantics and how
many past versions of iproute2 need to work, whereas user interface and
expectations of fixed, recent kernel/iproute2 are untouched.

Anyway, sure, it doesn't make sense to merge this before the fix is
final -- I'll resend then.

This prompts some questions though (answer this quick survey and win a
patch for netdev-FAQ.rst): when (and against which tree) do tests that
are fixed by a recent patch need to be submitted? Is it a problem if
the test is merged before the fix? Would a "dependency" note help?

> and I seem to recall a mention of there being some conflict with some of
> David A's changes.

That was for e28799e52a0a ("selftests: pmtu: Introduce
list_flush_ipv6_exception test case") on top of 438a9a856ba4 ("selftests: pmtu:
Add support for routing via nexthop objects"), but you already fixed the
conflict.

That test case, by the way, will also fail until we agree on the fix.

-- 
Stefano