Re: [Mesa-dev] [PATCH] gallium/i965g: hide that utterly broken driver better

2011-11-28 Thread Daniel Vetter
On Mon, Nov 28, 2011 at 03:17:09PM -0800, Jose Fonseca wrote:
> - Original Message -
> > And warn loudly in case people want to use it. Too many tester report
> > gpu hangs on irc and we rootcause this ...
> > 
> > Signed-Off-by: Daniel Vetter 
> 
> If you're doing this then why not just remove the drive all together, as 
> agreed a few days ago?

I simply missed that discussion. I also don't have any experience with
frobbing the mesa build system, so I'd surely miss something ...
-Daniel
-- 
Daniel Vetter
Mail: dan...@ffwll.ch
Mobile: +41 (0)79 365 57 48
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] gallium/i965g: hide that utterly broken driver better

2011-11-28 Thread Jose Fonseca
- Original Message -
> And warn loudly in case people want to use it. Too many tester report
> gpu hangs on irc and we rootcause this ...
> 
> Signed-Off-by: Daniel Vetter 

If you're doing this then why not just remove the drive all together, as agreed 
a few days ago?

Jose


- Forwarded Message -
> From: "Keith Whitwell" 
> To: "Kenneth Graunke" 
> Cc: mesa-dev@lists.freedesktop.org
> Sent: Friday, November 18, 2011 10:57:17 PM
> Subject: Re: [Mesa-dev] [PATCH 4/6] gallium: remove PIPE_CAP_GLSL and enable 
> GLSL unconditionally
> 
> 
> 
> - Original Message -
> > On 11/18/2011 11:27 AM, Marek Olšák wrote:
> > > Only i965g does not enable GLSL, but that driver has been
> > > unmaintained and
> > > bitrotting for quite a while anyway.
> > 
> > It doesn't even do GLSL?  I'm pretty shocked, I figured it at least
> > did
> > that.  Is it even worth keeping around in the tree?  Seems like
> > it's
> > just creating extra work for you guys, having to update it for
> > Gallium
> > changes...when ultimately, nobody's using it.
> >
> 
> I agree -- this was never finished & isn't likely to be either.
> 
> Keith
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] gallium/i965g: hide that utterly broken driver better

2011-11-28 Thread Chad Versace
On 11/28/2011 01:32 PM, Daniel Vetter wrote:
> And warn loudly in case people want to use it. Too many tester report
> gpu hangs on irc and we rootcause this ...
> 
> Signed-Off-by: Daniel Vetter 

Yes! This eliminate some future bug reports too.

Reviewed-by: Chad Versace 


Chad Versace
chad.vers...@linux.intel.com
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


Re: [Mesa-dev] [PATCH] gallium/i965g: hide that utterly broken driver better

2011-11-28 Thread Patrick Baggett
On Mon, Nov 28, 2011 at 3:32 PM, Daniel Vetter wrote:

> And warn loudly in case people want to use it. Too many tester report
> gpu hangs on irc and we rootcause this ...
>
> Signed-Off-by: Daniel Vetter 
> ---
>  configure.ac |9 -
>  1 files changed, 8 insertions(+), 1 deletions(-)
>
> diff --git a/configure.ac b/configure.ac
> index 8885a6d..4dee3ad 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -658,7 +658,7 @@ GALLIUM_DRIVERS_DEFAULT="r300,r600,swrast"
>  AC_ARG_WITH([gallium-drivers],
> [AS_HELP_STRING([--with-gallium-drivers@<:@=DIRS...@:>@],
> [comma delimited Gallium drivers list, e.g.
> -"i915,i965,nouveau,r300,r600,svga,swrast"
> +"i915,nouveau,r300,r600,svga,swrast"
> @<:@default=r300,r600,swrast@:>@])],
> [with_gallium_drivers="$withval"],
> [with_gallium_drivers="$GALLIUM_DRIVERS_DEFAULT"])
> @@ -2007,10 +2007,17 @@ if echo "$SRC_DIRS" | grep 'gallium' >/dev/null
> 2>&1; then
> echo "Winsys dirs: $GALLIUM_WINSYS_DIRS"
> echo "Driver dirs: $GALLIUM_DRIVERS_DIRS"
> echo "Trackers dirs:   $GALLIUM_STATE_TRACKERS_DIRS"
> +   if echo "$GALLIUM_DRIVERS_DIRS" | grep i965 > /dev/null 2>&1; then
> +  echo
> +  echo "WARNING: enabling i965 gallium driver"
> +  echo "the i965g driver is currently utterly broken, only
> for adventurours developers"
>

I think the word is "adventurous".


> +  echo
> +   fi
>  else
> echo "Gallium: no"
>  fi
>
> +
>  dnl Libraries
>  echo ""
>  echo "Shared libs: $enable_shared"
> --
> 1.7.7.1
>
> ___
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev