[Python-Dev] Making python C-API thread safe (try 2)
skip at pobox.com
Tue Sep 16 22:14:09 CEST 2003
>> Actually, it makes perfect sense. The reference count of Py_None is
>> never supposed to reach zero. If that happens it's because you have
>> a bug (too many Py_DECREFs or not enouch Py_INCREFs). Your version
>> of none_dealloc silently masks the error.
Harri> Yes, and there is no error if the reference count of Py_None
Harri> reaches zero. The Py_None reference count has no meaning.
Yes it does. If it goes to zero it means you have a bug in your code. Note
that there are plenty of situations where you INCREF or DECREF objects not
knowing (or caring) that the actual object you are messing with might be
Py_None. If I happen to incompletely test a function defined (in C)
def myfunc(arg1, arg2, arg3=None):
and have a reference count error in my code related to arg3 such that I
erroneously DECREF it, but never test the three arg case, your version of
none_dealloc() will silently miss the problem. The current none_dealloc()
will rightfully complain if myfunc() is called enough times, because
Py_None's reference count will go to zero.
Harri> By changing the way how Py_None is freed (by doing nothing)
Harri> Python would get simpler and faster.
I think you're wrong. There are lots of places in the current code base
where the INCREFs and DECREFs happen without concern for the actual object
passed. There are at least some places where a check for Py_None would
probably be warranted. In any case, I suspect that most INCREFs and DECREFs
of Py_None actually happen when the interpreter doesn't realize that Py_None
is being manipulated. By eliminating the few cases where you do know you're
messing with Py_None you probably won't reduce the number of Py_INCREF and
Py_DECREF calls substantially.
Harri> How can you have negative reference counts? Answer: You
Yes you can. From Include/object.h:
/* PyObject_HEAD defines the initial segment of every PyObject. */
#define PyObject_HEAD \
int ob_refcnt; \
struct _typeobject *ob_type;
Note that ob_refcnt is defined as an int, not an unsigned int. I'm not sure
if there are any ramifications to changing the type of that field. I don't
have time to inspect the code at the level necessary to answer
Harri> I think that you are completely missing the point of Python's
Harri> reference counting. :-) The idea is that when the count reaches
Harri> zero, then the object is deallocated. But if the object was never
Harri> allocated in the first place, why deallocate it then? That's why
Harri> having empty none_dealloc is beautiful.
Except for the case where the reference counting has an error. Let me
restate it this way: The C implementation of Python operationally defines a
reference count of zero for a statically allocated object (not just Py_None)
as a fatal error. That's as it should be, because any statically allocated
object should have a reference count of at least one.
Harri> I think that PyINCREF(Py_None) is ugly, and at least it is
Harri> completely unnecessary.
You're the person who keeps asking for it. Perhaps you should perform the
tests. If it works, submit a patch to SF. If not, dig deeper and figure
More information about the Python-list