From mboxrd@z Thu Jan 1 00:00:00 1970 Message-ID: <4B72A9DA.1090104@domain.hid> Date: Wed, 10 Feb 2010 13:43:06 +0100 From: Jan Kiszka MIME-Version: 1.0 References: <4B72A7E2.5020001@domain.hid> In-Reply-To: <4B72A7E2.5020001@domain.hid> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Xenomai-core] [Xenomai-git] Jan Kiszka : Make xnarch_init_timeconv an uninlined weak function List-Id: Xenomai life and development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Gilles Chanteperdrix Cc: Xenomai core Gilles Chanteperdrix wrote: > GIT version control wrote: >> Module: xenomai-jki >> Branch: for-upstream >> Commit: 6b40653e9c3c4a2433bb4e91344fc378eb860f75 >> URL: http://git.xenomai.org/?p=xenomai-jki.git;a=commit;h=6b40653e9c3c4a2433bb4e91344fc378eb860f75 >> >> Author: Jan Kiszka >> 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. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux