[issue11051] system calls per import

2012-02-19 Thread Éric Araujo
Changes by Éric Araujo : -- nosy: +eric.araujo ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.pyth

[issue11051] system calls per import

2012-02-18 Thread Charles-François Natali
Charles-François Natali added the comment: As noted by Antoine somewhere else (don't remember if it was on the bug tracker or mailing list), most of the startup time is due to site import: """ cf@neobox:~/python/cpython$ time ./python -c '' [44249 refs] real0m0.445s user0m0.376s sys

[issue11051] system calls per import

2011-12-07 Thread Terry J. Reedy
Terry J. Reedy added the comment: OK, I gather from the comment added in the second patch that you intentionally changed the logic to restart the loop more often. -- ___ Python tracker ___

[issue11051] system calls per import

2011-12-07 Thread STINNER Victor
STINNER Victor added the comment: By the way, _Py_stat() can fail because of a Python error: -1 result is not handled in import.c :-( It may leak to the "XXX undetected error" message. -- nosy: +haypo ___ Python tracker

[issue11051] system calls per import

2011-12-07 Thread Terry J. Reedy
Terry J. Reedy added the comment: I am a little puzzled by the patch. In logic, 'A and B' is equivalent to 'not A or not B'. But in the patch, -if (_Py_stat(filename, &statbuf) == 0 && /it exists */ -S_ISDIR(statbuf.st_mode)) /* it's a directory */ +if

[issue11051] system calls per import

2011-12-07 Thread Charles-François Natali
Charles-François Natali added the comment: > I would have appreciated had you considered my review before pushing > that change. Sorry, I didn't receive an email notification for your review, so I didn't know you had done one. I'll add a comment. -- __

[issue11051] system calls per import

2011-12-07 Thread Martin v . Löwis
Martin v. Löwis added the comment: I would have appreciated had you considered my review before pushing that change. -- ___ Python tracker ___ _

[issue11051] system calls per import

2011-12-07 Thread Roundup Robot
Roundup Robot added the comment: New changeset a541bda2f5e2 by Charles-François Natali in branch 'default': Issue #11051: Reduce the number of syscalls per import. http://hg.python.org/cpython/rev/a541bda2f5e2 -- nosy: +python-dev ___ Python tracker

[issue11051] system calls per import

2011-12-06 Thread Arfrever Frehtes Taifersar Arahesis
Changes by Arfrever Frehtes Taifersar Arahesis : -- nosy: +Arfrever ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscri

[issue11051] system calls per import

2011-12-05 Thread Antoine Pitrou
Antoine Pitrou added the comment: > Here's a trivial patch reducing the number of calls to open. Looks good to me. -- ___ Python tracker ___ ___

[issue11051] system calls per import

2011-12-05 Thread Martin v . Löwis
Martin v. Löwis added the comment: > First, I don't understand why we need to check both "foo.so" and > "foomodule.so". Because we always did, i.e. changing it now may break backwards compatibility. Now, as for why we always had *module.so also: it may be that calling an extension module foo.

[issue11051] system calls per import

2011-12-05 Thread Eric Snow
Changes by Eric Snow : -- nosy: +eric.snow ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue11051] system calls per import

2011-12-05 Thread Charles-François Natali
Charles-François Natali added the comment: Here's a trivial patch reducing the number of calls to open. before: """ $ strace -c -e open ./python -c "" % time seconds usecs/call callserrors syscall -- --- --- - - 100.000.49

[issue11051] system calls per import

2011-01-30 Thread Giampaolo Rodola'
Changes by Giampaolo Rodola' : -- nosy: +giampaolo.rodola ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http:

[issue11051] system calls per import

2011-01-30 Thread Antoine Pitrou
Antoine Pitrou added the comment: > > Something has gone out of control here. Why do we need to check so many > > alternative locations? > > What change do you propose? First, I don't understand why we need to check both "foo.so" and "foomodule.so". Second, I don't understand why we need to c

[issue11051] system calls per import

2011-01-30 Thread Georg Brandl
Changes by Georg Brandl : -- nosy: +georg.brandl ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue11051] system calls per import

2011-01-30 Thread Raymond Hettinger
Changes by Raymond Hettinger : -- nosy: +rhettinger ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail

[issue11051] system calls per import

2011-01-30 Thread Martin v . Löwis
Martin v. Löwis added the comment: > Something has gone out of control here. Why do we need to check so many > alternative locations? What change do you propose? -- ___ Python tracker ___

[issue11051] system calls per import

2011-01-30 Thread Nadeem Vawda
Changes by Nadeem Vawda : -- nosy: +nvawda ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.o

[issue11051] system calls per import

2011-01-28 Thread Eric Smith
Changes by Eric Smith : -- nosy: +eric.smith ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python

[issue11051] system calls per import

2011-01-28 Thread Antoine Pitrou
New submission from Antoine Pitrou : Here are the system calls when importing a single pure Python module: stat("/home/antoine/py3k/py3k/Lib/copyreg", 0x7fff1ed1f740) = -1 ENOENT (No such file or directory) open("/home/antoine/py3k/py3k/Lib/copyreg.cpython-32m.so", O_RDONLY) = -1 ENOENT (No su