Re: CVS commit: src/sys

[Mindaugas Rasiukevicius <rmind%netbsd.org@localhost>]

"Martin S. Weber" <Ephaeton%gmx.net@localhost> wrote:
> On Mon, May 16, 2011 at 08:18:08PM +0200, Martin Husemann wrote:
> > On Mon, May 16, 2011 at 07:29:26PM +0200, Martin S. Weber wrote:
> > > Reminds me of http://www.netbsd.org/ports/ -- remember
> > > sparc is "tier 2" aka "Go fix it yourself if you don't
> > > hurt me"...
> > 
> > Well, but the fix is simple in this case: turn it into a MD macro like
> > MAXDMAP() and allow other archs to provide a const implementation, while
> > sparc pays the price and has to fetch it from a variable.
> > 
> 
> or just hack it brutally
> 
> ( cvs rdiff -u -r1.308 -r1.309 src/sys/arch/sparc/sparc/machdep.c )
>  -- who cares about hacks in tier 2 architectures when the netbsd
> goals (http://www.netbsd.org/about/system.html - "well designed")
> only apply to tier 1 ports? (the goals page should be updated anyways
> to exclude examples such as EISA, TurboChannel, the alpha or pmap port
> etc)

The *modification* of those variables in such way is rather a hack.  Note
that I have asked sparc guys whether they are happy with such change.

Tell me more about "well designed".

-- 
Mindaugas