On Thu, Jul 5, 2018 at 7:24 AM Vlad Buslov <vla...@mellanox.com> wrote: > > Implement function that atomically checks if action exists and either takes > reference to it, or allocates idr slot for action index to prevent > concurrent allocations of actions with same index. Use EBUSY error pointer > to indicate that idr slot is reserved.
A dumb question: How could "concurrent allocations of actions with same index" happen as you already take idrinfo->lock for the whole tcf_idr_check_alloc()?? For me, it should be only one allocation could succeed, all others should fail. Maybe you are trying to prevent others treat it like existing one, but in that case you can just hold the idinfo->lock for all idr operations. And more importantly, upper layer is able to tell it is a creation or just replace, you don't have to check this in this complicated way. IOW, all of these complicated code should not exist.
