On 4/23/20 1:51 PM, Fabian Ebner wrote: > No functional change is intended. > > Signed-off-by: Fabian Ebner <f.eb...@proxmox.com> > --- > PVE/AbstractConfig.pm | 8 +------- > 1 file changed, 1 insertion(+), 7 deletions(-) > > diff --git a/PVE/AbstractConfig.pm b/PVE/AbstractConfig.pm > index beb10c7..f1b395c 100644 > --- a/PVE/AbstractConfig.pm > +++ b/PVE/AbstractConfig.pm > @@ -259,13 +259,7 @@ sub load_current_config { > sub lock_config_full { > my ($class, $vmid, $timeout, $code, @param) = @_; > > - my $filename = $class->config_file_lock($vmid); > - > - my $res = lock_file($filename, $timeout, $code, @param); > - > - die $@ if $@; > - > - return $res; > + return $class->lock_config_mode($vmid, $timeout, 0, $code, @param); > } > > sub create_and_lock_config { >
so I lost a bit track on the remaining patches from both of you. There seem some nits/unrelated hunks which got commented (e.g., patch guest-common 1/3). I'd appreciated if one of you could pick this up and send a nice v2 with those addressed and R-b tags where applicable. _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com https://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel