Skip to content

Conversation

dstogov
Copy link
Member

@dstogov dstogov commented Aug 18, 2014

New MM performance impact

mm

@laruence
Copy link
Member

👍

@76200
Copy link

76200 commented Aug 22, 2014

How about SF2?

@dstogov
Copy link
Member Author

dstogov commented Aug 23, 2014

what is SF?

Dmitry.

On Fri, Aug 22, 2014 at 3:38 PM, bartek notifications@github.com wrote:

How about SF2?


Reply to this email directly or view it on GitHub
#777 (comment).

@yosymfony
Copy link

SF2 = Symfony Framework 2

@php-pulls
Copy link

Comment on behalf of nikic at php.net:

Closing as this has been merged.

@php-pulls php-pulls closed this Aug 26, 2014
@hikari-no-yume
Copy link
Contributor

Ignore my deleted comment, it is probably not your fault.

@dstogov
Copy link
Member Author

dstogov commented Aug 27, 2014

Hi Andrea,

I don't know about any errors in new MM.
The line where it crashes may be helpful.

Most probably it's related to some memory corruption.
Try to run the same code with valgrind and USE_ZEND_ALLOC=0.
It it reports any errors, the problem must be unrelated to new MM.

Thanks. Dmtiry.

On Thu, Aug 28, 2014 at 2:03 AM, Andrea Faulds notifications@github.com
wrote:

[11:01pm] ajf: I wonder if an error I’m having is dstogov’s new memory
manager’s fault
[11:01pm] ajf: It’s really, really bizarre
[11:01pm] ajf: According to valgrind:
[11:01pm] ajf: 1) I try to decrement a bigint,
[11:01pm] ajf: 2) zend_bigint_init_alloc() is called to init and alloc a
new bigint,
[11:01pm] ajf: 3) zend_bigint_alloc() is called to alloc the new bigint,
[11:02pm] ajf: 3) it calls emalloc()
[11:02pm] ajf: 5) emalloc() calls zend_mm_alloc_heap()
[11:02pm] ajf: 6) emalloc() calls zend_mm_alloc_small()
[11:02pm] ajf: 7) segfault


Reply to this email directly or view it on GitHub
#777 (comment).

@hikari-no-yume
Copy link
Contributor

Yeah, @dstogov it looks like something else is going on, it just so happened the new MM make it segfault while memory silently corrupted with the old one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants