You are viewing a plain text version of this content. The canonical link for it is here.
Posted to modperl@perl.apache.org by Christian Hansen <ch...@ngmedia.com> on 2005/05/13 09:16:26 UTC

[MP2] 1.999024 fails on OS X

Program received signal EXC_BAD_ACCESS, Could not access memory.
0x014563fc in S_mess_alloc ()

(gdb) bt
#0  0x014563fc in S_mess_alloc ()
#1  0x014566e8 in Perl_vmess ()
#2  0x014571dc in Perl_vcroak ()
#3  0x014576b8 in Perl_croak_nocontext ()
#4  0x007dd9c8 in modperl_global_cleanup ()
#5  0x006a3f60 in run_cleanups (cref=0x1806228) at apr_pools.c:1952
#6  0x006a35f4 in apr_pool_clear (pool=0xbffffa1c) at apr_pools.c:694
#7  0x0000ac34 in main (argc=11, argv=0xbffffc40) at main.c:575

Looks like I'm hitting the same problem that Philippe M. Chiasson 
reported last month, 
http://article.gmane.org/gmane.comp.apache.mod-perl/18408

I'm using trunk, revision: 169956. With 1.999.21 all tests passes, so 
guess something changed in the namespace change.


Please let me know if I can be at further assistance.

Regards
Christian Hansen




1. Problem Description:

2. Used Components and their Configuration:

*** mod_perl version 1.999024

*** using /Users/chansen/src/mod_perl-2.0/lib/Apache2/BuildConfig.pm

*** Makefile.PL options:
   MP_APR_LIB     => aprext
   MP_APXS        => /usr/local/apache2/bin/apxs
   MP_COMPAT_1X   => 1
   MP_GENERATE_XS => 1
   MP_LIBNAME     => mod_perl
   MP_USE_DSO     => 1


*** /usr/local/apache2/bin/httpd -V
Server version: Apache/2.0.53
Server built:   May  6 2005 06:36:06
Server's Module Magic Number: 20020903:9
Architecture:   32-bit
Server compiled with....
  -D APACHE_MPM_DIR="server/mpm/prefork"
  -D APR_HAS_MMAP
  -D APR_HAVE_IPV6 (IPv4-mapped addresses enabled)
  -D APR_USE_SYSVSEM_SERIALIZE
  -D APR_USE_PTHREAD_SERIALIZE
  -D SINGLE_LISTEN_UNSERIALIZED_ACCEPT
  -D APR_HAS_OTHER_CHILD
  -D AP_HAVE_RELIABLE_PIPED_LOGS
  -D HTTPD_ROOT="/usr/local/apache2"
  -D SUEXEC_BIN="/usr/local/apache2/bin/suexec"
  -D DEFAULT_PIDLOG="logs/httpd.pid"
  -D DEFAULT_SCOREBOARD="logs/apache_runtime_status"
  -D DEFAULT_LOCKFILE="logs/accept.lock"
  -D DEFAULT_ERRORLOG="logs/error_log"
  -D AP_TYPES_CONFIG_FILE="conf/mime.types"
  -D SERVER_CONFIG_FILE="conf/httpd.conf"


*** (apr|apu)-config linking info

  -L/usr/local/lib -laprutil-0 -lgdbm -ldb-4.2 -lexpat -liconv
  -L/usr/local/lib -lapr-0 -lresolv  -lpthread



*** /usr/local/bin/perl -V
Summary of my perl5 (revision 5 version 8 subversion 5) configuration:
   Platform:
     osname=darwin, osvers=7.4.0, archname=darwin-thread-multi-2level
     uname='darwin mundus.corp.ngmedia.net 7.4.0 darwin kernel version 
7.4.0: wed may 12 16:58:24 pdt 2004; 
root:xnuxnu-517.7.7.obj~7release_ppc power macintosh powerpc '
     config_args='-ds -e -des -Dusethreads -Duseshrplib 
-Dperladmin=ch@ngmedia.com -Dcf_email=ch@ngmedia.com'
     hint=recommended, useposix=true, d_sigaction=define
     usethreads=define use5005threads=undef useithreads=define 
usemultiplicity=define
     useperlio=define d_sfio=undef uselargefiles=define usesocks=undef
     use64bitint=undef use64bitall=undef uselongdouble=undef
     usemymalloc=n, bincompat5005=undef
   Compiler:
     cc='cc', ccflags ='-fno-common -DPERL_DARWIN -no-cpp-precomp 
-fno-strict-aliasing -pipe -I/usr/local/include',
     optimize='-Os',
     cppflags='-no-cpp-precomp -fno-common -DPERL_DARWIN -no-cpp-precomp 
-fno-strict-aliasing -pipe -I/usr/local/include'
     ccversion='', gccversion='3.3 20030304 (Apple Computer, Inc. build 
1495)', gccosandvers=''
     intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=4321
     d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=8
     ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', 
lseeksize=8
     alignbytes=8, prototype=define
   Linker and Libraries:
     ld='env MACOSX_DEPLOYMENT_TARGET=10.3 cc', ldflags =' -L/usr/local/lib'
     libpth=/usr/local/lib /usr/lib
     libs=-lgdbm -ldbm -ldb -ldl -lm -lc
     perllibs=-ldl -lm -lc
     libc=/usr/lib/libc.dylib, so=dylib, useshrplib=true, 
libperl=libperl.dylib
     gnulibc_version=''
   Dynamic Linking:
     dlsrc=dl_dyld.xs, dlext=bundle, d_dlsymun=undef, ccdlflags=' '
     cccdlflags=' ', lddlflags=' -bundle -undefined dynamic_lookup 
-L/usr/local/lib'


Characteristics of this binary (from libperl):
   Compile-time options: MULTIPLICITY USE_ITHREADS USE_LARGE_FILES 
