Hi Phil, Sorry for slowness. On Wed, Oct 23, 2024 at 04:57:23PM +0200, Phil Sutter wrote: > Changes since v5: > - Extract the initial set of patches making netdev hooks name-based as > suggested by Florian. > - Drop Fixes: tag from patch 1: It is not correct (the pointless check > existed before that commit already) and it is rather an optimization > than fixing a bug. > > This series makes netdev hooks store the interface name spec they were > created for and establishes this stored name as the key identifier. The > previous one which is the hook's 'ops.dev' pointer is thereby freed to > vanish, so a vanishing netdev no longer has to drag the hook along with > it. (Patches 2-4) > > Furthermore, it aligns behaviour of netdev-family chains with that of > flowtables in situations of vanishing interfaces. When previously a > chain losing its last interface was torn down and deleted, it may now > remain in place (albeit with no remaining interfaces). (Patch 5) > > Patch 6 is a cleanup following patch 5, patches 1 and 7 are independent > code simplifications. Patch 1-4 can be integrated, they are relatively small. Patches 5-6 will need a rebase due to my fix in that path. Patch 7 is probably uncovering an issue with flowtable hardware offload support, because I suspect _UNBIND is not called from that path, I need to have a look. I am inclined to postpone this batch to the next development cycle.
