summaryrefslogtreecommitdiffstats
path: root/server
Commit message (Collapse)AuthorAgeFilesLines
* worker: rename functions that handle DisplayChannelClientMarc-André Lureau2015-11-101-18/+18
| | | | | | | Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move stream functions to a new stream.c fileMarc-André Lureau2015-11-104-50/+78
| | | | | | | Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move stream definitions to a new stream.h fileMarc-André Lureau2015-11-104-93/+125
| | | | | | Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: fix RED_STREAM_TIMOUT macro spellingMarc-André Lureau2015-11-101-3/+3
| | | | | Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* remove server/stream.h file added by mistake in previous commitFrediano Ziglio2015-11-101-139/+0
| | | | | Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: move image cache to displayMarc-André Lureau2015-11-105-91/+247
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker s/surfaces_dest/surface_depsMarc-André Lureau2015-11-104-23/+23
| | | | | Acked-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* syntax-check: Add missing #include <config.h>Pavel Grunt2015-11-095-0/+19
|
* red_dispatcher: Remove unused signal.h includePavel Grunt2015-11-091-1/+0
|
* worker: group add_memslotMarc-André Lureau2015-11-091-18/+18
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: remove unused NUM_CURSORS defineMarc-André Lureau2015-11-091-1/+0
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: make more functions staticMarc-André Lureau2015-11-091-32/+32
| | | | Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: remove unused functionMarc-André Lureau2015-11-091-7/+0
| | | | Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: rename {put,ref}_red_drawableMarc-André Lureau2015-11-091-9/+9
| | | | Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: s/destroy_surface/surface_unrefMarc-André Lureau2015-11-091-26/+28
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: remove unused parameter from remove_shadowFrediano Ziglio2015-11-091-2/+2
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: remove some useless countersMarc-André Lureau2015-11-091-13/+0
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* dispatcher: remove receive_data and send_data functionsMarc-André Lureau2015-11-092-35/+3
| | | | | | Use read_safe/write_safe instead which do the same stuff Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* server: create display and cursor channels in RedWorker constructorMarc-André Lureau2015-11-094-100/+48
| | | | | | | | Instead of requiring the dispatcher to send a message to the worker to create the display channel and cursor channel, just create them when the worker is created. Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: use RED_CHANNEL_CLIENT for dccMarc-André Lureau2015-11-061-52/+53
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* spicevmc: Return early when instance has no device statePavel Grunt2015-11-061-0/+5
| | | | | | | | | | | | | | | | | | | | | | Same approach as in spice_server_char_device_wakeup(). Avoid segmentation fault when the webdav channel (spice port channel) is used with the vnc display: #0 0x00007ffff7aab734 in spice_char_device_state_opaque_get (dev=0x0) at char_device.c:720 #1 0x00007ffff7b0850c in spice_server_port_event (sin=<optimized out>, event=<optimized out>) at spicevmc.c:578 #2 0x0000555555787ba4 in set_guest_connected (port=<optimized out>, guest_connected=1) at hw/char/virtio-console.c:89 #3 0x0000555555678d7c in control_out (len=<optimized out>, buf=0x55555775c3a0, vser=0x5555578d1540) at /home/pgrunt/RH/qemu/hw/char/virtio-serial-bus.c:404 #4 0x0000555555678d7c in control_out (vdev=0x5555578d1540, vq=0x555557941bc8) at /home/pgrunt/RH/qemu/hw/char/virtio-serial-bus.c:441 #5 0x000055555588eb98 in aio_dispatch (ctx=0x5555562e1a50) at aio-posix.c:160 #6 0x00005555558829ee in aio_ctx_dispatch (source=<optimized out>, callback=<optimized out>, user_data=<optimized out>) at async.c:226 #7 0x00007ffff2010e3a in g_main_context_dispatch () at /lib64/libglib-2.0.so.0 #8 0x000055555588d8fb in main_loop_wait () at main-loop.c:211 #9 0x000055555588d8fb in main_loop_wait (timeout=<optimized out>) at main-loop.c:256 #10 0x000055555588d8fb in main_loop_wait (nonblocking=<optimized out>) at main-loop.c:504 #11 0x000055555561b664 in main () at vl.c:1891
* worker: move shadow_new() and container_new()Marc-André Lureau2015-11-063-42/+56
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: use more DCC_TO_WORKERMarc-André Lureau2015-11-061-21/+21
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move some compress stats to displayMarc-André Lureau2015-11-063-104/+107
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: rename WORKER_FOREACH_DCC_SAFE macro to FOREACH_DCCMarc-André Lureau2015-11-061-20/+18
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: remove unused WORKER_TO_DCC macroMarc-André Lureau2015-11-061-6/+0
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: rename red_pipe_add_drawable* to dcc_add_drawable*Marc-André Lureau2015-11-061-7/+7
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move delta computationMarc-André Lureau2015-11-061-8/+9
| | | | | | Move delta computation from red_add_drawable to red_current_add_with_shadow. Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move MonitorsConfig to display channelMarc-André Lureau2015-11-053-41/+41
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: minor simplificationMarc-André Lureau2015-11-052-19/+24
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: don't process drawable if it can't be allocatedFrediano Ziglio2015-11-051-4/+6
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move DisplayChannel structMarc-André Lureau2015-11-052-32/+32
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* style: prefer early return to large if blocksJonathon Jongsma2015-11-041-22/+24
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Remove unnecessary typedef from pixmap-cache.hJonathon Jongsma2015-11-041-2/+0
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: Move some dpi code to display channelMarc-André Lureau2015-11-043-118/+123
| | | | | | | | This commit moves some DrawablePipeItem declarations to display-channel.h, though the function implementations remain in red_worker.c until they can be disentangled some more. Acked-by: Frediano Ziglio <fziglio@redhat.com>
* cursor: fix wrong logic when initializing the channelFabiano Fidêncio2015-11-041-1/+1
| | | | | | | | | | | | | | | | It's a regression introduced by commit e601e920bd5. The logic error was introduced when trying to achieve the following code[0]. but rewritten to prefer an early return, which was wrogly negated. [0]: if (cursor_is_connected(worker) && !COMMON_CHANNEL(worker->cursor_channel)->during_target_migrate) { red_channel_pipes_add_type(RED_CHANNEL(worker->cursor_channel), PIPE_ITEM_TYPE_CURSOR_INIT); } Signed-off-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Palette cache: Use correct marshal functionJonathon Jongsma2015-11-041-6/+8
| | | | | | | | | | | In order to invalidate a single palette cache item, we were using spice_marshall_msg_cursor_inval_one(), which is the marshal function used to send an invalidation message for the Cursor channel's cache. This didn't cause any problems because SPICE_MSG_CURSOR_INVAL_ONE and SPICE_MSG_DISPLAY_INVAL_PALETTE have the same message ID and parameters, but it's better to use the correct marshalling function. Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Various changes in RedWorker and CursorChannel related to error and warning ↵Marc-André Lureau2015-11-043-19/+27
| | | | | | | messages. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Move monitors_config to displayMarc-André Lureau2015-11-043-96/+96
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* s/process_drawable/process_drawMarc-André Lureau2015-11-041-4/+4
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: move renderer members to DisplayChannelMarc-André Lureau2015-11-031-16/+15
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: remove only assigned set_client_capabilities_pending fieldFrediano Ziglio2015-11-031-3/+0
| | | | | | | This field is only assigned and never readed so can be removed. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: move bitmap related to spice-bitmap-utilsMarc-André Lureau2015-11-037-167/+226
| | | | | | | Also remove some unused function parameters from bitmap_get_graduality_level() Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: remove unused worker parameterMarc-André Lureau2015-11-031-7/+7
| | | | | Acked-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Store QXLInstance in CursorItemMarc-André Lureau2015-11-031-31/+39
| | | | | | | | | | Doing so allows us to remove the extra QXLInstance parameter from cursor_item_unref() and makes the code a bit cleaner. Also add cursor_item_ref(). Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: make more of cursor privateMarc-André Lureau2015-11-035-64/+89
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: make cursor channel privateMarc-André Lureau2015-11-036-70/+94
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* Move pipe item enumerations out of red_worker.hMarc-André Lureau2015-11-023-18/+26
| | | | | | | | | | Move the cursor-specific pipe item types to cursor-channel.h, and the display-specific types to red_worker.c. Only leave the common definitions in red_worker.h. This prepares for splitting the display channel into a separate file. Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: move display_channel_client_create() to dcc_new()Marc-André Lureau2015-11-024-65/+83
| | | | | | | Move function from server/red_worker.c to new server/display-channel.c. Acked-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* tree: move that to a separate unitMarc-André Lureau2015-11-025-255/+311
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>