PERL_IMPLICIT_CONTEXT
   Built under darwin
   Compiled at Aug  4 2004 19:13:41
   %ENV:
     PERL_LWP_USE_HTTP_10="1"
   @INC:
     /usr/local/lib/perl5/5.8.5/darwin-thread-multi-2level
     /usr/local/lib/perl5/5.8.5
     /usr/local/lib/perl5/site_perl/5.8.5/darwin-thread-multi-2level
     /usr/local/lib/perl5/site_perl/5.8.5
     /usr/local/lib/perl5/site_perl/5.8.4/darwin-thread-multi-2level
     /usr/local/lib/perl5/site_perl/5.8.4
     /usr/local/lib/perl5/site_perl/5.8.3/darwin-thread-multi-2level
     /usr/local/lib/perl5/site_perl/5.8.3
     /usr/local/lib/perl5/site_perl/5.8.2/darwin-thread-multi-2level
     /usr/local/lib/perl5/site_perl/5.8.2
     /usr/local/lib/perl5/site_perl
     .

*** Packages of interest status:

Apache2         : -
Apache2::Request: -
CGI             : 3.05
LWP             : 5.76, 5.800, 5.803
mod_perl        : -
mod_perl2       : -


3. This is the core dump trace: (if you get a core dump):

This report was generated by t/REPORT on Fri May 13 06:52:50 2005 GMT.


Re: [MP2] 1.999024 fails on OS X

Posted by Christian Hansen <ch...@ngmedia.com>.
Stas Bekman wrote:
> 
> Christian, any difference if you test with perl w/o ithreads?

Yes, perl 5.8.6 compiled without ithreads works. perl 5.8.5 and 5.8.6 
with ithreads fails.

Regards
Christian Hansen


Re: [MP2] 1.999024 fails on OS X

Posted by Christian Hansen <ch...@ngmedia.com>.
Stas Bekman wrote:
> Christian Hansen wrote:
> 
>> I'm using trunk, revision: 169956. With 1.999.21 all tests passes, so 
>> guess something changed in the namespace change.
> 
> 
> With the same perl and apache versions/configuration?

Yes exactly the same, i did a retest before submitting this report.

> Christian, any difference if you test with perl w/o ithreads?

I'll get back to you as soon as i have compiled a perl without threads.


Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> In any case, go ahead and commit your fix for now to get it working on
> OSX. we will figure out the special case posted by Dominique after 2.0.0
> is released. No point to keep on postponing the release date.

Allright, I still think that my patch is correctly fixing the problem though.

But you are correct, we can discuss thus further later.

Comitted as r170747

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> In any case, go ahead and commit your fix for now to get it working on
> OSX. we will figure out the special case posted by Dominique after 2.0.0
> is released. No point to keep on postponing the release date.

Allright, I still think that my patch is correctly fixing the problem though.

But you are correct, we can discuss thus further later.

