lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 30 Nov 2009 11:00:41 -0800 From: ebiederm@...ssion.com (Eric W. Biederman) To: Octavian Purdila <opurdila@...acom.com> Cc: David Miller <davem@...emloft.net>, netdev@...r.kernel.org, jamal <hadi@...erus.ca>, Daniel Lezcano <dlezcano@...ibm.com>, Alexey Dobriyan <adobriyan@...il.com>, Patrick McHardy <kaber@...sh.net> Subject: Re: [PATCH 01/20] net: NETDEV_UNREGISTER_PERNET -> NETDEV_UNREGISTER_BATCH Octavian Purdila <opurdila@...acom.com> writes: > On Monday 30 November 2009 03:45:58 you wrote: > >> + /* Process any work delayed until the end of the batch */ >> + dev = list_entry(head->next, struct net_device, unreg_list); >> + call_netdevice_notifiers(NETDEV_UNREGISTER_BATCH, dev); >> >> - list_for_each_entry_safe(dev, aux, head, unreg_list) { >> - int new_net = 1; >> - list_for_each_entry(fdev, &pernet_list, unreg_list) { >> - if (net_eq(dev_net(dev), dev_net(fdev))) { >> - new_net = 0; >> - dev_put(dev); >> - break; >> - } >> - } >> - if (new_net) >> - list_move(&dev->unreg_list, &pernet_list); >> - } >> > > This implies that the devices we want to batch unregister needs to be part of > the same namespace. It might be worth mentioning that in the > unregister_netdevice_many() function comments. No it does not. For the route cache flush we have already done the delayed invalidate per namespace. So we only need to walk the route cache once and simply remove the entries that have been invalidated. For other cases if we really need it, I pass in the first entry of the unregistration list so we can recover the list if needed. The implication is that namespaces are not what is important when batching so it doesn't make sense to turn our list into a pretzel providing information that is not needed. > And, fortunately, it seems that all of the current batch unregister users > comply with this requirement. The case I am working on for this patchset is batching of the destruction of devices across namespace cleanup. This change reduces my total number of walks of the route cache from many down to one. Eric -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists