Page MenuHomeFreeBSD

agp: Do not attach to Intel GEN6+
ClosedPublic

Authored by dumbbell on Mar 8 2016, 10:25 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Oct 10, 1:43 AM
Unknown Object (File)
Tue, Sep 30, 10:09 AM
Unknown Object (File)
Tue, Sep 30, 8:45 AM
Unknown Object (File)
Tue, Sep 30, 8:44 AM
Unknown Object (File)
Tue, Sep 30, 8:19 AM
Unknown Object (File)
Tue, Sep 30, 6:31 AM
Unknown Object (File)
Mon, Sep 22, 6:00 AM
Unknown Object (File)
Sun, Sep 14, 6:44 PM
Subscribers

Details

Summary

The i915 video driver doesn't depend on agp(4) anymore for Sandybridge and later GPUs. Therefore, there is no need to attach agp(4) to those devices.

kib@, I noticed that the agp_g4x_res_spec variable was used by all GEN6+, but not by G4X. Is is expected?

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 2857
Build 2881: arc lint + arc unit

Event Timeline

dumbbell retitled this revision from to agp: Do not attach to Intel GEN6+.
dumbbell updated this object.
dumbbell edited the test plan for this revision. (Show Details)
dumbbell added reviewers: jhb, kib, kwm, zeising.
kib edited edge metadata.

About agp_g4x_res_spec, I think that it actually does not matter. I re-read GMCH datasheets for 965, G45 and SandyBridge. On all three chipsets, device 0:2:0 BAR at 0x10 is combined mmio and GTT, and BAR at 0x18 is the aperture. So the correct thing to do is to add a name for 965 BAR at 0x18 and add a second resource line to agp_i965_res_spec, which should be used by Gen4+.

I think that i965 and g4x res_specs where split due to my mistake.

This revision is now accepted and ready to land.Mar 9 2016, 12:21 PM

This patch works fine on my sandybridge system.

dumbbell edited edge metadata.

Fix agp_i965_res_spec

This revision now requires review to proceed.Mar 11 2016, 5:52 PM
kib edited edge metadata.

Can we leave the AGP_SNB registers definitions in the header ? I find it easier to consult our reg.h than to search datasheets.

This revision is now accepted and ready to land.Mar 12 2016, 4:21 AM
This revision was automatically updated to reflect the committed changes.