Milter-greylist

On Sun, Jan 30, 2011 at 04:05:44AM +0100, manu@... wrote:
> Francois Tigeot <ftigeot@...> wrote:
> 
> > I guess it's on line 133:
> The first call, on top of dumper(), right?

That's right.

> This means we launch two dumper threads, which is a good reason for
> getting into troubled.
> 
> Can you add debug messages at the beginning of dumper(), in places where
> dumper() is called (there should be only dumper_start()), and before the
> dumper_start() call (there should be only one, in main())?

I get some messages when I start milter-greylist:

673054832: calling dumper_start() from main, line 1711
673054832: calling dumper() from dumper_start, line 116
673055056: conf_retain called from dumper, line 134
conf.c:351 BUG: conf_retain called twice?

But there's nothing when it crashes.

> I have trouble to figure how we go there twice. If we don't understand,
> we will alwys have the possibility to use a flag to prevent the second
> execution, but I would like to understand.

All I can say for now is there has been a great deal of work on SMP and
threading between this DragonFly release and the previous one.

-- 
Francois Tigeot