[devel] lvm2 + device-mapper хочу обновить.

[devel] lvm2 + device-mapper хочу обновить.

[Pavlov Konstantin]

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

Всем привет.

У меня готовы сборки новых LVM2 = 2.02.09 и device-mapper = 1.02.09.
Из нюансов:
Изменен version script для libdevmapper. В device-mapper 1.02.02 была функция 
dm_strndup, которая позже (в 1.02.03) была удалена. Опять же, завязанных на 
эту функцию пакетов у нас всего два: LVM2 и dmsetup (который собирается из 
device-mapper):

$ apt-cache whatdepends "libdevmapper.so.1.00(DEVMAPPER_1.02.02)"
<libdevmapper.so.1.00(DEVMAPPER_1.02.02)>
  lvm2-2.02.01-alt2
    Требует: <libdevmapper.so.1.00(DEVMAPPER_1.02.02)>
      libdevmapper-1.02.02-alt3
  dmsetup-1.02.02-alt3
    Требует: <libdevmapper.so.1.00(DEVMAPPER_1.02.02)>
      libdevmapper-1.02.02-alt3

Т.е. фактически от этого изменения никто не пострадает, если device-mapper и 
lvm2 будут собраны в одну транзакцию.

Багу (ну, enhancement) на device-mapper я уже повесил (#10006). 
Если у кого есть возражения -- велкам.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

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

Re: [devel] device mapper version script

[Pavlov Konstantin]

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

On Friday 15 September 2006 19:55, Pavlov Konstantin wrote:
> Всем привет.
>
> У меня готовы сборки новых LVM2 = 2.02.09 и device-mapper = 1.02.09.
> Из нюансов:
> Изменен version script для libdevmapper. 

А вот и diff:

$ diff -urN ../device-mapper-1.02.02-alt-sym device-mapper-1.02.09-alt-sym
--- ../device-mapper-1.02.02-alt-sym    2006-09-15 20:07:18 +0400
+++ device-mapper-1.02.09-alt-sym       2006-09-15 20:07:22 +0400
@@ -29,7 +29,6 @@
                dm_task_set_ro;
                dm_task_set_sector;
                dm_task_set_uuid;
-       local:
                *;
 };

@@ -116,3 +115,27 @@
                dm_tree_preload_children;
                dm_tree_suspend_children;
 };
+
+DEVMAPPER_1.02.09 {
+    global:
+        _align_chunk;
+        _fini;
+        _init;
+        _new_chunk;
+        add_dev_node;
+        create_dir;
+        create_target;
+        dm_check_version;
+        dm_log;
+        dm_segtypes;
+        dm_strdup_aux;
+        dm_task_set_geometry;
+        dm_task_set_gid;
+        dm_task_set_mode;
+        dm_task_set_uid;
+        dm_tree_skip_lockfs;
+        rename_dev_node;
+        rm_dev_node;
+        update_devs;
+};
+

Правильно ли это?

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

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

Re: [devel] device mapper version script

[Sergey Vlasov]

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

On Fri, Sep 15, 2006 at 08:06:33PM +0400, Pavlov Konstantin wrote:
> On Friday 15 September 2006 19:55, Pavlov Konstantin wrote:
> > Всем привет.
> >
> > У меня готовы сборки новых LVM2 = 2.02.09 и device-mapper = 1.02.09.
> > Из нюансов:
> > Изменен version script для libdevmapper. 
> 
> А вот и diff:
> 
> $ diff -urN ../device-mapper-1.02.02-alt-sym device-mapper-1.02.09-alt-sym
> --- ../device-mapper-1.02.02-alt-sym    2006-09-15 20:07:18 +0400
> +++ device-mapper-1.02.09-alt-sym       2006-09-15 20:07:22 +0400
> @@ -29,7 +29,6 @@
>                 dm_task_set_ro;
>                 dm_task_set_sector;
>                 dm_task_set_uuid;
> -       local:

В окончательном пакете это делать не нужно - я предложил сделать это
временно, чтобы пакет хоть как-то собрался, и к нему можно было бы
применить утилиты для сравнения набора экспортируемых символов.

>                 *;
>  };
> 
> @@ -116,3 +115,27 @@

Где-то в промежутке должно было бы быть удаление dm_strdup (хотя, как
оказалось, несуществующие символы ld просто игнорирует).

Вообще история с dm_strdup следующая:

http://www.redhat.com/archives/dm-devel/2006-February/msg00057.html

Т.е., upstream сознательно разломал ABI.

>                 dm_tree_preload_children;
>                 dm_tree_suspend_children;
>  };
> +
> +DEVMAPPER_1.02.09 {
> +    global:
> +        _align_chunk;
> +        _fini;
> +        _init;
> +        _new_chunk;
> +        add_dev_node;
> +        create_dir;
> +        create_target;
> +        dm_check_version;
> +        dm_log;
> +        dm_segtypes;
> +        dm_strdup_aux;
> +        dm_task_set_geometry;
> +        dm_task_set_gid;
> +        dm_task_set_mode;
> +        dm_task_set_uid;
> +        dm_tree_skip_lockfs;
> +        rename_dev_node;
> +        rm_dev_node;
> +        update_devs;
> +};

Здесь два замечания:

1) Далеко не все эти символы нужно экспортировать - скорее всего,
   здесь должны быть только символы вида dm_*.  Более того, в
   оригинальных исходниках уже есть файл .exported_symbols, который
   применяется для явного указания экспортируемых символов (правда,
   там всё загоняется в одну версию с именем Base).  Очевидно, что в
   списке символов с версиями должны быть те и только те символы,
   которые перечислены в файле .exported_symbols (возможно, это даже
   не все символы вида dm_*).

2) В version script можно ещё указывать зависимости между версиями -
   написать, что новая версия базируется на предыдущей.  Хотя я не
   совсем понимаю смысл этого действия для варианта GNU - всё равно
   переносить символы из одной версии в другую нельзя; похоже, это
   было нужно в основном для исходного варианта от Sun.

   Впрочем, в существующем файле для версии 1.02.02 эти зависимости
   уже отсутствуют - не уверен, что от их добавления ничего не
   сломается.

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

Re: [devel] lvm2 + device-mapper хочу обновить.

[Dmitry V. Levin]

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

On Fri, Sep 15, 2006 at 07:55:45PM +0400, Pavlov Konstantin wrote:
> Всем привет.
> 
> У меня готовы сборки новых LVM2 = 2.02.09 и device-mapper = 1.02.09.
> Из нюансов:
> Изменен version script для libdevmapper.

В таком виде этот скрипт непригоден для публикации пакета.

[...]
> Багу (ну, enhancement) на device-mapper я уже повесил (#10006). 

Будет хорошо, если сперва вы его доработаете.


-- 
ldv

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

Re: [devel] device mapper version script

[Dmitry V. Levin]

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

On Fri, Sep 15, 2006 at 09:28:44PM +0400, Sergey Vlasov wrote:
[...]
> Т.е., upstream сознательно разломал ABI.

Вопрос немного в сторону: что, upstream совсем невменяем, что не в
состоянии самостоятельно поддерживать обратную совместимость и versioning?
Вроде бы и Drepper недалеко...


-- 
ldv

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

Re: [devel] device mapper version script

[Pavlov Konstantin]

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

On Friday 15 September 2006 21:28, Sergey Vlasov wrote:

> В окончательном пакете это делать не нужно - я предложил сделать это
> временно, чтобы пакет хоть как-то собрался, и к нему можно было бы
> применить утилиты для сравнения набора экспортируемых символов.

А, OK.

> >                 *;
> >  };
> >
> > @@ -116,3 +115,27 @@
>
> Где-то в промежутке должно было бы быть удаление dm_strdup (хотя, как
> оказалось, несуществующие символы ld просто игнорирует).
>
> Вообще история с dm_strdup следующая:
>
> http://www.redhat.com/archives/dm-devel/2006-February/msg00057.html
>
> Т.е., upstream сознательно разломал ABI.

Да, это тоже понятно.

