mirror of
https://github.com/XTXMarkets/ternfs.git
synced 2025-12-21 02:30:18 -06:00
01f9d5addf
The FUSE driver, up to now, had no way to know when the user had "explicitly" closed a file. Instead it linked the TernFS file on flush, which could cause nasty situation. The classic example is a fork causing the FD to a TernFS file being present in the forked process, and then the process dying causing a spurious flush. This commit adds a way to detect when a flush is due to a close(), which allows us to link the file only in the cases where that happened, which is a much better heuristic and close to what we do in the kernel module. This commit also contains various other improvements to make all tests pass under FUSE. The big remaining item is changing how files are read (they're currently read all upfront and then kept in memory).
50 lines
1.9 KiB
Python
Executable File
50 lines
1.9 KiB
Python
Executable File
#!/usr/bin/env python3
|
|
|
|
# Copyright 2025 XTX Markets Technologies Limited
|
|
#
|
|
# SPDX-License-Identifier: GPL-2.0-or-later
|
|
|
|
import sys
|
|
import os
|
|
from pathlib import Path
|
|
import subprocess
|
|
|
|
if len(sys.argv) < 2:
|
|
print(f'Usage: {sys.argv[0]} <build-type> [NINJA_ARG ...]', file=sys.stderr)
|
|
sys.exit(2)
|
|
|
|
if len(sys.argv) == 1:
|
|
build_type = 'release'
|
|
else:
|
|
build_type = sys.argv[1]
|
|
|
|
cpp_dir = Path(__file__).resolve().parent
|
|
repo_dir = cpp_dir.parent
|
|
|
|
build_dir = cpp_dir / 'build' / build_type
|
|
build_dir.mkdir(parents=True, exist_ok=True)
|
|
|
|
if build_type in ('ubuntu', 'ubuntudebug', 'ubuntusanitized', 'ubuntuvalgrind', 'alpine', 'alpinedebug') and 'IN_TERN_BUILD_CONTAINER' not in os.environ:
|
|
if build_type.startswith('alpine'):
|
|
container = 'ghcr.io/xtxmarkets/ternfs-alpine-build:2025-09-18-1'
|
|
else:
|
|
container = 'ghcr.io/xtxmarkets/ternfs-ubuntu-build:2025-09-18'
|
|
# See <https://groups.google.com/g/seastar-dev/c/r7W-Kqzy9O4>
|
|
# for motivation for `--security-opt seccomp=unconfined`,
|
|
# the `--pids-limit -1` is not something I hit but it seems
|
|
# like a good idea.
|
|
subprocess.run(
|
|
['docker', 'run', '--network', 'host', '-e', 'MAKE_PARALLELISM', '-e', 'http_proxy', '-e', 'https_proxy', '-e', 'no_proxy', '--pids-limit', '-1', '--security-opt', 'seccomp=unconfined', '--rm', '-i', '--mount', f'type=bind,src={repo_dir},dst=/ternfs', '-u', f'{os.getuid()}:{os.getgid()}', container, '/ternfs/cpp/build.py', build_type] + sys.argv[2:],
|
|
check=True,
|
|
)
|
|
else:
|
|
os.chdir(str(build_dir))
|
|
build_types = {
|
|
'ubuntu': 'release',
|
|
'ubuntudebug': 'debug',
|
|
'ubuntusanitized': 'sanitized',
|
|
'ubuntuvalgrind': 'valgrind',
|
|
}
|
|
subprocess.run(['cmake', '-G', 'Ninja', f'-DCMAKE_BUILD_TYPE={build_types.get(build_type, build_type)}', '../..'], check=True)
|
|
subprocess.run(['ninja'] + sys.argv[2:], check=True)
|