[devel] u: kernel-feat-swsusp-alt2 и suspend-alt3

[devel] u: kernel-feat-swsusp-alt2 и suspend-alt3

[Alex Yustasov]

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

Здравствуйте!
В инкоминг/сизифус отправились 

4b48f41ed9d96f95fb4d1ad405b67c06  kernel-feat-swsusp-2003.10.16-alt2.src.rpm
79106fc90c700b2701b0407b5ff12f22  suspend-0.16-alt3.src.rpm

kernel-feat-swsusp - патч 2.0-rc2

suspend:
Исправил packager в changelog alt2. Прошу прощения у Дмитрия Левина.
Просто у меня не собирался пакет в hasher'e. Выдавал не найден packager.
Я подумал, что это из-за того, что моей подписи нет в alt-gpgkeys.
Взял нормального packager'a. Но оказалось, что я вызывал 
hasher builddir suspend.spec

И еще одна проблема. Если повесить USB flash-ку на autofs, и монтировать
с опцией async (извращение, наверное), то после записи больших данных и 
# sync
и сразу
# service usb stop
модуль usb-storage не выгружается (занят), а uhci (usb-ohci, usb-uhci) 
выгружаются. Система зависает. Этого не происходит, если монтировать
с опцией sync, или async без autofs.
Добавил в /etc/suspend.conf параметр SWSUSP_MOUNT_USING_AUTOFS. 
Если равно "yes", то после sync будет пауза в 7 сек.

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

[devel] Re: u: kernel-feat-swsusp-alt2 и suspend-alt3

[Michael Shigorin]

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

On Thu, Oct 23, 2003 at 04:45:03PM +0300, Alex Yustasov wrote:
> Но оказалось, что я вызывал hasher builddir suspend.spec

Кстати, тоже недавно спросонья наступил -- там бы проверку, что
ли, сделать :)

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

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

Re: [devel] u: kernel-feat-swsusp-alt2 и suspend-alt3

[Andrey Rahmatullin]

Alex Yustasov пишет:
> Здравствуйте!
> В инкоминг/сизифус отправились 
> 
> 4b48f41ed9d96f95fb4d1ad405b67c06  kernel-feat-swsusp-2003.10.16-alt2.src.rpm
[...]

Общее впечатление от патча - он сломан. Весь.

