summaryrefslogtreecommitdiffstats
path: root/server
Commit message (Collapse)AuthorAgeFilesLines
* red channel: Remove unused red_channel_push_set_ackPavel Grunt2015-11-232-6/+0
| | | | | | | It is not needed since 8e7b22b7861d2859167767822f29749bfe3ecc16 Signed-off-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* red dispatcher: Remove unused declaration of red_dispatcher_set_mm_timePavel Grunt2015-11-231-1/+0
| | | | | | | It is not used since c541d7e29dc0053e6434d053a50454bb8a7121e9 Signed-off-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* dispatcher: Remove unused dispatcher_read_messagePavel Grunt2015-11-232-14/+0
| | | | | | | It is not needed since d3153f861065235c2a7a7120375be5db948c5a77 Signed-off-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move red_drawable_count to DisplayChannelFrediano Ziglio2015-11-232-7/+7
| | | | | Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com>
* reuse red_get_monotonic_time functionFrediano Ziglio2015-11-233-24/+8
| | | | | Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com>
* server: remove OpenGLMarc-André Lureau2015-11-218-143/+0
| | | | | | | | | | This is really not supported, requires X11, so better to remove it for now. Some day it might be revived, using DRM, .. Note for later, this could be removed too (not used by client): - spice-common/common/ogl_ctx Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* stream: remove red_stream_ prefix from mjpeg callbacksMarc-André Lureau2015-11-211-6/+6
| | | | | Author: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* stream: rename {red_stream_, }get_initial_bit_rateMarc-André Lureau2015-11-211-3/+2
| | | | | Author: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* stream: remove dcc parameter from stream_agent_stop()Marc-André Lureau2015-11-213-4/+5
| | | | | Author: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: move more free_glz_drawableMarc-André Lureau2015-11-205-76/+78
| | | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: Free encoders when client disconnectsPavel Grunt2015-11-203-0/+13
| | | | Signed-off-by: Pavel Grunt <pgrunt@redhat.com>
* dcc: Fix unused display_channel variablePavel Grunt2015-11-201-3/+2
| | | | | | | | | | Avoid problem when compiling with lz4 and without COMPRESS_STAT dcc.c: In function 'dcc_compress_image_lz4': dcc.c:657:21: error: unused variable 'display_channel' [-Werror=unused-variable] DisplayChannel *display_channel = DCC_TO_DC(dcc); Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: move stream_clip_item_unref()Marc-André Lureau2015-11-203-14/+18
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: merge red_draw_qxl_drawable in drawable_drawMarc-André Lureau2015-11-201-13/+9
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move dcc_pixmap_cache_addMarc-André Lureau2015-11-203-91/+92
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move dcc_freeze_glzMarc-André Lureau2015-11-203-11/+11
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: few function renamesMarc-André Lureau2015-11-201-9/+12
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: add display_channel_free_glz_drawables_to_free()Marc-André Lureau2015-11-205-26/+33
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move dcc_free_glz_drawable_instanceMarc-André Lureau2015-11-203-48/+52
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move display_channel_flush_all_surfacesMarc-André Lureau2015-11-203-19/+15
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move compress to dcc_compress_image()Frediano Ziglio2015-11-205-669/+681
| | | | | | 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: move glz_drawable_count to DisplayChannelFrediano Ziglio2015-11-202-7/+8
| | | | | Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move display_channel_wait_for_migrate_dataMarc-André Lureau2015-11-193-31/+32
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move dcc_start()Marc-André Lureau2015-11-194-223/+224
| | | | | Author: Marc-André Lureau <marcandre.lureau@gmail.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: start a DisplayChannelClient unitMarc-André Lureau2015-11-199-238/+312
| | | | | | 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: simplify GlzDrawableInstanceItem fields nameMarc-André Lureau2015-11-193-9/+9
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: move encoders to dcc-encodersMarc-André Lureau2015-11-196-591/+620
| | | | | | Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: painfully move display_channel_add_drawableMarc-André Lureau2015-11-184-546/+570
| | | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: add compression parameters to dccMarc-André Lureau2015-11-183-18/+31
| | | | | | | | | | | | | This allow different dcc to have different settings from default one. The parameters are copied initially from default settings but then they can change independently for each client. Even having a single client a future client is not affected by a previous setting on the old dcc. Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> [updated for the preferred compression] Signed-off-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* Rename red_display_free_* to dcc_free_*Jonathon Jongsma2015-11-181-11/+11
| | | | Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: simplify RedCompressBufMarc-André Lureau2015-11-182-89/+30
| | | | | | | | Make sure an allocated buffer is correctly referenced by the marshaller, and can't be free and reused by mistake. Simplify the code by using GSlice Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: move surfaces to DisplayChannelMarc-André Lureau2015-11-183-617/+576
| | | | | | | | Ok. this one was painful.Note that in some cases, DCC_TO_DC should be made safer (there used to be a if !dcc guard in some places, although that looks wrong anyway)... Acked-by: Pavel Grunt <pgrunt@redhat.com>
* Move stream creation and maintenance functions to stream.[ch]Jonathon Jongsma2015-11-184-370/+378
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: move stream_agent_stop and friendsMarc-André Lureau2015-11-184-177/+180
| | | | Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* worker: Remove unused red_show_treePavel Grunt2015-11-181-18/+0
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* display-channel: Keep macro definition in one lineMarc-André Lureau2015-11-181-2/+2
| | | | Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: simplify red_marshall_imageFrediano Ziglio2015-11-181-19/+15
| | | | | | | Remove some nested blocks using else if. Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com>
* worker: Move stream functions to stream.cJonathon Jongsma2015-11-184-100/+104
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* Move some tree item functions to tree.[ch]Jonathon Jongsma2015-11-173-55/+61
| | | | | | | | | | | Also rename some functions slightly: __find_shadow -> tree_item_find_shadow() __contained_by -> tree_item_contained_by() ring_of -> tree_item_container_items(); Acked-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Jonathon Jongsma <jjongsma@redhat.com>
* Move some more drawable functions to display channelJonathon Jongsma2015-11-162-28/+28
| | | | Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* worker: Rename current_clear() to current_remove_all()Jonathon Jongsma2015-11-161-8/+8
| | | | Acked-by: Pavel Grunt <pgrunt@redhat.com>
* Move dcc_push_stream_agent_clip() to display channelJonathon Jongsma2015-11-163-19/+20
| | | | | | rename to dcc_add_stream_agent_clip() Acked-by: Pavel Grunt <pgrunt@redhat.com>
* server: Duplicate typedef definitions are not allowed in C99Francois Gouget2015-11-163-4/+2
| | | | | | | This fixes some compilation errors with gcc 4.4.7 on RHEL 6. Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: Include stdint.h for int64_tFrancois Gouget2015-11-161-0/+1
| | | | | | | This fixes a compilation error with gcc 4.4 on RHEL 6. Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* server: Remove the display_channel_attach_stream() prototypeFrancois Gouget2015-11-131-3/+0
| | | | | | | It is unused. Signed-off-by: Francois Gouget <fgouget@codeweavers.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* worker: don't use weird RedCompressedBuf nbytes shiftingMarc-André Lureau2015-11-132-20/+30
| | | | | | | Signed-off-by: Marc-André Lureau <marcandre.lureau@gmail.com> Signed-off-by: Frediano Ziglio <fziglio@redhat.com> Acked-by: Pavel Grunt <pgrunt@redhat.com> Acked-by: Fabiano Fidêncio <fidencio@redhat.com>
* spicevmc: set state of DeviceInstance to NULLVictor Toso2015-11-131-0/+1
| | | | | | | After spice_char_device_state_destroy is called spicevmc should not keep reference to that memory. state->chardev_st and sin->st point to the same SpiceCharDeviceState and both should be set to NULL when it is destroyed.
* char-device: set to NULL freed pointers on destroyVictor Toso2015-11-131-0/+2
| | | | | | | | As SpiceCharDeviceState is only unref'ed on spice_char_device_state_destroy the same device could be destroyed more then once so the pointers that are freed should be set to NULL. Related: https://bugzilla.redhat.com/show_bug.cgi?id=1281455
* reds: Do not abort due to wrong headerPavel Grunt2015-11-131-1/+4
| | | | | | | | | | | | | | | | | | | | | Just prevent the buggy client from connecting. #0 0x00007fffe83b2a98 in raise () at /lib64/libc.so.6 #1 0x00007fffe83b469a in abort () at /lib64/libc.so.6 #2 0x00007ffff7b1533d in spice_logv (log_domain=0x7ffff7b87226 "Spice", log_level=SPICE_LOG_LEVEL_ERROR, strloc=0x7ffff7b92aba "reds.c:1373", function=0x7ffff7b94f40 <__FUNCTION__.31775> "reds_send_link_ack", format=0x7ffff7b871fe "assertion `%s' failed", args=args@entry=0x7fffffffcb68) at log.c:109 #3 0x00007ffff7b15468 in spice_log (log_domain=log_domain@entry=0x7ffff7b87226 "Spice", log_level=log_level@entry=SPICE_LOG_LEVEL_ERROR, strloc=strloc@entry=0x7ffff7b92aba "reds.c:1373", function=function@entry=0x7ffff7b94f40 <__FUNCTION__.31775> "reds_send_link_ack", format=format@entry=0x7ffff7b871fe "assertion `%s' failed") at log.c:123 #4 0x00007ffff7aee335 in reds_handle_read_link_done (link=0x555556b27c70) at reds.c:1373 #5 0x00007ffff7aee335 in reds_handle_read_link_done (opaque=0x555556b27c70) at reds.c:2139 #6 0x000055555588acc6 in qemu_iohandler_poll () #7 0x000055555588a8e1 in main_loop_wait () #8 0x0000555555614064 in main () Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1281442 Acked-by: Fabiano Fidêncio <fidencio@redhat.com> Acked-by: Frediano Ziglio <fziglio@redhat.com>
* char-device: free all memory pool when no clientsVictor Toso2015-11-131-0/+6
| | | | | | | | When no client is connect we should not need to keep the memory pool used by char-device. In most situations this is not significant but when using webdav this could mean freeing MAX_POOL_SIZE bytes Related: https://bugs.freedesktop.org/show_bug.cgi?id=91350
generated by cgit (git 2.34.1) at 2024-06-09 01:19:48 +0000