[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [lvm-devel] [PATCH 10/11] Remove redundant init_mirror_in_sync() call in init_lvm().



On Wed, 2008-12-10 at 22:50 +0000, Alasdair G Kergon wrote:
> On Wed, Dec 10, 2008 at 01:11:00PM -0500, Dave Wysochanski wrote:
> > _mirror_in_sync is statically initialized to 0 in lvm-globals.c,
> > so no need to set it in init_lvm().  Only other callers that may
> > set this are later in execution.
>  
> Can the statics be relied upon if a handle is created, then destroyed
> then a new handle created within the same process?
> 
> It basically depends if that initial value is relied upon anywhere, or
> if later code always sets this before using it.
> 

This is a good point.  We will have some global state and this needs
thought out.  We may need to export _apply_settings() or something like
it to reset the global state.  Or perhaps the global state should be
reset at the bottom of create_toolcontext()?

For the the handle re-use case of the existing lvmcmdlib.c, we do a
_apply_settings() in lvm_run_command() so the global state is reset. We
won't have lvm_run_command() to rely on for liblvm, but I guess we need
the equivalent.




[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]