develooper Front page | perl.perl5.porters | Postings from July 2013

[perl #118055] miniperl fails with SIGBUS on sparc (usethreads+use64bitint)

Thread Previous | Thread Next
From:
Father Chrysostomos via RT
Date:
July 7, 2013 01:02
Subject:
[perl #118055] miniperl fails with SIGBUS on sparc (usethreads+use64bitint)
Message ID:
rt-3.6.HEAD-2552-1373158911-39.118055-15-0@perl.org
On Mon Jun 24 18:16:21 2013, sprout wrote:
> On Mon Jun 24 15:23:27 2013, dom wrote:
> > On Mon, Jun 24, 2013 at 02:28:59PM -0700, Dave Mitchell via RT wrote:
> > > I've now cherry-picked the improved fix into maint-5.18.
> > > Does this mean this ticket can be closed?
> > 
> > Yes, I believe so - thank you!
> 
> Does this remaining issue change that?
> 
> On Mon May 20 00:20:46 2013, nicholas wrote:
> > struct pmop {
> >     BASEOP
> >     OP *	op_first;
> >     OP *	op_last;
> > #ifdef USE_ITHREADS
> >     IV          op_pmoffset;
> > #else
> >     REGEXP *    op_pmregexp;            /* compiled expression */
> > #endif
> > 
> > 
> > But really, that "IV" should be something else. "STRLEN", I think, or
> > even
> > just size_t, because it's being used as an array index, and arrays
> > can't
> > be 64 bits large on a 32 bit system.
> 
> We usually use PADOFFSET for array indices, but whatever.
> 
> > 
> > We can't change that for 5.18.x. But as well as the alignment thing,
> > we
> > should fix it for the future.
> 
> Any reason I shouldn’t change the IV in blead and remove the workaround
> from Perl_Slab_Alloc?
> 

I have done that in commit 784e50c.

-- 

Father Chrysostomos


---
via perlbug:  queue: perl5 status: open
https://rt.perl.org:443/rt3/Ticket/Display.html?id=118055

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About