summaryrefslogtreecommitdiffstats
path: root/server
Commit message (Collapse)AuthorAgeFilesLines
* char-device: fix usage of free/unref on WriteBufferVictor Toso2015-11-131-12/+22
| | | | | | | | | | | There are places were the could should definetly free the SpiceCharDeviceWriteBuffer and places that it should only unref it. The current use of spice_char_device_write_buffer_free was missleading. This patch creates the spice_char_device_write_buffer_unref and properly call these two functions. Related: https://bugs.freedesktop.org/show_bug.cgi?id=91350
* worker: fix compiling with COMPRESS_STAT enabledFrediano Ziglio2015-11-123-22/+22
| | | | | | 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: Remove usage of streaming_video global in DisplayChannelPavel Grunt2015-11-121-6/+6
| | | | | | | | | Avoid creating streams for every image due to not properly initialized DisplayChannel->stream_video field. Regression since: 4987df8e67f48cde13d39f4e81909f672ae33e29 Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: move some tree container functionsMarc-André Lureau2015-11-123-28/+33
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: rename a bit current_add functionsMarc-André Lureau2015-11-121-10/+10
| | | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: Remove duplicate DISPLAY_CLIENT_TIMEOUT defineJonathon Jongsma2015-11-111-1/+0
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move RED_WORKER_STAT, make it compile againMarc-André Lureau2015-11-113-169/+154
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: Move drawable utility functions to display channelJonathon Jongsma2015-11-112-92/+92
| | | | | | | | Functions that check the equality of a path, brush, etc are moved the display channel source file to prepare for moving the surfaces to the display channel. Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: Move is_opaque_item() to tree.hJonathon Jongsma2015-11-112-6/+6
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move drawable to displayMarc-André Lureau2015-11-112-68/+74
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move current to displayMarc-André Lureau2015-11-102-12/+20
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move red_destroy_surface_item()Marc-André Lureau2015-11-103-42/+49
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move stream to display channelMarc-André Lureau2015-11-105-320/+323
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move DisplayChannel/DisplayChannelClient functions togetherMarc-André Lureau2015-11-101-163/+164
| | | | | | 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>
* 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>