X-Git-Url: http://git.ozlabs.org/?p=petitboot;a=blobdiff_plain;f=discover%2Fipmi.c;h=f59974aaf637b623a413e890177bee9b0dc39a77;hp=710a47d1c6e8108a8c0d923a8b3df82bfa885dee;hb=eea8cf830616dbbd9024b7d704cf0ccf2dac4767;hpb=2e05b1dd0490ce7ecdd15f522e6c700bc6522062

diff --git a/discover/ipmi.c b/discover/ipmi.c
index 710a47d..f59974a 100644
--- a/discover/ipmi.c
+++ b/discover/ipmi.c
@@ -107,11 +107,21 @@ int ipmi_transaction(struct ipmi *ipmi, uint8_t netfn, uint8_t cmd,
 {
 	struct timeval start, now, delta;
 	struct pollfd pollfds[1];
+	struct flock lock;
 	int expired_ms, rc;
 
+	memset(&lock, 0, sizeof(lock));
+	lock.l_type = F_WRLCK;
+	lock.l_whence = SEEK_SET;
+	rc = fcntl(ipmi->fd, F_SETLKW, &lock);
+	if (rc == -1) {
+		pb_log("IPMI: error locking IPMI device: %m\n");
+		return rc;
+	}
+
 	rc = ipmi_send(ipmi, netfn, cmd, req_buf, req_len);
 	if (rc)
-		return rc;
+		goto out;
 
 	pollfds[0].fd = ipmi->fd;
 	pollfds[0].events = POLLIN;
@@ -132,11 +142,13 @@ int ipmi_transaction(struct ipmi *ipmi, uint8_t netfn, uint8_t cmd,
 		if (rc == 0) {
 			pb_log("IPMI: timeout waiting for response "
 					"(netfn %d, cmd %d)\n", netfn, cmd);
+			rc = -1;
 			break;
 		}
 
 		if (!(pollfds[0].revents & POLLIN)) {
 			pb_log("IPMI: unexpected fd status from poll?\n");
+			rc = -1;
 			break;
 		}
 
@@ -156,17 +168,24 @@ int ipmi_transaction(struct ipmi *ipmi, uint8_t netfn, uint8_t cmd,
 				expired_ms = (delta.tv_sec * 1000) +
 						(delta.tv_usec / 1000);
 
-				if (expired_ms >= timeout_ms)
+				if (expired_ms >= timeout_ms) {
+					rc = -1;
 					break;
+				}
 			}
 		} else {
 			pb_debug("IPMI: netfn(%x->%x), cmd(%x->%x)\n",
 					netfn, resp_netfn, cmd, resp_cmd);
-			return 0;
+			rc = 0;
+			goto out;
 		}
 	}
 
-	return -1;
+out:
+	lock.l_type = F_UNLCK;
+	if (fcntl(ipmi->fd, F_SETLKW, &lock) == -1)
+		pb_log("IPMI: error unlocking IPMI device: %m\n");
+	return rc ? -1 : 0;
 }
 
 static int ipmi_destroy(void *p)