Re: [ovs-dev] [PATCH v2 5/5] netdev-offload: Fix Clang's static analyzer 'Division by zero' warnings.

2023-10-25 Thread Simon Horman
On Mon, Oct 23, 2023 at 04:22:47PM +0200, Eelco Chaudron wrote: > When enabling DPDK with the configure the below, ovs-vswitchd will crash. > > ovs-vsctl set Open_vSwitch . other_config:n-offload-threads=0 > ovs-vsctl set Open_vSwitch . other_config:hw-offload=true > > This issue arises

Re: [ovs-dev] [PATCH v2 5/5] netdev-offload: Fix Clang's static analyzer 'Division by zero' warnings.

2023-10-23 Thread 0-day Robot
Bleep bloop. Greetings Eelco Chaudron, I am a robot and I have tried out your patch. Thanks for your contribution. I encountered some error that I wasn't expecting. See the details below. checkpatch: WARNING: The subject, ': ', is over 70 characters, i.e., 72. Subject: netdev-offload: Fix

[ovs-dev] [PATCH v2 5/5] netdev-offload: Fix Clang's static analyzer 'Division by zero' warnings.

2023-10-23 Thread Eelco Chaudron
When enabling DPDK with the configure the below, ovs-vswitchd will crash. ovs-vsctl set Open_vSwitch . other_config:n-offload-threads=0 ovs-vsctl set Open_vSwitch . other_config:hw-offload=true This issue arises because setting the 'n-offload-threads' value to zero is not a supported