cleanup of freelist management

[aimacintyre]

BPO	2862
Nosy	@gpshead, @pitrou, @tiran
Files	freelist_mgt_cleanup.patch: patch harmonising int/float freelist mgt with other type freelist mgt

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = 'https://github.com/gpshead'
closed_at = <Date 2008-07-06.03:36:25.396>
created_at = <Date 2008-05-15.13:45:19.803>
labels = ['interpreter-core', 'type-bug', 'tests', 'docs']
title = 'cleanup of freelist management'
updated_at = <Date 2008-07-06.03:36:25.351>
user = 'https://bugs.python.org/aimacintyre'

bugs.python.org fields:

activity = <Date 2008-07-06.03:36:25.351>
actor = 'gregory.p.smith'
assignee = 'gregory.p.smith'
closed = True
closed_date = <Date 2008-07-06.03:36:25.396>
closer = 'gregory.p.smith'
components = ['Documentation', 'Interpreter Core', 'Tests']
creation = <Date 2008-05-15.13:45:19.803>
creator = 'aimacintyre'
dependencies = []
files = ['10331']
hgrepos = []
issue_num = 2862
keywords = ['patch']
message_count = 4.0
messages = ['66858', '67809', '69019', '69314']
nosy_count = 4.0
nosy_names = ['gregory.p.smith', 'aimacintyre', 'pitrou', 'christian.heimes']
pr_nums = []
priority = 'high'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue2862'
versions = ['Python 2.6']

[aimacintyre]

In r60567, support for compacting the int & float freelists was added
with a function in the sys module to call the compaction routines.

Since then, other freelist clearing routines have been added to other
types and are called from the collect() function in the gc module.

The attached patch harmonises the in/float freelist compaction with the
other freelist clearing mechanisms. It does away with the function in
the sys module in favour of running the freelist clearing from gc.collect().

The signatures of the type specific freelist clearing routines have been
brought into line with the other type freelist clearing routines, with
the exception that they return the number of items _not_ free()ed,
rather than the number of items free()ed. I took this approach to try
and highlight the fact the current int/float freelist implementations
don't use pymalloc and not all items will always be free()ed when the
routine is called. This return is ignored when called from gc.collect()
in any case.

The patch includes doc updates and test updates to reflect the changes.
It has been tested on FreeBSD 6.3, surviving a default test run with no
unusual test failures as far as I can tell. It is not otherwise
rigorously tested.

The patch is against a trunk checkout that seems to be at r63156.

Ideally this patch, or similar modifications, should be applied before
the feature freeze for 2.6, so that freelist management is rationalised
to one way to do it.

[gpshead]

I agree with this patch and will commit it later this weekend if I hear
no objections. see the mailing list discussion.

[pitrou]

I agree with this patch and will commit it later this weekend if I hear
no objections.

Gregory, it seems there has been no objection on the ML :-)

[gpshead]

committed to 2.6 trunk in r64753.