Hi, IIya, thanks for the attention. If I create a zone for the first time, the new tp_cfg will be copied to the zone, see `ct_zone_alloc`. Then update_timeout_policy will check that the new copied tp equals to tp_cfg, so ofproto_ct_set_zone_timeout_policy will not got called. Or you can check tag v3.0.0 or before. There's no such issue for these versions.
Best regards. On Tue, May 14, 2024 at 5:11 AM Ilya Maximets <[email protected]> wrote: > On 5/6/24 06:05, Chandler Wu wrote: > > From 5b4d479a9083272e56c51ef9521e6ef665dd534d Mon Sep 17 00:00:00 2001 > > From: chandlerwu <[email protected]> > > Date: Mon, 6 May 2024 11:58:21 +0800 > > Subject: [PATCH] Subject:[PATCH] fix ct tp policy when create zone. > > > > Signed-off-by: chandlerwu <[email protected]> > > --- > > vswitchd/bridge.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c > > index 95a65fcdc..e60359b59 100644 > > --- a/vswitchd/bridge.c > > +++ b/vswitchd/bridge.c > > @@ -766,6 +766,8 @@ ct_zones_reconfigure(struct datapath *dp, struct > ovsrec_datapath *dp_cfg) > > if (!ct_zone) { > > ct_zone = ct_zone_alloc(zone_id, tp_cfg); > > hmap_insert(&dp->ct_zones, &ct_zone->node, > hash_int(zone_id, 0)); > > + ofproto_ct_set_zone_timeout_policy(dp->type, > ct_zone->zone_id, > > + &ct_zone->tp); > > } > > > > struct simap new_tp = SIMAP_INITIALIZER(&new_tp); > > Hi, Chandler Wu. Thanks for the patch! > > But could you, please, explain what is the issue you're trying to fix? > > Reading the code, it seems that update_timeout_policy() call later in > the function should correctly set the policy. Is that not happening? > > Best regards, Ilya Maximets. > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