Comitted as r170747

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
In any case, go ahead and commit your fix for now to get it working on 
OSX. we will figure out the special case posted by Dominique after 2.0.0 
is released. No point to keep on postponing the release date.

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
In any case, go ahead and commit your fix for now to get it working on 
OSX. we will figure out the special case posted by Dominique after 2.0.0 
is released. No point to keep on postponing the release date.

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> 
>> Stas Bekman wrote:
>>
>>> Philippe M. Chiasson wrote:
>>> [...]
>>>
>>>> After a long promised investigation session, I believe I have pinned
>>>> it down. It seems to have been introduced by change r160562 when the
>>>> anon_cnt initialization code was moved around as a result. Didn't have
>>>> enough time tonight to dig down as to _why_ this is breaking though...
>>>>
>>>> Not entirely sure why this breaks specifically on Darwin and all, but
>>>> the following patch seems to make the problem dissapear. Does it fix
>>>> the problem you are seeing too ?
>>>>
>>>> Index: src/modules/perl/modperl_handler.c
>>>> ===================================================================
>>>> --- src/modules/perl/modperl_handler.c  (revision 170139)
>>>> +++ src/modules/perl/modperl_handler.c  (working copy)
>>>> @@ -89,9 +89,6 @@
>>>>         modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>>>     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>>>     MP_MODGLOBAL_STORE_HV(gkey);
>>>> -
>>>> -    /* init the counter to 0 */
>>>> -    modperl_global_anon_cnt_init(p);
>>>> }
>>>>
>>>> /* allocate and populate the anon handler sub-struct */
>>>> Index: src/modules/perl/mod_perl.c
>>>> ===================================================================
>>>> --- src/modules/perl/mod_perl.c (revision 170139)
>>>> +++ src/modules/perl/mod_perl.c (working copy)
>>>> @@ -532,6 +532,9 @@
>>>>     modperl_global_init_server_rec(pconf, s);
>>>>
>>>>     modperl_tls_create_request_rec(pconf);
>>>> +
>>>> +    /* init the counter to 0 */
>>>> +    modperl_global_anon_cnt_init(pconf);
>>>> }
>>>>
>>>> /*
>>>
>>> Philippe is talking about
>>> http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
>>>
>>> http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
>>>
>>
>> Yup, that one.
>>
>>> But it can't be reverted , since it was a fix for a bug in
>>> vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
>>> as it wasn't suitable for the test suite (would have affected other
>>> tests). So it needs to stay that way and figured out what needs to be
>>> fixed elsewhere.
>>>
>>> Philippe, your patch can't be applied either, since it has exactly the
>>> same effect as reverting r160562, since you moved
>>> modperl_global_anon_cnt_init() to modperl_init_globals() which is called
>>> only for the top-level server
>>
>> Yes, but I am not reverting that change, keeping it as is, hash
>> initialization
>> and all, and only moving the modperl_global_anon_cnt_init() up so it's
>> called
>> only once. I don't think initializating that anon_cnt global more than
>> once is
>> a feature (and it registers it's destruction twice, causing the
>> segfault on OSX).
> 
> No, you are reverting it. Follow the calls trace, you've just moved the
> original case (before my change) to a different place.

Unless I am missing something terribly obvious, I believe you are talking
about this chunk of code:

    modperl_modglobal_key_t *gkey =
        modperl_modglobal_lookup(aTHX_ "ANONSUB");
    MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
    MP_MODGLOBAL_STORE_HV(gkey);

And that's what needs to be initialized in each server/vhost, correct?
And that still is. The only code I am suggesting to move around is the
modperl_global_anon_cnt_init() call

>> Are you saying that you need more than one distinct global anon_cnt in
>> some
>> vhost+anonsub+ithreads setup?
> 
> Yes. Otherwise an anon-sub registered at the server startup in a vhost
> +Parent doesn't work.

But what needs to be initialized for each vhost isn't the anon-sub counter
integer, but rather $PL_modglobal{ANONSUB}, right ?

> First it was moved from running from each vhost to just the base server:
> http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110177550408046&w=2
> The whole thread is here (i think it was OSX too):
> http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110168848916838&w=2
> 
> but then Dominique Quatravaux has encountered another bug, which you can
> see in this thread:
> http://marc.theaimsgroup.com/?t=111227919600001&r=1&w=2

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> 
>> Stas Bekman wrote:
>>
>>> Philippe M. Chiasson wrote:
>>> [...]
>>>
>>>> After a long promised investigation session, I believe I have pinned
>>>> it down. It seems to have been introduced by change r160562 when the
>>>> anon_cnt initialization code was moved around as a result. Didn't have
>>>> enough time tonight to dig down as to _why_ this is breaking though...
>>>>
>>>> Not entirely sure why this breaks specifically on Darwin and all, but
>>>> the following patch seems to make the problem dissapear. Does it fix
>>>> the problem you are seeing too ?
>>>>
>>>> Index: src/modules/perl/modperl_handler.c
>>>> ===================================================================
>>>> --- src/modules/perl/modperl_handler.c  (revision 170139)
>>>> +++ src/modules/perl/modperl_handler.c  (working copy)
>>>> @@ -89,9 +89,6 @@
>>>>         modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>>>     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>>>     MP_MODGLOBAL_STORE_HV(gkey);
>>>> -
>>>> -    /* init the counter to 0 */
>>>> -    modperl_global_anon_cnt_init(p);
>>>> }
>>>>
>>>> /* allocate and populate the anon handler sub-struct */
>>>> Index: src/modules/perl/mod_perl.c
>>>> ===================================================================
>>>> --- src/modules/perl/mod_perl.c (revision 170139)
>>>> +++ src/modules/perl/mod_perl.c (working copy)
>>>> @@ -532,6 +532,9 @@
>>>>     modperl_global_init_server_rec(pconf, s);
>>>>
>>>>     modperl_tls_create_request_rec(pconf);
>>>> +
>>>> +    /* init the counter to 0 */
>>>> +    modperl_global_anon_cnt_init(pconf);
>>>> }
>>>>
>>>> /*
>>>
>>> Philippe is talking about
>>> http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
>>>
>>> http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
>>>
>>
>> Yup, that one.
>>
>>> But it can't be reverted , since it was a fix for a bug in
>>> vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
>>> as it wasn't suitable for the test suite (would have affected other
>>> tests). So it needs to stay that way and figured out what needs to be
>>> fixed elsewhere.
>>>
>>> Philippe, your patch can't be applied either, since it has exactly the
>>> same effect as reverting r160562, since you moved
>>> modperl_global_anon_cnt_init() to modperl_init_globals() which is called
>>> only for the top-level server
>>
>> Yes, but I am not reverting that change, keeping it as is, hash
>> initialization
>> and all, and only moving the modperl_global_anon_cnt_init() up so it's
>> called
>> only once. I don't think initializating that anon_cnt global more than
>> once is
>> a feature (and it registers it's destruction twice, causing the
>> segfault on OSX).
> 
> No, you are reverting it. Follow the calls trace, you've just moved the
> original case (before my change) to a different place.

Unless I am missing something terribly obvious, I believe you are talking
about this chunk of code:

    modperl_modglobal_key_t *gkey =
        modperl_modglobal_lookup(aTHX_ "ANONSUB");
    MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
    MP_MODGLOBAL_STORE_HV(gkey);

And that's what needs to be initialized in each server/vhost, correct?
And that still is. The only code I am suggesting to move around is the
modperl_global_anon_cnt_init() call

>> Are you saying that you need more than one distinct global anon_cnt in
>> some
>> vhost+anonsub+ithreads setup?
> 
> Yes. Otherwise an anon-sub registered at the server startup in a vhost
> +Parent doesn't work.

But what needs to be initialized for each vhost isn't the anon-sub counter
integer, but rather $PL_modglobal{ANONSUB}, right ?

> First it was moved from running from each vhost to just the base server:
> http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110177550408046&w=2
> The whole thread is here (i think it was OSX too):
> http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110168848916838&w=2
> 
> but then Dominique Quatravaux has encountered another bug, which you can
> see in this thread:
> http://marc.theaimsgroup.com/?t=111227919600001&r=1&w=2

--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
> Stas Bekman wrote:
> 
>>Philippe M. Chiasson wrote:
>>[...]
>>
>>
>>>After a long promised investigation session, I believe I have pinned
>>>it down. It seems to have been introduced by change r160562 when the
>>>anon_cnt initialization code was moved around as a result. Didn't have
>>>enough time tonight to dig down as to _why_ this is breaking though...
>>>
>>>Not entirely sure why this breaks specifically on Darwin and all, but
>>>the following patch seems to make the problem dissapear. Does it fix
>>>the problem you are seeing too ?
>>>
>>>Index: src/modules/perl/modperl_handler.c
>>>===================================================================
>>>--- src/modules/perl/modperl_handler.c  (revision 170139)
>>>+++ src/modules/perl/modperl_handler.c  (working copy)
>>>@@ -89,9 +89,6 @@
>>>         modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>>     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>>     MP_MODGLOBAL_STORE_HV(gkey);
>>>-
>>>-    /* init the counter to 0 */
>>>-    modperl_global_anon_cnt_init(p);
>>> }
>>>
>>> /* allocate and populate the anon handler sub-struct */
>>>Index: src/modules/perl/mod_perl.c
>>>===================================================================
>>>--- src/modules/perl/mod_perl.c (revision 170139)
>>>+++ src/modules/perl/mod_perl.c (working copy)
>>>@@ -532,6 +532,9 @@
>>>     modperl_global_init_server_rec(pconf, s);
>>>
>>>     modperl_tls_create_request_rec(pconf);
>>>+
>>>+    /* init the counter to 0 */
>>>+    modperl_global_anon_cnt_init(pconf);
>>> }
>>>
>>> /*
>>
>>Philippe is talking about
>>http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
>>
>>http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
>>
> 
> 
> Yup, that one.
> 
> 
>>But it can't be reverted , since it was a fix for a bug in
>>vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
>>as it wasn't suitable for the test suite (would have affected other
>>tests). So it needs to stay that way and figured out what needs to be
>>fixed elsewhere.
>>
>>Philippe, your patch can't be applied either, since it has exactly the
>>same effect as reverting r160562, since you moved
>>modperl_global_anon_cnt_init() to modperl_init_globals() which is called
>>only for the top-level server:
> 
> 
> Yes, but I am not reverting that change, keeping it as is, hash initialization
> and all, and only moving the modperl_global_anon_cnt_init() up so it's called
> only once. I don't think initializating that anon_cnt global more than once is
> a feature (and it registers it's destruction twice, causing the segfault on OSX).

No, you are reverting it. Follow the calls trace, you've just moved the 
original case (before my change) to a different place.

> Are you saying that you need more than one distinct global anon_cnt in some
> vhost+anonsub+ithreads setup?

Yes. Otherwise an anon-sub registered at the server startup in a vhost 
+Parent doesn't work.

First it was moved from running from each vhost to just the base server:
http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110177550408046&w=2
The whole thread is here (i think it was OSX too):
http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110168848916838&w=2

but then Dominique Quatravaux has encountered another bug, which you can 
see in this thread:
http://marc.theaimsgroup.com/?t=111227919600001&r=1&w=2


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
> Stas Bekman wrote:
> 
>>Philippe M. Chiasson wrote:
>>[...]
>>
>>
>>>After a long promised investigation session, I believe I have pinned
>>>it down. It seems to have been introduced by change r160562 when the
>>>anon_cnt initialization code was moved around as a result. Didn't have
>>>enough time tonight to dig down as to _why_ this is breaking though...
>>>
>>>Not entirely sure why this breaks specifically on Darwin and all, but
>>>the following patch seems to make the problem dissapear. Does it fix
>>>the problem you are seeing too ?
>>>
>>>Index: src/modules/perl/modperl_handler.c
>>>===================================================================
>>>--- src/modules/perl/modperl_handler.c  (revision 170139)
>>>+++ src/modules/perl/modperl_handler.c  (working copy)
>>>@@ -89,9 +89,6 @@
>>>         modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>>     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>>     MP_MODGLOBAL_STORE_HV(gkey);
>>>-
>>>-    /* init the counter to 0 */
>>>-    modperl_global_anon_cnt_init(p);
>>> }
>>>
>>> /* allocate and populate the anon handler sub-struct */
>>>Index: src/modules/perl/mod_perl.c
>>>===================================================================
>>>--- src/modules/perl/mod_perl.c (revision 170139)
>>>+++ src/modules/perl/mod_perl.c (working copy)
>>>@@ -532,6 +532,9 @@
>>>     modperl_global_init_server_rec(pconf, s);
>>>
>>>     modperl_tls_create_request_rec(pconf);
>>>+
>>>+    /* init the counter to 0 */
>>>+    modperl_global_anon_cnt_init(pconf);
>>> }
>>>
>>> /*
>>
>>Philippe is talking about
>>http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
>>
>>http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
>>
> 
> 
> Yup, that one.
> 
> 
>>But it can't be reverted , since it was a fix for a bug in
>>vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
>>as it wasn't suitable for the test suite (would have affected other
>>tests). So it needs to stay that way and figured out what needs to be
>>fixed elsewhere.
>>
>>Philippe, your patch can't be applied either, since it has exactly the
>>same effect as reverting r160562, since you moved
>>modperl_global_anon_cnt_init() to modperl_init_globals() which is called
>>only for the top-level server:
> 
> 
> Yes, but I am not reverting that change, keeping it as is, hash initialization
> and all, and only moving the modperl_global_anon_cnt_init() up so it's called
> only once. I don't think initializating that anon_cnt global more than once is
> a feature (and it registers it's destruction twice, causing the segfault on OSX).

No, you are reverting it. Follow the calls trace, you've just moved the 
original case (before my change) to a different place.

> Are you saying that you need more than one distinct global anon_cnt in some
> vhost+anonsub+ithreads setup?

Yes. Otherwise an anon-sub registered at the server startup in a vhost 
+Parent doesn't work.

First it was moved from running from each vhost to just the base server:
http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110177550408046&w=2
The whole thread is here (i think it was OSX too):
http://marc.theaimsgroup.com/?l=apache-modperl-dev&m=110168848916838&w=2

but then Dominique Quatravaux has encountered another bug, which you can 
see in this thread:
http://marc.theaimsgroup.com/?t=111227919600001&r=1&w=2


-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> [...]
> 
>> After a long promised investigation session, I believe I have pinned
>> it down. It seems to have been introduced by change r160562 when the
>> anon_cnt initialization code was moved around as a result. Didn't have
>> enough time tonight to dig down as to _why_ this is breaking though...
>>
>> Not entirely sure why this breaks specifically on Darwin and all, but
>> the following patch seems to make the problem dissapear. Does it fix
>> the problem you are seeing too ?
>>
>> Index: src/modules/perl/modperl_handler.c
>> ===================================================================
>> --- src/modules/perl/modperl_handler.c  (revision 170139)
>> +++ src/modules/perl/modperl_handler.c  (working copy)
>> @@ -89,9 +89,6 @@
>>          modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>      MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>      MP_MODGLOBAL_STORE_HV(gkey);
>> -
>> -    /* init the counter to 0 */
>> -    modperl_global_anon_cnt_init(p);
>>  }
>>
>>  /* allocate and populate the anon handler sub-struct */
>> Index: src/modules/perl/mod_perl.c
>> ===================================================================
>> --- src/modules/perl/mod_perl.c (revision 170139)
>> +++ src/modules/perl/mod_perl.c (working copy)
>> @@ -532,6 +532,9 @@
>>      modperl_global_init_server_rec(pconf, s);
>>
>>      modperl_tls_create_request_rec(pconf);
>> +
>> +    /* init the counter to 0 */
>> +    modperl_global_anon_cnt_init(pconf);
>>  }
>>
>>  /*
> 
> Philippe is talking about
> http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
> 
> http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
> 

Yup, that one.

> But it can't be reverted , since it was a fix for a bug in
> vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
> as it wasn't suitable for the test suite (would have affected other
> tests). So it needs to stay that way and figured out what needs to be
> fixed elsewhere.
> 
> Philippe, your patch can't be applied either, since it has exactly the
> same effect as reverting r160562, since you moved
> modperl_global_anon_cnt_init() to modperl_init_globals() which is called
> only for the top-level server:

Yes, but I am not reverting that change, keeping it as is, hash initialization
and all, and only moving the modperl_global_anon_cnt_init() up so it's called
only once. I don't think initializating that anon_cnt global more than once is
a feature (and it registers it's destruction twice, causing the segfault on OSX).

Are you saying that you need more than one distinct global anon_cnt in some
vhost+anonsub+ithreads setup?
--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Philippe M. Chiasson wrote:
> [...]
> 
>> After a long promised investigation session, I believe I have pinned
>> it down. It seems to have been introduced by change r160562 when the
>> anon_cnt initialization code was moved around as a result. Didn't have
>> enough time tonight to dig down as to _why_ this is breaking though...
>>
>> Not entirely sure why this breaks specifically on Darwin and all, but
>> the following patch seems to make the problem dissapear. Does it fix
>> the problem you are seeing too ?
>>
>> Index: src/modules/perl/modperl_handler.c
>> ===================================================================
>> --- src/modules/perl/modperl_handler.c  (revision 170139)
>> +++ src/modules/perl/modperl_handler.c  (working copy)
>> @@ -89,9 +89,6 @@
>>          modperl_modglobal_lookup(aTHX_ "ANONSUB");
>>      MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>>      MP_MODGLOBAL_STORE_HV(gkey);
>> -
>> -    /* init the counter to 0 */
>> -    modperl_global_anon_cnt_init(p);
>>  }
>>
>>  /* allocate and populate the anon handler sub-struct */
>> Index: src/modules/perl/mod_perl.c
>> ===================================================================
>> --- src/modules/perl/mod_perl.c (revision 170139)
>> +++ src/modules/perl/mod_perl.c (working copy)
>> @@ -532,6 +532,9 @@
>>      modperl_global_init_server_rec(pconf, s);
>>
>>      modperl_tls_create_request_rec(pconf);
>> +
>> +    /* init the counter to 0 */
>> +    modperl_global_anon_cnt_init(pconf);
>>  }
>>
>>  /*
> 
> Philippe is talking about
> http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev
> 
> http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c
> 

Yup, that one.

> But it can't be reverted , since it was a fix for a bug in
> vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem,
> as it wasn't suitable for the test suite (would have affected other
> tests). So it needs to stay that way and figured out what needs to be
> fixed elsewhere.
> 
> Philippe, your patch can't be applied either, since it has exactly the
> same effect as reverting r160562, since you moved
> modperl_global_anon_cnt_init() to modperl_init_globals() which is called
> only for the top-level server:

Yes, but I am not reverting that change, keeping it as is, hash initialization
and all, and only moving the modperl_global_anon_cnt_init() up so it's called
only once. I don't think initializating that anon_cnt global more than once is
a feature (and it registers it's destruction twice, causing the segfault on OSX).

Are you saying that you need more than one distinct global anon_cnt in some
vhost+anonsub+ithreads setup?
--------------------------------------------------------------------------------
Philippe M. Chiasson m/gozer\@(apache|cpan|ectoplasm)\.org/ GPG KeyID : 88C3A5A5
http://gozer.ectoplasm.org/     F9BF E0C2 480E 7680 1AE5 3631 CB32 A107 88C3A5A5

Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
[...]
> After a long promised investigation session, I believe I have pinned
> it down. It seems to have been introduced by change r160562 when the
> anon_cnt initialization code was moved around as a result. Didn't have
> enough time tonight to dig down as to _why_ this is breaking though...
> 
> Not entirely sure why this breaks specifically on Darwin and all, but
> the following patch seems to make the problem dissapear. Does it fix
> the problem you are seeing too ?
> 
> Index: src/modules/perl/modperl_handler.c
> ===================================================================
> --- src/modules/perl/modperl_handler.c  (revision 170139)
> +++ src/modules/perl/modperl_handler.c  (working copy)
> @@ -89,9 +89,6 @@
>          modperl_modglobal_lookup(aTHX_ "ANONSUB");
>      MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>      MP_MODGLOBAL_STORE_HV(gkey);
> -
> -    /* init the counter to 0 */
> -    modperl_global_anon_cnt_init(p);
>  }
> 
>  /* allocate and populate the anon handler sub-struct */
> Index: src/modules/perl/mod_perl.c
> ===================================================================
> --- src/modules/perl/mod_perl.c (revision 170139)
> +++ src/modules/perl/mod_perl.c (working copy)
> @@ -532,6 +532,9 @@
>      modperl_global_init_server_rec(pconf, s);
> 
>      modperl_tls_create_request_rec(pconf);
> +
> +    /* init the counter to 0 */
> +    modperl_global_anon_cnt_init(pconf);
>  }
> 
>  /*

Philippe is talking about 
http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev

http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c

But it can't be reverted , since it was a fix for a bug in 
vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem, as 
it wasn't suitable for the test suite (would have affected other tests). 
So it needs to stay that way and figured out what needs to be fixed elsewhere.

Philippe, your patch can't be applied either, since it has exactly the 
same effect as reverting r160562, since you moved 
modperl_global_anon_cnt_init() to modperl_init_globals() which is called 
only for the top-level server:

void *modperl_config_srv_create(apr_pool_t *p, server_rec *s)
{
     modperl_config_srv_t *scfg = modperl_config_srv_new(p);

     if (!s->is_virtual) {

         /* give a chance to MOD_PERL_TRACE env var to set
          * PerlTrace. This place is the earliest point in mod_perl
          * configuration parsing, when we have the server object
          */
         modperl_trace_level_set_apache(s, NULL);

         /* Must store the global server record as early as possible,
          * because if mod_perl happens to be started from within a
          * vhost (e.g., PerlLoadModule) the base server record won't
          * be available to vhost and things will blow up
          */
         modperl_init_globals(s, p);
     }

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
Philippe M. Chiasson wrote:
[...]
> After a long promised investigation session, I believe I have pinned
> it down. It seems to have been introduced by change r160562 when the
> anon_cnt initialization code was moved around as a result. Didn't have
> enough time tonight to dig down as to _why_ this is breaking though...
> 
> Not entirely sure why this breaks specifically on Darwin and all, but
> the following patch seems to make the problem dissapear. Does it fix
> the problem you are seeing too ?
> 
> Index: src/modules/perl/modperl_handler.c
> ===================================================================
> --- src/modules/perl/modperl_handler.c  (revision 170139)
> +++ src/modules/perl/modperl_handler.c  (working copy)
> @@ -89,9 +89,6 @@
>          modperl_modglobal_lookup(aTHX_ "ANONSUB");
>      MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
>      MP_MODGLOBAL_STORE_HV(gkey);
> -
> -    /* init the counter to 0 */
> -    modperl_global_anon_cnt_init(p);
>  }
> 
>  /* allocate and populate the anon handler sub-struct */
> Index: src/modules/perl/mod_perl.c
> ===================================================================
> --- src/modules/perl/mod_perl.c (revision 170139)
> +++ src/modules/perl/mod_perl.c (working copy)
> @@ -532,6 +532,9 @@
>      modperl_global_init_server_rec(pconf, s);
> 
>      modperl_tls_create_request_rec(pconf);
> +
> +    /* init the counter to 0 */
> +    modperl_global_anon_cnt_init(pconf);
>  }
> 
>  /*

Philippe is talking about 
http://svn.apache.org/viewcvs.cgi?rev=160562&view=rev

http://svn.apache.org/viewcvs.cgi/perl/modperl/trunk/src/modules/perl/mod_perl.c?rev=160562&view=diff&r1=160562&r2=160561&p1=perl/modperl/trunk/src/modules/perl/mod_perl.c&p2=/perl/modperl/trunk/src/modules/perl/mod_perl.c

But it can't be reverted , since it was a fix for a bug in 
vhosts+anonsub+ithreads. I couldn't add a test reproducing the problem, as 
it wasn't suitable for the test suite (would have affected other tests). 
So it needs to stay that way and figured out what needs to be fixed elsewhere.

Philippe, your patch can't be applied either, since it has exactly the 
same effect as reverting r160562, since you moved 
modperl_global_anon_cnt_init() to modperl_init_globals() which is called 
only for the top-level server:

void *modperl_config_srv_create(apr_pool_t *p, server_rec *s)
{
     modperl_config_srv_t *scfg = modperl_config_srv_new(p);

     if (!s->is_virtual) {

         /* give a chance to MOD_PERL_TRACE env var to set
          * PerlTrace. This place is the earliest point in mod_perl
          * configuration parsing, when we have the server object
          */
         modperl_trace_level_set_apache(s, NULL);

         /* Must store the global server record as early as possible,
          * because if mod_perl happens to be started from within a
          * vhost (e.g., PerlLoadModule) the base server record won't
          * be available to vhost and things will blow up
          */
         modperl_init_globals(s, p);
     }

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com

Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Christian Hansen wrote:
> Philippe M. Chiasson wrote:
> 
>>
>> Just rebuilt myself a fresh blead-perl, fresh httpd-2.0-svn and
>> mod_perl-2.0-svn. All tests passed!
>>
>> My hunch is that's it's something about Apple's Perl, and I'll
>> investigate
>> more later on. For now, I guess the best advice is to try with a
>> self-built
>> Perl and see if that hepls.
> 
> 
> I have tested with perl 5.8.5, 5.8.6 with ithreads which fails, and
> 5.8.6 without ithreads which works. All built by myself.
> 
> I have no problems with MP 1.999.21 with ithreads, so my guess is that
> something changed after 1.999.21.
> 

After a long promised investigation session, I believe I have pinned
it down. It seems to have been introduced by change r160562 when the
anon_cnt initialization code was moved around as a result. Didn't have
enough time tonight to dig down as to _why_ this is breaking though...

Not entirely sure why this breaks specifically on Darwin and all, but
the following patch seems to make the problem dissapear. Does it fix
the problem you are seeing too ?

Index: src/modules/perl/modperl_handler.c
===================================================================
--- src/modules/perl/modperl_handler.c  (revision 170139)
+++ src/modules/perl/modperl_handler.c  (working copy)
@@ -89,9 +89,6 @@
         modperl_modglobal_lookup(aTHX_ "ANONSUB");
     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
     MP_MODGLOBAL_STORE_HV(gkey);
-
-    /* init the counter to 0 */
-    modperl_global_anon_cnt_init(p);
 }

 /* allocate and populate the anon handler sub-struct */
Index: src/modules/perl/mod_perl.c
===================================================================
--- src/modules/perl/mod_perl.c (revision 170139)
+++ src/modules/perl/mod_perl.c (working copy)
@@ -532,6 +532,9 @@
     modperl_global_init_server_rec(pconf, s);

     modperl_tls_create_request_rec(pconf);
+
+    /* init the counter to 0 */
+    modperl_global_anon_cnt_init(pconf);
 }

 /*


Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Christian Hansen wrote:
> Philippe M. Chiasson wrote:
> 
>>
>> Just rebuilt myself a fresh blead-perl, fresh httpd-2.0-svn and
>> mod_perl-2.0-svn. All tests passed!
>>
>> My hunch is that's it's something about Apple's Perl, and I'll
>> investigate
>> more later on. For now, I guess the best advice is to try with a
>> self-built
>> Perl and see if that hepls.
> 
> 
> I have tested with perl 5.8.5, 5.8.6 with ithreads which fails, and
> 5.8.6 without ithreads which works. All built by myself.
> 
> I have no problems with MP 1.999.21 with ithreads, so my guess is that
> something changed after 1.999.21.
> 

After a long promised investigation session, I believe I have pinned
it down. It seems to have been introduced by change r160562 when the
anon_cnt initialization code was moved around as a result. Didn't have
enough time tonight to dig down as to _why_ this is breaking though...

Not entirely sure why this breaks specifically on Darwin and all, but
the following patch seems to make the problem dissapear. Does it fix
the problem you are seeing too ?

Index: src/modules/perl/modperl_handler.c
===================================================================
--- src/modules/perl/modperl_handler.c  (revision 170139)
+++ src/modules/perl/modperl_handler.c  (working copy)
@@ -89,9 +89,6 @@
         modperl_modglobal_lookup(aTHX_ "ANONSUB");
     MP_TRACE_h(MP_FUNC, "init $PL_modglobal{ANONSUB} = []");
     MP_MODGLOBAL_STORE_HV(gkey);
-
-    /* init the counter to 0 */
-    modperl_global_anon_cnt_init(p);
 }

 /* allocate and populate the anon handler sub-struct */
