Re: [PATCH] drm: Cache the EDID value for a short time in i915

2009-04-01 Thread Eric Anholt
data from tell me what the current set of modes is. Once that's done, I expect this patch goes away. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Patch 1/3_v2]: [DRM/I915] : Sync the mode validation for INTERLACE/DBLSCAN

2009-04-01 Thread Eric Anholt
, mode_flags); + } + Why is this mysteriously in a new block? -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Patch 3/3_v2]: [DRM/I915] : Sync the default modes for LVDS output device

2009-04-01 Thread Eric Anholt
we're always using the fixed mode. Any other opinions on this? -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Patch 2/3_v2]: [DRM/I915] : Sync the panel fitting property with 2D driver

2009-04-01 Thread Eric Anholt
@@ if (intel_output-ddc_bus) intel_i2c_destroy(intel_output-ddc_bus); drm_connector_cleanup(connector); - kfree(connector); + drm_free(intel_output, sizeof(struct intel_output) + + sizeof(struct intel_lvds_priv), DRM_MEM_DRIVER); } -- Eric Anholt e

Re: [PATCH 6/6] drm/i915: Fix lock order reversal in GEM relocation entry copying. -- makes X hang

2009-03-31 Thread Eric Anholt
On Mon, 2009-03-30 at 12:00 +0200, Florian Mickler wrote: Hi! On Wed, 25 Mar 2009 14:45:10 -0700 Eric Anholt e...@anholt.net wrote: Signed-off-by: Eric Anholt e...@anholt.net Reviewed-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_gem.c | 187

Re: [PATCH] libdrm: speed up connector mode fetching

2009-03-31 Thread Eric Anholt
reprobe things? like we did in the X Server. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: DRM lock ordering fix series

2009-03-30 Thread Eric Anholt
lament that he can't run sysprof on his box. Getting ~10% of my CPU back by going 32-on-64 would be nice, but it's not worth not being able to usefully profile. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed

Re: DRM lock ordering fix series

2009-03-27 Thread Eric Anholt
On Fri, 2009-03-27 at 10:34 +0100, Andi Kleen wrote: Eric Anholt e...@anholt.net writes: Here's hopefully the final attempt at the lock ordering fix for GEM. The problem was introduced in .29 with the GTT mapping support. We hashed out a few potential fixes on the mailing list

Re: [PATCH 1/6] drm/i915: Fix lock order reversal in GTT pwrite path.

2009-03-27 Thread Eric Anholt
On Thu, 2009-03-26 at 17:43 -0700, Jesse Barnes wrote: On Wed, 25 Mar 2009 14:45:05 -0700 Eric Anholt e...@anholt.net wrote: Since the pagefault path determines that the lock order we use has to be mmap_sem - struct_mutex, we can't allow page faults to occur while the struct_mutex

Re: DRM lock ordering fix series

2009-03-27 Thread Eric Anholt
On Fri, 2009-03-27 at 09:19 -0700, Eric Anholt wrote: On Fri, 2009-03-27 at 10:34 +0100, Andi Kleen wrote: Eric Anholt e...@anholt.net writes: Here's hopefully the final attempt at the lock ordering fix for GEM. The problem was introduced in .29 with the GTT mapping support. We

Re: DRM lock ordering fix series

2009-03-27 Thread Eric Anholt
On Fri, 2009-03-27 at 19:10 +0100, Andi Kleen wrote: On Fri, Mar 27, 2009 at 09:36:45AM -0700, Eric Anholt wrote: You are aware that there is a fast path now (get_user_pages_fast) which is significantly faster? (but has some limitations) In the code I have, get_user_pages_fast

[no subject]

2009-03-27 Thread Eric Anholt
: Check for dev-primary-master before dereference. Eric Anholt (8): drm/i915: Change DCC tiling detection case to cover only mobile parts. drm/i915: Fix lock order reversal in GTT pwrite path. drm/i915: Make GEM object's page lists refcounted instead of get/free. drm/i915: Fix

Re: [PATCH 2/6] drm/i915: Make GEM object's page lists refcounted instead of get/free.

2009-03-26 Thread Eric Anholt
on the table we're looking it up from, anyway! So we end up with like 4 locked bus transactions per object in exec that could easily be reduced to 2 total, if we made there be a small object referencing lock covering the handle tables and object refcounts. -- Eric Anholt e...@anholt.net

[PATCH 1/6] drm/i915: Fix lock order reversal in GTT pwrite path.

2009-03-25 Thread Eric Anholt
to using get_user_pages to pin the user's memory, and map those pages atomically when copying it to the GPU. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c | 166 -- 1 files changed, 139 insertions(+), 27 deletions(-) diff --git

[PATCH 2/6] drm/i915: Make GEM object's page lists refcounted instead of get/free.

2009-03-25 Thread Eric Anholt
We've wanted this for a few consumers that touch the pages directly (such as the following commit), which have been doing the refcounting outside of get/put pages. --- drivers/gpu/drm/i915/i915_drv.h |3 +- drivers/gpu/drm/i915/i915_gem.c | 70 --- 2

[PATCH 4/6] drm/i915: Fix lock order reversal in shmem pread path.

2009-03-25 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c | 221 ++- 1 files changed, 195 insertions(+), 26 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index bdc7326..010af90 100644

[PATCH 3/6] drm/i915: Fix lock order reversal in shmem pwrite path.

2009-03-25 Thread Eric Anholt
Like the GTT pwrite path fix, this uses an optimistic path and a fallback to get_user_pages. Note that this means we have to stop using vfs_write and roll it ourselves. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c | 225

DRM lock ordering fix series

2009-03-25 Thread Eric Anholt
Here's hopefully the final attempt at the lock ordering fix for GEM. The problem was introduced in .29 with the GTT mapping support. We hashed out a few potential fixes on the mailing list and at OSTS. Peter's plan was to use get_user_pages, but it has significant CPU overhead (10% cost to text

[PATCH 6/6] drm/i915: Fix lock order reversal in GEM relocation entry copying.

2009-03-25 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net Reviewed-by: Keith Packard kei...@keithp.com --- drivers/gpu/drm/i915/i915_gem.c | 187 +++--- 1 files changed, 133 insertions(+), 54 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915

Re: [Intel-gfx] [PATCH] drm/i915: Fix TV get_modes to return modes count

2009-03-20 Thread Eric Anholt
and applied this series, but in the future please use separate commits for separate issues (for example with this commit, one would be returning the count, one would be int overflow, and one would be the clock accuracy check). -- Eric Anholt e...@anholt.net eric.anh...@intel.com

Re: [Intel-gfx] [PATCH] drm/i915: Check for dev-primary-master before dereference.

2009-03-20 Thread Eric Anholt
them? I think for any future performance debug like these did, we'd just expose values in debugfs (it keeps the debug code from interfering with app behavior, anyway). -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally

Re: [Patch 0/3]: [DRM/I915] : Sync the mode validation for INTERLACE/DBLSCAN

2009-03-20 Thread Eric Anholt
) { if (mode-status == MODE_OK) mode-status = connector_funcs-mode_valid(connector, -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Patch 3/3]: [DRM/I915] : Sync the default modes for LVDS output device

2009-03-20 Thread Eric Anholt
+ fixed_ret + ret; } /** -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- Apps built with the Adobe(R) Flex(R

Re: [Intel-gfx] [PATCH] drm: read EDID extensions from monitor (v2)

2009-03-20 Thread Eric Anholt
, struct edid *edid); extern void drm_mode_probed_add(struct drm_connector *connector, struct drm_display_mode *mode); extern void drm_mode_remove(struct drm_connector *connector, struct drm_display_mode *mode); -- Eric Anholt e...@anholt.net eric.anh...@intel.com

[PATCH] drm: Use a little stash on the stack to avoid kmalloc in most DRM ioctls.

2009-03-19 Thread Eric Anholt
The kmalloc was taking up about 1.5% of the CPU on an ioctl-heavy workload (x11perf -aa10text on 965). Initial results look like they have a corresponding improvement in performance for aa10text, but more numbers might not hurt. Thanks to ajax for pointing out this performance regression I'd

Re: [PATCH] drm: edid revision 0 is valid

2009-03-10 Thread Eric Anholt
w/ CVT calculation, etc. right now for an embedded Linux video project. Would there be any interest incorporating parts of that work? There's work going on for CEA-861 and syncing KMS against UMS. But certainly we love fixes when we don't even have to write them ourselves. -- Eric Anholt e

Re: [Intel-gfx] [PATCH] drm/i915: TV modes' parameters sync up with 2D driver

2009-03-06 Thread Eric Anholt
it my best shot, not being the author of the change. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- Open Source

Re: [Patch 1/5]: DRM/I915: Add the crt save/restore function for VT switch

2009-03-06 Thread Eric Anholt
talking about for user-mode backporting last week? -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- Open Source Business

Re: [Patch 4/5]: DRM/I915: Add the load-detection for crt detect

2009-03-06 Thread Eric Anholt
(intel_output, NULL, + dpms_mode); Indent this guy into lining up with the other arguments. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] make KMS resource mappings exclusive

2009-03-06 Thread Eric Anholt
of really useful userland tools for debugging that rely on read-only mapping of registers, we're pending this until we get that information into the kernel somehow. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed

Re: [Patch 3/5]: DRM/I915: Sync crt hotplug detection with intel video driver

2009-03-06 Thread Eric Anholt
Yakui yakui.z...@intel.com Signed-off-by: Eric Anholt e...@anholt.net (please use this format in the future) -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Intel-gfx] [PATCH] xf86-video-intel: support swapbuffers using page flipping

2009-03-06 Thread Eric Anholt
is getting a outside-of-this-client reference to it and avoid putting it in the BO cache if it gets unreferenced while still being scanned out. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [TIP,regression,i915] /dev/dri/card0 is no longer present

2009-03-03 Thread Eric Anholt
behavior). Give us a mapping, whether or not it's really WC. Failing the mapping is not OK. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: i915 X lockup

2009-02-28 Thread Eric Anholt
On Sat, 2009-02-28 at 00:47 -0800, Andrew Morton wrote: On Sat, 28 Feb 2009 09:31:28 +0100 Jiri Slaby jirisl...@gmail.com wrote: On 28.2.2009 01:20, Eric Anholt wrote: KMS support is not a feature of the server but of your 2D driver. You want 2.6.2, or things will be bad. I have

Re: i915 X lockup

2009-02-28 Thread Eric Anholt
On Sat, 2009-02-28 at 19:24 +0100, Bruno Prémont wrote: On Sat, 28 February 2009 Eric Anholt e...@anholt.net wrote: On Sat, 2009-02-28 at 00:47 -0800, Andrew Morton wrote: The kernel deadlocked on struct_mutex, did it not? That's a kernel bug regardless of what userspace you're running

Re: i915 X lockup

2009-02-27 Thread Eric Anholt
by using nomodesetting so that might be a quick thing to try... KMS support is not a feature of the server but of your 2D driver. You want 2.6.2, or things will be bad. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally

Re: [PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-25 Thread Eric Anholt
On Fri, 2009-02-20 at 08:36 +0100, Peter Zijlstra wrote: On Thu, 2009-02-19 at 18:04 -0800, Eric Anholt wrote: On Thu, 2009-02-19 at 23:26 +0100, Peter Zijlstra wrote: On Thu, 2009-02-19 at 22:02 +0100, Thomas Hellstrom wrote: It looks to me like the driver preferred locking order

Re: [PATCH 2/4] radeon: add r6xx/r7xx microcode

2009-02-24 Thread Eric Anholt
loading framework for DRM drivers instead of building it into the code. Maybe do it with the introduction of new microcode here? [1] http://lkml.org/lkml/2008/8/7/42 -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed

Re: [PATCH] drm: Take mmap_sem up front to avoid lock order violations.

2009-02-20 Thread Eric Anholt
bulk copies (ie. that will blow the L1 cache), do you? 16kb is the most common size (batchbuffers). 32k is popular on 915 (vertex), and varying between 0-128k on 965 (vertex). The pwrite path generally represents 10-30% of CPU consumption in CPU-bound apps. -- Eric Anholt e...@anholt.net

Re: [PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-19 Thread Eric Anholt
object caching in the future, which has the potential of saving steaming piles of memory. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] drm: Take mmap_sem up front to avoid lock order violations.

2009-02-18 Thread Eric Anholt
); + up_read(current-mm-mmap_sem); + pre_mutex_err: drm_free(object_list, sizeof(*object_list) * args-buffer_count, DRM_MEM_DRIVER); -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

[PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-17 Thread Eric Anholt
(copy_from/to_user()), such as i915_gem_pwrite(). Solve this by moving the easy things that needed struct_mutex with mmap_sem held to using a lock to cover just those data structures (offset hash and offset manager), and do trylock and reschedule in fault. Signed-off-by: Eric Anholt e...@anholt.net

Re: [PATCH 1/2] Remove Intel drivers from linux-core

2009-02-15 Thread Eric Anholt
. Signed-off-by: Pekka Paalanen p...@iki.fi --- If the patches are acceptable, I can push them myself to drm.git. Acked-by: Eric Anholt e...@anholt.net -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message

Re: [Mesa3d-dev] radeon-rewrite branch merging.

2009-02-11 Thread Eric Anholt
tree is mostly memory management and fbo handling right now (until I fix it, then we'll hopefully need a more complicated testcase than gears to actually hit it hard). -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally

Re: [Bug #12574] possible circular locking dependency detected

2009-02-10 Thread Eric Anholt
to the GTT mmap code. It's going to be a pain to fix (I tried getting the mmap_sem - struct_mutex path to go away, but the fact that mmap_sem is held over the fault handler pretty much kills that). It's high on the list, though. -- Eric Anholt e...@anholt.net eric.anh...@intel.com

Re: Gem GTT mmaps..

2009-02-10 Thread Eric Anholt
object to be freed sw_finish doesn't mean unmap (note that it doesn't actually unmap). If you want to actually unmap, that should be done with munmap. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message

Re: intel bufmgr concerns?

2009-02-04 Thread Eric Anholt
. What happens if a shared texture, for example, is used by multiple contexts from multiple threads simultaneously? Shared textures and renderbuffers don't have relocations. validate_index is protected by the lock held across bo_exec. check_aperture needs the same locking treatment. -- Eric Anholt

Re: Gem GTT mmaps..

2009-02-04 Thread Eric Anholt
. You do need to refcount the objects. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- Create and Deploy Rich Internet

[PATCH] drm: Fix lock order reversal between mmap_sem and struct_mutex.

2009-02-03 Thread Eric Anholt
()), such as i915_gem_pwrite(). Solve this by moving the one thing that needed struct_mutex with mmap_sem held to using a lock to cover just those data structures (offset hash and offset manager). Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/drm_gem.c |8 drivers/gpu

Re: [PATCH 2/3] drm: Split drm_map and drm_local_map

2009-02-02 Thread Eric Anholt
some remote plans and half finished patch to completely kill the drm_local_map_t typedef so I left those bits in. Signed-off-by: Benjamin Herrenschmidt b...@kernel.crashing.org Thanks for taking on this mess! Acked-by: Eric Anholt e...@anholt.net -- Eric Anholt e...@anholt.net

Re: [PATCH 3/3] drm: Make drm_local_map use a resource_size_t offset

2009-02-02 Thread Eric Anholt
is fixed at a 32-bit offset, while PCI resources may live above that, we ignore the map offset for maps of type _DRM_FRAMEBUFFER or _DRM_REGISTERS. It is assumed that each driver will have only one resource of each type. (I want to remember later what exact ABI problem was in question) -- Eric Anholt

Re: [PATCH 1/3] drm: Use resource_size_t for drm_get_resource_{start, len}

2009-02-02 Thread Eric Anholt
-- ___ Dri-devel mailing list Dri-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/dri-devel -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] drm: Drop unused and broken dri_library_name sysfs attribute.

2009-01-29 Thread Eric Anholt
this. Acked-by: Eric Anholt e...@anholt.net -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- This SF.net email

Re: [Intel-gfx] [PATCH] support tiled rendering in 2D driver

2009-01-27 Thread Eric Anholt
On Sat, 2009-01-24 at 11:01 -0800, Jesse Barnes wrote: On Friday, January 23, 2009 4:24 pm Eric Anholt wrote: +/** + * On some chips, pitch width has to be a power of two tile width, so + * calculate that here. + */ +unsigned long +I830GetFencePitch(I830Ptr pI830, unsigned long

[PATCH] drm: Rip out the racy, unused vblank signal code.

2009-01-27 Thread Eric Anholt
Schedule a vblank signal, kill the process, and we'll go walking over freed memory. Given that no open-source userland exists using this, nor have I ever heard of a consumer, just let this code die. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/drm_irq.c | 161

[PATCH] drm/i915: Skip SDVO/HDMI init when the chipset tells us it's not present.

2009-01-23 Thread Eric Anholt
This saves startup time from probing SDVO, and saves setting up HDMI outputs on G4X devices that don't have them. --- drivers/gpu/drm/i915/intel_display.c | 18 +++--- 1 files changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c

[PATCH] drm/i915: Suppress GEM teardown on X Server exit in KMS mode.

2009-01-23 Thread Eric Anholt
Fixes hangs when starting X for the second time. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c |3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 96316fd..3741101

Re: [Intel-gfx] [PATCH] support tiled rendering in 2D driver

2009-01-23 Thread Eric Anholt
will use the tiling state rather than the unset fence registers, and you'll get swizzled. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Intel-gfx] [PATCH] add fence register management to execbuf

2009-01-23 Thread Eric Anholt
code probably ought to live in i915_gem_tiling.c. Something for the next kernel, I guess. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Intel-gfx] [PATCH] support tiled rendering in 2D driver

2009-01-23 Thread Eric Anholt
On Fri, 2009-01-23 at 16:24 -0800, Eric Anholt wrote: On Fri, 2009-01-23 at 14:15 -0800, Jesse Barnes wrote: Set alignments, tile settings and flags correctly in the 2D driver to support tiled rendering. UXA's create pixmap function currently assumes the worst about the alignment

Re: [Intel-gfx] [RFC] VGA hotplug support for i915 kms

2009-01-17 Thread Eric Anholt
. Our experience with trying to do something sensible when we notice new things plugged in in the early randr 1.2 days was that it was a terrible idea that made users hate us. I suspect it would be even worse if we tried it in the kernel. -- Eric Anholt e...@anholt.net

Re: [PATCH] create IDR mutex for mode_config identifier allocation

2009-01-14 Thread Eric Anholt
configuration (mode lists etc.) */ + struct mutex idr_mutex; /* for IDR management */ struct idr crtc_idr; /* use this idr for all IDs, fb, crtc, connector, modes - just makes life easier */ /* this is limited to one for now */ int num_fb; -- Eric Anholt e...@anholt.net

Re: [PATCH] add debugfs init hooks

2009-01-14 Thread Eric Anholt
, watch the whitespace -- the kernel uses 8-space tabs, not 8 spaces. -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] add DRM_DEBUGFS config option to Kconfig

2009-01-14 Thread Eric Anholt
the debugfs filesystem + for the DRM and drivers. + config DRM_TDFX tristate 3dfx Banshee/Voodoo3+ depends on DRM PCI I'd rather see this not be an option -- if you've got both debugfs and drm in your kernel, you get the drm debugfs bits. -- Eric Anholt e...@anholt.net

Re: Add GEM to i915 broke compilation on MIPS

2009-01-14 Thread Eric Anholt
8a8d7ba9cac5f569c9b0426e583203bab1feeb73 Mon Sep 17 00:00:00 2001 From: Eric Anholt e...@anholt.net Date: Wed, 14 Jan 2009 17:16:25 -0800 Subject: [PATCH] drm: stash AGP include under the do-we-have-AGP ifdef This should fix the MIPS with DRM build. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu

Re: [RFC] add timeout to drmWaitVBlank to avoid hangs

2009-01-07 Thread Eric Anholt
=], [AC_CHECK_LIB([rt], [clock_gettime], [CLOCK_LIB=-lrt], [AC_MSG_ERROR([Couldn't find clock_gettime])])]) AC_SUBST([CLOCK_LIB]) -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] drm/i915: Don't oops when root asks to unpin an already unpinned buffer.

2009-01-06 Thread Eric Anholt
On Tue, 2009-01-06 at 14:22 +, Renato Caldas wrote: Hello, On Tue, Jan 6, 2009 at 12:34 AM, Eric Anholt e...@anholt.net wrote: Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff

Re: KMS with Xorg on G45 failing

2009-01-06 Thread Eric Anholt
happily submit a bug but thought the code was a little too new to harp on about especially if this is a known issue The segfault should be fixed as of: commit 342120be0956bfc12822d1ffbfbd8aaabf3e922f Author: Eric Anholt e...@anholt.net Date: Mon Jan 5 23:21:07 2009 -0800 Fix pin leakage

[PATCH] drm/i915: Don't allow objects to get bound while VT switched.

2009-01-05 Thread Eric Anholt
fallback performance. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c | 15 +-- 1 files changed, 9 insertions(+), 6 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index cc2ca55..14afc23 100644 --- a/drivers

[PATCH] drm/i915: Respect the other stolen memory sizes we know of.

2009-01-05 Thread Eric Anholt
fd.o bug #19336. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_dma.c | 46 --- drivers/gpu/drm/i915/i915_reg.h |8 ++- 2 files changed, 40 insertions(+), 14 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_dma.c b

[PATCH] drm/i915: Pin cursor bo and unpin old bo when setting cursor.

2009-01-05 Thread Eric Anholt
From: Kristian Høgsberg k...@redhat.com We also didn't track the cursor bo before and would leak a reference when the cursor image was change. Signed-off-by: Kristian Høgsberg k...@redhat.com Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/intel_display.c | 29

[PATCH] drm/i915: Don't oops when root asks to unpin an already unpinned buffer.

2009-01-05 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/i915_gem.c |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem.c b/drivers/gpu/drm/i915/i915_gem.c index 14afc23..e87db6f 100644 --- a/drivers/gpu/drm/i915/i915_gem.c +++ b

intel KMS and stability series

2009-01-05 Thread Eric Anholt
Here comes a series of patches I've been cleaning up over the last few days. The first couple are bugfixes that we've tripped over with 2.6.28 that I'll want to get merged to stable. The rest are getting KMS ready to go. Dropped from the series is airlied's vma patch, which caused lock ordering

[PATCH] drm/i915: Fix failure to take the mode_config lock in driver setup.

2009-01-05 Thread Eric Anholt
We're not really concerned with races during setup, but we do want to keep the warning around given how easy it is to misplace that lock. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/drm_crtc_helper.c |3 +++ drivers/gpu/drm/i915/intel_tv.c |2 ++ 2 files changed, 5

[PATCH] drm/i915: Non-mobile parts don't have integrated TV-out.

2009-01-05 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/intel_display.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index 14d82de..5824462 100644 --- a/drivers/gpu/drm/i915

[PATCH] drm/i915: Add support for integrated HDMI on G4X hardware.

2009-01-05 Thread Eric Anholt
This is ported directly from the userland 2D driver code. The HDMI audio bits aren't hooked up yet. Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/i915/Makefile|1 + drivers/gpu/drm/i915/i915_drv.h |2 + drivers/gpu/drm/i915/i915_reg.h | 17

Re: [Intel-gfx] [PATCH] drm/i915: use drm vma accounting functions to make sure VMs don't get lost

2009-01-02 Thread Eric Anholt
vm_area_struct *vma); extern unsigned long drm_core_get_map_ofs(struct drm_map * map); extern unsigned long drm_core_get_reg_ofs(struct drm_device *dev); extern unsigned int drm_poll(struct file *filp, struct poll_table_struct *wait); -- Eric Anholt e...@anholt.net eric.anh

[ANNOUNCE] libdrm-2.4.3

2008-12-22 Thread Eric Anholt
: ddf2876bd8b4b484a9c2a360a835aee8 libdrm-2.4.3.tar.gz SHA1: 465f3b7d26021225ca936043fbfadc2780f13653 libdrm-2.4.3.tar.gz -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH 2/13] drivers/gpu/drm/i915: Remove redundant test

2008-12-21 Thread Eric Anholt
...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH] Pin new and unpin old buffer when setting a mode.

2008-12-19 Thread Eric Anholt
On Wed, 2008-12-17 at 22:14 -0500, Kristian Høgsberg wrote: This removes the requirement for user space to pin a buffer before setting a mode that is backed by the pixels from that buffer. Signed-off-by: Kristian Høgsberg k...@redhat.com Committed. Thanks! -- Eric Anholt e...@anholt.net

Re: [PATCH 0/6] drm/i915: fix sparse warnings

2008-12-19 Thread Eric Anholt
deleting the function, and using the updated #6 patch. Thanks! -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [PATCH 1/3] drivers/gpu/drm: Move a dereference below a NULL test

2008-12-19 Thread Eric Anholt
? The original test is to check it dev-driver-unload exists, not it dev-driver exists. I think your test parameters are wrong. No, it was valid, but I think the better patch is: From 1e0a24cfe75a328db5a50dbcdaaf0eb461638b6b Mon Sep 17 00:00:00 2001 From: Eric Anholt e...@anholt.net Date: Fri, 19

Re: [PATCH] Rework DRM proc file handling

2008-12-19 Thread Eric Anholt
than reviewing a 1700-line patch :) -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: [Intel-gfx] [PATCH] Rework DRM proc file handling

2008-12-19 Thread Eric Anholt
On Sat, 2008-12-20 at 09:21 +1000, Dave Airlie wrote: On Sat, Dec 20, 2008 at 9:19 AM, Eric Anholt e...@anholt.net wrote: On Tue, 2008-12-09 at 14:00 -0500, Ben Gamari wrote: Hey everyone, This is the latest version of my procfs file handling patch. I have ported the old proc files

[PATCH] drm: Add a debug node for vblank state.

2008-12-19 Thread Eric Anholt
Signed-off-by: Eric Anholt e...@anholt.net --- drivers/gpu/drm/drm_irq.c |9 ++ drivers/gpu/drm/drm_proc.c | 63 include/drm/drmP.h |1 + 3 files changed, 73 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/drm_irq.c b

[PATCH] intel: Sync GEM ioctl comments for easier diffing against the kernel.

2008-12-17 Thread Eric Anholt
--- shared-core/i915_drm.h | 27 --- 1 files changed, 20 insertions(+), 7 deletions(-) diff --git a/shared-core/i915_drm.h b/shared-core/i915_drm.h index 976ff18..628f7f8 100644 --- a/shared-core/i915_drm.h +++ b/shared-core/i915_drm.h @@ -469,8 +469,12 @@ struct

[PATCH] intel: Rename plane[AB]* back to pipe[AB]*.

2008-12-17 Thread Eric Anholt
The values are really going to continue meaning pipe, not plane, and that's what they're called in the kernel copy of the header. Userland hasn't ever made the switch to pipe!=plane, since userland checks are based on DRM version, which is still stuck at 1.6. However, Mesa did start using

Re: [Intel-gfx] [PATCH] intel: Limit re-use cache size

2008-12-15 Thread Eric Anholt
; } } -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las

Re: [Intel-gfx] [PATCH] intel: Cache tiling/swizzle state in user mode. Reset tiling on reuse.

2008-12-15 Thread Eric Anholt
; +*swizzle_mode = bo_gem-swizzle_mode; return 0; } -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- SF.Net

Re: [Intel-gfx] [PATCH] Make i830_allocate_memory take tiling parameters.

2008-12-15 Thread Eric Anholt
) { + name, size, PITCH_NONE, GTT_PAGE_SIZE, + ALIGN_BOTH_ENDS, TILE_NONE)) == NULL) { ErrorF(Fail to alloc \n); return BadAlloc; } -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message

Re: [Intel-gfx] [PATCH] intel: When re-use cache is full, wait instead of allocating

2008-12-15 Thread Eric Anholt
-max_entries) + alloc_from_cache = 1; + else { + ret = ioctl(bufmgr_gem-fd, DRM_IOCTL_I915_GEM_BUSY, busy); + alloc_from_cache = (ret == 0 busy.busy == 0); + } if (alloc_from_cache) { bucket-head = bo_gem-next; -- Eric Anholt e...@anholt.net

Re: [Intel-gfx] [PATCH] [intel] Dump out memory usage information when the kernel fails to pin

2008-12-13 Thread Eric Anholt
available aperture, %dkb vs %dkb\n, total / 1024, (int)bufmgr_gem-gtt_size / 1024); return -1; -- 1.5.6.5 ___ Intel-gfx mailing list intel-...@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Eric

Re: [PATCH] libdrm merge of KMS bits

2008-12-12 Thread Eric Anholt
does would be awesome) -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part -- SF.Net email is Sponsored by MIX09, March 18-20

Re: [Intel-gfx] [PATCH] Reallocate frame buffer on resize

2008-12-11 Thread Eric Anholt
-virtualX; + else + fb_height = pScrn-virtualY; +} FbMemBox-x1 = 0; FbMemBox-x2 = pScrn-displayWidth; -- Eric Anholt e...@anholt.net eric.anh...@intel.com signature.asc Description: This is a digitally signed message part

Re: intel text mode restore problem

2008-12-10 Thread Eric Anholt
. -- Eric Anholt [EMAIL PROTECTED] [EMAIL PROTECTED] signature.asc Description: This is a digitally signed message part -- SF.Net email is Sponsored by MIX09, March 18-20, 2009 in Las Vegas, Nevada

Re: intel renderbuffer formats

2008-11-30 Thread Eric Anholt
by the hardware or the driver? Can the hardware do RGBA16? The public specs list r10g10b10a2, r16g16b16a16, and many others as available formats. -- Eric Anholt [EMAIL PROTECTED] [EMAIL PROTECTED] signature.asc Description: This is a digitally signed message part

Re: Handling buffer swaps in composited environments

2008-11-25 Thread Eric Anholt
(waiting only for getting its n-1 frame completed), getting damage posted as it swaps (flips front/back BO handles), and the compositor's picking up the last swapped frame when it wakes up to prepare the next frame to get displayed. -- Eric Anholt [EMAIL PROTECTED] [EMAIL

[PATCH] drm/i915: Respect GM965/GM45 bit-17-instead-of-bit-11 option for swizzling.

2008-11-25 Thread Eric Anholt
intrusive changes targeted at the next kernel release, not this one. Signed-off-by: Eric Anholt [EMAIL PROTECTED] --- drivers/gpu/drm/i915/i915_gem_tiling.c |7 --- drivers/gpu/drm/i915/i915_reg.h|1 + 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/drivers/gpu

Re: [Intel-gfx] [PATCH] [drm/i915] Retry execbuffer pinning after clearing the GTT

2008-11-25 Thread Eric Anholt
*/ + for (i = 0; i pinned; i++) + i915_gem_object_unpin(object_list[i]); + + /* evict everyone we can from the aperture */ + ret = i915_gem_evict_everything(dev); + if (ret) + goto err; } -- Eric

Re: [Intel-gfx] [PATCH] [drm/i915] In execbuffer, split handle lookup and relocation

2008-11-25 Thread Eric Anholt
++) { object_list[i]-pending_read_domains = 0; object_list[i]-pending_write_domain = 0; ret = i915_gem_object_pin_and_relocate(object_list[i], -- Eric Anholt [EMAIL PROTECTED] [EMAIL PROTECTED] signature.asc Description: This is a digitally signed

<    1   2   3   4   5   6   >