OK, it seems that my last hypothesis was the good one : mod_python is
doing things with thread states that are frowned upon by debug build.
The tests are only performed in the debug build, so that's why we had
no problem with the release build.
The tests are found in pystate.c, in function PyThreadState_Swap (line
297 in the current trunk revision) :
/* It should not be possible for more than one thread state
to be used for a thread. Check this the best we can in debug
builds.
*/
#if defined(Py_DEBUG) && defined(WITH_THREAD)
if (newts) {
PyThreadState *check = PyGILState_GetThisThreadState();
if (check && check->interp == newts->interp && check != newts)
Py_FatalError("Invalid thread state for this thread");
}
#endif
So it seems mod_python is trying to share the same thread state
between multiple threads, which "should not be possible". If someone
has an idea, please help me, because I'm not really up to date about
thread state management, so I'll have to read a fair bit of
documentation before understanding what's happening there.
Regards,
Nicolas
2006/5/20, Nicolas Lehuen <[EMAIL PROTECTED]>:
I've forgot to mention the platform : as usual for me, it's Windows XP SP2.
Regards,
Nicolas
2006/5/20, Nicolas Lehuen <[EMAIL PROTECTED]>:
> Hi Graham,
>
> After a few interesting weeks, I finally manage to get some time to
> help on mod_python.
>
> I've ran the tests on the latest Subversion revision (407968) with
> Apache 2.0.58 and Python 2.4.3, and all test pass, including the SSI
> ones.
>
> For a yet unknown reason, every test fail with a debug build of the
> Python trunk with the aformentioned "Invalid thread state for this
> thread" error.
>
> I don't know whether this is due to a mistake from me, a bug in the
> Python trunk, or if it is that the debug build is a little bit more
> cautious about thread state management and breaks earlier than the
> non-debug version when faced by a faulty mod_python thread state
> management code... I'll try to see if we still have the segfault that
> could be sometimes observed when the Apache server is stopped, as it
> may be related.
>
> Best regards,
> Nicolas
>
> 2006/5/9, Graham Dumpleton <[EMAIL PROTECTED]>:
> > Nicolas Lehuen wrote ..
> > > Hi Graham,
> > >
> > > The latest trunk version yields multiple segfaults and failures in
> > > different places :
> > >
> > > * Testing req.add_handler() for empty phase
> > > E
> > > * Testing req.add_handler() directory
> > > E
> > > * Testing interpreter per directive
> > > E
> > > * Testing phase status
> > > F
> > > * Testing server side include
> > > F
> >
> > Okay Nicolas, can you check out latest and give it a go again. Got rid of
> > obvious mistake of not deleting old variable definition. My checking was
> > not as rigourous as it should as I should have picked up that interpreter
> > name was wrong.
> >
> > The phase status example will probably still fail as don't understand that
> > one yet. It may be an auth setup issue in test suite configuration for that
> > specific test.
> >
> > Graham
> >
>
