Re: [ovs-dev] [ovs-dev v2 2/2] ofproto-dpif: fix meter use-after-free

2022-05-10 Thread Peng He
Hi, thanks for the detailed comments. will submit a new version. Eelco Chaudron 于2022年5月9日周一 16:35写道: > > > On 1 May 2022, at 4:17, Peng He wrote: > > > add a rcu_barrier before close_dpif_backer to ensure that > > all meters has been freed before id_pool_destory meter's > > id-pool. > > > > S

Re: [ovs-dev] [ovs-dev v2 2/2] ofproto-dpif: fix meter use-after-free

2022-05-09 Thread Eelco Chaudron
On 1 May 2022, at 4:17, Peng He wrote: > add a rcu_barrier before close_dpif_backer to ensure that > all meters has been freed before id_pool_destory meter's > id-pool. > > Signed-off-by: Peng He > Tested-by: David Marchand > --- > ofproto/ofproto-dpif.c | 2 ++ > 1 file changed, 2 insertion

Re: [ovs-dev] [ovs-dev v2 2/2] ofproto-dpif: fix meter use-after-free

2022-04-30 Thread 0-day Robot
Bleep bloop. Greetings Peng He, 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: ERROR: Author Peng He needs to sign off. WARNING: Unexpected sign-offs from developers who are not a

[ovs-dev] [ovs-dev v2 2/2] ofproto-dpif: fix meter use-after-free

2022-04-30 Thread Peng He
add a rcu_barrier before close_dpif_backer to ensure that all meters has been freed before id_pool_destory meter's id-pool. Signed-off-by: Peng He Tested-by: David Marchand --- ofproto/ofproto-dpif.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/ofproto/ofproto-dpif.c b/ofproto/ofproto-