I think it's better to fail and force a user to use --disable-glamor (or install glamor) than silently succeed without any acceleration for SI.
Marek On Mon, Sep 16, 2013 at 4:00 PM, Michel Dänzer <[email protected]> wrote: > On Mon, 2013-09-16 at 09:42 -0400, Alex Deucher wrote: >> It's required for newer asics, so enable it by >> default. >> >> Signed-off-by: Alex Deucher <[email protected]> >> --- >> configure.ac | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/configure.ac b/configure.ac >> index d1cf115..7cd30d2 100644 >> --- a/configure.ac >> +++ b/configure.ac >> @@ -101,9 +101,9 @@ CPPFLAGS="$CPPFLAGS $XORG_CFLAGS" >> AC_MSG_CHECKING([whether to include GLAMOR support]) >> AC_ARG_ENABLE(glamor, >> AS_HELP_STRING([--enable-glamor], >> - [Enable glamor, a new GL-based acceleration >> [default=no]]), >> + [Enable glamor, a new GL-based acceleration >> [default=auto]]), >> [GLAMOR="$enableval"], >> - [GLAMOR=no]) >> + [GLAMOR=auto]) >> AC_MSG_RESULT([$GLAMOR]) >> AM_CONDITIONAL(GLAMOR, test x$GLAMOR != xno) >> if test "x$GLAMOR" != "xno"; then > > I think this will cause configure to fail without --disable-glamor when > glamor isn't available? Probably need more changes similar to what's > done for udev support. > > > -- > Earthling Michel Dänzer | http://www.amd.com > Libre software enthusiast | Debian, X and DRI developer > > _______________________________________________ > xorg-driver-ati mailing list > [email protected] > http://lists.x.org/mailman/listinfo/xorg-driver-ati _______________________________________________ xorg-driver-ati mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-driver-ati
