Closed
Bug 901002
Opened 11 years ago
Closed 11 years ago
[8.1] Add Windows 8.1 to the OS list for gfx driver blacklisting
Categories
(Core :: Graphics, defect)
Tracking
()
RESOLVED
FIXED
mozilla26
People
(Reporter: bbondy, Assigned: bbondy)
References
Details
(Whiteboard: [8.1] )
Attachments
(1 file)
(deleted),
patch
|
bas.schouten
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
- Would it be better to always return DRIVER_OS_WINDOWS_8 for windows 8.1? (I don't think so but check checking)
- There are 3 APPEND_TO_DRIVER_BLOCKLIST_RANGE( DRIVER_OS_WINDOWS_8, should I be setting these for 8.1 a well? How do I tell?
Attachment #785058 -
Flags: review?(bas)
Updated•11 years ago
|
Summary: Add Windows 8.1 to the OS list for gfx driver blacklisting → [8.1] Add Windows 8.1 to the OS list for gfx driver blacklisting
Whiteboard: [8.1]
Assignee | ||
Comment 2•11 years ago
|
||
Hey Bas, been a couple weeks, review ping? :)
Updated•11 years ago
|
Attachment #785058 -
Flags: review?(bas) → review+
Comment 4•11 years ago
|
||
(In reply to Brian R. Bondy [:bbondy] from comment #1)
> Created attachment 785058 [details] [diff] [review]
> Patch v1
>
> - Would it be better to always return DRIVER_OS_WINDOWS_8 for windows 8.1?
> (I don't think so but check checking)
> - There are 3 APPEND_TO_DRIVER_BLOCKLIST_RANGE( DRIVER_OS_WINDOWS_8, should
> I be setting these for 8.1 a well? How do I tell?
I have no idea on either of these questions. Bjacob is still the most knowledgeable person on our blacklisting system. I suspect the answer to the second question is yes though.
Flags: needinfo?(bas)
Comment 6•11 years ago
|
||
(In reply to Brian R. Bondy [:bbondy] from comment #1)
> Created attachment 785058 [details] [diff] [review]
> Patch v1
>
> - Would it be better to always return DRIVER_OS_WINDOWS_8 for windows 8.1?
> (I don't think so but check checking)
I don't think so: it should be useful to be able to have blacklist rules that affect only Windows 8, or only Windows 8.1 ... unless you know something that I don't know, about Windows 8 and Windows 8.1 being really the same for all practical purposes here.
Also, our current blacklisting system is terribly fragile and unpredictable, and we know for instance that it is unsafe to add a new kind of FEATURE, but for all I know, it should be safe to add a new operating system version. We did it fairly recently for Windows 8 and we were fine.
> - There are 3 APPEND_TO_DRIVER_BLOCKLIST_RANGE( DRIVER_OS_WINDOWS_8, should
> I be setting these for 8.1 a well? How do I tell?
It's up to you: do you think that these blacklist rules should apply to Windows 8.1 as well, or not? If you don't know, the safe choice is probably to assume that Windows 8.1 is similar to Windows 8, therefore you need to duplicate these rules for Windows 8.1...
Flags: needinfo?(bjacob)
Assignee | ||
Comment 7•11 years ago
|
||
It could be different enough, but I have no way of knowing. I'll leave it as is for now.
Assignee | ||
Comment 8•11 years ago
|
||
Target Milestone: --- → mozilla26
Comment 9•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•