Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

On Fri, Apr 5, 2024 at 11:38 AM Vladislav Odintsov wrote: > > > On 5 Apr 2024, at 18:35, Ihar Hrachyshka wrote: > > On Thu, Apr 4, 2024 at 3:56 PM Vladislav Odintsov > wrote: > >> Thanks Ihar for the patch. >> >> It definitely triggers the bug mentioned in Fixes commit, but how do you >> like

Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

> On 5 Apr 2024, at 18:35, Ihar Hrachyshka wrote: > > On Thu, Apr 4, 2024 at 3:56 PM Vladislav Odintsov > wrote: >> Thanks Ihar for the patch. >> >> It definitely triggers the bug mentioned in Fixes commit, but how do you >> like next diff as an alternative? >> It

Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

On Thu, Apr 4, 2024 at 3:56 PM Vladislav Odintsov wrote: > Thanks Ihar for the patch. > > It definitely triggers the bug mentioned in Fixes commit, but how do you > like next diff as an alternative? > It seems a little easier to me, because it shows the real limit and the > situation where the

Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

Yes, this diff is from main. To trigger an initial bug it is enough to create a new ls/lr while all available tunnel ids are used for datapaths (4095). This is because we need to enter ovn_allocate_tnlid() with *hint=0 to trigger infinite loop. That is why I suggest just to create 4095 LSs and

Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

On Thu, Apr 4, 2024 at 3:56 PM Vladislav Odintsov wrote: > Thanks Ihar for the patch. > > It definitely triggers the bug mentioned in Fixes commit, but how do you > like next diff as an alternative? > It seems a little easier to me, because it shows the real limit and the > situation where the

Re: [ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

Thanks Ihar for the patch. It definitely triggers the bug mentioned in Fixes commit, but how do you like next diff as an alternative? It seems a little easier to me, because it shows the real limit and the situation where the problem was (separate ls-add): diff --git a/tests/ovn-northd.at

[ovs-dev] [PATCH ovn] Make tunnel ids exhaustion test case trigger the problem.

The original version of the scenario passed with or without the fix. Fixes: a1f165a7b807 ("northd: fix infinite loop in ovn_allocate_tnlid()") Signed-off-by: Ihar Hrachyshka --- tests/ovn-northd.at | 17 +++-- 1 file changed, 11 insertions(+), 6 deletions(-) diff --git