X-Git-Url: https://git.proxmox.com/?p=pve-common.git;a=blobdiff_plain;f=src%2FPVE%2FNetwork.pm;h=69051b9de048e5a6023902b9898af9cd5ab655b5;hp=ac2c168c0ef718b4ec3656d2b6d1259ea6579766;hb=0c0304c0d9a9a753b5d93a6c7b2239059a26d945;hpb=1b915170dbef65a1c41536f19f0572c3b0ed839f diff --git a/src/PVE/Network.pm b/src/PVE/Network.pm index ac2c168..69051b9 100644 --- a/src/PVE/Network.pm +++ b/src/PVE/Network.pm @@ -2,7 +2,7 @@ package PVE::Network; use strict; use warnings; -use PVE::Tools qw(run_command); +use PVE::Tools qw(run_command lock_file); use PVE::ProcFSTools; use PVE::INotify; use File::Basename; @@ -11,12 +11,6 @@ use POSIX qw(ECONNREFUSED); use Net::IP; -use Socket qw(IPPROTO_IP); - -use constant IFF_UP => 1; -use constant IFNAMSIZ => 16; -use constant SIOCGIFFLAGS => 0x8913; - # host network related utility functions our $ipv4_reverse_mask = [ @@ -56,6 +50,14 @@ our $ipv4_reverse_mask = [ ]; our $ipv4_mask_hash_localnet = { + '255.0.0.0' => 8, + '255.128.0.0' => 9, + '255.192.0.0' => 10, + '255.224.0.0' => 11, + '255.240.0.0' => 12, + '255.248.0.0' => 13, + '255.252.0.0' => 14, + '255.254.0.0' => 15, '255.255.0.0' => 16, '255.255.128.0' => 17, '255.255.192.0' => 18, @@ -71,15 +73,18 @@ our $ipv4_mask_hash_localnet = { '255.255.255.240' => 28, '255.255.255.248' => 29, '255.255.255.252' => 30, + '255.255.255.254' => 31, + '255.255.255.255' => 32, }; sub setup_tc_rate_limit { my ($iface, $rate, $burst, $debug) = @_; - system("/sbin/tc class del dev $iface parent 1: classid 1:1 >/dev/null 2>&1"); - system("/sbin/tc filter del dev $iface parent ffff: protocol all pref 50 u32 >/dev/null 2>&1"); - system("/sbin/tc qdisc del dev $iface ingress >/dev/null 2>&1"); - system("/sbin/tc qdisc del dev $iface root >/dev/null 2>&1"); + # these are allowed / expected to fail, e.g. when there is no previous rate limit to remove + eval { run_command("/sbin/tc class del dev $iface parent 1: classid 1:1 >/dev/null 2>&1"); }; + eval { run_command("/sbin/tc filter del dev $iface parent ffff: protocol all pref 50 u32 >/dev/null 2>&1"); }; + eval { run_command("/sbin/tc qdisc del dev $iface ingress >/dev/null 2>&1"); }; + eval { run_command("/sbin/tc qdisc del dev $iface root >/dev/null 2>&1"); }; return if !$rate; @@ -166,9 +171,20 @@ my $cond_create_bridge = sub { } }; +sub disable_ipv6 { + my ($iface) = @_; + return if !-d '/proc/sys/net/ipv6'; # ipv6 might be completely disabled + my $file = "/proc/sys/net/ipv6/conf/$iface/disable_ipv6"; + open(my $fh, '>', $file) or die "failed to open $file for writing: $!\n"; + print {$fh} "1\n" or die "failed to disable link-local ipv6 for $iface\n"; + close($fh); +} + my $bridge_add_interface = sub { my ($bridge, $iface, $tag, $trunks) = @_; + # drop link local address (it can't be used when on a bridge anyway) + disable_ipv6($iface); system("/sbin/brctl addif $bridge $iface") == 0 || die "can't add interface 'iface' to bridge '$bridge'\n"; @@ -176,8 +192,12 @@ my $bridge_add_interface = sub { if ($vlan_aware) { if ($tag) { - system("/sbin/bridge vlan add dev $iface vid $tag pvid untagged") == 0 || - die "unable to add vlan $tag to interface $iface\n"; + system({'/sbin/bridge'} 'bridge', 'vlan', 'del', 'dev', $iface, 'vid', '1-4094') == 0 + or die "failed to remove default vlan tags of $iface\n"; + system({'/sbin/bridge'} 'bridge', 'vlan', 'add', 'dev', $iface, 'vid', $tag, 'pvid', 'untagged') == 0 + or die "unable to add vlan $tag to interface $iface\n"; + + warn "Caution: Setting VLAN ID 1 on a VLAN aware bridge may be dangerous\n" if $tag == 1; } else { system("/sbin/bridge vlan add dev $iface vid 2-4094") == 0 || die "unable to add default vlan tags to interface $iface\n" if !$trunks; @@ -206,6 +226,7 @@ my $ovs_bridge_add_port = sub { $cmd .= " -- set Interface $iface type=internal" if $internal; system($cmd) == 0 || die "can't add ovs port '$iface'\n"; + disable_ipv6($iface); }; my $activate_interface = sub { @@ -223,7 +244,8 @@ sub tap_create { my $bridgemtu = &$read_bridge_mtu($bridge); eval { - PVE::Tools::run_command("/sbin/ifconfig $iface 0.0.0.0 promisc up mtu $bridgemtu"); + disable_ipv6($iface); + PVE::Tools::run_command(['/sbin/ip', 'link', 'set', $iface, 'up', 'promisc', 'on', 'mtu', $bridgemtu]); }; die "interface activation failed\n" if $@; } @@ -243,6 +265,8 @@ sub veth_create { } # up vethpair + disable_ipv6($veth); + disable_ipv6($vethpeer); &$activate_interface($veth); &$activate_interface($vethpeer); } @@ -253,7 +277,7 @@ sub veth_delete { if (-d "/sys/class/net/$veth") { run_command("/sbin/ip link delete dev $veth", outfunc => sub {}, errfunc => sub {}); } - + eval { tap_unplug($veth) }; } my $create_firewall_bridge_linux = sub { @@ -263,6 +287,7 @@ my $create_firewall_bridge_linux = sub { my ($fwbr, $vethfw, $vethfwpeer) = &$compute_fwbr_names($vmid, $devid); &$cond_create_bridge($fwbr); + disable_ipv6($fwbr); &$activate_interface($fwbr); copy_bridge_config($bridge, $fwbr); @@ -283,6 +308,7 @@ my $create_firewall_bridge_ovs = sub { my $bridgemtu = &$read_bridge_mtu($bridge); &$cond_create_bridge($fwbr); + disable_ipv6($fwbr); &$activate_interface($fwbr); &$bridge_add_interface($fwbr, $iface); @@ -291,7 +317,7 @@ my $create_firewall_bridge_ovs = sub { &$activate_interface($ovsintport); # set the same mtu for ovs int port - PVE::Tools::run_command("/sbin/ifconfig $ovsintport mtu $bridgemtu"); + PVE::Tools::run_command(['/sbin/ip', 'link', 'set', $ovsintport, 'mtu', $bridgemtu]); &$bridge_add_interface($fwbr, $ovsintport); }; @@ -371,6 +397,8 @@ sub tap_unplug { } &$cleanup_firewall_bridge($iface); + #cleanup old port config from any openvswitch bridge + eval {run_command("/usr/bin/ovs-vsctl del-port $iface", outfunc => sub {}, errfunc => sub {}) }; } sub copy_bridge_config { @@ -399,10 +427,13 @@ sub activate_bridge_vlan_slave { # create vlan on $iface is not already exist if (! -d "/sys/class/net/$ifacevlan") { - system("/sbin/ip link add link $iface name ${iface}.${tag} type vlan id $tag") == 0 || + system("/sbin/ip link add link $iface name $ifacevlan type vlan id $tag") == 0 || die "can't add vlan tag $tag to interface $iface\n"; } + # remove ipv6 link-local address before activation + disable_ipv6($ifacevlan); + # be sure to have the $ifacevlan up &$activate_interface($ifacevlan); @@ -437,29 +468,31 @@ sub activate_bridge_vlan { my @ifaces = (); my $dir = "/sys/class/net/$bridge/brif"; - PVE::Tools::dir_glob_foreach($dir, '((eth|bond)\d+(\.\d+)?)', sub { + PVE::Tools::dir_glob_foreach($dir, '(((eth|bond)\d+|en[^.]+)(\.\d+)?)', sub { push @ifaces, $_[0]; }); die "no physical interface on bridge '$bridge'\n" if scalar(@ifaces) == 0; - # add bridgevlan if it doesn't already exist - if (! -d "/sys/class/net/$bridgevlan") { - system("/sbin/brctl addbr $bridgevlan") == 0 || - die "can't add bridge $bridgevlan\n"; - } + lock_network(sub { + # add bridgevlan if it doesn't already exist + if (! -d "/sys/class/net/$bridgevlan") { + system("/sbin/brctl addbr $bridgevlan") == 0 || + die "can't add bridge $bridgevlan\n"; + } - # for each physical interface (eth or bridge) bind them to bridge vlan - foreach my $iface (@ifaces) { - activate_bridge_vlan_slave($bridgevlan, $iface, $tag); - } + # for each physical interface (eth or bridge) bind them to bridge vlan + foreach my $iface (@ifaces) { + activate_bridge_vlan_slave($bridgevlan, $iface, $tag); + } - #fixme: set other bridge flags + #fixme: set other bridge flags - # be sure to have the bridge up - system("/sbin/ip link set $bridgevlan up") == 0 || - die "can't up bridge $bridgevlan\n"; - + # remove ipv6 link-local address before activation + disable_ipv6($bridgevlan); + # be sure to have the bridge up + &$activate_interface($bridgevlan); + }); return $bridgevlan; } @@ -522,34 +555,32 @@ sub is_ip_in_cidr { return $cidr_obj->overlaps($ip_obj) == $Net::IP::IP_B_IN_A_OVERLAP; } -# struct ifreq { // FOR SIOCGIFFLAGS: -# char ifrn_name[IFNAMSIZ] -# short ifru_flags -# }; -my $STRUCT_IFREQ_SIOCGIFFLAGS = 'Z' . IFNAMSIZ . 's1'; -sub get_active_interfaces { - # Use the interface name list from /proc/net/dev - open my $fh, '<', '/proc/net/dev' - or die "failed to open /proc/net/dev: $!\n"; - # And filter by IFF_UP flag fetched via a PF_INET6 socket ioctl: - socket my $sock, PF_INET6, SOCK_DGRAM, &IPPROTO_IP - or die "failed to open socket\n"; - - my $ifaces = []; - while(defined(my $line = <$fh>)) { - next if $line !~ /^\s*([^:\s]+):/; - my $ifname = $1; - my $ifreq = pack($STRUCT_IFREQ_SIOCGIFFLAGS, $1, 0); - if (!defined(ioctl($sock, SIOCGIFFLAGS, $ifreq))) { - warn "failed to get interface flags for: $ifname\n"; - next; + +sub get_local_ip_from_cidr { + my ($cidr) = @_; + + my $cmd = ['/sbin/ip', 'address', 'show', 'to', $cidr, 'up']; + + my $IPs = []; + + my $code = sub { + my $line = shift; + + if ($line =~ m!^\s*inet(?:6)?\s+($PVE::Tools::IPRE)/\d+!) { + push @$IPs, $1; } - my ($name, $flags) = unpack($STRUCT_IFREQ_SIOCGIFFLAGS, $ifreq); - push @$ifaces, $1 if ($flags & IFF_UP); - } - close $fh; - close $sock; - return $ifaces; + }; + + PVE::Tools::run_command($cmd, outfunc => $code); + + return $IPs; +} + +sub lock_network { + my ($code, @param) = @_; + my $res = lock_file('/var/lock/pve-network.lck', 10, $code, @param); + die $@ if $@; + return $res; } 1;