Re: [PATCH] Use MACHINE_ARCH for boot loader

On Fri Aug 27 10, Alexander Best wrote:
> On Fri Aug 27 10, Kostik Belousov wrote:
> > On Thu, Aug 26, 2010 at 05:50:01PM -0700, Xin LI wrote:
> > > -----BEGIN PGP SIGNED MESSAGE-----
> > > Hash: SHA256
> > > 
> > > Hi,
> > > 
> > > The attached patch changes FreeBSD/x86 back to FreeBSD/i386 on i386 and
> > > FreeBSD/amd64 on amd64.
> > > 
> > > Comments welcome!  I'll commit it in by the weekend if there is no
> > > objection on this.
> > Change to FreeBSD/x86 was on purpose. And, since the same loader
> > can boot both i386 and amd64 kernels, I consider the current state
> > more logical.
> 
> yeah. i think jhb wanted this to happen. the old thread for this can be found
> here [1].

oops. forgot the reference. :( it's

[1] http://www.mail-archive.com/freebsd-hackers_at_freebsd.org/msg70613.html

...also r205662 seems to have introduced x86. the commit message might be
interesting.

cheers.
alex

> 
> there's also a PR with a patch submitted by myself which will make use of the
> keyword 'x86' throughout the whole loader(8) code.
> 
> the PR number for that is 147120. would be nice if someone would commit the
> patch. that is of course if in fact it has really been decided that for code
> that's being shared between amd64 and i386 the keyword 'x86' shall be used
> consistently.
> 
> however recently i've seen quite some commits to HEAD which make me believe
> that the developers (most of all jhb) want to make heavy use of the 'x86'
> keyword.
> 
> cheers.
> alex
> 
> > 
> > Later, kernel reports its architecture explicitely.
> 
> 
> 
> -- 
> a13x

-- 
a13x