since we iterate over the entries in /sys/block it makes sense to use this path
this should fix #1099 because udevadm does not take -n cciss!c0d0 (because it only looks in dev for this) but takes -p /sys/block/cciss!c0d0 Signed-off-by: Dominik Csapak <d.csa...@proxmox.com> --- PVE/Diskmanage.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/PVE/Diskmanage.pm b/PVE/Diskmanage.pm index 6cc5e1c..8382045 100644 --- a/PVE/Diskmanage.pm +++ b/PVE/Diskmanage.pm @@ -217,7 +217,7 @@ sub get_udev_info { my $info = ""; my $data = {}; eval { - run_command([$UDEVADM, 'info', '-n', $dev, '--query', 'all'], outfunc => sub { + run_command([$UDEVADM, 'info', '-p', $dev, '--query', 'all'], outfunc => sub { my ($line) = @_; $info .= "$line\n"; }); @@ -375,7 +375,7 @@ sub get_disks { $dev !~ m/^nvme\d+n\d+$/ && $dev !~ m/^cciss\!c\d+d\d+$/; - my $data = get_udev_info($dev); + my $data = get_udev_info("/sys/block/$dev"); return if !defined($data); my $devpath = $data->{devpath}; -- 2.1.4 _______________________________________________ pve-devel mailing list pve-devel@pve.proxmox.com http://pve.proxmox.com/cgi-bin/mailman/listinfo/pve-devel