Date: Mon, 11 Dec 2017 15:44:56 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: rgrimes@freebsd.org Cc: Conrad Meyer <cem@freebsd.org>, svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r326758 - in head/sys/i386: conf include Message-ID: <20171211154456.GB33210@FreeBSD.org> In-Reply-To: <201712111509.vBBF9Aha081695@pdx.rh.CN85.dnsmgr.net> References: <201712110432.vBB4WbnE021090@repo.freebsd.org> <201712111509.vBBF9Aha081695@pdx.rh.CN85.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Dec 11, 2017 at 07:09:10AM -0800, Rodney W. Grimes wrote: > [ Charset UTF-8 unsupported, converting... ] > > New Revision: 326758 > > URL: https://svnweb.freebsd.org/changeset/base/326758 > > > > Log: > > i386: Bump KSTACK_PAGES default to match amd64 > > > > Logically, extend r286288 to cover all threads, by default. > > > > The world has largely moved on from i386. Most FreeBSD users and > > developers ... > > This is a mistake, there is a huge world of i386 deployment, not all > the world needs, or even wants amd64, especially in the virtualization > world when you are running anything with less than 4G of memory, which > I would argue is a huge depolyement base. > > 64-bit code is simply wasteful on these platforms, and though maybe > software developers are ok with that waste, (smart) business people > are not. Big +1. Most of my hw is 32-bit and I like it that way. It works fine, it helps to maintains/check sw sanity, and remains our default reference platform. > > ... test on amd64 hardware. Then they should start doing a better job and test on i386 (at least) as well. > For better or worse, we have written a non-trivial amount of kernel > code that relies on stacks larger than 8 kB, and it "just works" on > amd64, so there has been little incentive to shrink it. If the code is broken, it should be fixed. "Just works on amd64" is not a good enough excuse to deprecate^Wput quick "fixes" for i386 that did not undergo sufficient review. ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20171211154456.GB33210>