Closed Bug 901002 Opened 6 years ago Closed 6 years ago

[8.1] Add Windows 8.1 to the OS list for gfx driver blacklisting

Categories

(Core :: Graphics, defect)

x86_64
Windows 8
defect
Not set

Tracking

()

RESOLVED FIXED
mozilla26

People

(Reporter: bbondy, Assigned: bbondy)

References

Details

(Whiteboard: [8.1] )

Attachments

(1 file)

No description provided.
Attached patch Patch v1Splinter Review
- 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)
Blocks: 855294
Blocks: 894039
No longer blocks: 894039
Blocks: 898795
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]
Hey Bas, been a couple weeks, review ping? :)
1 month + 1 week review ping :)
Flags: needinfo?(bas)
Attachment #785058 - Flags: review?(bas) → review+
(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)
Please see the previous comment.
Flags: needinfo?(bjacob)
(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)
It could be different enough, but I have no way of knowing. I'll leave it as is for now.
https://hg.mozilla.org/mozilla-central/rev/ed0014600bc7
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.