On 03/07/2011 09:10 PM, Michael Roth wrote:
+
+/* XXX: fd_read_poll should be suppressed, but an API change is
+   necessary in the character devices to suppress fd_can_read(). */
+int qemu_set_fd_handler3(void *ioh_record_list,
+                         int fd,
+                         IOCanReadHandler *fd_read_poll,
+                         IOHandler *fd_read,
+                         IOHandler *fd_write,
+                         void *opaque)

What's the reason to introduce this additional indirection (and with a void rather than opaque pointer)? A global iohandlers list would be fine in qemu-ioh.c (and it would be a worthwhile patch anyway for QEMU).

Paolo

Reply via email to