From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) by finch.gentoo.org (Postfix) with ESMTP id 738EC138A1F for ; Thu, 30 Jan 2014 20:44:21 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 136E9E0B4C; Thu, 30 Jan 2014 20:44:20 +0000 (UTC) Received: from smtp.gentoo.org (smtp.gentoo.org [140.211.166.183]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 796C8E0B4C for ; Thu, 30 Jan 2014 20:44:19 +0000 (UTC) Received: from big_daddy.dol-sen.ca (S010600222de111ff.vc.shawcable.net [96.49.5.156]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 545CC33F855 for ; Thu, 30 Jan 2014 20:44:18 +0000 (UTC) Date: Thu, 30 Jan 2014 12:40:07 -0800 From: Brian Dolbec To: gentoo-catalyst@lists.gentoo.org Subject: Re: [gentoo-catalyst] Re: Catalyst pull request Message-ID: <20140130124007.57bd04f1@big_daddy.dol-sen.ca> In-Reply-To: <20140130164611.GK14197@odin.tremily.us> References: <52E97F9E.5010906@gentoo.org> <20140129233454.GF14197@odin.tremily.us> <14fdc26924392b40107d9167cbdec5bd@tuxicoman.be> <20140130164611.GK14197@odin.tremily.us> Organization: Gentoo X-Mailer: Claws Mail 3.9.3 (GTK+ 2.24.22; x86_64-pc-linux-gnu) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-catalyst@lists.gentoo.org Reply-to: gentoo-catalyst@lists.gentoo.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Archives-Salt: 70b62d16-4de8-4aa8-9965-0f3d0490a150 X-Archives-Hash: ac25def60fe46ee93dac2ce768b941a3 On Thu, 30 Jan 2014 08:46:11 -0800 "W. Trevor King" wrote: > On Thu, Jan 30, 2014 at 09:56:03AM +0100, Guy Martin wrote: > > On 2014-01-30 00:34, W. Trevor King wrote: > > > On Wed, Jan 29, 2014 at 05:24:30PM -0500, Rick "Zero_Chaos" > > > Farina=20 > > >> On 01/29/2014 10:19 AM, Guy Martin wrote: > > >> > https://github.com/gmsoft-tuxicoman/catalyst/compare/6046e57...gms= oft?expand=3D1 > > > > > > I think 6046e57 is useful, but it's already in pending with > > > 70f7cfcand fcbcea4 [1]=E2=80=A6 > > > > This patch just make it easier to develop and test with a git > > branch. I think it's a good addition and will definitely not break > > anything. >=20 > Agreed on both counts. I'm just recommending the older 70f7cfc and > fcbcea4 over 6046e57. There isn't going to be any real development work in the stable 2.X branch. Just bug fixes until 3.0 is ready for release. So, it won't prove very useful in general. Plus I'm ready to commit the one in pending for the main development branch. >=20 > > > I've never used PALO [2], so I'm agnostic on 0f4c970. I agree > > > with dol-sen's #gentoo-releng comment that it would be better to > > > make this timeout configurable, but since this is code I'll never > > > use, I don't feel very strongly about it ;). > >=20 > > The timeout will be configurable when booting. The user always has > > the option to change the value. It's just there because when I > > install a box remotely, I can easily panic the kernel (like ctrl-d > > in the netboot) and I have no way to recover the box. >=20 > If you can't set this remotely already, I don't expect a user will be > able to change the value remotely either. You want to set this in > Catalyst instead, so I think it's fair to support users who want to > change (or disable) the timeout in Catalyst too. How about just a general kernel_opts field in the spec ie. kernel_opts: panic=3D30 foo=3Dbar That makes it possible to add more than just a panic=3D to the commandline while still being simple for catalyst. It is just one variable. The help comment above it in the example spec can list the options recommended. >=20 > > >> > Regarding commit 333808b, > > >=20 > > > I don't understand the first hunk in 333808b: > > > > > > =E2=80=A6 > > >=20 > > > Why create /tmp/kerncache if kerncache is disabled? > >=20 > > This directory is used later on in the kmerge.sh script for a few > > files. For example you have : > > /tmp/kerncache/${clst_kname}/${clst_kname}-${clst_version_stamp}.EXTRAV= ERSION > > /tmp/kerncache/${clst_kname}/${clst_kname}-${clst_version_stamp}.USE > >=20 > > I wasn't sure about the use of these files so I went for the safe > > route to create the directory. A proper fix might simply be to avoid > > creating these files at all if kerncache isn't used. >=20 > The proper fix route sounds better to me ;). Yes, if this is to go into the stable branch. It should be done properly. >=20 > > >> > the second hunk adds --selective=3Dn to emerge always merge the > > >> > kernel. When having two kernels in the livecd specs, the first > > >> > build works fine but the second one fails because the sources > > >> > are removed and not reemerged. > > >=20 > > > I'm not entirely sure how this is supposed to work, but I'm not > > > wild about --selective=3Dn. Are the two kernels you're adding to > > > the livecd both from the same package? =E2=80=A6 > >=20 > > Yes they are both from gentoo-sources. On hppa we have to compile a > > 32 and a 64bit kernel, provide them both to palo and it'll decide > > which one to boot base on the machine it's booting. Some hppa box > > can boot only 32 or 64 while some can boot both. >=20 > Got it, thanks. I'll poke around and see if I can come up with > something that works and preserves the @world entry. Can you point me > towards some example spec files for testing? >=20 > Thanks, > Trevor >=20 If you could address the above issues, I think the rest of the team would approve the changes for the 2.X branch. I can also add them to the pending branch queue for merge into master. I have done very little to the kernel code areas, so there should be few to no conflicts with the rewrite changes. Brian Dolbec