| Summary: | counters should be initialized at load time | ||||||
|---|---|---|---|---|---|---|---|
| Product: | DRI | Reporter: | Robert Noland <rnoland> | ||||
| Component: | DRM/other | Assignee: | Default DRI bug account <dri-devel> | ||||
| Status: | RESOLVED INVALID | QA Contact: | |||||
| Severity: | normal | ||||||
| Priority: | medium | ||||||
| Version: | DRI git | ||||||
| Hardware: | All | ||||||
| OS: | FreeBSD | ||||||
| Whiteboard: | |||||||
| i915 platform: | i915 features: | ||||||
| Attachments: |
|
||||||
|
Description
Robert Noland
2007-11-18 20:54:14 UTC
Created attachment 12621 [details] [review] Init counters at load time Since most of the module gets reset between lastclose and firstopen, it makes sense to me that the counters would as well. Setting up the list of counters at load time would make sense though. What is actually occurring and what the patch fixes is that when i915_driver_load is called, it adds additional counters ( += 4 ). I'm not certain that counters has ever been initialized at this point. When firstopen is called, counters is initialized to 6. We either need to move the counter initialization to load time, which this patch does or move i915s additions to counters into i915_driver_firstopen. The latter seems to be moving in the wrong direction when considering modesetting... Hi, Freedesktop's Bugzilla instance is EOLed and open bugs are about to be migrated to http://gitlab.freedesktop.org. To avoid migrating out of date bugs, I am now closing all the bugs that did not see any activity in the past year. If the issue is still happening, please create a new bug in the relevant project at https://gitlab.freedesktop.org/drm (use misc by default). Sorry about the noise! |
Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.