diff options
| author | Nataliia Solomko <natalirs1985@gmail.com> | 2025-01-28 11:56:18 +0200 |
|---|---|---|
| committer | Nataliia Solomko <natalirs1985@gmail.com> | 2025-02-03 14:39:09 +0200 |
| commit | e5c5d6778883ab1e73f9e0face6ad9b660b2efec (patch) | |
| tree | 197134f43fdd73caac71c42431a6ae8cc64044b0 /src | |
| parent | 1c8e9d5c409ba6b0a7e63cacb8e886bada036b4b (diff) | |
| download | vyos-1x-e5c5d6778883ab1e73f9e0face6ad9b660b2efec.tar.gz vyos-1x-e5c5d6778883ab1e73f9e0face6ad9b660b2efec.zip | |
T7066: VPP CPU workers should be calculated and verified
Diffstat (limited to 'src')
| -rwxr-xr-x | src/conf_mode/vpp.py | 17 |
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') |
