Re: [E-devel] E SVN: barbieri trunk/efl

[Cedric B. <ced...@fr...>]

On Tue, Jan 8, 2013 at 10:32 PM, Enlightenment SVN
<no-...@en...> wrote:
> Log:
> efl/edje: conditional usage of fixed-point, program/calc caches.
>
>   Weirdly enough raster turned PROGRAM cache on to fix a bug saying that
>   the other path was not tested, but from IN-EFL/edje/configure.ac CALC
>   cache was enabled and PROGRAM was disabled.
>
>   I kept the PROGRAM on and enabled CALC. Added fixed point as well.
>
>   Cedric, Raster: should we keep PROGRAM and CALC cache options if the
>   other path is not tested? Can't we assume they are always on? How much
>   memory these cache add? Can't this be reduced?

Yes, we need they both have a big memory impact and the other path was
working back in July with 1.7.x branch. You may choose the other path
when you have a good amount of CPU horse power, but not much memory.
This happen to be the case with some of the embedded target that I was
working on in the past, and I don't see them going away soon.

As for fixed point, it was also working back in July, didn't test it
since then. This issues reveal our lack of edje test suite and
benchmarking solution...

> Author:       barbieri
> Date:         2013-01-08 05:32:07 -0800 (Tue, 08 Jan 2013)
> New Revision: 82410
> Trac:         http://trac.enlightenment.org/e/changeset/82410
>
> Modified:
>   trunk/efl/configure.ac
>
> Modified: trunk/efl/configure.ac
> ===================================================================
> --- trunk/efl/configure.ac      2013-01-08 11:51:56 UTC (rev 82409)
> +++ trunk/efl/configure.ac      2013-01-08 13:32:07 UTC (rev 82410)
> @@ -3319,7 +3319,9 @@
>
>  ### Default values
>  want_multisense="no" # TODO: move to ecore_audio and enable
> -want_edje_program_cache="no"
> +
> +# TODO: should we keep or remove these?
> +want_edje_program_cache="yes"
>  want_edje_calc_cache="yes"
>  want_fixed_point="no"
>
> @@ -3349,7 +3351,13 @@
>
>  EFL_EVAL_PKGS([EDJE])
>
> -AC_DEFINE([EDJE_PROGRAM_CACHE], [1], [Cache result of program glob matches - this uses up extra ram with the gain of faster program matching])
> +AC_DEFINE_IF([EDJE_PROGRAM_CACHE], [test "${want_edje_program_cache}" = "yes"],
> +   [1], [Cache result of program glob matches])
> +AC_DEFINE_IF([EDJE_CALC_CACHE], [test "${want_edje_calc_cache}" = "yes"],
> +   [1], [Cache result of calc glob matches])
> +AC_DEFINE_IF([BUILD_EDJE_FP], [test "${want_fixed_point}" = "yes"],
> +   [1], [Use Fixed Point instead of FPU])
> +
>  AM_CONDITIONAL([ENABLE_MULTISENSE], [test "${want_multisense}" = "yes"])
>  AC_DEFINE_IF([ENABLE_MULTISENSE], [test "${want_multisense}" = "yes"],
>    [1], [Use Multisense])
>
>
> ------------------------------------------------------------------------------
> Master SQL Server Development, Administration, T-SQL, SSAS, SSIS, SSRS
> and more. Get SQL Server skills now (including 2012) with LearnDevNow -
> 200+ hours of step-by-step video tutorials by Microsoft MVPs and experts.
> SALE $99.99 this month only - learn more at:
> http://p.sf.net/sfu/learnmore_122512
> _______________________________________________
> enlightenment-svn mailing list
> enl...@li...
> https://lists.sourceforge.net/lists/listinfo/enlightenment-svn
>



--
Cedric BAIL