Index: src/modules/perl/mod_perl.c
===================================================================
--- src/modules/perl/mod_perl.c (revision 170139)
+++ src/modules/perl/mod_perl.c (working copy)
@@ -532,6 +532,9 @@
     modperl_global_init_server_rec(pconf, s);

     modperl_tls_create_request_rec(pconf);
+
+    /* init the counter to 0 */
+    modperl_global_anon_cnt_init(pconf);
 }

 /*


Re: [MP2] 1.999024 fails on OS X

Posted by Christian Hansen <ch...@ngmedia.com>.
Philippe M. Chiasson wrote:
> 
> After a long promised investigation session, I believe I have pinned
> it down. It seems to have been introduced by change r160562 when the
> anon_cnt initialization code was moved around as a result. Didn't have
> enough time tonight to dig down as to _why_ this is breaking though...
> 
> Not entirely sure why this breaks specifically on Darwin and all, but
> the following patch seems to make the problem dissapear. Does it fix
> the problem you are seeing too ?

Yes, tested with 5.8.5 and 5.8.6 compiled with ithreads, all tests pass.

Thanks for hunting this down, much appreciated.


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@perl.apache.org
For additional commands, e-mail: dev-help@perl.apache.org


Re: [MP2] 1.999024 fails on OS X

Posted by Christian Hansen <ch...@ngmedia.com>.
Philippe M. Chiasson wrote:
> 
> After a long promised investigation session, I believe I have pinned
> it down. It seems to have been introduced by change r160562 when the
> anon_cnt initialization code was moved around as a result. Didn't have
> enough time tonight to dig down as to _why_ this is breaking though...
> 
> Not entirely sure why this breaks specifically on Darwin and all, but
> the following patch seems to make the problem dissapear. Does it fix
> the problem you are seeing too ?

Yes, tested with 5.8.5 and 5.8.6 compiled with ithreads, all tests pass.

Thanks for hunting this down, much appreciated.


Re: [PMX:####] Re: [MP2] 1.999024 fails on OS X

Posted by Christian Hansen <ch...@ngmedia.com>.
Philippe M. Chiasson wrote:
> 
> Just rebuilt myself a fresh blead-perl, fresh httpd-2.0-svn and
> mod_perl-2.0-svn. All tests passed!
> 
> My hunch is that's it's something about Apple's Perl, and I'll investigate
> more later on. For now, I guess the best advice is to try with a self-built
> Perl and see if that hepls.

I have tested with perl 5.8.5, 5.8.6 with ithreads which fails, and 
5.8.6 without ithreads which works. All built by myself.

I have no problems with MP 1.999.21 with ithreads, so my guess is that 
something changed after 1.999.21.



[PMX:####] Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Philippe M. Chiasson wrote:
Philippe M. Chiasson wrote:
> Stas Bekman wrote:
> 
>>Christian Hansen wrote:
>>
>>
>>>Program received signal EXC_BAD_ACCESS, Could not access memory.
>>>0x014563fc in S_mess_alloc ()
>>>
>>>(gdb) bt
>>>#0  0x014563fc in S_mess_alloc ()
>>>#1  0x014566e8 in Perl_vmess ()
>>>#2  0x014571dc in Perl_vcroak ()
>>>#3  0x014576b8 in Perl_croak_nocontext ()
>>>#4  0x007dd9c8 in modperl_global_cleanup ()
>>>#5  0x006a3f60 in run_cleanups (cref=0x1806228) at apr_pools.c:1952
>>>#6  0x006a35f4 in apr_pool_clear (pool=0xbffffa1c) at apr_pools.c:694
>>>#7  0x0000ac34 in main (argc=11, argv=0xbffffc40) at main.c:575
>>>
>>>Looks like I'm hitting the same problem that Philippe M. Chiasson
>>>reported last month,
>>>http://article.gmane.org/gmane.comp.apache.mod-perl/18408
> 
> 
> Yup, that's what I've been seeing.
> 
> 
>>Philippe, any chance you could do the promised investigation you have
>>promised at the above URL?
> 
> 
> It's on my radar...
> 
> 

> Stas Bekman wrote:
> 
>>Christian Hansen wrote:
>>
>>
>>>Program received signal EXC_BAD_ACCESS, Could not access memory.
>>>0x014563fc in S_mess_alloc ()
>>>
>>>(gdb) bt
>>>#0  0x014563fc in S_mess_alloc ()
>>>#1  0x014566e8 in Perl_vmess ()
>>>#2  0x014571dc in Perl_vcroak ()
>>>#3  0x014576b8 in Perl_croak_nocontext ()
>>>#4  0x007dd9c8 in modperl_global_cleanup ()
>>>#5  0x006a3f60 in run_cleanups (cref=0x1806228) at apr_pools.c:1952
>>>#6  0x006a35f4 in apr_pool_clear (pool=0xbffffa1c) at apr_pools.c:694
>>>#7  0x0000ac34 in main (argc=11, argv=0xbffffc40) at main.c:575
>>>
>>>Looks like I'm hitting the same problem that Philippe M. Chiasson
>>>reported last month,
>>>http://article.gmane.org/gmane.comp.apache.mod-perl/18408
> 
> 
> Yup, that's what I've been seeing.
> 
> 
>>Philippe, any chance you could do the promised investigation you have
>>promised at the above URL?
> 

Just rebuilt myself a fresh blead-perl, fresh httpd-2.0-svn and
mod_perl-2.0-svn. All tests passed!

My hunch is that's it's something about Apple's Perl, and I'll investigate
more later on. For now, I guess the best advice is to try with a self-built
Perl and see if that hepls.


Re: [MP2] 1.999024 fails on OS X

Posted by "Philippe M. Chiasson" <go...@ectoplasm.org>.
Stas Bekman wrote:
> Christian Hansen wrote:
> 
>> Program received signal EXC_BAD_ACCESS, Could not access memory.
>> 0x014563fc in S_mess_alloc ()
>>
>> (gdb) bt
>> #0  0x014563fc in S_mess_alloc ()
>> #1  0x014566e8 in Perl_vmess ()
>> #2  0x014571dc in Perl_vcroak ()
>> #3  0x014576b8 in Perl_croak_nocontext ()
>> #4  0x007dd9c8 in modperl_global_cleanup ()
>> #5  0x006a3f60 in run_cleanups (cref=0x1806228) at apr_pools.c:1952
>> #6  0x006a35f4 in apr_pool_clear (pool=0xbffffa1c) at apr_pools.c:694
>> #7  0x0000ac34 in main (argc=11, argv=0xbffffc40) at main.c:575
>>
>> Looks like I'm hitting the same problem that Philippe M. Chiasson
>> reported last month,
>> http://article.gmane.org/gmane.comp.apache.mod-perl/18408

Yup, that's what I've been seeing.

> Philippe, any chance you could do the promised investigation you have
> promised at the above URL?

It's on my radar...



Re: [MP2] 1.999024 fails on OS X

Posted by Stas Bekman <st...@stason.org>.
Christian Hansen wrote:
> 
> Program received signal EXC_BAD_ACCESS, Could not access memory.
> 0x014563fc in S_mess_alloc ()
> 
> (gdb) bt
> #0  0x014563fc in S_mess_alloc ()
> #1  0x014566e8 in Perl_vmess ()
> #2  0x014571dc in Perl_vcroak ()
> #3  0x014576b8 in Perl_croak_nocontext ()
> #4  0x007dd9c8 in modperl_global_cleanup ()
> #5  0x006a3f60 in run_cleanups (cref=0x1806228) at apr_pools.c:1952
> #6  0x006a35f4 in apr_pool_clear (pool=0xbffffa1c) at apr_pools.c:694
> #7  0x0000ac34 in main (argc=11, argv=0xbffffc40) at main.c:575
> 
> Looks like I'm hitting the same problem that Philippe M. Chiasson 
> reported last month, 
> http://article.gmane.org/gmane.comp.apache.mod-perl/18408

Philippe, any chance you could do the promised investigation you have 
promised at the above URL?

> I'm using trunk, revision: 169956. With 1.999.21 all tests passes, so 
> guess something changed in the namespace change.

With the same perl and apache versions/configuration?

> Please let me know if I can be at further assistance.

Christian, any difference if you test with perl w/o ithreads?

-- 
__________________________________________________________________
Stas Bekman            JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/     mod_perl Guide ---> http://perl.apache.org
mailto:stas@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org   http://ticketmaster.com