[Qemu-devel] [PATCH 4/9] Introduce qemu_set_fd_write_handler()

[Juan Quintela]

Patch consist:

    s/qemu_set_fd_handler(fd, NULL, write, o)/
      qemu_set_fd_write_handler(fd, write, o)/

    s/qemu_set_fd_handler2(fd, NULL, NULL, write, o)/
      qemu_set_fd_write_handler(fd, write, o)/

Signed-off-by: Juan Quintela <address@hidden>
---
 audio/alsaaudio.c   |    4 ++--
 audio/ossaudio.c    |    2 +-
 migration-tcp.c     |    2 +-
 migration-unix.c    |    2 +-
 migration.c         |    2 +-
 net/socket.c        |    2 +-
 qemu-char.h         |    4 +++-
 usb-linux.c         |    2 +-
 vl.c                |   11 +++++++++++
 vnc-auth-vencrypt.c |    2 +-
 10 files changed, 23 insertions(+), 10 deletions(-)

diff --git a/audio/alsaaudio.c b/audio/alsaaudio.c
index 2455ee4..894f02b 100644
--- a/audio/alsaaudio.c
+++ b/audio/alsaaudio.c
@@ -273,8 +273,8 @@ static int alsa_poll_helper (snd_pcm_t *handle, struct 
pollhlp *hlp, int mask)
             if (conf.verbose) {
                 dolog ("POLLOUT %d %d\n", i, pfds[i].fd);
             }
-            err = qemu_set_fd_handler (pfds[i].fd, NULL,
-                                       alsa_poll_handler, hlp);
+            err = qemu_set_fd_write_handler (pfds[i].fd, alsa_poll_handler,
+                                             hlp);
         }
         if (conf.verbose) {
             dolog ("Set handler events=%#x index=%d fd=%d err=%d\n",
diff --git a/audio/ossaudio.c b/audio/ossaudio.c
index 4c91a46..4d0cd36 100644
--- a/audio/ossaudio.c
+++ b/audio/ossaudio.c
@@ -146,7 +146,7 @@ static int oss_poll_out (HWVoiceOut *hw)
 {
     OSSVoiceOut *oss = (OSSVoiceOut *) hw;

-    return qemu_set_fd_handler (oss->fd, NULL, oss_helper_poll_out, NULL);
+    return qemu_set_fd_write_handler (oss->fd, oss_helper_poll_out, NULL);
 }

 static int oss_poll_in (HWVoiceIn *hw)
diff --git a/migration-tcp.c b/migration-tcp.c
index 66f126f..c141322 100644
--- a/migration-tcp.c
+++ b/migration-tcp.c
@@ -123,7 +123,7 @@ MigrationState *tcp_start_outgoing_migration(Monitor *mon,
             ret = -(s->get_error(s));

         if (ret == -EINPROGRESS || ret == -EWOULDBLOCK)
-            qemu_set_fd_handler2(s->fd, NULL, NULL, tcp_wait_for_connect, s);
+            qemu_set_fd_write_handler(s->fd, tcp_wait_for_connect, s);
     } while (ret == -EINTR);

     if (ret < 0 && ret != -EINPROGRESS && ret != -EWOULDBLOCK) {
diff --git a/migration-unix.c b/migration-unix.c
index fe9a820..b50ccb2 100644
--- a/migration-unix.c
+++ b/migration-unix.c
@@ -118,7 +118,7 @@ MigrationState *unix_start_outgoing_migration(Monitor *mon,
            ret = -(s->get_error(s));

         if (ret == -EINPROGRESS || ret == -EWOULDBLOCK)
-           qemu_set_fd_handler2(s->fd, NULL, NULL, unix_wait_for_connect, s);
+           qemu_set_fd_write_handler(s->fd, unix_wait_for_connect, s);
     } while (ret == -EINTR);

     if (ret < 0 && ret != -EINPROGRESS && ret != -EWOULDBLOCK) {
diff --git a/migration.c b/migration.c
index 787147a..c310e4b 100644
--- a/migration.c
+++ b/migration.c
@@ -332,7 +332,7 @@ ssize_t migrate_fd_put_buffer(void *opaque, const void 
*data, size_t size)
         ret = -(s->get_error(s));

     if (ret == -EAGAIN)
-        qemu_set_fd_handler2(s->fd, NULL, NULL, migrate_fd_put_notify, s);
+        qemu_set_fd_write_handler(s->fd, migrate_fd_put_notify, s);

     return ret;
 }
diff --git a/net/socket.c b/net/socket.c
index 5772202..e746b19 100644
--- a/net/socket.c
+++ b/net/socket.c
@@ -316,7 +316,7 @@ static NetSocketState *net_socket_fd_init_stream(VLANState 
*vlan,
     if (is_connected) {
         net_socket_connect(s);
     } else {
-        qemu_set_fd_handler(s->fd, NULL, net_socket_connect, s);
+        qemu_set_fd_write_handler(s->fd, net_socket_connect, s);
     }
     return s;
 }
diff --git a/qemu-char.h b/qemu-char.h
index 70e4bf9..76cacd9 100644
--- a/qemu-char.h
+++ b/qemu-char.h
@@ -110,6 +110,8 @@ int qemu_set_fd_handler(int fd,
 int qemu_set_fd_read_handler(int fd,
                              IOHandler *fd_read,
                              void *opaque);
-
+int qemu_set_fd_write_handler(int fd,
+                              IOHandler *fd_read,
+                              void *opaque);

 #endif
diff --git a/usb-linux.c b/usb-linux.c
index b7ac3e9..9c871ce 100644
--- a/usb-linux.c
+++ b/usb-linux.c
@@ -940,7 +940,7 @@ static int usb_host_open(USBHostDevice *dev, int bus_num,
                 prod_name);

     /* USB devio uses 'write' flag to check for async completions */
-    qemu_set_fd_handler(dev->fd, NULL, async_complete, dev);
+    qemu_set_fd_write_handler(dev->fd, async_complete, dev);

     usb_device_attach(&dev->dev);
     return 0;
diff --git a/vl.c b/vl.c
index c855167..6e75347 100644
--- a/vl.c
+++ b/vl.c
@@ -2665,6 +2665,17 @@ int qemu_set_fd_read_handler(int fd,
     return qemu_set_fd_handler2(fd, NULL, fd_read, NULL, opaque);
 }

+int qemu_set_fd_write_handler(int fd,
+                              IOHandler *fd_write,
+                              void *opaque)
+{
+    if(fd_write == NULL) {
+        fprintf(stderr, "qemu_set_fd_write_handler: NULL write handler\n");
+        exit(1);
+    }
+    return qemu_set_fd_handler2(fd, NULL, NULL, fd_write, opaque);
+}
+
 #ifdef _WIN32
 /***********************************************************/
 /* Polling handling */
diff --git a/vnc-auth-vencrypt.c b/vnc-auth-vencrypt.c
index 2999c2d..7b3fd57 100644
--- a/vnc-auth-vencrypt.c
+++ b/vnc-auth-vencrypt.c
@@ -73,7 +73,7 @@ static int vnc_start_vencrypt_handshake(struct VncState *vs) {
            if (!gnutls_record_get_direction(vs->tls.session))
                qemu_set_fd_read_handler(vs->csock, vnc_tls_handshake_io, vs);
            else
-               qemu_set_fd_handler(vs->csock, NULL, vnc_tls_handshake_io, vs);
+               qemu_set_fd_write_handler(vs->csock, vnc_tls_handshake_io, vs);
            return 0;
        }
        VNC_DEBUG("Handshake failed %s\n", gnutls_strerror(ret));
-- 
1.6.6.1