Fixes issue SkyLandTW#5 (High system load)

Cotix · Apr 22, 2016 · 9cae7aa · 9cae7aa
1 parent 987593c
commit 9cae7aa
Showing 1 changed file with 10 additions and 10 deletions.
diff --git a/src/clevo-indicator.c b/src/clevo-indicator.c
@@ -276,6 +276,12 @@ static int main_ec_worker(void)
 {
 	setuid(0);
 	system("modprobe ec_sys");
+	FILE* io_fd = fopen("/sys/kernel/debug/ec/ec0/io", "r");
+	if (io_fd <= 0) 
+	{
+		printf("unable to read EC from sysfs: %s\n", strerror(errno));
+		exit(EXIT_FAILURE);
+	}
 	while (share_info->exit == 0)
 	{
 		// check parent
@@ -293,14 +299,9 @@ static int main_ec_worker(void)
 			share_info->manual_prev_fan_duty = new_fan_duty;
 		}
 		// read EC
-		int io_fd = open("/sys/kernel/debug/ec/ec0/io", O_RDONLY, 0);
-		if (io_fd < 0)
-		{
-			printf("unable to read EC from sysfs: %s\n", strerror(errno));
-			exit (EXIT_FAILURE);
-		}
+		rewind(io_fd);
 		unsigned char buf[EC_REG_SIZE];
-		ssize_t len = read(io_fd, buf, EC_REG_SIZE);
+		ssize_t len = fread(buf, 1, EC_REG_SIZE, io_fd);
 		switch (len)
 		{
 		case -1:
@@ -320,7 +321,6 @@ static int main_ec_worker(void)
 		default:
 			printf("wrong EC size from sysfs: %ld\n", len);
 		}
-		close(io_fd);
 		// auto EC
 		if (share_info->auto_duty == 1)
 		{
@@ -335,9 +335,9 @@ static int main_ec_worker(void)
 				share_info->auto_duty_val = next_duty;
 			}
 		}
-		//
-		usleep(500);
+		usleep(500000);
 	}
+	fclose(io_fd);
 	printf("worker quit\n");
 	return EXIT_SUCCESS;
 }