merged thanks -steve
On Fri, 2008-07-11 at 15:12 +0200, Fabio M. Di Nitto wrote: > On Fri, 11 Jul 2008, Christine Caulfield wrote: > > > Fabio M. Di Nitto wrote: > >> > >> Hi Steven, > >> > >> as discussed in a previous thread logsys_config_subsys_set does not allow > >> to set a lower priority (for example switching from LOG_LEVEL_DEBUG to > >> LOG_LEVEL_INFO). > >> > >> This behaviour can be problematic if we ask people to switch debugging on > >> and then they can't turn it off. > >> > >> The patch in attachment allows logsys to roll back to whatever priority > >> the > >> user wants. > >> > >> Please apply > >> > >> > > The problem with that patch is that it's the opposite of the one I posted > > and > > breaks (or rather leaves broken) the initialisation code. > > > > If a logging priority is set in the configuration file, then that is set > > into > > logsys during config processing. When the lcrso modules are subsequently > > loaded they will reset the logging level back to their hard-coded default. > > So > > the config options are useless. > > > > Chrissie > > > > The patch in attachment fixes the problem at the root by not forcing > priority re-init if the subsystem is already configured. > > Please apply > > Fabio > > -- > I'm going to make him an offer he can't refuse. > _______________________________________________ Openais mailing list > [email protected] > https://lists.linux-foundation.org/mailman/listinfo/openais _______________________________________________ Openais mailing list [email protected] https://lists.linux-foundation.org/mailman/listinfo/openais
