On 7/1/25 1:41 PM, Roi Dayan via dev wrote:
> From: Eli Britstein <el...@nvidia.com>
> 
> A group has a hash_map that contains the array of its buckets.
> Revalidator threads access it to check if some bucket is alive, while
> the main thread may modify the group, re-allocating this array. This race
> condition may lead to invalid memory access.

Hi, Eli and Roi.

Groups are actually never modified, they are re-created.  AFAIR, the
group_modify() callback is only used from the ofproto_group_mod_finish()
for the newly created group that cannot be accessed by any other
threads as it's not published yet.

Could you explain in more detail the code path that leads to invalid
memory access in you case?

Best regards, Ilya Maximets.
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to