Jan Kiszka wrote:
> Gilles Chanteperdrix wrote:
>> GIT version control wrote:
>>> Module: xenomai-jki
>>> Branch: for-upstream
>>> Commit: 6b40653e9c3c4a2433bb4e91344fc378eb860f75
>>> Author: Jan Kiszka <jan.kis...@siemens.com>
>>> Date: Wed Feb 10 13:24:29 2010 +0100
>>> Make xnarch_init_timeconv an uninlined weak function
>>> Otherwise the wrong set of time conversion variables might get
>>> initialized when using > 1 skin libraries.
>> If that would be possible, then it is the conversion variables which
>> should made be weak, not the function.
>> The way I see it, the posix and native skins currently get a different
>> set of variables and functions, which works, but with your change, since
>> there is only one function, only one set of variable gets initialized by
>> the two function calls. And one skin just broke.
>> Or am I missing something? Does the patch fix a problem you really had?
> Frankly, I wasn't able to test in the field yet as replacing the libs
> there is non-trivial. But I was able to observe that only one set of
> functions is used - which is logical considering the weak marks. And
> this breaks due to the static inline initialization.
> However, let's mark both functions and variables weak to fix the issue
> and avoid leaving unused variables around. Will update my patch in a minute.
Ok. Merged, thanks.
Xenomai-core mailing list