Stephen Ma <stephenm@employees.org> escribió en artículo:
:>>>>> "Geoff" == Geoff Wing <gcw@pobox.com> writes:
:Note that on NetBSD 1.5Y and later, since we have kernel support for
:MTRR (i.e. HAS_MTRR_BUILTIN is defined), HAS_MTRR_SUPPORT will not be
:defined. Thus, "memrange.h" is not included, which means bsd_video.c
:builds with no problem on NetBSD-current.

Oh, you're quite right, I misread that bit and jumped the gun a bit.
I saw MTRR_BUILTIN and kept reading it as MTRR_SUPPORT.  Apologies.

:However, breaking the build for 1.5 is not good. We have a few options
:for 1.5: either define HasNetBSDApertureDriver to NO for 1.5; or
:disable the LKM MTRR support for 1.5, or do something tricky with the
:toplevel Makefile to enable the aperture driver support only if the
:aperture package is installed.

Unless someone can think of a good trick for the last one - maybe
presuming lots of basic things, e.g. /var/db/pkg & pkgtools stuff,
I think that's a bit excessive.  I guess it would make more sense
to have a message spewed out about changes needs for building X as
part of the aperture pkg and have it disabled for X by default
for 1.5X and prior.

Regards,
Geoff