Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(320)

Issue 6485058: Disable npot texture support on AMD cards on XP. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 9 months ago by John Bauman
Modified:
11 years, 8 months ago
Reviewers:
dgkoch, nicolas
CC:
angleproject-review_googlegroups.com
Base URL:
https://angleproject.googlecode.com/svn/trunk
Visibility:
Public.

Description

Disable npot texture support on AMD cards on XP. We had been seeing some BSODs due to this. BUG= Committed: https://code.google.com/p/angleproject/source/detail?r=1264

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+21 lines, -19 lines) Patch
M src/libEGL/Display.h View 2 chunks +14 lines, -0 lines 0 comments Download
M src/libEGL/Display.cpp View 2 chunks +7 lines, -3 lines 0 comments Download
M src/libEGL/Surface.cpp View 1 chunk +0 lines, -16 lines 0 comments Download

Messages

Total messages: 6
John Bauman
11 years, 9 months ago (2012-08-24 00:31:50 UTC) #1
John Bauman
ping
11 years, 8 months ago (2012-08-27 16:13:32 UTC) #2
dgkoch
The patch seems fine enough from a code POV. But killing NPOT on all AMD ...
11 years, 8 months ago (2012-08-27 17:33:52 UTC) #3
John Bauman
On 2012/08/27 17:33:52, dgkoch wrote: > The patch seems fine enough from a code POV. ...
11 years, 8 months ago (2012-08-27 17:39:26 UTC) #4
dgkoch
On 2012/08/27 17:39:26, John Bauman wrote: > On 2012/08/27 17:33:52, dgkoch wrote: > > The ...
11 years, 8 months ago (2012-08-27 17:42:21 UTC) #5
John Bauman
11 years, 8 months ago (2012-08-27 17:44:34 UTC) #6
On 2012/08/27 17:42:21, dgkoch wrote:
> On 2012/08/27 17:39:26, John Bauman wrote:
> > On 2012/08/27 17:33:52, dgkoch wrote:
> > > The patch seems fine enough from a code POV. But killing NPOT on all AMD
> cards
> > > on XP seems a bit extreme.  It can't be narrowed down any further?
> > 
> > No, the BSOD can happen on all AMD cards, with any NPOT texture that has
more
> > than 1 miplevel. Chrome doesn't expose this extension to web content (except
> > through pepper, and that's probably pretty uncommon), so getting rid of it
> won't
> > affect us much.
> 
> It's not limited to certain drivers either (new/old?).  One would think using
> npot textures on d3d9 would be pretty common..

Every driver version we know of has the bug. I'm not sure how common NPOT
textures with multiple miplevels are, but maybe our specific usecase makes us
somewhat more likely to hit the bug than otherwise.
> 
> Anyhow LGTM if that's what you need to do.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b