[devel] libopenobex-1.3 symbol versioning patch
Sergey Vlasov
=?iso-8859-1?q?vsu_=CE=C1_altlinux=2Eru?=
Сб Авг 12 19:49:01 MSD 2006
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 думает, не забить ли уже на всё это
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя : =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Тип : application/pgp-signature
Размер : 191 байтов
Описание: =?iso-8859-1?q?=CF=D4=D3=D5=D4=D3=D4=D7=D5=C5=D4?=
Url : <http://lists.altlinux.org/pipermail/devel/attachments/20060812/7c55a7c8/attachment-0001.bin>
Подробная информация о списке рассылки Devel