[devel] webkitgtk TEXTREL

[devel] webkitgtk TEXTREL

[Alexey Shabalin]

Вечер добрый.
Прошу помощи у зала.
В новом webkitgtk выделили отдельную библиотеку libjavascriptcoregtk,
в которой есть ассемблерная вставка.
И теперь появилась ошибка TEXTREL.
Багу в апстрим повесил, но вдруг кто-то в состоянии исправить, не
дожидаясь апстрима.
http://git.altlinux.org/tasks/56980/build/100/i586/log

-- 
Alexey Shabalin

Re: [devel] webkitgtk TEXTREL

[Dmitry V. Levin]

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

On Fri, Oct 21, 2011 at 08:49:46PM +0400, Alexey Shabalin wrote:
> Вечер добрый.
> Прошу помощи у зала.
> В новом webkitgtk выделили отдельную библиотеку libjavascriptcoregtk,
> в которой есть ассемблерная вставка.
> И теперь появилась ошибка TEXTREL.
> Багу в апстрим повесил, но вдруг кто-то в состоянии исправить, не
> дожидаясь апстрима.
> http://git.altlinux.org/tasks/56980/build/100/i586/log

В двух файлах наблюдается следующее:

#if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
#define SYMBOL_STRING_RELOCATION(name) #name "@plt"
#elif OS(DARWIN) || (CPU(X86_64) && COMPILER(MINGW) && !GCC_VERSION_AT_LEAST(4, 5, 0))
#define SYMBOL_STRING_RELOCATION(name) "_" #name
#elif CPU(X86) && COMPILER(MINGW)
#define SYMBOL_STRING_RELOCATION(name) "@" #name "@4"
#else
#define SYMBOL_STRING_RELOCATION(name) #name
#endif

Попробуйте заменить
#if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
на
#if (OS(LINUX) || OS(FREEBSD)) && (CPU(X86) || CPU(X86_64))
и посмотреть, что из этого выйдет.
В любом случае бага зарыта здесь.


-- 
ldv

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

Re: [devel] webkitgtk TEXTREL

[Alexey Shabalin]

21 октября 2011 г. 21:13 пользователь Dmitry V. Levin
<ldv@altlinux.org> написал:
> On Fri, Oct 21, 2011 at 08:49:46PM +0400, Alexey Shabalin wrote:
>> Вечер добрый.
>> Прошу помощи у зала.
>> В новом webkitgtk выделили отдельную библиотеку libjavascriptcoregtk,
>> в которой есть ассемблерная вставка.
>> И теперь появилась ошибка TEXTREL.
>> Багу в апстрим повесил, но вдруг кто-то в состоянии исправить, не
>> дожидаясь апстрима.
>> http://git.altlinux.org/tasks/56980/build/100/i586/log
>
> В двух файлах наблюдается следующее:
>
> #if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
> #define SYMBOL_STRING_RELOCATION(name) #name "@plt"
> #elif OS(DARWIN) || (CPU(X86_64) && COMPILER(MINGW) && !GCC_VERSION_AT_LEAST(4, 5, 0))
> #define SYMBOL_STRING_RELOCATION(name) "_" #name
> #elif CPU(X86) && COMPILER(MINGW)
> #define SYMBOL_STRING_RELOCATION(name) "@" #name "@4"
> #else
> #define SYMBOL_STRING_RELOCATION(name) #name
> #endif
>
> Попробуйте заменить
> #if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
> на
> #if (OS(LINUX) || OS(FREEBSD)) && (CPU(X86) || CPU(X86_64))
> и посмотреть, что из этого выйдет.
> В любом случае бага зарыта здесь.

Да, так сборка заканчивается успешно.
Насколько корректо это решение?
Спасибо.

-- 
Alexey Shabalin

Re: [devel] webkitgtk TEXTREL

[Sergey Vlasov]

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

On Fri, Oct 21, 2011 at 09:13:25PM +0400, Dmitry V. Levin wrote:
> On Fri, Oct 21, 2011 at 08:49:46PM +0400, Alexey Shabalin wrote:
> > Вечер добрый.
> > Прошу помощи у зала.
> > В новом webkitgtk выделили отдельную библиотеку libjavascriptcoregtk,
> > в которой есть ассемблерная вставка.
> > И теперь появилась ошибка TEXTREL.
> > Багу в апстрим повесил, но вдруг кто-то в состоянии исправить, не
> > дожидаясь апстрима.
> > http://git.altlinux.org/tasks/56980/build/100/i586/log
> 
> В двух файлах наблюдается следующее:
> 
> #if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
> #define SYMBOL_STRING_RELOCATION(name) #name "@plt"
> #elif OS(DARWIN) || (CPU(X86_64) && COMPILER(MINGW) && !GCC_VERSION_AT_LEAST(4, 5, 0))
> #define SYMBOL_STRING_RELOCATION(name) "_" #name
> #elif CPU(X86) && COMPILER(MINGW)
> #define SYMBOL_STRING_RELOCATION(name) "@" #name "@4"
> #else
> #define SYMBOL_STRING_RELOCATION(name) #name
> #endif
> 
> Попробуйте заменить
> #if (OS(LINUX) || OS(FREEBSD)) && CPU(X86_64)
> на
> #if (OS(LINUX) || OS(FREEBSD)) && (CPU(X86) || CPU(X86_64))
> и посмотреть, что из этого выйдет.
> В любом случае бага зарыта здесь.

Похоже, просто так исправить это не получится, поскольку в
webkit/Source/JavaScriptCore/jit/JSInterfaceJIT.h в #elif CPU(X86)
наблюдается

        static const RegisterID regT3 = X86Registers::ebx;

в то время как для выполнения вызова через PLT требуется, чтобы в %ebx
находился адрес GOT.  Так что придётся ещё разбираться, в какие места
нужно добавить получение адреса GOT перед такими вызовами.

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