[issue17835] test_io broken on PPC64 Linux

2014-09-29 Thread James Spurin
James Spurin added the comment: With both the kernel parameters defined and undefined, I get the following output - # /local/0/opt/python-3.4.1/bin/python Python 3.4.1 (default, Sep 29 2014, 13:31:39) [GCC 4.4.7 20120313 (Red Hat 4.4.7-4)] on linux Type "help", "copyright", "credits" or "licens

[issue17835] test_io broken on PPC64 Linux

2014-09-25 Thread Charles-François Natali
Charles-François Natali added the comment: Let's try with this instead: >>> from socket import socket, SO_SNDBUF, SOL_SOCKET >>> s = socket() >>> s.getsockopt(SOL_SOCKET, SO_SNDBUF) -- ___ Python tracker _

[issue17835] test_io broken on PPC64 Linux

2014-09-25 Thread James Spurin
James Spurin added the comment: fcntl doesnt seem to like the parameter you mentioned - # cat /etc/redhat-release Red Hat Enterprise Linux Server release 6.5 (Santiago) # /local/0/opt/python-3.4.1/bin/python Python 3.4.1 (default, Sep 24 2014, 12:23:21) [GCC 4.4.7 20120313 (Red Hat 4.4.7-4)] on

[issue17835] test_io broken on PPC64 Linux

2014-09-24 Thread Charles-François Natali
Charles-François Natali added the comment: > In this case, the issues are being caused by the following kernel parameters > that we have for our default build - > > # > ## TIBCO network tuning # > # > net.core.rmem_default = 33554432 > net.core.wme

[issue17835] test_io broken on PPC64 Linux

2014-09-24 Thread James Spurin
James Spurin added the comment: I encountered similar issues to those discussed in this issue whilst compiling 3.4.1 on 'Red Hat Enterprise Linux Server release 6.5 (Santiago)' In particular, the following tests were failing - [root@lonlx90800 ~]# /local/0/python-3.4.1/bin/python3 /local/0/py

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: This should be fixed now. David, please re-open if the failure still occurs. -- resolution: -> fixed stage: -> committed/rejected status: open -> closed versions: +Python 2.7, Python 3.3 ___ Python tracker

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Roundup Robot
Roundup Robot added the comment: New changeset 4b4ed1e11fd0 by Antoine Pitrou in branch '3.3': Issue #17835: Fix test_io when the default OS pipe buffer size is larger than one million bytes. http://hg.python.org/cpython/rev/4b4ed1e11fd0 New changeset de35eae9048a by Antoine Pitrou in branch 'd

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: > The patch is correct, however I think it's a bit overkill, especially > since it's Linux only. > Choosing a fixed large value (3 or 4 MB) just consumes a bit more > memory and should be more than enough. Ok, so let's say 4MB + 1 then. --

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Charles-François Natali
Charles-François Natali added the comment: > Ok, here is a new patch updating PIPE_MAX_SIZE with a proper value if > possible. The patch is correct, however I think it's a bit overkill, especially since it's Linux only. Choosing a fixed large value (3 or 4 MB) just consumes a bit more memory and

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread David Edelsohn
David Edelsohn added the comment: The PIPE_MAX_SIZE patch also fixes the failure. -- ___ Python tracker ___ ___ Python-bugs-list maili

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ok, here is a new patch updating PIPE_MAX_SIZE with a proper value if possible. -- keywords: +patch Added file: http://bugs.python.org/file30008/pipe_max_size.patch ___ Python tracker

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Charles-François Natali
Charles-François Natali added the comment: >> Why not simply increase the amount of data written instead of limiting >> the pipe size? > > Hmm, indeed :-) > >> By the way, there's support.PIPE_MAX_SIZE for that purpose. > > Hardwired to 3 MB. I wonder if it may broken soon. On Linux, for non roo

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread David Edelsohn
David Edelsohn added the comment: The patch limiting the pipe size resolves the test_io failure. Either of the approaches should work. -- ___ Python tracker ___

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: > Why not simply increase the amount of data written instead of limiting > the pipe size? Hmm, indeed :-) > By the way, there's support.PIPE_MAX_SIZE for that purpose. Hardwired to 3 MB. I wonder if it may broken soon. --

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Charles-François Natali
Charles-François Natali added the comment: Why not simply increase the amount of data written instead of limiting the pipe size? By the way, there's support.PIPE_MAX_SIZE for that purpose. -- ___ Python tracker __

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ok, what does the following patch do for you: diff --git a/Lib/test/test_io.py b/Lib/test/test_io.py --- a/Lib/test/test_io.py +++ b/Lib/test/test_io.py @@ -3168,7 +3168,7 @@ class SignalsTest(unittest.TestCase): select = support.import_module("select")

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread David Edelsohn
David Edelsohn added the comment: >>> r, w = os.pipe() >>> fcntl.fcntl(r, 1031, 1000) 65536 -- ___ Python tracker ___ ___ Python-bugs-

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: Ah, right. That number is the pipe buffer size (1032 is F_GETPIPE_SZ). It's 65536 here, so when the test tries to write 1 million bytes on a pipe, the write blocks as expected (you can read the comments to understand why the test is doint that). But with a 1 M

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread David Edelsohn
David Edelsohn added the comment: >>> import fcntl, os >>> r, w = os.pipe() >>> fcntl.fcntl(w, 1032) 1048576 -- ___ Python tracker ___ ___

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread Antoine Pitrou
Antoine Pitrou added the comment: What does the following say for you? >>> import fcntl, os >>> r, w = os.pipe() >>> fcntl.fcntl(w, 1032) -- nosy: +pitrou ___ Python tracker ___

[issue17835] test_io broken on PPC64 Linux

2013-04-24 Thread David Edelsohn
New submission from David Edelsohn: Unoptimized debug build (configured using --with-pydebug). gcc -pthread -c -Wno-unused-result -g -O0 -Wall -Wstrict-prototypes -m64 test_interrupted_write_retry_buffered (test.test_io.CSignalsTest) ... ERROR test_interrupted_write_retry_text (test.test_io.CSig