Commit 58af37d6 authored by Jason Madden's avatar Jason Madden

Add to release notes a reference about the PyPy cpyext memory leak bug.

Also, this comes on the heels of adding back __weakref__ to Semaphore.
This becomes a field in the struct and the field is cleared in
tp_dealloc. It seems like this would trigger the memory leak bug when
taking references to a subclass, but testing seems to show that
doesn't happen. I guess PyPy handles that specially:

>>>> objgraph.show_growth()
>>>> _ = [weakref.ref(gevent.thread.LockType()) for i in range(1000)]; del _
>>>> objgraph.show_growth()
>>>>

[skip ci]
parent e8ec0f54
...@@ -48,8 +48,21 @@ through 2.5.1, 2.6.0, 2.6.1, and pre-release versions of 2.7.0. ...@@ -48,8 +48,21 @@ through 2.5.1, 2.6.0, 2.6.1, and pre-release versions of 2.7.0.
:class:`gevent.lock.Semaphore`. Whether or not these matter will :class:`gevent.lock.Semaphore`. Whether or not these matter will
depend on the workload of each application. depend on the workload of each application.
.. note:: Released versions of PyPy through at least 2.6.1 have `a
bug`_ that can cause a memory leak when subclassing
objects that are implemented in Cython, as are the two
things mentioned above. The `Semaphore` class is
subclassed to become :class:`gevent.thread.LockType`,
which in turn is used as the basis for
:class:`threading.Lock`. The `Semaphore` object is coded
carefully to avoid this leak, assuming it is deallocated
when not acquired (which should be the typical case). The
``c-ares`` package has not been audited for this issue.
.. _cffi 1.3.0: https://bitbucket.org/cffi/cffi/src/ad3140a30a7b0ca912185ef500546a9fb5525ece/doc/source/whatsnew.rst?at=default .. _cffi 1.3.0: https://bitbucket.org/cffi/cffi/src/ad3140a30a7b0ca912185ef500546a9fb5525ece/doc/source/whatsnew.rst?at=default
.. _1.2.0: https://cffi.readthedocs.org/en/latest/whatsnew.html#v1-2-0 .. _1.2.0: https://cffi.readthedocs.org/en/latest/whatsnew.html#v1-2-0
.. _a bug: https://bitbucket.org/pypy/pypy/issues/2149/memory-leak-for-python-subclass-of-cpyext
Improved subprocess support Improved subprocess support
=========================== ===========================
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment