summaryrefslogtreecommitdiff
path: root/src/lib/evas/cserve2/evas_cs2_client.c (follow)
AgeCommit message (Collapse)Author
2013-12-26efl: Unified eina critical manro to CRI.Daniel Juyung Seo
Being annoyed by different types of eina critical macros - CRI, CRIT, CRITICAL -, I concluded to unify them to one. Discussed on IRC and finally, CRI was chosen to meet the consistency with other macros - ERR, WRN, INF, DBG - in terms of the number of characters. If there is any missing bits, please let me know.
2013-12-09Evas/cserve2: Fix crash during shutdownJean-Philippe Andre
If an image failed to load, and cserve2 returned an error message, then the File_Entry was freed, but not removed from the hash. Solution: remove entry from the hash, let the callback free the data.
2013-11-29evas/cserve2: Fix interrupt on select in edje_ccJean-Philippe Andre
Summary: Block SIGCHLD during select(). This fixes a bug with edje_cc when EVAS_CSERVE2=1: Fixes T464. select() used to return prematurately with EINTR because the app received some unexpected signals. In particular SIGCHLD is received when a child terminates, but this is not a reason to cancel the image load. In theory, all blocked signals in pselect() should be pending until pselect returns, so any SIGCHLD should still trigger the app's signal handler. Reviewers: cedric CC: raster, cedric Maniphest Tasks: T464 Differential Revision: https://phab.enlightenment.org/D357
2013-10-29evas/cserve2: Fix CID 1039286 (Resource leak)Jean-Philippe Andre
Close socket in case of connection failure.
2013-10-29evas/cserve2: Fix new Coverity issuesJean-Philippe Andre
Nothing extraordinary here. Most potential crashes are extremely unlikely. - Fix CID 1113444 - Fix CID 1113442 - Fix CID 1113441 (Logically dead code, can not be NULL) - Fix CID 1113440: Explicit null dereferenced This is actually an impossible situation. Fixed by checking for nullity and printing out some error messages instead of just crashing. - Fix CID 1113439: Dereference after null check Logically impossible code as both idxpath and datapath must be either set or null at the same time. Change the if logic to tell Coverity there's no bug. - Fix CID 1113438 (Argument cannot be negative) Fix wrong check of return value from shm_open. - Fix CID 1113437 (Argument cannot be negative) Fix wrong check of return value from shm_open. - Fix CID 1113436 (Dereference null return value) This case really shouldn't happen. But the extra check does not hurt. - Fix CID 1113435 (Dereference before null check) Check for nullity after map open. - Fix CID 1113434 (Extra sizeof expression) Debug buggy debug tool :) - Fix CID 1113433 (Uninitialized scalar variable) Insignificant issue: only prints wrong debug logs :) - Fix CID 1113431 (Uninitialized scalar value) Check if (!found) only to print out logs. Not a big deal if found was invalid. - Fix CID 1039462 (Logically dead code)
2013-10-28evas/cserve2: Merge Glyph_Data and mempool indexJean-Philippe Andre
Glyphs were previously using 3 shared buffers, now reduce to 2: - Memory pool (mempool) containing the glyph drawable data - Index table (Shared_Index / array) containing only the indexes of the buffers in the mempool - Glyph_Data table (array) containing the glyphs descriptors AS WELL as the buffer indexes. So, we just merge the two index tables into one by using directly objects of type Glyph_Data for the referencing of the mempool buffers.
2013-10-28evas/cserve2: Add debug and reduce number of GLYPHS_USED messagesJean-Philippe Andre
One socket message was sent per each glyph used ... which means a LOT of messages when text is being redrawn. Reduce this flow of messages by triggering send() only when 50+ items are being used. Btw, USED is a bit useless as there is no UNUSED equivalent. Also, slightly improve debug logs.
2013-10-28evas/cserve2: Use Eina_Refcount instead of intJean-Philippe Andre
Use the refcount macros instead of manually handling integers. Not sure if it really helps since the refcounting in cs2 client is a bit special :)
2013-10-28evas/cserve2: prevent unwanted remap of strings tableJean-Philippe Andre
In the client, string_get() can cause a remapping of the strings index & mempool. This means that all pointers to string data are invalid past that call. Solution: add a safe_get() function that prevents remap during search. It might prove faster also, but will return NULL more often.
2013-10-28evas/cserve2: Fix font reloading after cserve2 restart.Jean-Philippe Andre
Well it LOOKS like it's working properly. Clients can safely keep running after cserve2 crashed and restarted.
2013-10-28evas/cserve2: Fix refcount for glyphs & glyph buffersJean-Philippe Andre
Maybe a little overkill on the iterations (ref/unref), but at least we can really track down which glyphs are currently being used.
2013-10-28evas/cserve2: Reconnect to cserve2 in case of server crashJean-Philippe Andre
Try to reconnect to cserve2 if the socket connection was lost. Resend some messages if necessary. Images reload seems to be working. Actually, the images don't change over time, so the clients just keep the previous references to their images. FONT RELOAD IS NOT WORKING: - Crashes - Invalid glyph data - Infinite loop in _glyph_map_remap_check() Root cause: When new glyphs are requested from the server, they are added to the mempool. So it is necessary to remap the font. Unfortunately, in case of server reboot, we did not keep the mempool so the old glyphs that were not requested again will not be valid.
2013-10-28evas/cserve2: Fix CList usageJean-Philippe Andre
In some error cases, the list could become a cycle. Foreach would then loop forever!
2013-10-28evas/cserve2: Fix invalid file referencing in clientJean-Philippe Andre
Fixes elementary_test "Bg Image": - Various load_opts (jpeg geometry) where not handled properly by the client.
2013-10-28evas/cserve2: Fix images indexing (File_Entry stuff)Jean-Philippe Andre
High-level problem: cserve2 does not support load_opts properly when opening an image. As a result, when (pre)loading a JPEG file with specific load options (eg. w, h, region and orientation), the image buffer might have the wrong dimensions. So, we need to use load_opts when computing file hash key. And, pass these load options to the loader slave, and use them while OPENING the image. This will set properly the geometry. Fixes test "Preload and Prescale" in elementary_test.
2013-10-28evas/cserve2: Fallback to normal cache for animated gifsJean-Philippe Andre
Pass around "animated" flag for images that can be animated. Fallback to local cache if the image is animated. Implementing support for animated images in cserve2 does not seem to make a lot of sense considering each frame must be requested independently in real time,... and to be honest there doesn't seem to be any valid use case anyway :)
2013-10-28evas/cserve2: Reuse file entries when possibleJean-Philippe Andre
For some reason, a new File_Entry was created whenever a new image is loaded, even if that file was already opened by the client.
2013-10-28evas/cserve2: Reduce debug logs (s/INF/DBG)Jean-Philippe Andre
2013-10-28evas/cserve2: Fix font hintingJean-Philippe Andre
cserve2 was not handling font hints properly in the server side Add some consistency checks and separate glyphs per hinting value
2013-10-28evas/cserve2: Reset pointers to NULL after closeJean-Philippe Andre
2013-10-28evas/cserve2: Add TIMEOUT macro for cs2 clientJean-Philippe Andre
This will help debugging as we can disable the timeout on demand.
2013-10-28evas/cserve2: Small performance fixesJean-Philippe Andre
2013-10-28evas/cserve2: Fix crashes in E17/cserve2Jean-Philippe Andre
When remapping the glyph data buffer, we need to reposition the glyph pointers as well.
2013-10-28evas/cserve2: Scan glyphs and add to fashJean-Philippe Andre
This will save (just) a few socket waits.
2013-10-28evas/cserve2: Remap glyph mempool if resizedJean-Philippe Andre
2013-10-28evas/cserve2: Use only one Glyph_Map per fontJean-Philippe Andre
In the new cs2 model, only one shared buffer is allocated for a font, and it resizes on demand.
2013-10-28evas/cserve2: Remap indexes when count is wrongJean-Philippe Andre
When advertised count is wrong, try to remap the index. This will avoid errors with string_get().
2013-10-28evas/cserve2: Add valid flag on Image_Data and File_DataJean-Philippe Andre
Images and Files can be discovered by the client scanner before they are valid (aka. loaded or opened). We want clients to ignore all shared objects that are not ready yet, as they are in an undefined state (values and memory might be invalid).
2013-10-28evas/cserve2: Remap string entries when size changedJean-Philippe Andre
Lots of files can't be found by the client when the strings table changed, because it was not remapped properly.
2013-10-28evas/cserve2: Optimize shared stringsJean-Philippe Andre
Shared string indexes are not repacked, since they live in a memory pool (where the indexes can be reused). So, the index in the table is equal to their ID. Add initial test to check if the item at index n has the ID n.
2013-10-28evas/cserve2: Client-side scanning of Font EntriesJean-Philippe Andre
2013-10-28evas/cserve2: Store Glyph_Data in shared mempoolsJean-Philippe Andre
- Create one Glyph_Data array per Font_Entry - Copy Font_Source descriptors in shared array - Copy font entries in shared index - Send font index path over socket - Merge Font_Source and Font_Data in shared memory - Send font index path on client connect - Repack font index after free - Pass font shm name to clients
2013-10-28evas/cserve2: Store glyphs in shared arraysJean-Philippe Andre
Also, change the internal structure of glyph listing: - Use lists instead of inlists Switch to use Glyph_Data instead of Glyph_Entry
2013-10-28evas/cserve2: Fix Eina_File leak at shutdownJean-Philippe Andre
cserve2_shutdown was not called
2013-10-28evas/cserve2: Scan shared indexes in client sideJean-Philippe Andre
Read Image descriptors from the shared arrays But, accessing the shared index can be a bit expensive, so try to read from the socket before scanning the index, without blocking.
2013-10-28evas/cserve2: Add _shared_index_remap_check()Jean-Philippe Andre
Check consistency of index header and remap if necessary.
2013-10-28evas/cserve2: Share paths of index tables with clientsJean-Philippe Andre
- Send paths to all connected clients - Send paths upon connection from a new client - Client side: - Fast find opened images - Fast find loaded images - Optimize load by skipping socket wait Also, remap indexes when needed (file size changed) When index file grows, the clients are not notified, so they must constantly check that their mapping matches the current array size.
2013-10-28evas/cserve2: Prepare shared indexes in clientsJean-Philippe Andre
- Move required struct definitions to common evas_cs2.h - Open/close shm and map arrays to memory Pretty stupid for now as it will close & reopen the shm files even when unchanged. - Check shm size and array header consistency - Implement message send - Send messages from server when calling repack()
2013-08-05Don't call fcntl functions if we do not HAVE_FCNTL support built in.Chris Michael
Signed-off-by: Chris Michael <cp.michael@samsung.com>
2013-07-16fix clang nonnull complaintCarsten Haitzler (Rasterman)
2013-07-04evas/cserve2: merge SETOPTS into OPENJean-Philippe Andre
When opening an image, OPEN and SETOPTS were always sent consecutively. Merging these two messages may improve the performance a bit (not measured), but also simplify the whole thing. Note: cserve2 debug tools have not been fixed yet.
2013-07-04evas/cserve2: use Evas_Image_Load_Opts directlyJean-Philippe Andre
Image_Data contained a similar struct to the standard Evas struct, let's not duplicate it uselessly.
2013-07-02evas/cserve2: return in case of select timeout.Jean-Philippe Andre
Signed-off-by: Cedric Bail <cedric.bail@samsung.com>
2013-07-02evas/cserve2: add some more error debug (client side).Jean-Philippe Andre
Signed-off-by: Cedric Bail <cedric.bail@samsung.com>
2013-07-02evas/cserve2: avoid useless SETOPT messages.Jean-Philippe Andre
Most of the time, the SETOPT message is sent only to carry 0 data. Let's add a flag to the OPEN message to avoid useless socket calls. Server-side: implement with message faking. Also, fix image ID logic on client side: - alloc data entry asap, - assign image_id during call to OPEN Signed-off-by: Cedric Bail <cedric.bail@samsung.com>
2013-07-02evas/cserve2: minor fixes (error paths).Jean-Philippe Andre
Signed-off-by: Cedric Bail <cedric.bail@samsung.com>
2013-06-20cserve2: fix minor memleaks with fontsJean-Philippe Andre
Leaks happen when fonts fail to load
2013-06-20cserve2: Handle interrupts when failing to read from serverJean-Philippe Andre
If we don't do that, the client will end up in an infinite loop from where it just can't exit, if the server is dead/stuck.
2013-06-20cserve2: prevent infinite loop during connectionJean-Philippe Andre
In case of EACCESS, there is no point trying forever In case of EINTR (interrupt), we can consider for now that any signal will prevent the use of cserve2. If we don't do that, Ctrl+C simply won't work :)
2013-06-04evas/cserve2: add lots of error checks in the client side.Jean-Philippe Andre
It seems some images are fully loaded even when we request only preload. The load handler will take care of both cases anyway. Signed-off-by: Cedric Bail <cedric.bail@samsung.com>