Re: module: Ensure a module's state is set accordingly during module coming cleanup code

2016-11-23 Thread Jessica Yu
+++ Miroslav Benes [16/11/16 16:49 +0100]: On Wed, 9 Nov 2016, Jessica Yu wrote: +++ Rusty Russell [26/10/16 11:24 +1030]: > Aaron Tomlin writes: > > In load_module() in the event of an error, for e.g. unknown module > > parameter(s) specified we go to perform some module coming clean up > > o

Re: module: Ensure a module's state is set accordingly during module coming cleanup code

2016-11-16 Thread Miroslav Benes
On Wed, 9 Nov 2016, Jessica Yu wrote: > +++ Rusty Russell [26/10/16 11:24 +1030]: > > Aaron Tomlin writes: > > > In load_module() in the event of an error, for e.g. unknown module > > > parameter(s) specified we go to perform some module coming clean up > > > operations. At this point the module

Re: module: Ensure a module's state is set accordingly during module coming cleanup code

2016-11-09 Thread Jessica Yu
+++ Rusty Russell [26/10/16 11:24 +1030]: Aaron Tomlin writes: In load_module() in the event of an error, for e.g. unknown module parameter(s) specified we go to perform some module coming clean up operations. At this point the module is still in a "formed" state when it is actually going away.

Re: [RFC PATCH 1/2] module: Ensure a module's state is set accordingly during module coming cleanup code

2016-10-25 Thread Rusty Russell
Aaron Tomlin writes: > In load_module() in the event of an error, for e.g. unknown module > parameter(s) specified we go to perform some module coming clean up > operations. At this point the module is still in a "formed" state > when it is actually going away. > > This patch updates the module's

[RFC PATCH 1/2] module: Ensure a module's state is set accordingly during module coming cleanup code

2016-10-20 Thread Aaron Tomlin
In load_module() in the event of an error, for e.g. unknown module parameter(s) specified we go to perform some module coming clean up operations. At this point the module is still in a "formed" state when it is actually going away. This patch updates the module's state accordingly to ensure anyon