"Martin S. Weber" <ephae...@gmx.net> 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