> >                 dm_tree_preload_children;
> >                 dm_tree_suspend_children;
> >  };
> > +
> > +DEVMAPPER_1.02.09 {
> > +    global:
> > +        _align_chunk;
> > +        _fini;
> > +        _init;
> > +        _new_chunk;
> > +        add_dev_node;
> > +        create_dir;
> > +        create_target;
> > +        dm_check_version;
> > +        dm_log;
> > +        dm_segtypes;
> > +        dm_strdup_aux;
> > +        dm_task_set_geometry;
> > +        dm_task_set_gid;
> > +        dm_task_set_mode;
> > +        dm_task_set_uid;
> > +        dm_tree_skip_lockfs;
> > +        rename_dev_node;
> > +        rm_dev_node;
> > +        update_devs;
> > +};
>
> Здесь два замечания:
>
> 1) Далеко не все эти символы нужно экспортировать - скорее всего,
>    здесь должны быть только символы вида dm_*.  Более того, в
>    оригинальных исходниках уже есть файл .exported_symbols, который
>    применяется для явного указания экспортируемых символов (правда,
>    там всё загоняется в одну версию с именем Base).  Очевидно, что в
>    списке символов с версиями должны быть те и только те символы,
>    которые перечислены в файле .exported_symbols (возможно, это даже
>    не все символы вида dm_*).
>
> 2) В version script можно ещё указывать зависимости между версиями -
>    написать, что новая версия базируется на предыдущей.  Хотя я не
>    совсем понимаю смысл этого действия для варианта GNU - всё равно
>    переносить символы из одной версии в другую нельзя; похоже, это
>    было нужно в основном для исходного варианта от Sun.
>
>    Впрочем, в существующем файле для версии 1.02.02 эти зависимости
>    уже отсутствуют - не уверен, что от их добавления ничего не
>    сломается.

Понятно, принял к сведению.

On Saturday 16 September 2006 00:48, Dmitry V. Levin wrote:
> Вопрос немного в сторону: что, upstream совсем невменяем, что не в
> состоянии самостоятельно поддерживать обратную совместимость и versioning?
> Вроде бы и Drepper недалеко...

Это я проверю. Собственно, у меня почему-то есть предубеждение, что не-мой 
пакет запакован "правильно". Хотя пора бы уже понять обратное. :)

> В таком виде этот скрипт непригоден для публикации пакета.
>
> [...]
>
> > Багу (ну, enhancement) на device-mapper я уже повесил (#10006).
>
> Будет хорошо, если сперва вы его доработаете.

Обязательно, для того сюда и написал.

(Это мое первое столкнование с подобного рода вещами).

Спасибо.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

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

Re: [devel] device mapper version script

[Dmitry V. Levin]

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

On Fri, Sep 15, 2006 at 09:28:44PM +0400, Sergey Vlasov wrote:
> 1) Далеко не все эти символы нужно экспортировать - скорее всего,
>    здесь должны быть только символы вида dm_*.  Более того, в
>    оригинальных исходниках уже есть файл .exported_symbols, который
>    применяется для явного указания экспортируемых символов (правда,
>    там всё загоняется в одну версию с именем Base).  Очевидно, что в
>    списке символов с версиями должны быть те и только те символы,
>    которые перечислены в файле .exported_symbols (возможно, это даже
>    не все символы вида dm_*).

Конечно, ничего помимо перечисленного в .exported_symbols экспортировать
не надо.

> 2) В version script можно ещё указывать зависимости между версиями -
>    написать, что новая версия базируется на предыдущей.  Хотя я не
>    совсем понимаю смысл этого действия для варианта GNU - всё равно
>    переносить символы из одной версии в другую нельзя; похоже, это
>    было нужно в основном для исходного варианта от Sun.

Я почему-то этого не сделал.  Не помню почему.

>    Впрочем, в существующем файле для версии 1.02.02 эти зависимости
>    уже отсутствуют - не уверен, что от их добавления ничего не
>    сломается.

Можно добавить, сломаться не должно.


-- 
ldv

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

Re: [devel] device mapper version script

[Dmitry V. Levin]

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

On Sat, Sep 16, 2006 at 02:12:19AM +0400, Pavlov Konstantin wrote:
> On Saturday 16 September 2006 00:48, Dmitry V. Levin wrote:
> > Вопрос немного в сторону: что, upstream совсем невменяем, что не в
> > состоянии самостоятельно поддерживать обратную совместимость и versioning?
> > Вроде бы и Drepper недалеко...
> 
> Это я проверю.

