ecore-drm2: Merge initial modesetting with atomic flip code

ecore-drm2: Use just DRM_MODE_ATOMIC_ALLOW_MODESET for initial setting
of mode

This fixes an issue where gl_drm engine would end up flickering
everytime a frame was being set.

Thanks derek ;)

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add support for Atomic Pageflips

This patch modifies our ecore_drm2_fb_flip code to use Atomic/Nuclear
pageflips.

NB: Works perfectly under software drm engine .. some flickering with the
gl_drm engine that needs investigating.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to support setting DPMS levels via Atomic

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to use Atomic for Edid if available

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to setup Plane Atomic state for Outputs

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to setup Connector Atomic state for Outputs

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to setup Crtc Atomic state for Outputs

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to free Atomic state on shutdown

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to fill Plane Atomic state

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to fill Connector Atomic state

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to fill Atomic Crtc State

This patch adds code to enable Atomic Modesetting support (via ioctl)
and to fill in Atomic Crtc state during startup.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add code to check if Atomic Modesettting is usable

This code will detect the drm driver name and check that the kernel
itself is new enough to use Atomic Modesetting. This is needed as some
drivers (i915) do not handle Atomic Modesetting propertly without a
new enough kernel.

Signed-off-by: Chris Michael <cp.michael@samsung.com>

ecore-drm2: Add private structures to support Atomic Modesetting

Signed-off-by: Chris Michael <cp.michael@samsung.com>

elementary: Remove unused but set variable in test_image

Signed-off-by: Chris Michael <cp.michael@samsung.com>

elm: remove dulicate ELM_POPUP_ACTION_BUTTON_MAX definition

Summary: ELM_POPUP_ACTION_BUTTON_MAX is already defined in the included header file, elm_widget_popup.h.

Reviewers: cedric, jpeg, minkyu, Hermet

Reviewed By: Hermet

Subscribers: seoz, jehun.lim

Differential Revision: https://phab.enlightenment.org/D4312

Update gitignore.

Eo: introducing libeo_dbg.so.

This has been in the making for a very long time. Thanks to Marcel for
reminding me to do it.

What is it?
This is a tool to help application developers debug their apps with
everything Eo. Eo is strict, but not as strict as it can be. Many strict
tests and debug are very expensive to implement, and we have so many
hot-paths that even basic "ifs" to check if debugging is enabled  will
add significant overhead to normal running applications. This is why I
created this library. All the expensive tests and bookkeeping should be
wrapped around with "#ifdef EO_DEBUG". With this change, libeo.so is
compiled twice, once normally, and once with this define set (as
libeo_dbg.so). This means that normal eo code will not be affected, but
if you decide to debug your application, all you need to do is:

LD_PRELOAD=/path/to/libeo_dbg.so ./app

Or use the convenient wrapper:
eo_debug ./app

Which will load the debug heavy version.

What's currently there: at the moment, EO_DEBUG enables xref and
data_xref and stricter tests when fetching object data.
In the future, I also plan introducing "zombie objects", which
essentially mean that objects are never really deleted, so you can query
them long after they are gone to get more information on what they were.
So if for example you have an object id that you want to query after the
object has been deleted, you can.

I also plan on having a way to disable/enable certain debug mode
features via env vars, and maybe make the test suite link against this
one instead of the normal one, and possibly add more internal hooks for
the test suite to better inspect internal state?

