summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorNataliia Solomko <natalirs1985@gmail.com>2025-01-28 11:56:18 +0200
committerNataliia Solomko <natalirs1985@gmail.com>2025-02-03 14:39:09 +0200
commite5c5d6778883ab1e73f9e0face6ad9b660b2efec (patch)
tree197134f43fdd73caac71c42431a6ae8cc64044b0 /src
parent1c8e9d5c409ba6b0a7e63cacb8e886bada036b4b (diff)
downloadvyos-1x-e5c5d6778883ab1e73f9e0face6ad9b660b2efec.tar.gz
vyos-1x-e5c5d6778883ab1e73f9e0face6ad9b660b2efec.zip
T7066: VPP CPU workers should be calculated and verified
Diffstat (limited to 'src')
-rwxr-xr-xsrc/conf_mode/vpp.py17
1 files changed, 15 insertions, 2 deletions
diff --git a/src/conf_mode/vpp.py b/src/conf_mode/vpp.py
index 923ac7f31..7da40a21a 100755
--- a/src/conf_mode/vpp.py
+++ b/src/conf_mode/vpp.py
@@ -381,10 +381,22 @@ def verify(config):
if 'cpu' in config['settings']:
if (
'corelist_workers' in config['settings']['cpu']
- and 'main_core' not in config['settings']['cpu']
- ):
+ or 'workers' in config['settings']['cpu']
+ ) and 'main_core' not in config['settings']['cpu']:
raise ConfigError('"cpu main-core" is required but not set!')
+ cpus = int(get_core_count())
+ if 'workers' in config['settings']['cpu']:
+ # number of worker threads must be not more than
+ # available CPUs in the system - 2 (1 for main thread and at least 1 for system processes)
+ workers = int(config['settings']['cpu']['workers'])
+ available_workers = cpus - 2
+ if workers > available_workers:
+ raise ConfigError(
+ f'The system does not have enough CPUs for {workers} VPP workers '
+ f'(reduce to {available_workers} or less)'
+ )
+
verify_memory(config['settings'])
# Check if deleted interfaces are not xconnect memebrs
@@ -428,6 +440,7 @@ def generate(config):
def apply(config):
+
# Open persistent config
# It is required for operations with interfaces
persist_config = JSONStorage('vpp_conf')