[Qemu-commits] [qemu/qemu] 7c2876: qio: rename qio_task_thread_result

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 7c28768fef4bbd7601b26c4efca4ced9c0651356
      
https://github.com/qemu/qemu/commit/7c28768fef4bbd7601b26c4efca4ced9c0651356
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M io/task.c

  Log Message:
  -----------
  qio: rename qio_task_thread_result

It is strange that it was called gio_task_thread_result.  Rename it to
follow the naming rule of the file.

Reviewed-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: 315409c7115b9f6b1b027899c214f4a1f706a7c5
      
https://github.com/qemu/qemu/commit/315409c7115b9f6b1b027899c214f4a1f706a7c5
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M include/io/channel.h
    M io/channel.c

  Log Message:
  -----------
  qio: introduce qio_channel_add_watch_{full|source}

Firstly, introduce an internal qio_channel_add_watch_full(), which
enhances qio_channel_add_watch() that context can be specified.

Then add a new API wrapper qio_channel_add_watch_source() to return a
GSource pointer rather than a tag ID.

Note that the _source() call will keep a reference of GSource so that
callers need to unref them explicitly when finished using the GSource.

Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: 938c8b79e5249d41469151e30268cfaf88bdb588
      
https://github.com/qemu/qemu/commit/938c8b79e5249d41469151e30268cfaf88bdb588
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M include/io/net-listener.h
    M io/net-listener.c

  Log Message:
  -----------
  qio: store gsources for net listeners

Originally we were storing the GSources tag IDs.  That'll be not enough
if we are going to support non-default gcontext for QIO code.  Switch to
GSources without changing anything real.  Now we still always pass in
NULL, which means the default gcontext.

Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: a17536c594bfed94d05667b419f747b692f5fc7f
      
https://github.com/qemu/qemu/commit/a17536c594bfed94d05667b419f747b692f5fc7f
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M include/io/task.h
    M io/channel-socket.c
    M io/dns-resolver.c
    M io/task.c
    M tests/test-io-task.c

  Log Message:
  -----------
  qio: non-default context for threaded qtask

qio_task_run_in_thread() allows main thread to run blocking operations
in the background. However it has an assumption on that it's always
working with the default context. This patch tries to allow the threaded
QIO task framework to run with non-default gcontext.

Currently no functional change so far, so the QIOTasks are still always
running on main context.

Reviewed-by: Daniel P. Berrange <address@hidden>
Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: 8005fdd8fa1c754492776f6da14893932faa057a
      
https://github.com/qemu/qemu/commit/8005fdd8fa1c754492776f6da14893932faa057a
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M chardev/char-socket.c
    M include/io/channel-socket.h
    M io/channel-socket.c
    M migration/socket.c
    M tests/test-io-channel-socket.c

  Log Message:
  -----------
  qio: non-default context for async conn

We have worked on qio_task_run_in_thread() already.  Further, let
all the qio channel APIs use that context.

Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: 1939ccdaa61ce6a1f57d83277b3d41d3a9ad3c58
      
https://github.com/qemu/qemu/commit/1939ccdaa61ce6a1f57d83277b3d41d3a9ad3c58
  Author: Peter Xu <address@hidden>
  Date:   2018-03-06 (Tue, 06 Mar 2018)

  Changed paths:
    M chardev/char-socket.c
    M include/io/channel-tls.h
    M io/channel-tls.c
    M migration/tls.c
    M nbd/client.c
    M nbd/server.c
    M tests/test-io-channel-tls.c
    M ui/vnc-auth-vencrypt.c
    M ui/vnc-ws.c

  Log Message:
  -----------
  qio: non-default context for TLS handshake

A new parameter "context" is added to qio_channel_tls_handshake() is to
allow the TLS to be run on a non-default context.  Still, no functional
change.

Signed-off-by: Peter Xu <address@hidden>
Signed-off-by: Daniel P. Berrangé <address@hidden>


  Commit: 3ef91576b96de7051dacc2132cddfb486b46e863
      
https://github.com/qemu/qemu/commit/3ef91576b96de7051dacc2132cddfb486b46e863
  Author: Peter Maydell <address@hidden>
  Date:   2018-03-08 (Thu, 08 Mar 2018)

  Changed paths:
    M chardev/char-socket.c
    M include/io/channel-socket.h
    M include/io/channel-tls.h
    M include/io/channel.h
    M include/io/net-listener.h
    M include/io/task.h
    M io/channel-socket.c
    M io/channel-tls.c
    M io/channel.c
    M io/dns-resolver.c
    M io/net-listener.c
    M io/task.c
    M migration/socket.c
    M migration/tls.c
    M nbd/client.c
    M nbd/server.c
    M tests/test-io-channel-socket.c
    M tests/test-io-channel-tls.c
    M tests/test-io-task.c
    M ui/vnc-auth-vencrypt.c
    M ui/vnc-ws.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/berrange/tags/qio-next-pull-request' 
into staging

# gpg: Signature made Wed 07 Mar 2018 11:24:41 GMT
# gpg:                using RSA key BE86EBB415104FDF
# gpg: Good signature from "Daniel P. Berrange <address@hidden>"
# gpg:                 aka "Daniel P. Berrange <address@hidden>"
# Primary key fingerprint: DAF3 A6FD B26B 6291 2D0E  8E3F BE86 EBB4 1510 4FDF

* remotes/berrange/tags/qio-next-pull-request:
  qio: non-default context for TLS handshake
  qio: non-default context for async conn
  qio: non-default context for threaded qtask
  qio: store gsources for net listeners
  qio: introduce qio_channel_add_watch_{full|source}
  qio: rename qio_task_thread_result

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/854a4436dd31...3ef91576b96d