[Patches] [ python-Patches-1492828 ] Improvements to ceval.c

[SourceForge.net]

Patches item #1492828, was opened at 2006-05-22 05:15
Message generated for change (Comment added) made by rhettinger
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1492828&group_id=5470

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: Performance
Group: Python 2.5
>Status: Closed
>Resolution: Rejected
Priority: 5
Private: No
Submitted By: mrjbq7 (mrjbq7)
Assigned to: Raymond Hettinger (rhettinger)
Summary: Improvements to ceval.c

Initial Comment:
>From Raymond Hettinger, submitting here to keep track of for 
NeedForSpeed sprint.

Here are some customizations to your Python build:
 
First, make sure that WITH_TSC and WITH_THREAD are not defined in the 
build.

Then, attached diff to disable the tracing code, remove NOPs, speed-up 
absolute jumps, and increase the signal check interval.

----------------------------------------------------------------------

>Comment By: Raymond Hettinger (rhettinger)
Date: 2007-04-02 20:59

Message:
Logged In: YES 
user_id=80475
Originator: NO

Sorry, all of these are useless for the general distribution.

* Increasing the ticker rollovers will give a tiny speed-up but would hurt
the responsiveness of multi-threaded applications.

* The tracing functions are needed in the general distribution and should
not be shut-off.

* The NOP case is a fail-safe; not essential, but not detrimental either. 
The peephole optimizer introduces NOPS and then clears them out.  If
someone makes a change leaving them in or if they are generating their own
bytecode, we don't want the eval-loop to fail.

* The JUMP_ABSOLUTE step needs to make a full trip through the eval-loop
or else it won't be possible to break out of a "while 1: pass".




While

----------------------------------------------------------------------

Comment By: Tim Peters (tim_one)
Date: 2006-05-22 10:39

Message:
Logged In: YES 
user_id=31435

Assigned to Raymond.  Raymond is there something of general
use here?  As a standalone patch, it sucks ;-)

----------------------------------------------------------------------

Comment By: mrjbq7 (mrjbq7)
Date: 2006-05-22 06:00

Message:
Logged In: YES 
user_id=1172546

Okay, now I checked the box "upload and attach file".  Thats a terrible
UI.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=305470&aid=1492828&group_id=5470