[pve-devel] [PATCH qemu-server 1/2] add hv_stimer, hv_synic cpu flags option

[Alexandre DERUMIER]

>>this does not work, because with that regex we need to define 
>>+hv_synic;+hv_stimer 
>>but this does not work of course 

oh yes, indeed, we don't need the +  for hv_... flags.

>>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? 

I personnaly prefer the first one.




----- Mail original -----
De: "Dominik Csapak" <d.csapak at proxmox.com>
À: "pve-devel" <pve-devel at pve.proxmox.com>
Envoyé: Mercredi 27 Juin 2018 10:29:41
Objet: Re: [pve-devel] [PATCH qemu-server 1/2] add hv_stimer, hv_synic cpu flags option

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 at pve.proxmox.com 
https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel