Re: [devel] libopenobex-1.3 symbol versioning patch

[Sergey Vlasov <vsu@altlinux.ru>]

[-- Attachment #1: Type: text/plain, Size: 3154 bytes --]

On Sat, Aug 12, 2006 at 06:36:34PM +0400, Alexey Tourbin wrote:
> Кстати, если флаги не критичные, типа O_NOATIME, т.е. если старая
> библиотека на эти флаги не реагирует и более-менее нормально работает,
> тогда можно ничего не делать.

В данном случае флаги критичные.  Причём OBEX_EV_REQCHECK - это флаг,
который библиотека передаёт в callback, поэтому новую версию придётся
вешать на OBEX_Init.

> У меня сходу не получилось при помощи одного только version script
> повесить один символ на два интерфейса.  Кажется, надо прямо в код
> вставлять всякие asm/.symver, это недавно было в
> 
> 	Date: Wed, 9 Aug 2006 03:16:59 +0400
> 	From: "Dmitry V. Levin"
> 	Subject: Re: [devel] rpmelfsymv
> 	To: ALT Devel discussion list
> 	Message-ID: <20060808231659.GA11505@basalt.office.altlinux.org>
> 
> В общем муть какая-то.

Именно.

Совсем просто даже через asm не получается:

__asm__(".symver real_OBEX_Init,OBEX_Init@@OPENOBEX_1.2");
__asm__(".symver real_OBEX_Init,OBEX_Init@OPENOBEX_1.0");

Независимо от порядка получаю:

{standard input}: Assembler messages:
{standard input}:12: Error: multiple versions [`OBEX_Init@OPENOBEX_1.0'|`OBEX_Init@@OPENOBEX_1.2'] for symbol `real_OBEX_Init'

Приходится делать так:

__asm__(".symver real_OBEX_Init,OBEX_Init@@OPENOBEX_1.2");
__asm__(".globl old_OBEX_Init");
__asm__(".set old_OBEX_Init, real_OBEX_Init");
__asm__(".symver old_OBEX_Init,OBEX_Init@OPENOBEX_1.0");

Т.е., для каждой версии необходим отдельный промежуточный символ.  И
настоящую функцию приходится переименовывать, иначе потом при линковке
получается multiple definition of `OBEX_Init'.

Что ещё хуже - в libalsa я нашёл следующий кусок:

#ifdef __powerpc64__
# define symbol_version(real, name, version) 			\
	__asm__ (".symver " #real "," #name "@" #version);	\
	__asm__ (".symver ." #real ",." #name "@" #version)
# define default_symbol_version(real, name, version) 		\
	__asm__ (".symver " #real "," #name "@@" #version);	\
	__asm__ (".symver ." #real ",." #name "@@" #version)
#else
# define symbol_version(real, name, version) \
	__asm__ (".symver " #real "," #name "@" #version)
# define default_symbol_version(real, name, version) \
	__asm__ (".symver " #real "," #name "@@" #version)
#endif

И даже хуже:

#ifdef __powerpc64__
#define use_default_symbol_version(real, name, version) \
	__asm__ (".weak " #name); 			\
	__asm__ (".weak ." #name); 			\
	__asm__ (".set " #name "," #real);		\
	__asm__ (".set ." #name ",." #real)
#else
#if defined(__alpha__) || defined(__mips__)
#define use_default_symbol_version(real, name, version) \
        __asm__ (".weak " #name); \
        __asm__ (#name " = " #real)
#else
#define use_default_symbol_version(real, name, version) \
	__asm__ (".weak " #name); \
	__asm__ (".set " #name "," #real)
#endif

Т.е., на powerpc64 у каждой функции не один, а два символа, на которые
надо вешать версии, а на alpha и mips не работает .set.

Но примеров вешания разных версий на один символ нет даже там.  Вообще
такое кто-нибудь делает?

> * at продолжает засорять @devel
* vsu думает, не забить ли уже на всё это

[-- Attachment #2: Type: application/pgp-signature, Size: 191 bytes --]