Pablo Neira Ayuso <pablo@xxxxxxxxxxxxx> wrote: > Updating netdev basechain is broken in many ways. > > Keeping a list of pending hooks to be added/deleted in the transaction > object does not mix well with table flag updates (ie. setting dormant > flag in table) which operate on the existing basechain hook list. > Instead, add/delete hook to/from the basechain hook list and allocate > one transaction object per new device to refers to the hook to > add/delete. > > Add an 'inactive' flag that is set on to identify devices that has been > already deleted, so double deletion in one batch is not possible. Do you think it makes sense to remove dormant flag support for the netdev family? It would avoid the register/unregister entanglements and might reduce headaches down the road. IOW, do you think dormant flag toggling is useful for netdev family?