P.S: The amount of errors spewed out when running it on elementary_test
makes me wish I wrote this earlier. :(

@feature

modules: emotion: fix event name after EFL Canvas Video convert

In commit 7b90e1147442d6a8023422400bffd77e2815fe0b the event name changed but
this module was not updated. Luckily we are building it on Jenkins or it would
have gone unnoticed for a long time.

examples: emotion: fix event naming after convert to EFL Canvas Object

In commit 7b90e1147442d6a8023422400bffd77e2815fe0b the event names changed
but examples have not been updated.

elm_image: Move back download events to legacy only

The events for (async) image download will be redefined by
@cedric later. So, remove them from eo now and only keep their
legacy implementation.

Also, improve elm_test example and add docs.

photocam: add missing EOLIAN prefix

scrollable_interface: fixed gravity_set() API.

Summary: elm_scroller_gravity_set()  API is not working.

Test Plan: elementary_test -> scroller3

Reviewers: jaehwan, SanghyeonLee, Hermet, jypark, raster, cedric

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4252

Revert "edje/edje_cc: use strncpy() instead of strcpy()."

This reverts commit ab1a72f5e7df6fe0adef54bdcddd9867a2ebe3a6.

just dismiss in coverity if the code is not actually wrong

Revert "edje edje_embryo: use strncpy()."

This reverts commit 06bd8dcf330fe31891475c92aa340d4886f47e2b.

just dismiss in coverity if the code is not actually wrong

Revert "edje edje_cc_out: use strncpy()."

This reverts commit 6de3b2c5d36993cf3dbe94e8fbefd04043f91740.

just dismiss in coverity if the code is not actually wrong

ecore,ecore_con: simplify destructor by linking future life cycle with object.

Updating slovenian translation

emotion: convert Emotion_Object into Efl.Canvas.Video

Reviewers: singh.amitesh, raster, jpeg, cedric

Differential Revision: https://phab.enlightenment.org/D3994

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

emile: fix typos.

eet: fix gnutls support with newer version.

tests: elm_win: fix compiler warning about signed vs. unsigned comparison

The variable i only goes from 0 to 4 here but it gets compared against a normal
int so we make sure i also is a normal int.

elm: progressbar: hide unit if unit_format_func is invalid

In commit 03882d558824f657f5c5a54fcd2d632eafeafd87 this was introduced but in
the end the visible signal was send in all cases. This looks like an oversight
to me. Fixed.

evas: model_save: remove unused label after error handling change

In commit 8272d1492796657978c5591971768d37d4e15a7e the error handling was
moved upwards and the need for the goto label removed. Catching up here and
removing the label as well.

evas: remove unnecessary check for clip coords.

Summary: (dst_clip_w <= 0 || dst_clip_h <= 0) is already checked.

Reviewers: jpeg, cedric, Hermet

Reviewed By: Hermet

Subscribers: conr2d

Differential Revision: https://phab.enlightenment.org/D4303

elm_config: add null check before usage

elm_atspi: remove redundant null checking

The type is always _ADDED or _REMOVED, therefore atspi_desc
is always "add" or "remove"; no need to have null checking for it.

evas software_engine: ++safety code.

edje edje_cc_out: use strncpy().

This change is not much meaningful but avoids an annoying coverity detection.

edje edje_embryo: use strncpy().

This change is not meaningful but avoids an annoying coverity detection.

edje/edje_cc: use strncpy() instead of strcpy().

strncpy() is better for security.
Also, this change avoids annoying coverity detection.

evas engine: correct null check

check null first, before access it.

Revert "evas engine: return as soon as possible if the surface or context is not valid."

This reverts commit 2f158ebe65cc3d007c1eae0e25590f092931ee25.

misunderstood ^ operator.
pinged by jpeg.

previous code was correct.

evas engine: return as soon as possible if the surface or context is not valid.

And later, the surface and context are accessed.
So both of arguments should be valid necessarily.

eo-cxx: Add progress to future and promise

edje_edit : add null check and close eet

Signed-off-by: Jaehyun Cho <jae_hyun.cho@samsung.com>

edje_decc: modified fclose location

modified fclose location. because of file pointer can become NULL.

Signed-off-by: Jaehyun Cho <jae_hyun.cho@samsung.com>

ecore_con,elput: fix warnings

Summary:
elput: fix warning for unused write result
         ecore_con: fix warning for unused asprintf result

Reviewers: iscaro, devilhorns, cedric

Reviewed By: cedric

Subscribers: cedric, seoz, jpeg

Differential Revision: https://phab.enlightenment.org/D4308

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

elm_progressbar: show status when unit_format_func is set

Summary:
 - elm,state,units,visible signal is emitted only when
   unit is set, but not when unit_format_func is set.
 - Since default unit has been set, this signal is emitted
   but signal will not be emitted after unit is set to NULL.

Test Plan:
 1. Create a progressbar.
 2. elm_progressbar_unit_format_set(obj, NULL);
 3. set unit_format_func by elm_progressbar_unit_format_function_set()
    and observe elm.text.status part visible.

Reviewers: Hermet, jpeg, cedric

Reviewed By: cedric

Subscribers: D-TAU, eunue, conr2d, cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4210

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

atspi: fix parent-child relationship for elm_list and elm_toolbar

Summary:
This patch provides proper parent-child relationship for elm_list and elm_toolbar
while atsapi_mode is set for icon and end element.

This patch is moved from:
    bf188e59431ad9c4ca877b2632884d3d430de6b1

Change-Id: Iae855aacf29bef3808a0b5ec159f46cbf0f4539d

Reviewers: stanluk, cedric

Reviewed By: cedric

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4259

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

atspi: fix state macros

Summary:
This patch changes the value of 1 to 1ULL in STATE_TYPE macros
to signal the compiler that the value must be considered
as a unsigned long long, it has to be done cause state_set variable
can be longer than 32 bits.

This patch is moved. Orginal commit hash:
    a559e473c21c8da7c4e5a87b9c8583ce519cc35e

Change-Id: Ida89f3be185736f61543d37010d0f5cb8d80a751

Reviewers: cedric, stanluk

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4260

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eet: fix memory leak

Summary:
If ef is null, have to return before _set_material_to_eet_file_from_mesh api.
New momory will be allocated in the api.

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4305

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

eet: set alpha_texture for ETC1+Alpha

Summary:
alpha_texture need to be set EINA_TRUE to support ETC1+Alpha

@fix

Test Plan:
Create an EDC file with png image with Alpha.
           compress image with ETC1 and ETC2.
           Observe Alpha is properly applied in both case.

Reviewers: jpeg, Hermet, cedric

Reviewed By: cedric

Subscribers: conr2d

Differential Revision: https://phab.enlightenment.org/D4307

Signed-off-by: Cedric BAIL <cedric@osg.samsung.com>

Evas: Don't access members of a null struct. Fixes T4616

libproxy: dlopen() and make it runtime optional.

ecore_file: Fix improper way of comparing in ecore_file_path_dir_exists @fix

Summary: There is wrong comparing while using strcmp function. Should be inverted.

Reviewers: cedric, raster, NikaWhite

Reviewed By: NikaWhite

Subscribers: cedric, NikaWhite, jpeg

Differential Revision: https://phab.enlightenment.org/D4291

eina_cpu, evas: Remove _eina_cpu_fast_core_get, don't set render thread affinity

We've decided it would be best to just let the scheduler do its job.

eina - eina file map populate fallback to use log int not int... for size

this makes the size match other sizes in eina_file. doesn't affect
linux but i think bsd's get hit.

tests: ecore: switch from fprintf to ck_asser_msg macro for custom error message

The ck_assert_msg macro can do this for us already in a failed case.

eldbus: fix null value usage

Evas font: force freetype v35 ttf interpreter

Ref T4623

v40 bytecode interpreter is official as of freetype 2.7.
The results don't look so good at the moment. The text looks and glyph
positioning seem worse than they were with the previous v35 interpreter.

So, in the meantime we'll keep using v35, just so everything looks
normal again.

Although the v40 is relevant since around 2.6.3, I rather not do any
FREETYPE_MINOR checks in this patch, because distributions might ship
previous versions with the other (v38) interpreter enabled.

ecore_con_local: fix memory leak.

after socket function call, close file descriptor.

embryo : Prevent buffer overflow in embryo_cc_sc2

Reviewers: jypark

Reviewed By: jypark

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4304

efreet xml parse - handle 0 sized xml specially to avoid crash

fixes T4493

eina - redo a lot of the static inlines to have better instr cache usage

this moves a lot of logic that is rare away from the linear/flat asm
path of code so we et fewer l1 cache misses when executing chuncks of
our code. this also reduces the code size and takes some funcs like in
eina_inline_lock_posix.x and makes them real functions to reduce code
size thus better l1 cache usage - only for new/free of locks.
spinlocks, semaphores etc. as these will have no advantage being
inlined but simply bloat out code size instead.

overall this actually reduces efl lib binary sizes 0.4%, so that's a
good sign.

this passes make check and i think i got it right... let me know if i
didn't. i'm also not sure i should just keep the static inlines and
not make the formerly static inline funcs full EAPI ones now... good q.

edje_edit: add null checking

ecore: fix the parameter of strerror

elm fileselector: add null checking before strlen

eldbus: null checking before using the pointer

ecore: fix warning in timer test.

elementary: patch got pushed before the complete fix.

elementary: be more conservative on the amount of data requested from disk.

eina: populate memory in the right limit.

ecore: remove useless ecore_parent.eo.

eina_cpu: Fix coverity defects in eina_cpu_map_init

Fixes a potential fd leak on failure and wrong core to
speed mapping on systems with > 31 cpu cores

CID: 1362860
CID: 1362859
CID: 1362857

edje_edit: make tweens able to work with image sets

it's important because some tweens of loaded edje group might use image sets
instead of images, so now making edje_edit API to work with them correctly

@fix

render_thread: Attempt to set affinity to a random fast core

We've been pinning the render thread for every EFL process to core 0.
This is a bit silly in the first place, but some big.LITTLE arm systems,
such as exynos 5422, have the LITTLE cores first.

On those systems we put all the render threads on a slow core.

This attempts to fix that by using a random core from the pool of fast
cores.

If we can't determine which cores are fast (ie: we're not on a
linux kernel with cpufreq enabled) then we'll continue doing what we've
always done - pin to core 0.

