Re: [devel] [git update] packages/mpg123: heads/master

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Sun, Jan 06, 2008 at 06:28:23PM +0300, Alexey Morsov wrote:
> Update of /people/swi/packages/mpg123.git
> 
> Changes statistics since `1.0.1-alt1' follows:
>  mpg123.spec |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> Changelog since `1.0.1-alt1' follows:
> commit 5ce736c13634c5f090749c7d8873982428ff8451
> Author: Alexey Morsov <samurai@home>
> Date:   Sun Jan 6 18:27:40 2008 +0300
> 
>     fix textrel method for ix86
> 
> Full diff since `1.0.1-alt1' follows:
> diff --git a/mpg123.spec b/mpg123.spec
> index abefde2..44cb106 100644
> --- a/mpg123.spec
> +++ b/mpg123.spec
> @@ -18,6 +18,10 @@ Packager: Alexey Tourbin <at@altlinux>
>  # this pathc absolute in 1.0.0 (with whole file)
>  #Patch1: mpg123-0.66-8bit.alt.patch
>  
> +%ifarch %ix86
> +%set_verify_elf_method textrel=relaxed
> +%endif

Нет, такое изменение без предварительного обсуждения не годится.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Michael Shigorin]

On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
> > +%ifarch %ix86
> > +%set_verify_elf_method textrel=relaxed
> > +%endif
> Нет, такое изменение без предварительного обсуждения не годится.

Помнится, там был ассемблер?

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Sun, Jan 06, 2008 at 07:37:39PM +0200, Michael Shigorin wrote:
> On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
> > > +%ifarch %ix86
> > > +%set_verify_elf_method textrel=relaxed
> > > +%endif
> > Нет, такое изменение без предварительного обсуждения не годится.
> 
> Помнится, там был ассемблер?

Обычно
%ifarch %ix86
%set_verify_elf_method textrel=relaxed
%endif
означает, что для %ix86 апстрим использует (кривой) ассемблерный код,
который мантейнер не смог (или не захотел) отключить.

Поскольку в предыдущей сборке этого костыля не было, нужны веские
основания для того, чтобы этот костыль появился.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Tourbin]

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

On Sun, Jan 06, 2008 at 07:37:39PM +0200, Michael Shigorin wrote:
> On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
> > > +%ifarch %ix86
> > > +%set_verify_elf_method textrel=relaxed
> > > +%endif
> > Нет, такое изменение без предварительного обсуждения не годится.
> 
> Помнится, там был ассемблер?

Надо ассемблер отрубить.  Особенно неграмотный ассемблер.  Ассемблер
давал ощутимый выигрыш на декодировании mp3 лет десять назад, во времена
когда был популярен P166MMX.  Теперь процессоры стали настолько мощнее,
что в деле декодирования mp3 не стоит экономить их загрузку, она всё
равно мизерная.

Кстати, при большом load затыки в проигрывании mp3 вызывает скорее
дисковая подсистема ядра, нежели чем что-то ещё. :(  Приходится
активизировать 'last resort' способ в виде 'sudo renice тры-ты-ты'. :(

И вообще, насколько я понимаю, стандартным способом декодирования mp3
сейчас является libmad.  В этом смысле оригинальные эти подвыподверты
mpg123 никакого энтузиазма не вызывают.

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

Re: [devel] [git update] packages/mpg123: heads/master

[Michael Shigorin]

On Sun, Jan 06, 2008 at 10:41:04PM +0300, Alexey Tourbin wrote:
> > > > +%ifarch %ix86
> > > > +%set_verify_elf_method textrel=relaxed
> > > > +%endif
> > > Нет, такое изменение без предварительного обсуждения не
> > > годится.
> > Помнится, там был ассемблер?
> Надо ассемблер отрубить.

Ради textrel?  Я бы не стал, но дело твоё.

> Особенно неграмотный ассемблер.  Ассемблер давал ощутимый
> выигрыш на декодировании mp3 лет десять назад, во времена когда
> был популярен P166MMX.  Теперь процессоры стали настолько
> мощнее, что в деле декодирования mp3 не стоит экономить их
> загрузку, она всё равно мизерная.

Сейчас есть Lite.

[угу, но это другое]

> И вообще, насколько я понимаю, стандартным способом
> декодирования mp3 сейчас является libmad.

Даже не знаю, xmms играет через libmpg123 вроде...
:)

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[devel] textrel vs asm: +1 for asm-centric default

[Michael Shigorin]

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

On Sun, Jan 06, 2008 at 10:08:37PM +0300, Dmitry V. Levin wrote:
> > Помнится, там был ассемблер?
> Обычно
> %ifarch %ix86
> %set_verify_elf_method textrel=relaxed
> %endif
> означает, что для %ix86 апстрим использует (кривой)
> ассемблерный код, который мантейнер не смог (или не захотел)
> отключить.

Дим, если тебе достаточно CPU, чтоб отключать ассемблер -- 
это ещё не значит, что стоит настаивать, чтобы так делали все.

Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
Если случай mp3 скорее неактуален большинству аудитории
(кроме Lite разве что, хотя для меня на месте майнтейнера
и это было бы достаточным поводом не делать прыжков в ширину)
-- то всяких недопереоптимизированных кодеков и мультимедийных
библиотек у нас вроде как наблюдается.

Предлагаю предложения "отключить ассемблер" по умолчанию
_аргументировать_ либо сопровождать патчем по исправлению оного.

Либо разъяснить нам, начиная с упёртого меня, чем так плохо иметь
TEXTREL в подобных программах (конечных) и библиотеках, кроме
того, что релокации замедляют процесс линковки (причём на 486 
со 125-мегабайтным диском меня это не волновало, в отличие от
возможности слушать стерео вместо моно или 44 вместо 22).

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Tue, Jan 08, 2008 at 12:59:39AM +0200, Michael Shigorin wrote:
> On Sun, Jan 06, 2008 at 10:41:04PM +0300, Alexey Tourbin wrote:
> > > > > +%ifarch %ix86
> > > > > +%set_verify_elf_method textrel=relaxed
> > > > > +%endif
> > > > Нет, такое изменение без предварительного обсуждения не
> > > > годится.
> > > Помнится, там был ассемблер?
> > Надо ассемблер отрубить.
> 
> Ради textrel?  Я бы не стал, но дело твоё.

Зачем там понадобился _такой_ ассемблер?

> > И вообще, насколько я понимаю, стандартным способом
> > декодирования mp3 сейчас является libmad.
> 
> Даже не знаю, xmms играет через libmpg123 вроде...
> :)

Кстати, это интересный вопрос.

$ apt-cache whatdepends libmad.so.0 |fgrep -c libmad.so.0
35

$ apt-cache showpkg libmpg123.so
Package: libmpg123.so
Versions: 

Reverse Depends: 
Dependencies: 
Provides: 
Reverse Provides: 
kdemultimedia-arts 3.5.8-alt1

P.S.  Кстати, libmad не очень хорошо собран...
P.P.S. Имеет ли смысл s/mpg123/mpg321/ ?


-- 
ldv

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

Re: [devel] textrel vs asm

[Dmitry V. Levin]

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

On Tue, Jan 08, 2008 at 01:05:25AM +0200, Michael Shigorin wrote:
> On Sun, Jan 06, 2008 at 10:08:37PM +0300, Dmitry V. Levin wrote:
> > > Помнится, там был ассемблер?
> > Обычно
> > %ifarch %ix86
> > %set_verify_elf_method textrel=relaxed
> > %endif
> > означает, что для %ix86 апстрим использует (кривой)
> > ассемблерный код, который мантейнер не смог (или не захотел)
> > отключить.
> 
> Дим, если тебе достаточно CPU, чтоб отключать ассемблер -- 
> это ещё не значит, что стоит настаивать, чтобы так делали все.

Дело в том, что появление в новой версии программы кривых ассемблерных
вставок ещё не повод их включать (по крайней мере, в случае с mpg123
картина выглядит именно такой).

> Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> Если случай mp3 скорее неактуален большинству аудитории
> (кроме Lite разве что, хотя для меня на месте майнтейнера
> и это было бы достаточным поводом не делать прыжков в ширину)
> -- то всяких недопереоптимизированных кодеков и мультимедийных
> библиотек у нас вроде как наблюдается.

У нас есть mpeg-декодеры получше чем mpg123.

> Предлагаю предложения "отключить ассемблер" по умолчанию
> _аргументировать_ либо сопровождать патчем по исправлению оного.

Просто всякий раз, прежде чем "включить ассемблер", надо проверить,
приносит ли это измеримые преимущества, и стоят ли они тех рисков,
на которые приходится идти из-за включения.

> Либо разъяснить нам, начиная с упёртого меня, чем так плохо иметь
> TEXTREL в подобных программах (конечных) и библиотеках, кроме
> того, что релокации замедляют процесс линковки (причём на 486 
> со 125-мегабайтным диском меня это не волновало, в отличие от
> возможности слушать стерео вместо моно или 44 вместо 22).

TEXTREL - это дакая гадость, которая делает текстовый сегмент не совсем
текстовым; фактически dynamic linker вынужден патчить сам текстовый
сегмент, см. dsohowto; в результате атаковать доступный на запись
текстовый сегмент проще, чем недоступный.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Денис Смирнов]

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

On Tue, Jan 08, 2008 at 12:59:39AM +0200, Michael Shigorin wrote:

>>> Помнится, там был ассемблер?
>> Надо ассемблер отрубить.
MS> Ради textrel?  Я бы не стал, но дело твоё.

У меня есть подозрение что на современных процессорах gcc, вполне
возможно, даст лучший результат чем сто лет назад написанная вставка на
asm.

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
 * ott хочет lisp machine 
 * gvy hands emacs to ott 
<ott> gvy это не машинаа
<gvy> ott, самосвал? ;-)
<ott> gvy это тормоз

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

Re: [devel] textrel vs asm

[Michael Shigorin]

On Tue, Jan 08, 2008 at 02:22:01AM +0300, Dmitry V. Levin wrote:
> > Либо разъяснить нам, начиная с упёртого меня, чем так плохо иметь
> > TEXTREL в подобных программах (конечных) и библиотеках, кроме
> > того, что релокации замедляют процесс линковки (причём на 486 
> > со 125-мегабайтным диском меня это не волновало, в отличие от
> > возможности слушать стерео вместо моно или 44 вместо 22).
> TEXTREL - это дакая гадость, которая делает текстовый сегмент
> не совсем текстовым; фактически dynamic linker вынужден патчить
> сам текстовый сегмент, см. dsohowto; в результате атаковать
> доступный на запись текстовый сегмент проще, чем недоступный.

Довод; спасибо, не знал.

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

[devel] [wiki] Re: textrel vs asm

[Michael Shigorin]

On Tue, Jan 08, 2008 at 02:22:01AM +0300, Dmitry V. Levin wrote:
> TEXTREL - это дакая гадость, которая делает текстовый сегмент
> не совсем текстовым; фактически dynamic linker вынужден патчить
> сам текстовый сегмент, см. dsohowto; в результате атаковать
> доступный на запись текстовый сегмент проще, чем недоступный.

Добавил к http://wiki.sisyphus.ru/devel/TextRel

-- 
 ---- WBR, Michael Shigorin <mike@altlinux.ru>
  ------ Linux.Kiev http://www.linux.kiev.ua/

Re: [devel] textrel vs asm

[led]

Tuesday, 08 January 2008 01:22:01 Dmitry V. Levin написав:
> > Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> > Если случай mp3 скорее неактуален большинству аудитории
> > (кроме Lite разве что, хотя для меня на месте майнтейнера
> > и это было бы достаточным поводом не делать прыжков в ширину)
> > -- то всяких недопереоптимизированных кодеков и мультимедийных
> > библиотек у нас вроде как наблюдается.
>
> У нас есть mpeg-декодеры получше чем mpg123.

Например?

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[led]

Tuesday, 08 January 2008 07:14:50 Денис Смирнов написав:
> On Tue, Jan 08, 2008 at 12:59:39AM +0200, Michael Shigorin wrote:
> >>> Помнится, там был ассемблер?
> >>
> >> Надо ассемблер отрубить.
>
> MS> Ради textrel?  Я бы не стал, но дело твоё.
>
> У меня есть подозрение что на современных процессорах gcc, вполне
> возможно, даст лучший результат чем сто лет назад написанная вставка на
> asm.

Вы слишком высокого мнения о gcc и о "результатах", которые он "даёт". Если не 
хотите расстраиваться - лучше не дизасемблируйте и не смотрите:)

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[led]

Sunday, 06 January 2008 21:41:04 Alexey Tourbin написав:
> On Sun, Jan 06, 2008 at 07:37:39PM +0200, Michael Shigorin wrote:
> > On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
> > > > +%ifarch %ix86
> > > > +%set_verify_elf_method textrel=relaxed
> > > > +%endif
> > >
> > > Нет, такое изменение без предварительного обсуждения не годится.
> >
> > Помнится, там был ассемблер?
>
> Надо ассемблер отрубить.  Особенно неграмотный ассемблер.  Ассемблер
> давал ощутимый выигрыш на декодировании mp3 лет десять назад, во времена
> когда был популярен P166MMX.  Теперь процессоры стали настолько мощнее,
> что в деле декодирования mp3 не стоит экономить их загрузку, она всё
> равно мизерная.
>
> Кстати, при большом load затыки в проигрывании mp3 вызывает скорее
> дисковая подсистема ядра, нежели чем что-то ещё. :(  Приходится
> активизировать 'last resort' способ в виде 'sudo renice тры-ты-ты'. :(
>
> И вообще, насколько я понимаю, стандартным способом декодирования mp3
> сейчас является libmad.  В этом смысле оригинальные эти подвыподверты
> mpg123 никакого энтузиазма не вызывают.

Ага, а стандартным" способом декодиравания doc-файлов является od(1). :)

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[led]

Tuesday, 08 January 2008 01:12:53 Dmitry V. Levin написав:
> On Tue, Jan 08, 2008 at 12:59:39AM +0200, Michael Shigorin wrote:
> > On Sun, Jan 06, 2008 at 10:41:04PM +0300, Alexey Tourbin wrote:
> > > > > > +%ifarch %ix86
> > > > > > +%set_verify_elf_method textrel=relaxed
> > > > > > +%endif
> > > > >
> > > > > Нет, такое изменение без предварительного обсуждения не
> > > > > годится.
> > > >
> > > > Помнится, там был ассемблер?
> > >
> > > Надо ассемблер отрубить.
> >
> > Ради textrel?  Я бы не стал, но дело твоё.
>
> Зачем там понадобился _такой_ ассемблер?
>
> > > И вообще, насколько я понимаю, стандартным способом
> > > декодирования mp3 сейчас является libmad.
> >
> > Даже не знаю, xmms играет через libmpg123 вроде...
> >
> > :)
>
> Кстати, это интересный вопрос.
>
> $ apt-cache whatdepends libmad.so.0 |fgrep -c libmad.so.0
> 35
>
> $ apt-cache showpkg libmpg123.so
> Package: libmpg123.so
> Versions:
>
> Reverse Depends:
> Dependencies:
> Provides:
> Reverse Provides:
> kdemultimedia-arts 3.5.8-alt1

А разве libmpg123.so не месяц назад только появился в природе?


-- 
___
Led

Re: [devel] textrel vs asm

[Dmitry V. Levin]

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

On Tue, Jan 08, 2008 at 10:21:01PM +0200, led@altlinux.ru wrote:
> Tuesday, 08 January 2008 01:22:01 Dmitry V. Levin написав:
> > > Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> > > Если случай mp3 скорее неактуален большинству аудитории
> > > (кроме Lite разве что, хотя для меня на месте майнтейнера
> > > и это было бы достаточным поводом не делать прыжков в ширину)
> > > -- то всяких недопереоптимизированных кодеков и мультимедийных
> > > библиотек у нас вроде как наблюдается.
> >
> > У нас есть mpeg-декодеры получше чем mpg123.
> 
> Например?

Я не проводил специального тестирования, но этих декодеров в Сизифе много,
я даже сразу не скажу сколько именно, так что я верю что среди них есть
что-то получше чем этот mpg123.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Tue, Jan 08, 2008 at 10:28:54PM +0200, led@altlinux.ru wrote:
> Tuesday, 08 January 2008 01:12:53 Dmitry V. Levin написав:
[...]
> > Кстати, это интересный вопрос.
> >
> > $ apt-cache whatdepends libmad.so.0 |fgrep -c libmad.so.0
> > 35
> >
> > $ apt-cache showpkg libmpg123.so
> > Package: libmpg123.so
> > Versions:
> >
> > Reverse Depends:
> > Dependencies:
> > Provides:
> > Reverse Provides:
> > kdemultimedia-arts 3.5.8-alt1
> 
> А разве libmpg123.so не месяц назад только появился в природе?

По крайней мере, в Сизифе libmpg123.so находится в не совсем
библиотечной форме.
А что это за нововведение, чем оно лучше многих других?


-- 
ldv

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

Re: [devel] textrel vs asm

[Dmitry V. Levin]

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

On Tue, Jan 08, 2008 at 10:54:07PM +0200, led@altlinux.ru wrote:
> Tuesday, 08 January 2008 22:45:01 Dmitry V. Levin написав:
> > On Tue, Jan 08, 2008 at 10:21:01PM +0200, led@altlinux.ru wrote:
> > > Tuesday, 08 January 2008 01:22:01 Dmitry V. Levin написав:
> > > > > Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> > > > > Если случай mp3 скорее неактуален большинству аудитории
> > > > > (кроме Lite разве что, хотя для меня на месте майнтейнера
> > > > > и это было бы достаточным поводом не делать прыжков в ширину)
> > > > > -- то всяких недопереоптимизированных кодеков и мультимедийных
> > > > > библиотек у нас вроде как наблюдается.
> > > >
> > > > У нас есть mpeg-декодеры получше чем mpg123.
> > >
> > > Например?
> >
> > Я не проводил специального тестирования, но этих декодеров в Сизифе много,
> > я даже сразу не скажу сколько именно, так что я верю что среди них есть
> > что-то получше чем этот mpg123.
> 
> Да, с верой не поспоришь:) Да вот только "много" есть, а "получше" - таки 
> нет:)

Не может быть! :)  Если серьёзно, то лучше посмотреть на результаты
сравнительного тестирования.


-- 
ldv

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

Re: [devel] textrel vs asm

[led]

Tuesday, 08 January 2008 22:45:01 Dmitry V. Levin написав:
> On Tue, Jan 08, 2008 at 10:21:01PM +0200, led@altlinux.ru wrote:
> > Tuesday, 08 January 2008 01:22:01 Dmitry V. Levin написав:
> > > > Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> > > > Если случай mp3 скорее неактуален большинству аудитории
> > > > (кроме Lite разве что, хотя для меня на месте майнтейнера
> > > > и это было бы достаточным поводом не делать прыжков в ширину)
> > > > -- то всяких недопереоптимизированных кодеков и мультимедийных
> > > > библиотек у нас вроде как наблюдается.
> > >
> > > У нас есть mpeg-декодеры получше чем mpg123.
> >
> > Например?
>
> Я не проводил специального тестирования, но этих декодеров в Сизифе много,
> я даже сразу не скажу сколько именно, так что я верю что среди них есть
> что-то получше чем этот mpg123.

Да, с верой не поспоришь:) Да вот только "много" есть, а "получше" - таки 
нет:)

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[led]

Tuesday, 08 January 2008 22:48:38 Dmitry V. Levin написав:
> On Tue, Jan 08, 2008 at 10:28:54PM +0200, led@altlinux.ru wrote:
> > Tuesday, 08 January 2008 01:12:53 Dmitry V. Levin написав:
>
> [...]
>
> > > Кстати, это интересный вопрос.
> > >
> > > $ apt-cache whatdepends libmad.so.0 |fgrep -c libmad.so.0
> > > 35
> > >
> > > $ apt-cache showpkg libmpg123.so
> > > Package: libmpg123.so
> > > Versions:
> > >
> > > Reverse Depends:
> > > Dependencies:
> > > Provides:
> > > Reverse Provides:
> > > kdemultimedia-arts 3.5.8-alt1
> >
> > А разве libmpg123.so не месяц назад только появился в природе?
>
> По крайней мере, в Сизифе libmpg123.so находится в не совсем
> библиотечной форме.
> А что это за нововведение, чем оно лучше многих других?

mpg123 всегда был лучшым свободным (более-менее) декодером mp3 и таким он и 
сейчас остаётся. Раньше у него был ТОЛЬКО бинарник mpg123 и лицензия 
не "совсем хорошая". В mplayer очень давно уже используется 
сильно "похаченный" и оптимизированный код из mpg123 в качестве штатного 
mp3-декодера. Сейчас появилась libmpg123 и с "хорошей" лицензией:)

P.S. Даже виндузятники (те, которые вынуждены слушать приличную музыку в mp3), 
использовали mpg123-плагин к Winamp'у, так как лучшего, пожалуй, и нет (по 
крайней мере, из бесплатных).

-- 
___
Led

Re: [devel] textrel vs asm

[led]

Tuesday, 08 January 2008 22:53:56 Dmitry V. Levin написав:
> On Tue, Jan 08, 2008 at 10:54:07PM +0200, led@altlinux.ru wrote:
> > Tuesday, 08 January 2008 22:45:01 Dmitry V. Levin написав:
> > > On Tue, Jan 08, 2008 at 10:21:01PM +0200, led@altlinux.ru wrote:
> > > > Tuesday, 08 January 2008 01:22:01 Dmitry V. Levin написав:
> > > > > > Я до сих пор не вижу в TEXTREL _такой_ большой проблемы.
> > > > > > Если случай mp3 скорее неактуален большинству аудитории
> > > > > > (кроме Lite разве что, хотя для меня на месте майнтейнера
> > > > > > и это было бы достаточным поводом не делать прыжков в ширину)
> > > > > > -- то всяких недопереоптимизированных кодеков и мультимедийных
> > > > > > библиотек у нас вроде как наблюдается.
> > > > >
> > > > > У нас есть mpeg-декодеры получше чем mpg123.
> > > >
> > > > Например?
> > >
> > > Я не проводил специального тестирования, но этих декодеров в Сизифе
> > > много, я даже сразу не скажу сколько именно, так что я верю что среди
> > > них есть что-то получше чем этот mpg123.
> >
> > Да, с верой не поспоришь:) Да вот только "много" есть, а "получше" - таки
> > нет:)
>
> Не может быть! :)  Если серьёзно, то лучше посмотреть на результаты
> сравнительного тестирования.

...или отслеживать эти "результаты тестирования" и отзывы, а также сравнивать 
самому последние лет 7:)

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Morsov]

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

On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
 >  
> > +%ifarch %ix86
> > +%set_verify_elf_method textrel=relaxed
> > +%endif
> 
> Нет, такое изменение без предварительного обсуждения не годится.
Насколько я понял это из-за загружаемых библиотек, .so которые mpg123
подгружает во время работы (через libltdl) для вывода звука (oss,alsa,esd...)
- он с ними не линкуется.

asm вставок в коде я не заметил.



-- 
С уважением,
Алексей Морсов
программист ЗАО "ИК "Риком-Траст"

Jabber: samurai@www.fondmarket.ru
ICQ: 196766290
www.ricom.ru
www.fondmarket.ru

ALT Linux Team Member
email: swi@altlinux.ru
web: www.altlinux.ru, www.sisyphus.ru



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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Wed, Jan 09, 2008 at 12:50:57PM +0300, Alexey Morsov wrote:
> On Sun, Jan 06, 2008 at 07:12:26PM +0300, Dmitry V. Levin wrote:
>  >  
> > > +%ifarch %ix86
> > > +%set_verify_elf_method textrel=relaxed
> > > +%endif
> > 
> > Нет, такое изменение без предварительного обсуждения не годится.
> Насколько я понял это из-за загружаемых библиотек, .so которые mpg123
> подгружает во время работы (через libltdl) для вывода звука (oss,alsa,esd...)
> - он с ними не линкуется.

i586-alt-linux-gcc -shared  .libs/compat.o .libs/parse.o .libs/frame.o
.libs/format.o .libs/decode_2to1.o .libs/decode_4to1.o
.libs/decode_ntom.o .libs/equalizer.o .libs/icy.o .libs/id3.o
.libs/layer1.o .libs/layer2.o .libs/layer3.o .libs/optimize.o
.libs/readers.o .libs/tabinit.o .libs/stringbuf.o .libs/libmpg123.o
.libs/decode_i386.o .libs/dct64_i386.o .libs/decode_i586_dither.o
.libs/dnoise.o  -lm  -march=i586 -mtune=i686 -Wl,-soname
-Wl,libmpg123.so.0 -Wl,-version-script -Wl,.libs/libmpg123.ver -o
.libs/libmpg123.so.0.0.0
/usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.

> asm вставок в коде я не заметил.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Morsov]

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

On Wed, Jan 09, 2008 at 01:05:11PM +0300, Dmitry V. Levin wrote:
> 
> i586-alt-linux-gcc -shared  .libs/compat.o .libs/parse.o .libs/frame.o
> .libs/format.o .libs/decode_2to1.o .libs/decode_4to1.o
> .libs/decode_ntom.o .libs/equalizer.o .libs/icy.o .libs/id3.o
> .libs/layer1.o .libs/layer2.o .libs/layer3.o .libs/optimize.o
> .libs/readers.o .libs/tabinit.o .libs/stringbuf.o .libs/libmpg123.o
> .libs/decode_i386.o .libs/dct64_i386.o .libs/decode_i586_dither.o
> .libs/dnoise.o  -lm  -march=i586 -mtune=i686 -Wl,-soname
> -Wl,libmpg123.so.0 -Wl,-version-script -Wl,.libs/libmpg123.ver -o
> .libs/libmpg123.so.0.0.0
> /usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.
> 
> > asm вставок в коде я не заметил.

Я не очень силен в данных вещах. Т.е. хотите сказать что к обращениям к
коды коорый в неслинкованных .so это отношения не имеет?


-- 
С уважением,
Алексей Морсов
программист ЗАО "ИК "Риком-Траст"

Jabber: samurai@www.fondmarket.ru
ICQ: 196766290
www.ricom.ru
www.fondmarket.ru

ALT Linux Team Member
email: swi@altlinux.ru
web: www.altlinux.ru, www.sisyphus.ru



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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Wed, Jan 09, 2008 at 01:17:26PM +0300, Alexey Morsov wrote:
> On Wed, Jan 09, 2008 at 01:05:11PM +0300, Dmitry V. Levin wrote:
> > 
> > i586-alt-linux-gcc -shared  .libs/compat.o .libs/parse.o .libs/frame.o
> > .libs/format.o .libs/decode_2to1.o .libs/decode_4to1.o
> > .libs/decode_ntom.o .libs/equalizer.o .libs/icy.o .libs/id3.o
> > .libs/layer1.o .libs/layer2.o .libs/layer3.o .libs/optimize.o
> > .libs/readers.o .libs/tabinit.o .libs/stringbuf.o .libs/libmpg123.o
> > .libs/decode_i386.o .libs/dct64_i386.o .libs/decode_i586_dither.o
> > .libs/dnoise.o  -lm  -march=i586 -mtune=i686 -Wl,-soname
> > -Wl,libmpg123.so.0 -Wl,-version-script -Wl,.libs/libmpg123.ver -o
> > .libs/libmpg123.so.0.0.0
> > /usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.
> > 
> > > asm вставок в коде я не заметил.
> 
> Я не очень силен в данных вещах. Т.е. хотите сказать что к обращениям к
> коды коорый в неслинкованных .so это отношения не имеет?

Вы лог сборки mpg123 вообще смотрели?

Там написано примерно следующее:
i586-alt-linux-gcc -pipe -Wall -O2 -march=i586 -mtune=i686
-fomit-frame-pointer -ffast-math -frename-registers -I. -I. -I../../src -c
decode_i586_dither.S -fPIC -DPIC -o .libs/decode_i586_dither.o

В результате
$ gcc -shared .libs/decode_i586_dither.o
/usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Wed, Jan 09, 2008 at 01:34:51PM +0300, Dmitry V. Levin wrote:
> On Wed, Jan 09, 2008 at 01:17:26PM +0300, Alexey Morsov wrote:
> > On Wed, Jan 09, 2008 at 01:05:11PM +0300, Dmitry V. Levin wrote:
> > > 
> > > i586-alt-linux-gcc -shared  .libs/compat.o .libs/parse.o .libs/frame.o
> > > .libs/format.o .libs/decode_2to1.o .libs/decode_4to1.o
> > > .libs/decode_ntom.o .libs/equalizer.o .libs/icy.o .libs/id3.o
> > > .libs/layer1.o .libs/layer2.o .libs/layer3.o .libs/optimize.o
> > > .libs/readers.o .libs/tabinit.o .libs/stringbuf.o .libs/libmpg123.o
> > > .libs/decode_i386.o .libs/dct64_i386.o .libs/decode_i586_dither.o
> > > .libs/dnoise.o  -lm  -march=i586 -mtune=i686 -Wl,-soname
> > > -Wl,libmpg123.so.0 -Wl,-version-script -Wl,.libs/libmpg123.ver -o
> > > .libs/libmpg123.so.0.0.0
> > > /usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.
> > > 
> > > > asm вставок в коде я не заметил.
> > 
> > Я не очень силен в данных вещах. Т.е. хотите сказать что к обращениям к
> > коды коорый в неслинкованных .so это отношения не имеет?
> 
> Вы лог сборки mpg123 вообще смотрели?
> 
> Там написано примерно следующее:
> i586-alt-linux-gcc -pipe -Wall -O2 -march=i586 -mtune=i686
> -fomit-frame-pointer -ffast-math -frename-registers -I. -I. -I../../src -c
> decode_i586_dither.S -fPIC -DPIC -o .libs/decode_i586_dither.o
> 
> В результате
> $ gcc -shared .libs/decode_i586_dither.o
> /usr/bin/ld: warning: creating a DT_TEXTREL in a shared object.

--with-cpu=i586_dither вы написали самостоятельно?
Если да, то где обоснование или хотя бы упоминание,
если нет, то где ссылка на источник?

Я посмотрел на вывод от
srpmcmp mpg123-0.59s-alt0.9.src.rpm mpg123-1.0.1-alt1.src.rpm
Ой, и зачем я это сделал?..


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Morsov]

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

On Wed, Jan 09, 2008 at 01:53:38PM +0300, Dmitry V. Levin wrote:
> --with-cpu=i586_dither вы написали самостоятельно?
> Если да, то где обоснование или хотя бы упоминание,
> если нет, то где ссылка на источник?
сам. :( 
Просто 
--with-cpu=i586_dither  Use code optimized for i586 processors with
dithering (noise shaping),

вот и подумал что хорошо б оптимизации (видимо плохо подумал).

> 
> Я посмотрел на вывод от
> srpmcmp mpg123-0.59s-alt0.9.src.rpm mpg123-1.0.1-alt1.src.rpm
> Ой, и зачем я это сделал?..
Ну... помнится секцию %build я еще в 0.66 старую выкинул. Ибо у апстрима 
сменилось все в сборке и та что была не подходила...

:( Вообщем я понял что налажал :(
Извините. Уж так обрадовался что оно пашет... 


-- 
С уважением,
Алексей Морсов
программист ЗАО "ИК "Риком-Траст"

Jabber: samurai@www.fondmarket.ru
ICQ: 196766290
www.ricom.ru
www.fondmarket.ru

ALT Linux Team Member
email: swi@altlinux.ru
web: www.altlinux.ru, www.sisyphus.ru



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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Wed, Jan 09, 2008 at 02:16:11PM +0300, Alexey Morsov wrote:
> On Wed, Jan 09, 2008 at 01:53:38PM +0300, Dmitry V. Levin wrote:
> > --with-cpu=i586_dither вы написали самостоятельно?
> > Если да, то где обоснование или хотя бы упоминание,
> > если нет, то где ссылка на источник?
> сам. :( 
> Просто 
> --with-cpu=i586_dither  Use code optimized for i586 processors with
> dithering (noise shaping),
> 
> вот и подумал что хорошо б оптимизации (видимо плохо подумал).

Я думаю что лучше пока отключить эту оптимизацию.
Там по умолчанию для i586 есть ассемблерная оптимизация, которая не
порождает TEXTREL'ов, но оптимизация i586_dither по умолчанию не включена,
в файле INSTALL написано почему.

> > Я посмотрел на вывод от
> > srpmcmp mpg123-0.59s-alt0.9.src.rpm mpg123-1.0.1-alt1.src.rpm
> > Ой, и зачем я это сделал?..
> Ну... помнится секцию %build я еще в 0.66 старую выкинул. Ибо у апстрима 
> сменилось все в сборке и та что была не подходила...
> 
> :( Вообщем я понял что налажал :(
> Извините. Уж так обрадовался что оно пашет... 

Тогда давайте будем исправлять.
У вас http://git.altlinux.org/people/swi/packages/mpg123.git
находится в актуальном состоянии?


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Morsov]

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

On Wed, Jan 09, 2008 at 03:35:59PM +0300, Dmitry V. Levin wrote:
> 
> Я думаю что лучше пока отключить эту оптимизацию.
> Там по умолчанию для i586 есть ассемблерная оптимизация, которая не
> порождает TEXTREL'ов, но оптимизация i586_dither по умолчанию не включена,
> в файле INSTALL написано почему.
Угу. т.е. все таки =i586? не generic_fpu ? для x86_64 собирается именно
generic_fpu.
> 
> Тогда давайте будем исправлять.
> У вас http://git.altlinux.org/people/swi/packages/mpg123.git
> находится в актуальном состоянии?
Да. из него и собирал.


-- 
С уважением,
Алексей Морсов
программист ЗАО "ИК "Риком-Траст"

Jabber: samurai@www.fondmarket.ru
ICQ: 196766290
www.ricom.ru
www.fondmarket.ru

ALT Linux Team Member
email: swi@altlinux.ru
web: www.altlinux.ru, www.sisyphus.ru

NP: Rush - Far Cry


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

Re: [devel] [git update] packages/mpg123: heads/master

[Dmitry V. Levin]

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

On Wed, Jan 09, 2008 at 03:57:53PM +0300, Alexey Morsov wrote:
> On Wed, Jan 09, 2008 at 03:35:59PM +0300, Dmitry V. Levin wrote:
> > 
> > Я думаю что лучше пока отключить эту оптимизацию.
> > Там по умолчанию для i586 есть ассемблерная оптимизация, которая не
> > порождает TEXTREL'ов, но оптимизация i586_dither по умолчанию не включена,
> > в файле INSTALL написано почему.
> Угу. т.е. все таки =i586? не generic_fpu ? для x86_64 собирается именно
> generic_fpu.

%ifarch там тоже не нужен.
В configure.ac уже есть соответствующий код, посмотрите сами.

> > Тогда давайте будем исправлять.
> > У вас http://git.altlinux.org/people/swi/packages/mpg123.git
> > находится в актуальном состоянии?
> Да. из него и собирал.

Тогда намекните Славе, чтобы он прошёлся по спекфайлу.


-- 
ldv

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

Re: [devel] [git update] packages/mpg123: heads/master

[Alexey Morsov]

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

On Wed, Jan 09, 2008 at 04:43:14PM +0300, Dmitry V. Levin wrote:
> %ifarch там тоже не нужен.
> В configure.ac уже есть соответствующий код, посмотрите сами.
Прибрал, пересобрал. 

> 
> > > Тогда давайте будем исправлять.
> > > У вас http://git.altlinux.org/people/swi/packages/mpg123.git
> > > находится в актуальном состоянии?
> > Да. из него и собирал.
> 
> Тогда намекните Славе, чтобы он прошёлся по спекфайлу.
Э. простите какому именно? Тот что Slava Semushin ? :)


-- 
С уважением,
Алексей Морсов
программист ЗАО "ИК "Риком-Траст"

Jabber: samurai@www.fondmarket.ru
ICQ: 196766290
www.ricom.ru
www.fondmarket.ru

ALT Linux Team Member
email: swi@altlinux.ru
web: www.altlinux.ru, www.sisyphus.ru

NP: Steve Hackett - Darktown Riot


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

Re: [devel] [git update] packages/mpg123: heads/master

[Konstantin A. Lepikhov]

<цитата от="led@altlinux.ru">
<skip>
> mpg123 всегда был лучшым свободным (более-менее) декодером mp3 и таким он
> и
> сейчас остаётся. Раньше у него был ТОЛЬКО бинарник mpg123 и лицензия
> не "совсем хорошая". В mplayer очень давно уже используется
> сильно "похаченный" и оптимизированный код из mpg123 в качестве штатного
> mp3-декодера. Сейчас появилась libmpg123 и с "хорошей" лицензией:)
о качестве кода mplayer уже давно слагают легенды. Они даже на с пишут как
на ассемблере. Поэтому пример неудачный.

>
> P.S. Даже виндузятники (те, которые вынуждены слушать приличную музыку в
> mp3),
> использовали mpg123-плагин к Winamp'у, так как лучшего, пожалуй, и нет (по
> крайней мере, из бесплатных).
виндузятникам простительно не знать про libmad и madplay.

-- 
WBR et al.

Re: [devel] [git update] packages/mpg123: heads/master

[led]

Wednesday, 09 January 2008 16:37:11 Konstantin A. Lepikhov написав:
> <цитата от="led@altlinux.ru">
> <skip>
>
> > mpg123 всегда был лучшым свободным (более-менее) декодером mp3 и таким он
> > и
> > сейчас остаётся. Раньше у него был ТОЛЬКО бинарник mpg123 и лицензия
> > не "совсем хорошая". В mplayer очень давно уже используется
> > сильно "похаченный" и оптимизированный код из mpg123 в качестве штатного
> > mp3-декодера. Сейчас появилась libmpg123 и с "хорошей" лицензией:)
>
> о качестве кода mplayer уже давно слагают легенды. Они даже на с пишут как
> на ассемблере. Поэтому пример неудачный.

А ты код читай, а не легенды о нём:)

>
> > P.S. Даже виндузятники (те, которые вынуждены слушать приличную музыку в
> > mp3),
> > использовали mpg123-плагин к Winamp'у, так как лучшего, пожалуй, и нет
> > (по крайней мере, из бесплатных).
>
> виндузятникам простительно не знать про libmad и madplay.

Тебе непростительно не знать, что такое libmad и дл чего он был сделан:)

-- 
___
Led

Re: [devel] [git update] packages/mpg123: heads/master

[Konstantin A. Lepikhov]

<цитата от="led@altlinux.ru">
> Wednesday, 09 January 2008 16:37:11 Konstantin A. Lepikhov написав:
>> <цитата от="led@altlinux.ru">
>> <skip>
>>
>> > mpg123 всегда был лучшым свободным (более-менее) декодером mp3 и таким
>> он
>> > и
>> > сейчас остаётся. Раньше у него был ТОЛЬКО бинарник mpg123 и лицензия
>> > не "совсем хорошая". В mplayer очень давно уже используется
>> > сильно "похаченный" и оптимизированный код из mpg123 в качестве
>> штатного
>> > mp3-декодера. Сейчас появилась libmpg123 и с "хорошей" лицензией:)
>>
>> о качестве кода mplayer уже давно слагают легенды. Они даже на с пишут
>> как
>> на ассемблере. Поэтому пример неудачный.
>
> А ты код читай, а не легенды о нём:)
ниасилил. А многие как, nick вообще ушли, хлопнув дверью.

>
>>
>> > P.S. Даже виндузятники (те, которые вынуждены слушать приличную музыку
>> в
>> > mp3),
>> > использовали mpg123-плагин к Winamp'у, так как лучшего, пожалуй, и нет
>> > (по крайней мере, из бесплатных).
>>
>> виндузятникам простительно не знать про libmad и madplay.
>
> Тебе непростительно не знать, что такое libmad и дл чего он был сделан:)
так открой же тайну, о мудрый джедай!

-- 
WBR et al.

Re: [devel] [git update] packages/mpg123: heads/master

[Денис Смирнов]

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

On Tue, Jan 08, 2008 at 10:24:40PM +0200, led@altlinux.ru wrote:

> Вы слишком высокого мнения о gcc и о "результатах", которые он "даёт". Если не 
> хотите расстраиваться - лучше не дизасемблируйте и не смотрите:)

Я частенько смотрю на вывод gcc -S. Иногда меня вывод очень радует, иногда
просто пугает :)

-- 
С уважением, Денис

http://freesource.info
----------------------------------------------------------------------------
"Нет программы -- нет проблемы" (c)
		-- mike in #5012

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