Dmitry Eremin-Solenikov(lumag) replied on github web page:

.travis.yml
line 7
@@ -53,9 +53,6 @@ env:
         - CONF=""
         - CONF="--disable-abi-compat"
         - CONF="--enable-deprecated"
-        - CONF="--enable-schedule-sp"
-        - CONF="--enable-schedule-iquery"
-        - CONF="--enable-schedule-scalable"
         - CONF="--enable-dpdk-zero-copy"


Comment:
This would mean that we have now 4 options to test, instead of just one. What 
about renaming 'default' scheduler to some sensible name and adding 
`--with-scheduler-default=FOO`? Then we will have just one combination to test.

> Bill Fischofer(Bill-Fischofer-Linaro) wrote:
> Perhaps `--enable-schedule-dynamic` (default) to indicate run time selection 
> that can be overridden by a specific static selection at `./configure` time 
> via the other `--enable-schedule-xxx` options?


>> Petri Savolainen(psavol) wrote:
>> Run time selection is good addition and can be even the default behavior. 
>> Anyway, I'd like to keep the build time selection also, since usually one 
>> application uses the same scheduler always. Fixing the scheduler at build 
>> time avoids accidentally usage of different schedulers in different servers 
>> (or VMs or test environments) and thus avoid problems/debugging caused by 
>> that.
>> 


https://github.com/Linaro/odp/pull/467#discussion_r167562856
updated_at 2018-02-12 14:01:42

Reply via email to