edje example: fix broken batman svg resource.

correct missing '/' at the end of svg tag.

eina_cpu: Fix error handling in eina_cpu_fast_core_get

I introduced a crash on linux systems where the cpufreq directory
doesn't exist.  This fixes it.

efl_net_dialer_http: use libproxy.

query libproxy in a thread (since it's nasty and blocks), then apply
proxy to curl and execute the request.

efl_net_dialer_tcp: use libproxy settings.

libproxy allows various means to configure a proxy, will load from
gnome and kde configuration settings, envvars, macos and even windows
registry.

curl still doesn't use it, but we can make that later.

efl_net_dialer_websocket: remove unused variable.

efl-future will null the pointer, so pd is not needed anymore.

tests: ecore: relax the timing precision for the promise timeout test

This test has been failing on Jenkins again and again. After adding the debug
a while ago it now shows that the value is between 0.01 and 0.02 in all cases
I have seen. Relaxing the timeout here a bit to make it pass in situation where
our CI is under load.

eo: fix headr to header

if you enable EO_DEBUG you cannot build eo, since the field is called
header not headr

eo: fix reference to constants

they are renamed from the prefix EO_ to EFL_

eo: fix callback cmp

Summary:
as told in _eina_stringshared_key_cmp in eina_hash.c:

originally we want to do this:
   return key1 - key2;
but since they are ptrs and an int can't store the different of 2 ptrs in
either 32 or 64bit (signed hasn't got enough range for the diff of 2
32bit values regardless of their type... we'd need 33bits or 65bits)

So changing this to the same logic.

Reviewers: tasn, raster

Subscribers: cedric, jpeg

Differential Revision: https://phab.enlightenment.org/D4298

edje_svg_loader: Initialize char array.

Initialize char array before calling strcmp().

Revert "render_thread: Attempt to set affinity to a random fast core"

This reverts commit a17ac66f0a0b089dde0b2e550523b0d59ec97f52.

This change broke every efl build locally, as well as on Jenkins, for me.

CLD     lib/ethumb_client/libethumb_client.la
/bin/sh: line 2: 10047 Segmentation fault      (core dumped) EFL_RUN_IN_TREE=1
../src/bin/edje/edje_cc -id . -fd . -id ./tests/emotion/data tests/emotion/data/theme.edc
tests/emotion/data/theme.edj
Makefile:52584: recipe for target 'tests/emotion/data/theme.edj' failed
make[4]: *** [tests/emotion/data/theme.edj] Error 139
make[4]: *** Waiting for unfinished jobs....
/bin/sh: line 2: 10088 Segmentation fault      (core dumped) EFL_RUN_IN_TREE=1
../src/bin/edje/edje_cc -id . -fd . -id ./modules/ethumb/emotion modules/ethumb/emotion/template.edc
modules/ethumb/emotion/template.edj
Makefile:52590: recipe for target 'modules/ethumb/emotion/template.edj' failed
make[4]: *** [modules/ethumb/emotion/template.edj] Error 139
/bin/sh: line 2: 10119 Segmentation fault      (core dumped) EFL_RUN_IN_TREE=1
../src/bin/edje/edje_cc -id . -fd . -id ../src/modules/elementary/prefs/
modules/elementary/prefs/elm_prefs_swallow.edc modules/elementary/prefs/elm_prefs_swallow.edj
Makefile:52614: recipe for target 'modules/elementary/prefs/elm_prefs_swallow.edj' failed
make[4]: *** [modules/elementary/prefs/elm_prefs_swallow.edj] Error 139

evas_object_image: fix alpha_set bug.

if image object's first alpha value is false, evas_object_image_alpha_set function did not work.
opaque_valid is always 1 even though has_alpha value changed.

elm_menu: fix callback deletion from NULL object

Reviewers: cedric, raster, Hermet

Reviewed By: Hermet

Subscribers: jpeg, reutskiy.v.v

Differential Revision: https://phab.enlightenment.org/D4294

efl_net_dialer_tcp: add SOCKS proxy support.

SOCKS is implemented in its own thread using synchronous/blocking
primitives, which simplifies the code a lot -- as well as simulate the
usage of Ecore_Thread as our users will likely do.

Since SOCKSv4a and SOCKSv5 allow name resolution, the whole
getaddrinfo() is done in the same thread, when needed, instead of a
separate thread to do that, which should also save some resources.

Instead of the legacy ECORE_CON_SOCKS_V4 and ECORE_CON_SOCKS_V5, now
we use socks_proxy, all_proxy and no_proxy. This matches our other
dialers http/websocket (which will use http_proxy, all_proxy and
no_proxy). If desired it's easy to add back support for those
variables, but I think we should just deprecate them. (The legacy code
will keep unchanged, thus direct users of ecore_con_server will still
use those -- just the previous users of ecore_con_server will be
converted to use the new API).

efl_net_dialer_http: do eos on error.

efl_net_dialer: improve proxy docs.

Document some proxy behavior like done by CURL, so we'll follow that
standard, with $http_proxy, $socks_proxy, $all_proxy and $no_proxy.

also add some missing @since.

efl_promise: check NULL before calling.

we do not check any of success, failure or progress, so we must check
if they are valid before calling.

This fixed a bug in efl_net_dialer_tcp where it uses a null failure
cb and was SEGV.

eina_thread: don't leak on pthread_cancel().

if we cancel a thread we should still free the context, so use the new
EINA_THREAD_CLEANUP_PUSH() and EINA_THREAD_CLEANUP_POP()

elm: Fix edit popup menu order

Cut/Copy/Paste is the standard order most other places.
@fix

ecore,ecore_con: fix migration to efl_future.

Efl_Future actually work with weak reference. So you do not need to
set things to NULL, but you actually need to register the memory location
of the future with efl_future_use.

Revert "eina: Actually call eina_cpu_init() and eina_cpu_shutdown()"

This reverts commit 1881b0d34302b8c29d4b086715eeda9e576ee60a.

This is called from an array constructed with pre-processor macros and
I didn't notice.

evas-drm: update drm engine recovery from all buffers used

Use the relatively new buffer release API to free up buffers instead
of immediately going nuclear and blowing away all buffer ages.

eina_cpu: Fix broken E_API macro

Oops, I cut and paste it from a wrong file.

Thanks to Vincent Torri for the quick catch.