On 9/1/22 22:17, Dumitru Ceara wrote:
> This implementation was removed by accident between V4 [0] and V5 [1]
> of a7c7d4519e50 ("controller: avoid recomputes triggered by SBDB
> Port_Binding updates.").
>
> [0]
>
This implementation was removed by accident between V4 [0] and V5 [1]
of a7c7d4519e50 ("controller: avoid recomputes triggered by SBDB
Port_Binding updates.").
[0]
https://patchwork.ozlabs.org/project/ovn/patch/20220819151853.2149088-1-xsimo...@redhat.com/
[1]
Hi,
If I recall correctly, the openvswitch-dpdk package was obsoleted
in 2016 in favor of having only one package with dpdk enabled.
Of course, upstream can decide whether it makes sense to follow
that or not.
Another thing is that openvswitch-fedora.spec should work much
better with RHEL-8
Using [NETNS|OVS]_DAEMONIZE will start tcpdump in the background,
and it will also make sure it gets killed in corner cases.
For the check_pkt_len tests, we also kill tcpdump between individual
tests in the same test case to avoid confusion when analyzing results.
This also required some changes
When OVS fails to find an OpenFlow port for a packet received
from the upcall it just prints the warning like this:
|INFO|received packet on unassociated datapath port N
However, during the flow translation more information is available
as if the recirculation id wasn't found or it was a
'openvswitch' and 'openvswitch-dpdk' packages are not available
starting with RHEL 8. The only option to get an official OVS
build is to use FDP repository, which is not available with the
default RHEL subscription.
Also, the spec file is outdated. We need python3, not python2.
Some headers are
On 8/31/22 21:45, Mike Pattrick wrote:
> On Thu, Aug 25, 2022 at 6:14 AM Daniel Ding wrote:
>>
>> If ovs-tcpdump received HUP or TERM signal, mirror and mirror
>> interface should be destroyed. This often happens, when
>> controlling terminal is closed, like ssh session closed, and
>> other users
On Thu, Sep 1, 2022 at 3:33 PM Eelco Chaudron wrote:
> >> -AT_CHECK([cat p3.pcap | awk '{print $NF}' | uniq -c | awk
> >> '{$1=$1;print}'], [0], [dnl
> >> +AT_CHECK([cat p3.pcap | awk 'NF{print $NF}' | uniq -c | awk
> >> '{$1=$1;print}'], [0], [dnl
> >
> > I don't see the relation with
On 1 Sep 2022, at 14:11, David Marchand wrote:
> On Wed, Aug 31, 2022 at 1:21 PM Eelco Chaudron wrote:
>>
>> Using NETNS_DAEMONIZE will start tcpdump in the background, and it will
>> also make sure it gets killed in corner cases.
>>
>> For the check_pkt_len tests, we also kill tcpdump between
Signed-off-by: Dumitru Ceara
---
Note: this change is just cosmetic, no functionality should be affected
by it.
---
utilities/ovn-nbctl.c | 50 ++-
1 file changed, 26 insertions(+), 24 deletions(-)
diff --git a/utilities/ovn-nbctl.c
On Wed, Aug 31, 2022 at 1:21 PM Eelco Chaudron wrote:
>
> Using NETNS_DAEMONIZE will start tcpdump in the background, and it will
> also make sure it gets killed in corner cases.
>
> For the check_pkt_len tests, we also kill tcpdump between individual
> tests in the same test case to avoid
Packets that could not be transmitted because the TXQ are full should be
taken into account in the global ovs_tx_failure_drops as it was the case
before commit 29b94e12d57d ("netdev-dpdk: Refactor the DPDK transmit
path.").
netdev_dpdk_eth_tx_burst() returns the number of packets that were *not*
Hi Mike,
Mike Pattrick, Aug 31, 2022 at 18:09:
> On Wed, Aug 3, 2022 at 4:58 AM Robin Jarry wrote:
> > dropped += netdev_dpdk_eth_tx_burst(dev, qid, pkts, cnt);
> > +stats.tx_failure_drops += dropped;
>
> Hello Robin,
>
> I think this will double count tx_failure_drops if there were any
> > On 8/25/22 14:52, David Marchand wrote:
> > > Debian packaging target builds OVS against a packaged DPDK version.
> > > As a result, it is not relevant in the dpdk-latest branch which follows
> > > DPDK development branch.
> > >
> > > Signed-off-by: David Marchand
> > > ---
> > >
On Wed, Aug 31, 2022 at 2:29 PM Ilya Maximets wrote:
>
> On 8/25/22 14:52, David Marchand wrote:
> > Debian packaging target builds OVS against a packaged DPDK version.
> > As a result, it is not relevant in the dpdk-latest branch which follows
> > DPDK development branch.
> >
> > Signed-off-by:
15 matches
Mail list logo