When compiling drivers on Windows, instances have been seen where a
temporary directory fails to get cleaned up due to
ERROR_SHARING_VIOLATION (32).
This issue was not seen on operating systems other than Windows.
This patch eliminates the use of tempfile.TemporaryDirectory, which
was triggering the issue.
For completeness, the error message is included below.
--- stderr ---
Traceback (most recent call last):
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 631, in _rmtree_unsafe
os.rmdir(path)
~~~~~~~~^^^^^^
PermissionError: [WinError 32] The process cannot access the file because it is
being used by another process:
'V:\\github\\dpdk15\\dpdk\\build\\buildtools\\tmpnrqvi9nc'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 909, in onexc
_os.unlink(path)
~~~~~~~~~~^^^^^^
PermissionError: [WinError 5] Access is denied:
'V:\\github\\dpdk15\\dpdk\\build\\buildtools\\tmpnrqvi9nc'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "V:\github\dpdk15\dpdk\buildtools\gen-pmdinfo-cfile.py", line 13, in
<module>
with tempfile.TemporaryDirectory(dir=tmp_root) as temp:
~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 950, in __exit__
self.cleanup()
~~~~~~~~~~~~^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 954, in cleanup
self._rmtree(self.name, ignore_errors=self._ignore_cleanup_errors)
~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 934, in _rmtree
_shutil.rmtree(name, onexc=onexc)
~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 790, in rmtree
return _rmtree_unsafe(path, onexc)
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 635, in _rmtree_unsafe
onexc(os.rmdir, path, err)
~~~~~^^^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 924, in onexc
cls._rmtree(path, ignore_errors=ignore_errors,
~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
repeated=(path == name))
^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\tempfile.py",
line 934, in _rmtree
_shutil.rmtree(name, onexc=onexc)
~~~~~~~~~~~~~~^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 790, in rmtree
return _rmtree_unsafe(path, onexc)
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 635, in _rmtree_unsafe
onexc(os.rmdir, path, err)
~~~~~^^^^^^^^^^^^^^^^^^^^^
File "C:\Program
Files\WindowsApps\PythonSoftwareFoundation.Python.3.13_3.13.1008.0_x64__qbz5n2kfra8p0\Lib\shutil.py",
line 631, in _rmtree_unsafe
os.rmdir(path)
~~~~~~~~^^^^^^
PermissionError: [WinError 32] The process cannot access the file because it is
being used by another process:
'V:\\github\\dpdk15\\dpdk\\build\\buildtools\\tmpnrqvi9nc'
[53/450] Compiling C object lib/librte_pcapng.a.p/pcapng_rte_pcapng.c.obj
ninja: build stopped: subcommand failed.
v3:
- removed parameter tmp_root from gen-pmdinfo-cfile.py
- modified gen-pmdinfo-cfile.py to create '@PRIVATE_DIR@' when it does not
exist.
v2:
- removed use of Python's tempfile.TemporaryDirectory and used
'@PRIVATE_DIR@' in meson's custom_target command instead.
Andre Muezerie (1):
buildtools: avoid break due to failure to cleanup temporary directory
buildtools/gen-pmdinfo-cfile.py | 26 +++++++++++++-------------
buildtools/meson.build | 2 +-
drivers/meson.build | 2 +-
3 files changed, 15 insertions(+), 15 deletions(-)
--
2.49.0.vfs.0.2
