this does not work, because with that regex we need to define
+hv_synic;+hv_stimer
but this does not work of course
i believe the correct regex would be
qr/(?:[+-](pcid|spec-ctrl)|hv_(stimer|synic))/
or maybe let the +/- optional and add it for pcid/spec-ctlr/future flags?
On 06/24/2018 08:47 PM, Alexandre Derumier wrote:
---
PVE/QemuServer.pm | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/PVE/QemuServer.pm b/PVE/QemuServer.pm
index d6efb3a..1e4ac5d 100644
--- a/PVE/QemuServer.pm
+++ b/PVE/QemuServer.pm
@@ -155,7 +155,7 @@ my $cpu_vendor_list = {
max => 'default',
};
-my $cpu_flag = qr/[+-](pcid|spec-ctrl)/;
+my $cpu_flag = qr/[+-](pcid|spec-ctrl|hv_stimer|hv_synic)/;
my $cpu_fmt = {
cputype => {
@@ -174,7 +174,7 @@ my $cpu_fmt = {
flags => {
description => "List of additional CPU flags separated by ';'."
. " Use '+FLAG' to enable, '-FLAG' to disable a flag."
- . " Currently supported flags: 'pcid', 'spec-ctrl'.",
+ . " Currently supported flags: 'pcid', 'spec-ctrl',
'hv_stimer', 'hv_synic'.",
format_description => '+FLAG[;-FLAG...]',
type => 'string',
pattern => qr/$cpu_flag(;$cpu_flag)*/,
_______________________________________________
pve-devel mailing list
pve-devel@pve.proxmox.com
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
