Re: [ovs-dev] [PATCH V6 05/18] netdev-dpdk: Introduce rte flow query function

2019-12-28 Thread 贺鹏
Hi, there is a potential memory leak in ovs-dpdk hw offload, when remove a dpdk port in datapath, the dpif will call dpif_port_del and will eventually remove the netdev from the map *port_to_netdev* (in netdev-offload). Meanwhile, a resulted datapath reconfiguration will flush all the marked

Re: [ovs-dev] [ovs-dev 7/7] netdev-offload-dpdk: Add dpdk offload meter action

2019-12-28 Thread 0-day Robot
Bleep bloop. Greetings Tonghao Zhang, 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: Line has non-spaces leading whitespace WARNING: Line has trailing whitespace #223

Re: [ovs-dev] [ovs-dev 5/7] netdev-dpif: Implement netdev meter offload

2019-12-28 Thread 0-day Robot
Bleep bloop. Greetings Tonghao Zhang, 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: Line has non-spaces leading whitespace WARNING: Line has trailing whitespace #34

Re: [ovs-dev] [ovs-dev 3/7] dpif-provider: Add dpif provider meter api

2019-12-28 Thread 0-day Robot
Bleep bloop. Greetings Tonghao Zhang, 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: Line is 84 characters long (recommended limit is 79) #95 FILE: lib/dpif.c:1997:

Re: [ovs-dev] [ovs-dev 2/7] netdev-offload: Pass dpif as an argument to flow_put

2019-12-28 Thread 0-day Robot
Bleep bloop. Greetings Tonghao Zhang, 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: Line is 81 characters long (recommended limit is 79) #78 FILE: