Woohoo, we kill agp_memory. I'm in favour!
Do we need to keep the sg_table around, or can we just temporary allocate
it?
This fits nicely into my plans, i915_gem_gtt.c has been a candidate to
eliminate a few of the more expensive agp routines. Thanks, I'll look more
closely at the series next wee
On Sat, 6 Nov 2010 14:53:32 -0700, Eric Anholt wrote:
> This is not known to fix any particular bugs we have, but the spec
> says to do it, and the BIOS hadn't already set it up on my system.
Whilst you haven't directly identified any bug caused by these two,
adhering to the spec sounds like a s
This is not known to fix any particular bugs we have, but the spec
says to do it, and the BIOS hadn't already set it up on my system.
Signed-off-by: Eric Anholt
---
drivers/gpu/drm/i915/i915_reg.h | 13 +
drivers/gpu/drm/i915/intel_display.c |6 ++
2 files changed, 19
Not known to fix any current bugs.
Signed-off-by: Eric Anholt
---
drivers/gpu/drm/i915/i915_reg.h |2 ++
drivers/gpu/drm/i915/intel_display.c |4
2 files changed, 6 insertions(+), 0 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
ind
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/i915/i915_drv.h |9 +++--
drivers/gpu/drm/i915/i915_gem_gtt.c | 63 --
2 files changed, 50 insertions(+), 22 deletions(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
inde
No more drm_*_agp in i915_gem.c!
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/i915/i915_drv.h |2 ++
drivers/gpu/drm/i915/i915_gem.c | 14 +++---
drivers/gpu/drm/i915/i915_gem_gtt.c | 28
3 files changed, 33 insertions(+), 11 deletions(-)
Just some minor shuffling to get rid of any agp traces in the
exported functions.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c | 120 --
include/drm/intel-gtt.h | 12
2 files changed, 80 insertions(+), 52 deletions(-)
diff --
This still uses the agp functions to actually reinstate the mappings
(with a gross hack to make agp cooperate), but it wires everything
up correctly for the switchover.
The call to agp_rebind_memory can be dropped because all non-kms drivers
do all their rebinding on EnterVT.
v2: Be more paranoid
Its only user, intel-gtt.c is now gone.
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/char/agp/generic.c | 20
include/linux/agp_backend.h |1 -
2 files changed, 0 insertions(+), 21 deletions(-)
diff --git a/drivers/char/agp/generic.c b/drivers/char/agp/g
The intel drm calls the chipset functions now directly. Userspace
never called the corresponding ioctl, hence it can be killed, too.
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/char/agp/agp.h |1 -
drivers/char/agp/compat_ioctl.c |1 -
drivers/char/agp/compat_ioctl
This is required to restore gtt mappings on resume when agp is gone.
The right way to do this would be to make sturct drm_mm_node embeddable
and use the allocation list maintained by the drm memory manager. But
that's a bigger project. Getting rid of the per bo agp_mem will save
more memory than t
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c|7 +++
drivers/gpu/drm/i915/i915_gem.c |9 -
include/drm/intel-gtt.h |2 +-
3 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c
No longer used.
Cc: Dave Airlie
Signed-off-by: Daniel Vetter
---
drivers/gpu/drm/drm_agpsupport.c |6 --
include/drm/drmP.h |1 -
2 files changed, 0 insertions(+), 7 deletions(-)
diff --git a/drivers/gpu/drm/drm_agpsupport.c b/drivers/gpu/drm/drm_agpsupport.c
index 25
Now the intel-gtt.c rewrite is complete!
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c | 150 +-
1 files changed, 61 insertions(+), 89 deletions(-)
diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c
index 8fff75a..f6
... and a few other defines.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c|5 -
drivers/gpu/drm/i915/i915_gem.c |1 -
include/drm/intel-gtt.h | 12
include/linux/intel-gtt.h | 20
4 files changed, 12 insertions(+
Still a separate agp_bridge_driver because of the i81x-only
dedicated vram support.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c | 197 +++---
1 files changed, 71 insertions(+), 126 deletions(-)
diff --git a/drivers/char/agp/intel-gtt.c b/dr
Initialization is still done with the old code with a few
added things sprinkled in to make the intel_fake_agp helper
functions work.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c | 152 +-
1 files changed, 60 insertions(+), 92 deletions(-
Used for the now dead agp type_to_mask stuff.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c |6 --
1 files changed, 0 insertions(+), 6 deletions(-)
diff --git a/drivers/char/agp/intel-gtt.c b/drivers/char/agp/intel-gtt.c
index 8122aca..b677713 100644
--- a/drivers/char/a
i830_check_flags already disallows it, so no need to implement it
in the write_entry function. Seems to be a remnant from i810 support.
Signed-off-by: Daniel Vetter
---
drivers/char/agp/intel-gtt.c |8 +---
1 files changed, 1 insertions(+), 7 deletions(-)
diff --git a/drivers/char/agp/i
Hi all,
The first few patches are a bunch of leftover cleanups from the intel-gtt
rewrite in .36.
Tested on my i815, i855, i945 and ironlake.
Some items still on my todo-list for the gtt:
- Embed drm_mm_node.
- Kill /dev/agpgart for kms-only chips. Requires some small rework of the
init sequen
On Saturday 06 November 2010 07:14:03 Arkadiusz Miskiewicz wrote:
> AFAIK unreleased libdrm is required but someone forgot to release it and
> bump dependency in intel driver configure
Thanks, the problem seemed familiar ;)
I'll try to update and check again!
Regards
___
Fixes corruption with glBufferSubData on my machine,
Can someone review and push?
(Resent with code comment fixed)
--
Peter Clifton
Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA
Tel: +44 (0)7729 980173 - (No signal in
Fixes corruption with glBufferSubData on my machine,
Can someone review and push?
--
Peter Clifton
Electrical Engineering Division,
Engineering Department,
University of Cambridge,
9, JJ Thomson Avenue,
Cambridge
CB3 0FA
Tel: +44 (0)7729 980173 - (No signal in the lab!)
Tel: +44 (0)1223 748328
> AFAIK unreleased libdrm is required but someone forgot to release it and bump
> dependency in intel driver configure.
Seems to happen reliably every release ;)
- Clemens
___
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedeskt
On Sat, 06 Nov 2010 01:44:59 +, Peter Clifton wrote:
> Presumably this says something bad about my app, or its benchmarking
> code. Any clues as to what that is?
>
> Without the commit, I can use other X11 apps whilst my PCB program is
> benchmarking. With it, the X11 server graphics almost c
25 matches
Mail list logo
