Giampaolo Rodola' added the comment:
Just received this report on psutil bug tracker:
https://code.google.com/p/psutil/issues/detail?id=455
It seems os.prlimit() is affected by the same problem:
import resource
resource.RLIM_INFINITY
-1
--
resolution: fixed -
status: closed - open
Giampaolo Rodola' added the comment:
s/os.prlimit/resource.prlimit
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
___
Python-bugs-list
Antoine Pitrou added the comment:
How is that a problem? In any case, this shouldn't have anything to do with
prlimit(), please open another issue.
--
resolution: - fixed
status: open - closed
___
Python tracker rep...@bugs.python.org
Christian Heimes added the comment:
Tests are passing on all buildbots for quite some time now.
--
resolution: - fixed
stage: commit review - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
Roundup Robot added the comment:
New changeset 87d41a5a9077 by Christian Heimes in branch 'default':
Issue #16595: prlimit() needs Linux kernel 2.6.36+
http://hg.python.org/cpython/rev/87d41a5a9077
--
___
Python tracker rep...@bugs.python.org
Christian Heimes added the comment:
The buildbot is
Linux-2.6.35-vs2.3.0.36.32-gentoo-i686-Intel-R-_Core-TM-2_CPU_6600_@_2.40GHz-with-gentoo-2.1
but prlimit() requires 2.6.36+. I didn't expect to see a combination of glibc
with prlimit() and Kernel without prlimit(). According to man prlimit
STINNER Victor added the comment:
Or we should extend with supress(OSerror, errno=errno.ENOSYS): ... :-)
(Just kidding, ignored tests must be marked as skipped.)
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
Charles-François Natali added the comment:
I didn't expect to see a combination of glibc with prlimit() and Kernel
without prlimit(). According to man prlimit it's not suppose to fail with
ENOSYS, too.
Yeah, we've seen this several times on some buildbots.
Note that actually, any syscall
Charles-François Natali added the comment:
The test is failing with ENOSYS on one of the buildbots:
==
ERROR: test_prlimit (test.test_resource.ResourceTest)
--
Roundup Robot added the comment:
New changeset 796c21e27a92 by Christian Heimes in branch 'default':
Issue #16595: Add prlimit() to resource module
http://hg.python.org/cpython/rev/796c21e27a92
--
nosy: +python-dev
___
Python tracker
Christian Heimes added the comment:
Let's see how the buildbots respond. I may have to disable some tests for
ancient Kernel versions.
--
stage: patch review - commit review
status: open - pending
___
Python tracker rep...@bugs.python.org
Antoine Pitrou added the comment:
+self.assertRaises(PermissionError, resource.prlimit,
+ 1, resource.RLIMIT_AS)
Please skip this when run as root.
+self.assertEqual(resource.prlimit(0, resource.RLIMIT_AS), (-1, -1))
What if it's not that value by
Christian Heimes added the comment:
Roundup bot hangs. I have addressed your concerns in
http://hg.python.org/cpython/rev/8c77117f41a9
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
Christian Heimes added the comment:
Thanks for your tests. I may have to rework my test scenario a bit.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
Giampaolo Rodola' added the comment:
From my original post which unfortunately was ignored :( :
http://mail.python.org/pipermail/python-ideas/2012-June/015323.html
...for completeness perhaps it also makes sense to expose Linux-specific RLIMIT
constants:
RLIMIT_MSGQUEUE
RLIMIT_NICE
Vajrasky Kok added the comment:
Fedora 18, this is the result after applying your patch and execute your unit
test:
[sky@localhost cpython]$ ./python Lib/test/test_resource.py
test_args (__main__.ResourceTest) ... ok
test_fsize_enforced (__main__.ResourceTest) ... ok
test_fsize_ismax
Christian Heimes added the comment:
Does anybody want to review the code before I commit it?
--
nosy: +haypo, pitrou
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
Christian Heimes added the comment:
Updated patch, now raises PermissionError on EPERM.
--
Added file: http://bugs.python.org/file30685/prlimit2.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
Changes by Christian Heimes li...@cheimes.de:
Removed file: http://bugs.python.org/file28184/prlimit.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
Changes by Arfrever Frehtes Taifersar Arahesis arfrever@gmail.com:
--
nosy: +Arfrever
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
New submission from Christian Heimes:
Linux has an additional function to setrlimit and getrlimit which supports to
query and/or modify resource limits of another process.
http://www.kernel.org/doc/man-pages/online/pages/man2/getrlimit.2.html
The patch implements resource.prlimit(pid,
Christian Heimes added the comment:
prlimit() needs glibc 2.13+ (thanks Arfrever).
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue16595
___
___
22 matches
Mail list logo
