Messages in this thread | | | Date | Fri, 26 Sep 2003 22:26:44 +0200 | From | Andrea Arcangeli <> | Subject | Re: log-buf-len dynamic |
| |
On Thu, Sep 25, 2003 at 02:40:15PM +0100, marcelo@parcelfarce.linux.theplanet.co.uk wrote: > On Mon, 22 Sep 2003, Andrea Arcangeli wrote: > > > Hi, > > > > I'm rejecting on the log-buf-len feature in 2.4.23pre5, the code in > > mainline is worthless for any distributor, shipping another rpm package > > just for the bufsize would be way overkill. > > Andrea, > > As Willy stated previously this is useful for people who want to change > the log buf size without having to change the code manually, and I think > this is a useful and non intrusive change. > > Do you see any problem with this?
as wrote in a later email, technically the dynamic solution obsoletes the static one, but only as far as the kernel is concerned.
It happens that to enable the dynamic solution you've to change 1 line to lilo.conf/grub.lst. since 2.4 is already being used in production distributed scenarios, there is at least a psycology value in providing a static solution that doesn't involve lilo.conf changes, so people can deploy the feature without having to edit conf files at all (with grub they don't need to run lilo either).
So I will rewrite my patch in a way that doesn't reject the static setting for next -aa (together with some other tons of pending things, but the vm merge is already completed on my tree, so I'm in sync again with pre5 on that side and that was top priority to fixup to be sure nothing was missing, thanks for merging the watermarks already btw ;).
Andrea - If you prefer relying on open source software, check these links: rsync.kernel.org::pub/scm/linux/kernel/bkcvs/linux-2.[45]/ http://www.cobite.com/cvsps/ svn://svn.kernel.org/linux-2.[46]/trunk - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |