This patch introduces a new handle_console_tty function. This function
performs read/write from/to console tty.

Signed-off-by: Bhupinder Thakur <bhupinder.tha...@linaro.org>
Reviewed-by: Stefano Stabellini <sstabell...@kernel.org>
Acked-by: Wei Liu <wei.l...@citrix.com>
---
CC: Ian Jackson <ian.jack...@eu.citrix.com>
CC: Wei Liu <wei.l...@citrix.com>
CC: Stefano Stabellini <sstabell...@kernel.org>
CC: Julien Grall <julien.gr...@arm.com>

Changes since v5:
- Split this change in a separate patch.

 tools/console/daemon/io.c | 35 +++++++++++++++++++----------------
 1 file changed, 19 insertions(+), 16 deletions(-)

diff --git a/tools/console/daemon/io.c b/tools/console/daemon/io.c
index c361b42..5c6da31 100644
--- a/tools/console/daemon/io.c
+++ b/tools/console/daemon/io.c
@@ -1130,6 +1130,24 @@ static void maybe_add_console_tty_fd(struct console *con)
        }
 }
 
+static void handle_console_tty(struct console *con)
+{
+       if (con->master_fd != -1 && con->master_pollfd_idx != -1) {
+               if (fds[con->master_pollfd_idx].revents &
+                   ~(POLLIN|POLLOUT|POLLPRI))
+                       console_handle_broken_tty(con, 
domain_is_valid(con->d->domid));
+               else {
+                       if (fds[con->master_pollfd_idx].revents &
+                           POLLIN)
+                               handle_tty_read(con);
+                       if (fds[con->master_pollfd_idx].revents &
+                           POLLOUT)
+                               handle_tty_write(con);
+               }
+       }
+       con->master_pollfd_idx = -1;
+}
+
 void handle_io(void)
 {
        int ret;
@@ -1260,22 +1278,7 @@ void handle_io(void)
 
                        handle_console_ring(con);
 
-                       if (con->master_fd != -1 && con->master_pollfd_idx != 
-1) {
-                               if (fds[con->master_pollfd_idx].revents &
-                                   ~(POLLIN|POLLOUT|POLLPRI))
-                                       console_handle_broken_tty(con,
-                                                  domain_is_valid(d->domid));
-                               else {
-                                       if (fds[con->master_pollfd_idx].revents 
&
-                                           POLLIN)
-                                               handle_tty_read(con);
-                                       if (fds[con->master_pollfd_idx].revents 
&
-                                           POLLOUT)
-                                               handle_tty_write(con);
-                               }
-                       }
-
-                       con->master_pollfd_idx = -1;
+                       handle_console_tty(con);
 
                        if (d->last_seen != enum_pass)
                                shutdown_domain(d);
-- 
2.7.4


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
https://lists.xen.org/xen-devel

Reply via email to