On Fri, 08 May 2009 15:03:52 -0400
"Anurag S. Maskey" <Anurag.Maskey at Sun.COM> wrote:

> logging.c:69-70
> - these changes have already been pushed.  you're missing a pull, merge, 
> commit, recommit, i think.

Last time I put back this nonsense thinking it fixed 8796.  I'm undoing it.

> 
> sysevent_events.c:169-170
> - missing pthread_attr_destroy() before pfail().

pfail() exit()s.

> 
> dlpi_events.c
> - good catch on this early free()ing.  my head was still spinning 
> looking at the log
> 
> events.h
> - the actual walkcontext() call is commented in ncu_phys.c.  Let's have 
> a bug to resolve the deadlock issue in ncu_phys.c

Okay, I'll write a bug about syslog_stack().

                mph

> 
> Anurag
> 
> 
> Michael Hunter wrote:
> > webrev at /net/coupe.eng/builds/mph/nwam1_work/webrev/
> >
> > for
> >
> >   8778 pthread_attr_destroy is only called 1/4 of the time
> >   8796 NWAM Daemon generating bogus messages...
> >
> > Sorry for the internal webrev.
> >
> >                 mph
> > _______________________________________________
> > nwam-dev mailing list
> > nwam-dev at opensolaris.org
> > http://mail.opensolaris.org/mailman/listinfo/nwam-dev
> >   

Reply via email to