Интересно, как вы планируете это сделать? :)

> Собственно, у меня почему-то есть предубеждение, что не-мой 
> пакет запакован "правильно". Хотя пора бы уже понять обратное. :)

Вообще-то versioning в этом пакете сделал я для того, чтобы не надо было
пересобирать пакет lilo по каждому чиху.  Надеюсь что для 1.02.02
versioning адекватен.  Для обновления на следующую версию нужно просто
завести ещё одну версию в version script'е и добавить туда новые символы
из .exported_symbols.  А вот как быть с символами, которые upstream
перестал экспортировать, я не знаю, это зависит.


-- 
ldv

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

Re: [devel] device mapper version script

[Pavlov Konstantin]

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

On Saturday 16 September 2006 02:31, Dmitry V. Levin wrote:
> > > Вопрос немного в сторону: что, upstream совсем невменяем, что не в
> > > состоянии самостоятельно поддерживать обратную совместимость и
> > > versioning? Вроде бы и Drepper недалеко...
> >
> > Это я проверю.
>
> Интересно, как вы планируете это сделать? :)

Сравнить "наш" versioning и "их", к примеру. 

(Я не удаленность Дреппера от апстрима проверять собрался :)

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

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

Re: [devel] device mapper version script

[Dmitry V. Levin]

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

On Sat, Sep 16, 2006 at 02:52:46AM +0400, Pavlov Konstantin wrote:
> On Saturday 16 September 2006 02:31, Dmitry V. Levin wrote:
> > > > Вопрос немного в сторону: что, upstream совсем невменяем, что не в
> > > > состоянии самостоятельно поддерживать обратную совместимость и
> > > > versioning? Вроде бы и Drepper недалеко...
> > >
> > > Это я проверю.
> >
> > Интересно, как вы планируете это сделать? :)
> 
> Сравнить "наш" versioning и "их", к примеру. 

Нет смысла сравнивать, versioning заведомо отличается.

upstream позволяет себе менять soname с каждой минорной версией:
LIB_VERSION := $(shell cat $(top_srcdir)/VERSION | \
                awk -F '.' '{printf "%s.%s",$$1,$$2}')

Я не понимаю, почему они так себя ведут, но меня это не устраивает.

> (Я не удаленность Дреппера от апстрима проверять собрался :)

А жаль, боюсь что всё дело в этом. :)


-- 
ldv

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

Re: [devel] device mapper version script

[Sergey Vlasov]

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

On Sat, Sep 16, 2006 at 02:31:08AM +0400, Dmitry V. Levin wrote:
> Вообще-то versioning в этом пакете сделал я для того, чтобы не надо было
> пересобирать пакет lilo по каждому чиху.  Надеюсь что для 1.02.02
> versioning адекватен.  Для обновления на следующую версию нужно просто
> завести ещё одну версию в version script'е и добавить туда новые символы
> из .exported_symbols.  А вот как быть с символами, которые upstream
> перестал экспортировать, я не знаю, это зависит.

Формально в данном случае надо полностью удалять из библиотеки версию
DEVMAPPER_1.02.02 (а символы оттуда перенести в более позднюю версию).
Вроде бы на текущий момент это не вызовет больших разрушений,
поскольку этой версией пользуется только пакет lvm2, для обновления
которого всё это и затеяно.

Возможно, имеет смысл пройтись по промежуточным релизам, просмотреть
изменения в .exported_symbols и завести для них отдельные версии (на
случай, если в будущем разработчики опять выкинут что-то подобное).

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

Re: [devel] device mapper version script

[Pavlov Konstantin]

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

Всем привет еще раз.

В общем, таки добрался я еще раз посмотреть device-mapper и lvm2.
Что из этого вышло, можно видеть в git.alt/people/thresh/device-mapper.git и 
git.alt/people/thresh/lvm2.git

device-mapper версии 1.02.12, lvm2 версии 2.02.12.

-- 
Pavlov Konstantin,
ALT Linux Team,
VideoLAN Team,
jid: thresh@altlinux.org

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