X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=blobdiff_plain;f=discover%2Fipmi.c;h=38423b2647319fe7a3b040adc61a293857f9c9fd;hp=54d4e0636f37093b089fc4a7748ca9e7db8533d4;hb=8ede209928e7bf7c06a11eee9c51a551193ba8e9;hpb=420f0e2c5d8338b42047c58f055cdb944685ead7 diff --git a/discover/ipmi.c b/discover/ipmi.c index 54d4e06..38423b2 100644 --- a/discover/ipmi.c +++ b/discover/ipmi.c @@ -4,6 +4,7 @@ #include #include #include +#include #include #include @@ -183,7 +184,8 @@ int ipmi_transaction(struct ipmi *ipmi, uint8_t netfn, uint8_t cmd, out: lock.l_type = F_UNLCK; - fcntl(ipmi->fd, F_SETLKW, &lock); + if (fcntl(ipmi->fd, F_SETLKW, &lock) == -1) + pb_log("IPMI: error unlocking IPMI device: %m\n"); return rc ? -1 : 0; } @@ -219,3 +221,101 @@ bool ipmi_present(void) return !access(ipmi_devnode, R_OK | W_OK); } +/* Reads and applies an IPMI interface config override, which closely follows + * the format of an interface config struct as described in lib/types */ +int parse_ipmi_interface_override(struct config *config, uint8_t *buf, + uint16_t len) +{ + struct interface_config *ifconf; + char *ipstr, *gatewaystr; + uint8_t hwsize, ipsize; + int addr_type, i = 0; + socklen_t addr_len; + + /* Get 1-byte hardware address size and ip address size */ + memcpy(&hwsize, &buf[i], sizeof(hwsize)); + i += sizeof(hwsize); + memcpy(&ipsize, &buf[i], sizeof(ipsize)); + i += sizeof(ipsize); + + if (!hwsize || !ipsize) { + pb_log("%s: Empty response\n", __func__); + return -1; + } + + /* At the moment only support 6-byte MAC addresses */ + if (hwsize != sizeof(ifconf->hwaddr)) { + pb_log("Unsupported HW address size in network override: %u\n", + hwsize); + return -1; + } + + /* Sanity check the IP address size */ + if (ipsize == 4) { + addr_type = AF_INET; + addr_len = INET_ADDRSTRLEN; + } else if (ipsize == 16) { + addr_type = AF_INET6; + addr_len = INET6_ADDRSTRLEN; + } else { + pb_log("Unsupported IP address size: %u\n", ipsize); + return -1; + } + + /* Everything past here is the interface config */ + ifconf = talloc_zero(config, struct interface_config); + if (!ifconf) { + pb_log("Failed to allocate network override\n"); + return -1; + } + + /* Hardware Address */ + memcpy(ifconf->hwaddr, &buf[i], hwsize); + i += hwsize; + + /* Check 1-byte ignore and method flags */ + ifconf->ignore = !!buf[i++]; + ifconf->method = !!buf[i++]; + + if (ifconf->method == CONFIG_METHOD_STATIC) { + if (ipsize + ipsize + 1 > len - i) { + pb_log("Expected data greater than buffer size\n"); + talloc_free(ifconf); + return -1; + } + + /* IP address */ + ipstr = talloc_array(ifconf, char, addr_len); + if (!inet_ntop(addr_type, &buf[i], ipstr, addr_len)) { + pb_log("Failed to convert ipaddr: %m\n"); + talloc_free(ifconf); + return -1; + } + i += ipsize; + + /* IP address subnet */ + ifconf->static_config.address = talloc_asprintf(ifconf, + "%s/%u", ipstr, buf[i]); + i++; + + /* Gateway address */ + gatewaystr = talloc_array(ifconf, char, addr_len); + if (!inet_ntop(addr_type, &buf[i], gatewaystr, addr_len)) { + pb_log("Failed to convert gateway: %m\n"); + talloc_free(ifconf); + return -1; + } + ifconf->static_config.gateway = gatewaystr; + i += ipsize; + } + + pb_log("Applying IPMI network interface override\n"); + + /* Replace any existing interface config */ + talloc_free(config->network.interfaces); + config->network.n_interfaces = 1; + config->network.interfaces = talloc(config, struct interface_config *); + config->network.interfaces[0] = ifconf; + + return 0; +}