[Devel] Re: [PATCH] net: Marking struct pernet_operations __net_initdata was inappropriate
David Miller
davem at davemloft.net
Fri Oct 26 22:55:16 PDT 2007
From: ebiederm at xmission.com (Eric W. Biederman)
Date: Fri, 26 Oct 2007 17:45:33 -0600
>
> It is not safe to to place struct pernet_operations in a special section.
> We need struct pernet_operations to last until we call unregister_pernet_subsys.
> Which doesn't happen until module unload.
>
> So marking struct pernet_operations is a disaster for modules in two ways.
> - We discard it before we call the exit method it points to.
> - Because I keep struct pernet_operations on a linked list discarding
> it for compiled in code removes elements in the middle of a linked
> list and does horrible things for linked insert.
>
> So this looks safe assuming __exit_refok is not discarded
> for modules.
>
> Signed-off-by: Eric W. Biederman <ebiederm at xmission.com>
Applied, thanks Eric.
Although juding by his comments I though that Denis had different
plans in mind to fix this.
_______________________________________________
Containers mailing list
Containers at lists.linux-foundation.org
https://lists.linux-foundation.org/mailman/listinfo/containers
More information about the Devel
mailing list