1. kernel-feat-swsusp-2003.10.16-alt2 с инкрементным патчем внутри
(2.0rc1 against 2.4.22 + 2.0rc2 against 2.0rc1) в принципе не патчит
ванильное 2.4.22 ядро. А вот обычный (не инкрементный) патч 2.0-rc2
(http://prdownloads.sourceforge.net/swsusp/swsusp-2.0-rc2-whole.bz2?download) 

патчит нормально.
2. XFS-ные куски патча не прикладываются, если не приложен сам XFS-патч,
а video-splash-ные - без video-splash.
3. Сам патч конфликтует по drivers/usb/host/usb-ohci.c с
kernel-fix-drivers-usb-2003.09.29-alt1. Конкретно с
90_08_disable_all_usb.patch. Еще конкретнее - оба пытаются добавить пару
строчек в тело одной коротенькой функциии:

   static int __init ohci_hcd_init (void)
   {
         return pci_module_init (&ohci_pci_driver);
   }

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

Re: [devel] u: kernel-feat-swsusp-alt2 и suspend-alt3

[Sergey Vlasov]

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

On Sat, Oct 25, 2003 at 12:42:16AM +0600, Andrey Rahmatullin wrote:
> Alex Yustasov пишет:
> >Здравствуйте!
> >В инкоминг/сизифус отправились 
> >
> >4b48f41ed9d96f95fb4d1ad405b67c06  
> >kernel-feat-swsusp-2003.10.16-alt2.src.rpm
> [...]
> 
> Общее впечатление от патча - он сломан. Весь.
> 
> 1. kernel-feat-swsusp-2003.10.16-alt2 с инкрементным патчем внутри
> (2.0rc1 against 2.4.22 + 2.0rc2 against 2.0rc1) в принципе не патчит
> ванильное 2.4.22 ядро. А вот обычный (не инкрементный) патч 2.0-rc2
> (http://prdownloads.sourceforge.net/swsusp/swsusp-2.0-rc2-whole.bz2?download) 
> 
> патчит нормально.
> 2. XFS-ные куски патча не прикладываются, если не приложен сам XFS-патч,
> а video-splash-ные - без video-splash.

Для таких случаев в kernel-build-tools уже предусмотрено условное
приложение патчей.  В данном случае, например, патчи
04_swsusp-xfs-option-patch и 05_swsusp-xfs-option-fix следует
переложить в подкаталог 04_kernel-feat-fs-xfs, а патч
06_swsusp-1.1-rc10-bootsplash-option - в подкаталог
06_kernel-feat-drivers-video-splash.  При этом одновременно будет
обеспечен и правильный порядок приложения патчей - независимо от
порядка %add_patch_list сначала будут применены патчи, на которые есть
такие условные зависимости.

> 3. Сам патч конфликтует по drivers/usb/host/usb-ohci.c с
> kernel-fix-drivers-usb-2003.09.29-alt1. Конкретно с
> 90_08_disable_all_usb.patch. Еще конкретнее - оба пытаются добавить пару
> строчек в тело одной коротенькой функциии:
> 
>   static int __init ohci_hcd_init (void)
>   {
>         return pci_module_init (&ohci_pci_driver);
>   }
> 
> С этим можно что-то сделать? Ведь либо придется делать один патч
> зависящим от другого, либо склеивать их, либо делать их конфликтующими,
> и в любом случае не будет выбора между прикладыванием одного из патчей
> либо обоих. Или как-то можно?

С этим несколько хуже - в частности, сейчас в kernel-build-tools нет
средств для наложения патчей при условии _отсутствия_ какого-либо
пакета с патчами с списке.  Хотя можно извернуться - например, в
10_kernel-fix-drivers-usb/xxx отменить мешающий кусок, потом в 11_yyy
наложить свой патч, а далее в 12_kernel-fix-drivers-usb/zzz вернуть
отменённые изменения на место (уже объединённые с другими).  Но это уж
как-то совсем навороченно выглядит.

Хотя на самом деле kernel-feat-swsusp-2003.10.16-alt2 как раз должен
накладываться после kernel-fix-drivers-usb-2003.09.29-alt1 - а вот в
обратном порядке уже не получится.  Можно зафиксировать этот порядок -
для этого в /usr/src/kernel/patches/kernel-feat-swsusp надо создать
подкаталог kernel-fix-drivers-usb (фактически в действие вступит всё
тот же механизм условного применения патчей - хотя каталог и пустой,
но порядок применения всё равно меняется).

Наконец, зависимость на kernel-fix-* не настолько неприятна - вот от
жёстких зависимостей на другие kernel-feat-* желательно избавляться
(если они не нужны по смыслу патча).

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

Re: [devel] u: kernel-feat-swsusp-alt2 и suspend-alt3

[Alex Yustasov]

On Fri, Oct 24, 2003 at 11:30:57PM +0400, Sergey Vlasov wrote:
<skip>

> > Общее впечатление от патча - он сломан. Весь.
> > 
> > 1. kernel-feat-swsusp-2003.10.16-alt2 с инкрементным патчем внутри
> > (2.0rc1 against 2.4.22 + 2.0rc2 against 2.0rc1) в принципе не патчит
> > ванильное 2.4.22 ядро. А вот обычный (не инкрементный) патч 2.0-rc2
> > (http://prdownloads.sourceforge.net/swsusp/swsusp-2.0-rc2-whole.bz2?download) 

Если честно, то я об этом не думал. 

> > патчит нормально.
> > 2. XFS-ные куски патча не прикладываются, если не приложен сам XFS-патч,
> > а video-splash-ные - без video-splash.
> 
> Для таких случаев в kernel-build-tools уже предусмотрено условное
> приложение патчей.  В данном случае, например, патчи
> 04_swsusp-xfs-option-patch и 05_swsusp-xfs-option-fix следует
> переложить в подкаталог 04_kernel-feat-fs-xfs, а патч
> 06_swsusp-1.1-rc10-bootsplash-option - в подкаталог
> 06_kernel-feat-drivers-video-splash.  При этом одновременно будет
> обеспечен и правильный порядок приложения патчей - независимо от
> порядка %add_patch_list сначала будут применены патчи, на которые есть
> такие условные зависимости.

Попробовал. В kernel-feat-fs-xfs в /include/linux/sched.h добавляется
#define PF_FSTRANS. swsusp патчи для "xfs-есть" и "xfs-нет" будут разные.
То что получилось пока, лежит на 
http://snmon.at.tut.by/swsusp/kernel-feat-swsusp-2003.10.16-alt3.src.rpm
Сегодня уже больше ничего не смогу сделать.
Если так пройдет, то наверно получится сделать, чтобы можно было прикладывать
на ванильное ядро и ядро с feat-fix-ами.