[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: (Oops, closing was my intent of my previous comment, but I forgot it, thanks Berker.) -- ___ Python tracker

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

Berker Peksag added the comment: This has been fixed in all active branches (2.7, 3.6 and master) so I think we can close it as 'fixed'. Thanks, Nir! -- nosy: +berker.peksag resolution: -> fixed stage: patch review -> resolved status: open -> closed type: ->

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: The bug has been fixed in Python 2.7, 3.6 and the master branch. Thank you Nir Soffer for the bug report and the fix! -- versions: -Python 3.5, Python 3.8 ___ Python tracker

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: New changeset 830daae1c82ed33deef0086b7b6323e5be0b0cc8 by Victor Stinner (Nir Soffer) in branch '2.7': [2.7] bpo-32186: Release the GIL during fstat and lseek calls (#4651)

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: New changeset 8bcd41040a5f1f9b48a86d0e21f196e4b1f90e4b by Victor Stinner (Miss Islington (bot)) in branch '3.6': bpo-32186: Release the GIL during lseek and fstat (GH-4652) (#4661)

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

Change by Roundup Robot : -- pull_requests: +4571 ___ Python tracker ___

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: New changeset 6a89481680b921e7b317c29877bdda9a6031e5ad by Victor Stinner (Nir Soffer) in branch 'master': bpo-32186: Release the GIL during lseek and fstat (#4652)

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

STINNER Victor added the comment: We already release the GIL when calling lseek() in fileio.c, in the portable_lseek() function. So it makes sense to also do it in _io_FileIO_readall_impl() in the same file. os.lseek() also releases the GIL. I found another

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

Change by Nir Soffer : -- keywords: +patch pull_requests: +4563 stage: -> patch review ___ Python tracker ___

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

Change by Nir Soffer : -- pull_requests: +4564 ___ Python tracker ___ ___ Python-bugs-list

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

Nir Soffer added the comment: Forgot to mention - reproducible with python 2.7. Similar issues exists in python 3, but I did not try to reproduce since we are using python 2.7. I posted patches for both 2.7 and master: - https://github.com/python/cpython/pull/4651 -

[issue32186] io.FileIO hang all threads if fstat blocks on inaccessible NFS server

New submission from Nir Soffer : Using io.FileIO can hang all threads when accessing an inaccessible NFS server. To reproduce this, you need to open the file like this: fd = os.open(filename, ...) fio = io.FileIO(fd, "r+", closefd=True) Inside fileio_init, there is a