Description

colesbury

opened on Aug 4, 2022

Issue body actions

Bug report

On Linux systems, when configured with a default pipe capacity of 4096, the following test fail:

• test_fcntl.test_fcntl_f_pipesize
• test_subprocess.test_pipesizes

From a recent fcntl(2) Linux man page:

F_SETPIPE_SZ:
...
Attempts to set the pipe capacity below the page size are silently rounded up to the page size.

There's a check that attempts to skip the tests if the pipe capacity is 512 bytes, but that's less than the smallest page size on x86.

Since this feature appears to be Linux specific, the check should:

1. Use os.sysconf('SC_PAGESIZE') as a minimum
2. Fix the typos ("SkitTest") in test_fcntl_f_pipesize and test_pipesizes

Linked PRs

• gh-95672 fix typo SkitTest to SkipTest #102119
• [3.11] gh-95672 fix typo SkitTest to SkipTest (gh-102119) #102121
• [3.10] gh-95672 fix typo SkitTest to SkipTest (gh-102119) #102122
• gh-95672 skip fcntl when pipesize is smaller than pagesize #102163
• gh-95672: Update memory_watchdog to use test.support.get_pagesize #102365
• gh-95672: Fix versionadded indentation of get_pagesize in test.rst #102455

Activity

1. colesbury

   added

   type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

   testsTests in the Lib/test dirTests in the Lib/test dir

   3.11only security fixesonly security fixes

   3.10 (EOL)end of lifeend of life

   3.12only security fixesonly security fixes

   on Aug 4, 2022
2. corona10

   self-assigned this

   on Feb 18, 2023
3. corona10 commented on Feb 18, 2023

   

   corona10

   on Feb 18, 2023

   Member

   More actions

   Same as #95675 (comment)
   I will introduce this issue into the internal OSS sprint as the good first bug.
   I will delegate the issue to my mentees.
   Thank you for understanding!
4. hyeongyun0916 commented on Feb 20, 2023

   

   hyeongyun0916

   on Feb 20, 2023

   Contributor

   More actions

   I'm looking this issue 👀
5. corona10

   assigned

   hyeongyun0916

   on Feb 20, 2023
6. corona10

   added a commit that references this issue on Feb 21, 2023

   gh-95672 fix typo SkitTest to SkipTest (gh-102119)

   d5c7954
7. miss-islington

   added 2 commits that reference this issue on Feb 21, 2023

   pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119)

   bbe277e

   pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119)

   d3f8606
8. corona10

   added 2 commits that reference this issue on Feb 22, 2023

   [3.11] gh-95672 fix typo SkitTest to SkipTest (gh-102119) (#102121)

   5d47150

   [3.10] gh-95672 fix typo SkitTest to SkipTest (gh-102119) (gh-102122)

   bac3fe7
9. carljm

   added a commit that references this issue on Feb 23, 2023

   Merge branch 'main' into lockfreecp

   b257a16
10. corona10

    added 2 commits that reference this issue on Mar 1, 2023

    gh-95672 skip fcntl when pipesize is smaller than pagesize (gh-102163)

    2f62a5d

    gh-95672: Update memory_watchdog to use test.support.get_pagesize (gh…

    6059743
11. corona10 commented on Mar 2, 2023

    

    corona10

    on Mar 2, 2023

    Member

    More actions

    @hyeongyun0916 Thanks for the hardwork!
12. corona10

    closed this as completedon Mar 2, 2023
13. corona10

    added a commit that references this issue on Mar 6, 2023

    gh-95672: Fix versionadded indentation of get_pagesize in test.rst (g…

    d3ca042
14. carljm

    added 2 commits that reference this issue on Mar 6, 2023

    Merge branch 'main' into nodeaddict

    33dda2a

    Merge branch 'main' into inlinecomp2

    ecb313c
15. python-sidebar

    added 2 commits that reference this issue on Sep 1, 2024

    pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119)

    8ac8bdd

    pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119)

    da08f1d
16. JelleZijlstra

    added a commit that references this issue on Sep 10, 2024

    pythongh-95672 fix typo SkitTest to SkipTest (pythongh-102119)

    065e45d

Issue actions