From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 24 Oct 2003 23:30:57 +0400 From: Sergey Vlasov To: ALT Devel discussion list Subject: Re: [devel] u: =?koi8-r?Q?kernel-feat-swsu?= =?koi8-r?Q?sp-alt2_=C9?= suspend-alt3 Message-ID: <20031024193057.GB2439@sirius.home> Mail-Followup-To: ALT Devel discussion list References: <20031023134503.GA15941@server.by> <3F997288.809@altlinux.ru> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="CdrF4e02JqNVZeln" Content-Disposition: inline In-Reply-To: <3F997288.809@altlinux.ru> X-BeenThere: devel@altlinux.ru X-Mailman-Version: 2.1.2 Precedence: list Reply-To: ALT Devel discussion list List-Id: ALT Devel discussion list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 24 Oct 2003 19:31:10 -0000 Archived-At: List-Archive: List-Post: --CdrF4e02JqNVZeln Content-Type: text/plain; charset=koi8-r Content-Disposition: inline Content-Transfer-Encoding: 8bit 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-* желательно избавляться (если они не нужны по смыслу патча). --CdrF4e02JqNVZeln Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.3 (GNU/Linux) iD8DBQE/mX3xW82GfkQfsqIRAuMMAJ9mcGFN2qLBkcBWDRSkifoMGP0/9gCeIb7/ W7H+EKSJ8ib6UEAwVV4KR6w= =FLVf -----END PGP SIGNATURE----- --CdrF4e02JqNVZeln--