Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1448381

[PATCH v3 2/3] tools/power/acpi/acpidbg: Use new flushing mechanism

Show all headers | View raw

This patch converts tools/power/acpi/tools/acpidbg/acpidbg to use the new
flushing mechanism.

Signed-off-by: Lv Zheng <lv.zheng@intel.com>
---
 tools/power/acpi/tools/acpidbg/acpidbg.c |   51 ++++--------------------------
 1 file changed, 7 insertions(+), 44 deletions(-)

diff --git a/tools/power/acpi/tools/acpidbg/acpidbg.c b/tools/power/acpi/tools/acpidbg/acpidbg.c
index a88ac45..f5542b9 100644
--- a/tools/power/acpi/tools/acpidbg/acpidbg.c
+++ b/tools/power/acpi/tools/acpidbg/acpidbg.c
@@ -15,7 +15,9 @@
 #include <stdbool.h>
 #include <fcntl.h>
 #include <assert.h>
+#include <sys/ioctl.h>
 #include <linux/circ_buf.h>
+#include <uapi/linux/acpi-ioctls.h>
 
 #define ACPI_AML_FILE		"/sys/kernel/debug/acpi/acpidbg"
 #define ACPI_AML_SEC_TICK	1
@@ -83,7 +85,6 @@ static const char *acpi_aml_file_path = ACPI_AML_FILE;
 static unsigned long acpi_aml_mode = ACPI_AML_INTERACTIVE;
 static bool acpi_aml_exit;
 
-static bool acpi_aml_batch_drain;
 static unsigned long acpi_aml_batch_state;
 static char acpi_aml_batch_prompt;
 static char acpi_aml_batch_roll;
@@ -239,11 +240,9 @@ static int acpi_aml_write_batch_log(int fd, struct circ_buf *crc)
 
 	p = &crc->buf[crc->tail];
 	len = circ_count_to_end(crc);
-	if (!acpi_aml_batch_drain) {
-		len = write(fd, p, len);
-		if (len < 0)
-			perror("write");
-	}
+	len = write(fd, p, len);
+	if (len < 0)
+		perror("write");
 	if (len > 0)
 		crc->tail = (crc->tail + len) & (ACPI_AML_BUF_SIZE - 1);
 	return len;
@@ -270,10 +269,7 @@ static void acpi_aml_loop(int fd)
 	if (acpi_aml_mode == ACPI_AML_BATCH) {
 		acpi_aml_log_state = ACPI_AML_LOG_START;
 		acpi_aml_batch_pos = acpi_aml_batch_cmd;
-		if (acpi_aml_batch_drain)
-			acpi_aml_batch_state = ACPI_AML_BATCH_READ_LOG;
-		else
-			acpi_aml_batch_state = ACPI_AML_BATCH_WRITE_CMD;
+		acpi_aml_batch_state = ACPI_AML_BATCH_WRITE_CMD;
 	}
 	acpi_aml_exit = false;
 	while (!acpi_aml_exit) {
@@ -330,39 +326,6 @@ static void acpi_aml_loop(int fd)
 	}
 }
 
-static bool acpi_aml_readable(int fd)
-{
-	fd_set rfds;
-	struct timeval tv;
-	int ret;
-	int maxfd = 0;
-
-	tv.tv_sec = 0;
-	tv.tv_usec = ACPI_AML_USEC_PEEK;
-	FD_ZERO(&rfds);
-	maxfd = acpi_aml_set_fd(fd, maxfd, &rfds);
-	ret = select(maxfd+1, &rfds, NULL, NULL, &tv);
-	if (ret < 0)
-		perror("select");
-	if (ret > 0 && FD_ISSET(fd, &rfds))
-		return true;
-	return false;
-}
-
-/*
- * This is a userspace IO flush implementation, replying on the prompt
- * characters and can be turned into a flush() call after kernel implements
- * .flush() filesystem operation.
- */
-static void acpi_aml_flush(int fd)
-{
-	while (acpi_aml_readable(fd)) {
-		acpi_aml_batch_drain = true;
-		acpi_aml_loop(fd);
-		acpi_aml_batch_drain = false;
-	}
-}
-
 void usage(FILE *file, char *progname)
 {
 	fprintf(file, "usage: %s [-b cmd] [-f file] [-h]\n", progname);
@@ -426,7 +389,7 @@ int main(int argc, char **argv)
 	acpi_aml_set_fl(STDOUT_FILENO, O_NONBLOCK);
 
 	if (acpi_aml_mode == ACPI_AML_BATCH)
-		acpi_aml_flush(fd);
+		ioctl(fd, ACPI_IOCTL_DEBUGGER_FLUSH);
 	acpi_aml_loop(fd);
 
 exit:
-- 
1.7.10

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 0/3] ACPI / debugger: Add kernel flushing support Lv Zheng <lv.zheng@intel.com> - 2016-07-14 05:00 +0200
  [PATCH 1/3] debugfs: Add .fsync() callback proxy support Lv Zheng <lv.zheng@intel.com> - 2016-07-14 05:00 +0200
    RE: [PATCH 1/3] debugfs: Add .fsync() callback proxy support "Zheng, Lv" <lv.zheng@intel.com> - 2016-07-19 10:20 +0200
  [PATCH v2 2/2] tools/power/acpi/acpidbg: Use new flushing mechanism Lv Zheng <lv.zheng@intel.com> - 2016-07-19 12:10 +0200
  [PATCH v2 0/2] ACPI / debugger: Add kernel flushing support Lv Zheng <lv.zheng@intel.com> - 2016-07-19 12:10 +0200
    [PATCH v2 1/2] ACPI / debugger: Add kernel flushing support Lv Zheng <lv.zheng@intel.com> - 2016-07-19 12:10 +0200
      Re: [PATCH v2 1/2] ACPI / debugger: Add kernel flushing support "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-21 15:40 +0200
        RE: [PATCH v2 1/2] ACPI / debugger: Add kernel flushing support "Zheng, Lv" <lv.zheng@intel.com> - 2016-07-22 02:40 +0200
  [PATCH] tools/power/acpi/tools/acpidbg: Add multi-commands support in batch mode Lv Zheng <lv.zheng@intel.com> - 2016-07-20 10:20 +0200
    Re: [PATCH] tools/power/acpi/tools/acpidbg: Add multi-commands support in batch mode "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-07-21 15:50 +0200
      RE: [PATCH] tools/power/acpi/tools/acpidbg: Add multi-commands  support in batch mode "Zheng, Lv" <lv.zheng@intel.com> - 2016-07-22 02:30 +0200
  [PATCH v3 2/3] tools/power/acpi/acpidbg: Use new flushing mechanism Lv Zheng <lv.zheng@intel.com> - 2016-07-22 06:20 +0200
  [PATCH v3 0/3] ACPI / debugger: Add kernel flushing support Lv Zheng <lv.zheng@intel.com> - 2016-07-22 06:20 +0200
    [PATCH v3 1/3] ACPI / debugger: Add kernel flushing support Lv Zheng <lv.zheng@intel.com> - 2016-07-22 06:20 +0200
    [PATCH v3 3/3] tools/power/acpi/acpidbg: Add multi-commands support in batch mode Lv Zheng <lv.zheng@intel.com> - 2016-07-22 06:20 +0200

csiph-web