Re: [PATCH] drm: Silence the warning for headless machines

2009-10-14 Thread Dave Airlie
stuff reported such failure is a requirement. Dave. Signed-off-by: Chris Wilson ch...@chris-wilson.co.uk Cc: Dave Airlie airl...@redhat.com ---  drivers/gpu/drm/drm_crtc_helper.c |   16 +++-  1 files changed, 7 insertions(+), 9 deletions(-) diff --git a/drivers/gpu/drm

Re: [PATCH] drm/kms: add support for memory bandwidth considerations

2009-10-14 Thread Dave Airlie
On Wed, Oct 14, 2009 at 2:33 AM, Eric Anholt e...@anholt.net wrote: On Tue, 2009-10-13 at 13:30 +1000, Dave Airlie wrote: From: Dave Airlie airl...@redhat.com Some GPUs (mostly in the lower server end of the market) have memory bandwidth limitations that hit in the real world, however memory

[PATCH] drm/kms: add support for memory bandwidth considerations

2009-10-12 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Some GPUs (mostly in the lower server end of the market) have memory bandwidth limitations that hit in the real world, however memory bw calcs require bit depth to be worked out and at the moment, the KMS interface doesn't know about bit depth until we set

[PATCH] libdrm: add drmModeGetConnectorWithDepth API for memory bw concerned drivers.

2009-10-12 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This API should be used going forward instead of drmModeGetConnector. Signed-off-by: Dave Airlie airl...@redhat.com --- libdrm/xf86drmMode.c |9 +++-- libdrm/xf86drmMode.h |9 + 2 files changed, 16 insertions(+), 2 deletions(-) diff

[PATCH] drm/radeon/kms: add support for encoder cloning.

2009-10-12 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com The RN50 really needs this since its a single crtc card, however other gpus may benefit from it as well. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/radeon_encoders.c| 38 ++- drivers/gpu/drm/radeon

[PATCH] drm/radeon/kms: add support for encoder cloning (v2)

2009-10-12 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com The RN50 really needs this since its a single crtc card, however other gpus may benefit from it as well. Changes since v1: add the clones after initialising all the encoders. Userspace DDX changes are required as it reports connector clones over randr

Re: [PATCHES] radeon KMS updates

2009-10-11 Thread Dave Airlie
patches: 1 - make sure lvds panels have valid native mode info. if not, report lvds as disconnected 2 - fix up crtc dpms ordering based on recommendation from bios docs 3 - add support for AdjustDisplayPll table which is used to adjust the clock to meet special requirements of different

[PATCH] drm/radeon/kms: fix use of index before proper initialisation.

2009-10-11 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Alex I suspect the DDX needs a similiar patch. With the adjust table introduction, we need to fill out index before getting the table version. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/atombios_crtc.c |2 +- 1 files

Re: [PATCHES] radeon KMS updates

2009-10-11 Thread Dave Airlie
1 - make sure lvds panels have valid native mode info. if not, report lvds as disconnected 2 - fix up crtc dpms ordering based on recommendation from bios docs 3 - add support for AdjustDisplayPll table which is used to adjust the clock to meet special requirements of different encoders

[PATCH] drm/radeon/kms: fix vline register for second head.

2009-10-07 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Both r100/r600 had this wrong, use the macro to extract the register to relocate. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/r100.c|2 +- drivers/gpu/drm/radeon/r600_cs.c |2 +- 2 files changed, 2 insertions(+), 2

[git pull] more drm kms code

2009-10-07 Thread Dave Airlie
c1176d6f03e1085797ce83648a2c76ae15a2b515 Merge: 0eca52a d4ac6a0 Author: Dave Airlie airl...@redhat.com Date: Thu Oct 8 14:03:05 2009 +1000 Merge branch 'drm-next' of ../drm-next into drm-linus conflict in radeon since new init path merged with vga arb code. Conflicts

Re: [PATCH] drm/radeon/kms: Convert RS600 to new init path

2009-10-06 Thread Dave Airlie
On Thu, Oct 1, 2009 at 6:09 AM, Jerome Glisse jgli...@redhat.com wrote: New init path allow to simply asic initialization and make easier to trace what happen on each different asic. We are removing most callback. Do a massive RS600 register cleanup to clarify RS600 register, we are still bit

[PATCH 2/2] drm/fb: add setcmap and fix 8-bit support.

2009-10-04 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This adds support for the setcmap api and fixes the 8bpp support at least on radeon hardware. It adds a new load_lut hook which can be called once the color map is setup. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/drm_fb_helper.c

[PATCH 1/2] drm/radeon/kms: respect single crtc cards, only create one crtc. (v2)

2009-10-04 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Also add single crtc for RN50 chips. changes in v2: fix vblank init to respect single crtc flag fix r100 mode bandwidth to respect single crtc flag Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/r100.c |8

Re: [PATCH] drm/radeon/kms: respect single crtc cards, only create one crtc.

2009-10-02 Thread Dave Airlie
       {0x1002, 0x5a61, PCI_ANY_ID, PCI_ANY_ID, 0, 0, CHIP_RS400|RADEON_IS_IGP|RADEON_IS_IGPGART}, \ Original radeons (R100) are also single crtc cards. 0x5144, 0x5145, 0x5146, 0x5147 They already had the flag set from ages ago. there was bug in r100.c also to make this patch work, its

kms and memory bandwidth limitations

2009-10-01 Thread Dave Airlie
So I was looking at the memory bandwidth limitations that ajax added to the X server, and how we could add the concept to KMS. Now the X server is in a lucky position as when it gets the modes it also knows what bit depth its going to run in, and it can use that at the mode validation time to

[PATCH] drm/radeon/kms: respect single crtc cards, only create one crtc.

2009-10-01 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com Also add single crtc for RN50 chips. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/r100.c |8 +--- drivers/gpu/drm/radeon/radeon_display.c |6 +- drivers/gpu/drm/radeon/radeon_encoders.c

Re: [PATCH] drm/i915: add _PAGE_IOMAP to GTT mapping attributes

2009-09-30 Thread Dave Airlie
On 09/30/09 03:13, Jan Beulich wrote: This is for avoiding translation of the passed in address in virtual environments like Xen. Ah, thanks. I suspected one of these was missing and was about to go hunting for it. Do think other DRI drivers are missing this too? Hm,

Re: [PATCH] drm/radeon/kms: Convert RS690/RS740 to new init path.

2009-09-30 Thread Dave Airlie
On Tue, Sep 29, 2009 at 10:32 PM, Jerome Glisse jgli...@redhat.com wrote: Also cleanup register specific to RS690/RS740. Signed-off-by: Jerome Glisse jgli...@redhat.com ---  drivers/gpu/drm/radeon/radeon.h      |   10 +  drivers/gpu/drm/radeon/radeon_asic.h |   41 ++--  

[PATCH] drm/ttm: add pool wc/uc page allocator

2009-09-28 Thread Dave Airlie
pulled from pools implemented. ] Signed-off-by: Jerome Glisse jgli...@redhat.com Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/ttm/Makefile |2 +- drivers/gpu/drm/ttm/ttm_memory.c |5 +- drivers/gpu/drm/ttm/ttm_page_alloc.c | 341

[no subject]

2009-09-28 Thread Dave Airlie
of the architecture is now in place and more code sharing can happen btw various asics. Signed-off-by: Jerome Glisse jgli...@redhat.com Signed-off-by: Dave Airlie airl...@linux.ie commit d39c3b895876427c5083a936e00f3f5b7f0fc1b3 Author: Jerome Glisse jgli...@redhat.com Date: Mon Sep 28 18:34:43

[PATCH] drm/ttm: fix refcounting in ttm global code.

2009-09-28 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie the global refcount wasn't being increased after the first reference. this caused an oops on unload on a multi-gpu card. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/ttm/ttm_global.c |2 +- 1 files changed, 1 insertions(+), 1 deletions

[git pull] drm

2009-09-28 Thread Dave Airlie
to new init path, change are smaller than previous one as most of the architecture is now in place and more code sharing can happen btw various asics. Signed-off-by: Jerome Glisse jgli...@redhat.com Signed-off-by: Dave Airlie airl...@linux.ie commit

[PATCH] drm/radeon/kms: remove unneeded master create/destroy functions.

2009-09-23 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com We shouldn't need these at all in radeon kms mode. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/radeon_drv.c |5 --- drivers/gpu/drm/radeon/radeon_kms.c | 49 --- 2 files changed, 0 insertions

[PATCH] [RFC] drm/radeon/kms: don't require up to 64k allocations.

2009-09-23 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This avoids needing to do a kmalloc PAGE_SIZE for the main indirect buffer chunk, it adds an accessor for all reads from the chunk and caches a single page at a time for subsequent reads. TODO: r600 fixups. --- drivers/gpu/drm/radeon/r100.c | 188

Re: [PATCH] [RFC] drm/radeon/kms: don't require up to 64k allocations.

2009-09-23 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This avoids needing to do a kmalloc PAGE_SIZE for the main indirect buffer chunk, it adds an accessor for all reads from the chunk and caches a single page at a time for subsequent reads. FWIW, this works on my PowerBook but seems to drop

[PATCH] drm/radeon/kms: don't require up to 64k allocations. (v2)

2009-09-23 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This avoids needing to do a kmalloc PAGE_SIZE for the main indirect buffer chunk, it adds an accessor for all reads from the chunk and caches a single page at a time for subsequent reads. changes since v1: Use a two page pool which should be the most common

Re: [origin tree build failure] [PATCH] Re: [git pull] drm tree.

2009-09-22 Thread Dave Airlie
Subject: [PATCH] vgaarb: wrap the client register API so we can disable VGA ARB. This provides an dummy register function so everything builds if VGA arb is turned off. Signed-off-by: Dave Airlie airl...@redhat.com --- include/linux/vgaarb.h | 11 ++- 1 files

Re: preventing GPU reset DoS

2009-09-22 Thread Dave Airlie
On Wed, Sep 23, 2009 at 5:13 AM, Pauli Nieminen suok...@gmail.com wrote: Hi! I have been thinking GPU reset as possible DoS attack from user-space.Problem here is that display doesn't work anymore at all if attacker chooses to run a application that constantly causes GPU hang. It would be of

[git pull] two radeon fixes.

2009-09-22 Thread Dave Airlie
alexdeuc...@gmail.com commit c214271563c00f2721c5111e27b53bf06dabc6e4 Author: Dave Airlie airl...@linux.ie Date: Tue Sep 22 08:50:10 2009 +1000 drm/radeon: consolidate family flags used in pciids. having these separate was pointless and introduced a bug when one got updated

[PATCH] drm/kms: start adding command line interface using fb.

2009-09-22 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie [note this requires an fb patch posted to linux-fbdev-devel already] This uses the normal video= command line option to control the kms output setup at boot time. It is used to override the autodetection done by kms. video= normally takes a framebuffer

Re: [origin tree build failure] [PATCH] Re: [git pull] drm tree.

2009-09-21 Thread Dave Airlie
builds to work until I figure out any Kconfig magic. From 8a874578cbf8b07b988e666c15fa0ba767f3c1cb Mon Sep 17 00:00:00 2001 From: Dave Airlie airl...@redhat.com Date: Tue, 22 Sep 2009 13:53:00 +1000 Subject: [PATCH] vgaarb: wrap the client register API so we can disable VGA ARB. This provides

[PATCH 3/3] drm/radeon/kms: move around new init path code to avoid posting at init

2009-09-18 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie We really don't want to post the card at init, it takes a relatively long time and isn't required, so split the resume path into a startup path called by both init/resume and separate resume entry point to do posting. Signed-off-by: Dave Airlie airl

[PATCH 2/3] radeon/r600: fix some issues with suspend/resume.

2009-09-18 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie a) don't zero gart table on gart enable b) move pinning shader object into resume path c) unpin shader object on suspend d) set cp ready to false after cp shutdown on suspend. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/r600.c

Re: [PATCH] drm: fix drm_fb_helper handling of kernel crtcs

2009-09-18 Thread Dave Airlie
its initial config. Ideally it shouldn't even include CRTCs in its initial config if they're not in use, but my old fix for that no longer works. At any rate, this fixes a real bug I was seeing where after a console blank, both pipes would come back on, even though only one had been enabled

Re: [PATCH] drm: update crtc x/y when only fb changes

2009-09-18 Thread Dave Airlie
From: Ben Skeggs bske...@redhat.com applied to drm-next. Dave. -- Come build with us! The BlackBerryreg; Developer Conference in SF, CA is the only developer event you need to attend this year. Jumpstart your

Re: [PATCH] drm/radeon/kms: Move radeon_get_clock_info() call out of radeon_clocks_init().

2009-09-18 Thread Dave Airlie
From: Michel Dänzer daen...@vmware.com Someone on IRC reported problems after commit 95a8f1bf4f48b434c9f839ab5a0773f66b39d7c6 ('drm/radeon/kms: Move radeon_clocks_init() call back after getting VRAM info.'). And indeed, at least some ASIC vram_info hooks use the clock info obtained by

Re: [PATCH] drm/radeon/r600: don't do interrupts

2009-09-18 Thread Dave Airlie
From 8568f7c5f5729f9fb7a645b5e77835c52bb3abed Mon Sep 17 00:00:00 2001 From: Alex Deucher alexdeuc...@gmail.com Date: Thu, 17 Sep 2009 14:25:12 -0400 Subject: [PATCH] drm/radeon/r600: don't do interrupts Interrupts are not supported yet. This prevents things like mesa from trying to use

Re: [PATCH] drm/radeon/kms: don't fail if we fail to init GPU acceleration

2009-09-18 Thread Dave Airlie
kernel failed to initialize KMS. This should allow to give a working KMS setup in all case (even with non functionning accel). Signed-off-by: Jerome Glisse jgli...@redhat.com Applied thanks, Dave. -- Come build

Re: [PATCH] drm/radeon/kms: Move radeon_get_clock_info() call out of

2009-09-18 Thread Dave Airlie
An rv350 specific issue then? I booted it on my desktop rv370 and I'm sure Michel has booted it on his rv350 macbook, so might be a bios issue or soemthing else. Dave. -- Come build with us! The BlackBerryreg;

[PATCH] drm/r600/kms: fixup number of loops per blit calculation.

2009-09-17 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie Some people were seeing *ERROR* radeon: writting more dword to ring than expected after certain blits, the loops calculation didn't take into account that we do a separate blit for the remainder after doing the aligned blits. Signed-off-by: Dave Airlie airl

drm/radeon/kms: disable VGA rendering engine before taking over VRAM

2009-09-17 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie Before we use any of VRAM, we need to disable the VGA rendering engine, this render text mode into a graphical framebuffer for scanout, however it does this on vblank, and can end up overwriting the GART table and r600 shader objects. Signed-off-by: Dave Airlie

Re: [PATCH] drm/radeon/kms: Use surfaces for scanout / cursor byte swapping on big endian.

2009-09-16 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:10 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com diff --git a/include/drm/radeon_drm.h b/include/drm/radeon_drm.h index 2ba61e1..341c21a 100644

Re: [PATCH] drm/radeon/kms: Use depth 16 for console.

2009-09-16 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:20 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:03 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com Now

Re: [PATCH] drm/radeon/kms: Use depth 16 for console.

2009-09-16 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 17:16 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:20 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:03 +1000, Dave Airlie

Re: [PATCH] drm/radeon/kms: Use surfaces for scanout / cursor byte swapping on big endian.

2009-09-16 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 17:13 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:10 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com diff

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-15 Thread Dave Airlie
Are you saying Yes, it is right to carry version information in the drm.h file? No I'm still in no way convinced of this, the fact Thomas doesn't see it as a requirement either, and *no* other drm driver does it, is all pointing towards its unnecessary. You seem to think its obvious but we've

Re: [PATCH] drm/kms: Force full modeset if depth/bpp changes.

2009-09-15 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com This one confuses me, the bit depth and stuff should all be setup in the set base function, I wonder is there a bug somewhere that we aren't dealing with. Dave. Signed-off-by: Michel Dänzer daen...@vmware.com

Re: [PATCH] drm/radeon/kms: Use depth 16 for console.

2009-09-15 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com Now that we can handle 16 bpp on big endian as well, we can save VRAM like this and probably also improve console output speed. The console only uses a limited number of colours anyway. (8 bpp might be even

Re: [PATCH] drm/radeon/kms: Use surfaces for scanout / cursor byte swapping on big endian.

2009-09-15 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com @@ -200,6 +201,7 @@ void radeon_object_kunmap(struct radeon_object *robj)        }        robj-kptr = NULL;        spin_unlock(robj-tobj.lock); +       radeon_object_check_tiling(robj, 0, 0);        

Re: [PATCH] drm/radeon/kms: Use depth 16 for console.

2009-09-15 Thread Dave Airlie
2009/9/16 Michel Dänzer mic...@daenzer.net: On Wed, 2009-09-16 at 08:03 +1000, Dave Airlie wrote: 2009/9/16 Michel Dänzer mic...@daenzer.net: From: Michel Dänzer daen...@vmware.com Now that we can handle 16 bpp on big endian as well, we can save VRAM like this and probably also

[PATCH] drm/radeon/kms: reprogram format in set base.

2009-09-15 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie This should in theory fix the problem with a mode set being required for adjusting the color depth. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/radeon_legacy_crtc.c | 29 +++ 1 files changed, 29 insertions

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-14 Thread Dave Airlie
Hello Luc and Dave:    Thank you very much for your comment on the UniChrome DRM. And sorry for the trouble I made. Based on your comment, we modify our UniChrome patch as below(I would like to call it Ver1.5 because it's for reference not for fomal submittion): . The attached

[PATCH] drm/radeon/kms: IB locking dumps out a lockdep ordering issue

2009-09-13 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie We sometimes lock IB then the ring and sometimes the ring then the IB. This is mostly due to the IB locking not being well defined about what data in the structs it actually locks. Define what I believe is the correct behaviour and gets rid of the lock dep

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-11 Thread Dave Airlie
No thats where you got it wrong, a driver should never *require* version of interface at runtime == version of interface at build time. We rarely make incompatible major number changes in the kernel drivers, (radeon kms being the first in my memory). DRM drivers ship in the kernel,

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-10 Thread Dave Airlie
These patches break both free drivers out there. They not only break the API, they also require some of these ioctls to be used correctly for correct initialisation. There seems to be no attempt at working with these two drivers to fix this specific issue. I'm looking for the API break but

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-10 Thread Dave Airlie
As a first answer, without going in depth, as i just returned from my thursday constitutional. Do you have an explanation as to why this commit never made it to the kernel? Because it probably wasn't noticed, feel free to resend it. I'm not sure why you need a version inside the

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-10 Thread Dave Airlie
Because it probably wasn't noticed, feel free to resend it. I'm not sure why you need a version inside the via_drm.h but I'm willing to accept that the via driver development process is messed up enough to require it. No other driver has needed it. How do graphics drivers tell

Re: [Patch 0/2] [VIA UniChrome DRM] Patch system hang issue caused by 3D scaling+ACPI

2009-09-10 Thread Dave Airlie
What should the canonical source of such versioning information be? * This header file defines the interface, and this versioning included in the same headerfile should then niquely identify this interface. * driver builds against this header and should then require this version of the

Re: [PATCH 2/3] drm/radeon/kms: move mtrr range add and memory information

2009-09-10 Thread Dave Airlie
/* + * Printing helpers + */ +#define PINF(s, arg...) printk(KERN_INFO radeon s, ##arg) +#define PWRN(s, arg...) \ + printk(KERN_WARNING radeon (WR:%s:%d) s, __FILE__, __LINE__, ##arg) +#define PERR(s, arg...) \ + printk(KERN_ERR radeon (ER:%s:%d) s, __FILE__, __LINE__,

[PATCH] drm/radeon/kms: add initial connector properties

2009-09-09 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This adds: coherent mode: TMDS coherent mode for atom cards. scaling mode: LVDS scaler mode load detect: DAC load detection, DVI-I, VGA, TV tmds pll: legacy TMDS pll selection tv standard: TV standard selection. TODO: other TV ones? dvi subconnector selection

Re: [PATCH] drm/radeon/kms: Don't try to process irq when we are unloading

2009-09-09 Thread Dave Airlie
On Wed, 9 Sep 2009, Jerome Glisse wrote: If module is being unloaded we should not try to handle irq especialy we should not call into drm helper or we could hard hang the computer free_irq will call the irq handler to make sure we behave properly. Should you not just ack all irqs here before

[git pull] drm-fixes for final

2009-09-06 Thread Dave Airlie
|2 +- 5 files changed, 70 insertions(+), 69 deletions(-) commit a54775c8758a754186bc6adbfc518b1e9f8f1e4e Author: Dave Airlie airl...@redhat.com Date: Mon Sep 7 15:26:19 2009 +1000 drm/radeon/kms: add LTE/GTE discard + rv515 two sided stencil register. This adds some rv350

Re: [Patch 4/4] DRM/I915: Add the default mode for CRT output without EDID

2009-09-02 Thread Dave Airlie
On Thu, 2009-09-03 at 09:33 +0800, yakui.z...@intel.com wrote: From: Zhao Yakui yakui.z...@intel.com Add the default mode for the CRT output without EDID. But we only add the mode which visible part is equal to or less than 1024x768. Use 800x600 as the default mode for consistency with X

[PATCH] drm/radeon/kms: block depthxy offset from use from userspace.

2009-08-31 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This could be used to bypass CS checks. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/reg_srcs/r300 |1 - drivers/gpu/drm/radeon/reg_srcs/rs600 |1 - drivers/gpu/drm/radeon/reg_srcs/rv515 |1 - 3 files changed, 0

Re: [Patch] DRM: Parse the detailed time info in CEA-EDID

2009-08-30 Thread Dave Airlie
it will skip the CEA-data block and parse the detailed timing info. Signed-off-by: Zhao Yakui yakui.z...@intel.com --- Pushed to drm-next. Dave. -- Let Crystal Reports handle the reporting - Free Crystal Reports

Re: [PATCH 1/3] drm/crtc_helper: replace modeset fail path with something simpler

2009-08-30 Thread Dave Airlie
Signed-off-by: Maarten Maathuis madman2...@gmail.com -- David Airlie, Software Engineer http://www.skynet.ie/~airlied / airlied at skynet.ie Linux kernel - DRI, VAX / pam_smb / ILUG On Thu, 27 Aug 2009, Maarten Maathuis wrote: - The previous system was not very transparent, nor flexible. -

Re: [PATCH] drm: dereference of tmp in drm_proc_create_files()

2009-08-30 Thread Dave Airlie
tmp allocation may fail, prevent a dereference. Thanks, pushed to drm-next. Dave. Signed-off-by: Roel Kluin roel.kl...@gmail.com --- diff --git a/drivers/gpu/drm/drm_proc.c b/drivers/gpu/drm/drm_proc.c index bbd4b3d..dc967af 100644 --- a/drivers/gpu/drm/drm_proc.c +++

Re: [PATCH] r128: Add test for initialisation to all ioctls that require it

2009-08-30 Thread Dave Airlie
Almost all r128's private ioctls require that the CCE state has already been initialised. However, most do not test that this has been done, and will proceed to dereference a null pointer. This may result in a security vulnerability, since some ioctls are unprivileged. This adds a macro

Re: [PATCH 3/3] radeon: Use request_firmware()

2009-08-30 Thread Dave Airlie
KMS support by Dave Airlie airl...@redhat.com. For Radeon 100- to 500-series, firmware blobs look like: struct { __be32 datah; __be32 datal; } cp_ucode[256]; For Radeon 600-series, there are two separate firmware blobs: __be32 me_ucode[PM4_UCODE_SIZE * 3

[PATCH] drm/kms: add explicit encoder disable function and detach harder.

2009-08-30 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com For shared tv-out and VGA encoders, we really need to know if the encoder is just being switched off temporarily in blanking or if we are really disabling it hard. Also we need to try harder to disconnect encoders from unused connectors so we can share more

Re: Linux 2.6.31-rc7

2009-08-26 Thread Dave Airlie
On Wed, Aug 26, 2009 at 4:26 PM, Eric Anholte...@anholt.net wrote: On Wed, 2009-08-26 at 13:47 +1000, Dave Airlie wrote: On Wed, Aug 26, 2009 at 1:33 PM, Linus Torvaldstorva...@linux-foundation.org wrote: On Wed, 26 Aug 2009, Zhenyu Wang wrote: In my experience, the BIOS setup

[PATCH] drm/ttm: consolidate cache flushing code in one place.

2009-08-26 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This merges the TTM and drm cache flushing into one file in the drm core. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/drm_cache.c | 51 ++- drivers/gpu/drm/ttm/ttm_tt.c | 67

Re: [PATCH 3/3] radeon: Use request_firmware()

2009-08-26 Thread Dave Airlie
On Mon, Aug 24, 2009 at 3:58 AM, Ben Hutchingsb...@decadent.org.uk wrote: Based on a patch by Jaswinder Singh Rajput . For Radeon 100- to 500-series, firmware blobs look like: struct { __be32 datah; __be32 datal; } cp_ucode[256]; For Radeon 600-series, firmware blobs look like: __be32

[PATCH] drm/rs600/690: use autogenerated safe register tables.

2009-08-26 Thread Dave Airlie
From: Dave Airlie airl...@redhat.com This ports rs690 to the safe reg tables and makes rs600 also use the same table. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/radeon/Makefile |5 + drivers/gpu/drm/radeon/radeon_asic.h |6 +- drivers/gpu/drm/radeon

Re: [PATCH 3/3] radeon: Use request_firmware()

2009-08-26 Thread Dave Airlie
On Thu, Aug 27, 2009 at 11:07 AM, Dave Airlieairl...@gmail.com wrote: On Mon, Aug 24, 2009 at 3:58 AM, Ben Hutchingsb...@decadent.org.uk wrote: Based on a patch by Jaswinder Singh Rajput . For Radeon 100- to 500-series, firmware blobs look like: struct { __be32 datah; __be32 datal; }

Re: [PATCH 1/3] drm/crtc_helper: replace modeset fail path with something simpler

2009-08-26 Thread Dave Airlie
On Wed, 19 Aug 2009, Maarten Maathuis wrote: - The previous system was not very transparent, nor flexible. - This is needed to be able to fix a few bugs in the mechanism. This fails checkpatch.pl, please fix whitespace. ERROR: trailing whitespace #75: FILE:

checkpatch.pl requirement

2009-08-26 Thread Dave Airlie
I've been a bit lax on this in the past but its only going to cause me further headaches if I keep fixing stuff up. So any patches you want to go upstream need to show 0 errors on scripts/checkpatch.pl from the Linux tree. I will accept some 80 char line warnings if the code is at least more

Re: TTM pool allocator

2009-08-26 Thread Dave Airlie
Hi, Attached new version of pool allocator not much changed. - Use kmap_atomic_prot on highmem page to clear them if necessary (please check that i do use proper pgprot) - Force populate on cache change (discussed previously the reason for this). - Allocate structure to hold the pool

Re: Intel 915 ... GEM and the massive changes all over the place

2009-08-25 Thread Dave Airlie
Was it necessary? Yes. Intel needs to hire Alan Cox. At least he had the balls to throw in the towel. they did. no, thank you. Dave. Thanks a lot . -- Let Crystal Reports handle the reporting - Free

[PATCH] drm/mm: add ability to dump mm lists via debugfs (v3)

2009-08-25 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie This adds code to the drm_mm to talk to debugfs, and adds support to radeon to add the VRAM and GTT mm lists to debugfs. I tested with spinlock debugging and it doesn't give out. changes since v2: use proper locking. Signed-off-by: Dave Airlie airl

Re: Linux 2.6.31-rc7

2009-08-25 Thread Dave Airlie
On Wed, Aug 26, 2009 at 1:33 PM, Linus Torvaldstorva...@linux-foundation.org wrote: On Wed, 26 Aug 2009, Zhenyu Wang wrote: In my experience, the BIOS setup doesn't reflect what outputs should be used at runtime, and certainly not the correct configuration of the enabled outputs.  For

Re: Linux 2.6.31-rc7

2009-08-25 Thread Dave Airlie
On Wed, Aug 26, 2009 at 2:13 PM, Linus Torvaldstorva...@linux-foundation.org wrote: On Wed, 26 Aug 2009, Dave Airlie wrote: If you actually detected things _right_, none of this would be an issue. But you don't. And you seem to have a really hard time even admitting that. You try to re

[git pull] minor drm fix for kms busy.

2009-08-23 Thread Dave Airlie
Author: Michel Dänzer daen...@vmware.com Date: Sat Aug 22 17:38:23 2009 +0200 drm/radeon/kms: Fix radeon_gem_busy_ioctl harder. It was mixing up TTM placement values and flags. Signed-off-by: Michel Dänzer daen...@vmware.com Signed-off-by: Dave Airlie airl...@redhat.com

Re: [PATCH] drm/mm: add ability to dump mm lists via debugfs (v2)

2009-08-21 Thread Dave Airlie
2009/8/21 Thomas Hellström tho...@shipmail.org: Dave Airlie wrote: From: Dave Airlie airl...@linux.ie This adds code to the drm_mm to talk to debugfs, and adds support to radeon to add the VRAM and GTT mm lists to debugfs. changes since v1: don't bother with free list just add used/free

Re: Lost patch

2009-08-20 Thread Dave Airlie
2009/8/20 Thomas Hellström tho...@shipmail.org: Dave, The patch titled [PATCH] drm: Fix sysfs device confusion seems to have been lost somewhere. It's not drm-next and is a prerequisite for the ttm sysfs patches. Funny, Jbarnes have replied by a Reviewed-by, but  I can't find it in my

Re: [PATCH] drm: Fix sysfs device confusion.

2009-08-20 Thread Dave Airlie
On Thu, Aug 20, 2009 at 5:52 PM, Thomas Hellstromthellst...@vmware.com wrote: The drm sysfs class suspend / resume methods could not distinguish between different device types wich could lead to illegal type casts. Use struct device_type and make sure the class suspend / resume callbacks are

Re: [PATCH] drm/kms/radeon: Add kconfig to specify the default modesetting

2009-08-20 Thread Dave Airlie
On Thu, Aug 20, 2009 at 11:26 PM, Takashi Iwaiti...@suse.de wrote: This patch adds a Kconfig item to specify the default radeon modeset value.  Also fixed the description of CONFIG_DRM_RADEON_KMS. Since KMS gives problems on some Radeon devices, it's safer to disable it as default on distro

[PATCH] drm/mm: add ability to dump mm lists via debugfs (v2)

2009-08-20 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie This adds code to the drm_mm to talk to debugfs, and adds support to radeon to add the VRAM and GTT mm lists to debugfs. changes since v1: don't bother with free list just add used/free to main list add totals in pages Signed-off-by: Dave Airlie airl

[PATCH] drm/mm: add ability to dump mm lists via debugfs.

2009-08-20 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie This adds code to the drm_mm to talk to debugfs, and adds support to radeon to add the VRAM and GTT mm lists to debugfs. It allows you to see the used and free lists. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/drm/drm_mm.c| 23

Re: [PATCH] ttm: Make parts of a struct ttm_bo_device global.

2009-08-19 Thread Dave Airlie
On Wed, Aug 19, 2009 at 12:51 AM, Thomas Hellstromthellst...@vmware.com wrote: Common resources, like memory accounting and swap lists should be global and not per device. Introduce a struct ttm_bo_global to accomodate this, and register it with sysfs. Add a small sysfs interface to return the

Re: [PATCH] ttm: Make parts of a struct ttm_bo_device global.

2009-08-19 Thread Dave Airlie
On Thu, Aug 20, 2009 at 3:11 PM, Dave Airlieairl...@gmail.com wrote: On Wed, Aug 19, 2009 at 12:51 AM, Thomas Hellstromthellst...@vmware.com wrote: Common resources, like memory accounting and swap lists should be global and not per device. Introduce a struct ttm_bo_global to accomodate

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-18 Thread Dave Airlie
+#undef set_base +  struct drm_prop_enum_list {        int type;        char *name; @@ -342,6 +344,34 @@ void drm_framebuffer_cleanup(struct drm_framebuffer *fb)  EXPORT_SYMBOL(drm_framebuffer_cleanup);  /** + * drm_crtc_async_flip - do a set_base call from a work queue + * @work: work

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-18 Thread Dave Airlie
On Wed, Aug 19, 2009 at 2:12 AM, Keith Whitwellkei...@vmware.com wrote: I think the bug in question was because somebody (Jon Smirl??) removed the empty apparently unused poll implementation from the drm fd, only to discover that the X server was actually polling the fd. If this code adds

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-18 Thread Dave Airlie
On Wed, Aug 19, 2009 at 8:03 AM, Luc Verhaegenl...@skynet.be wrote: On Wed, Aug 19, 2009 at 07:03:41AM +1000, Dave Airlie wrote: On Wed, Aug 19, 2009 at 2:12 AM, Keith Whitwellkei...@vmware.com wrote: I think the bug in question was because somebody (Jon Smirl??) removed the empty

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-18 Thread Dave Airlie
On Wed, Aug 19, 2009 at 9:31 AM, Luc Verhaegenl...@skynet.be wrote: On Wed, Aug 19, 2009 at 09:22:10AM +1000, Dave Airlie wrote: On Wed, Aug 19, 2009 at 9:12 AM, Luc Verhaegenl...@skynet.be wrote: On Wed, Aug 19, 2009 at 09:07:55AM +1000, Dave Airlie wrote: On Wed, Aug 19, 2009 at 8:03 AM

[git pull] drm-fixes for -final

2009-08-18 Thread Dave Airlie
insertions(+), 254 deletions(-) commit 5ef5f72febfea420ce58f670bad83830a5e5e3de Author: Dave Airlie airl...@redhat.com Date: Mon Aug 17 13:11:23 2009 +1000 drm/kms: teardown crtc correctly when fb is destroyed. If userspace destroys a framebuffer that is in use on a crtc, don't

[PATCH] PCI/vgaarb: cleanup some warnings + cleanup some comments.

2009-08-18 Thread Dave Airlie
From: Dave Airlie airl...@linux.ie Fix some warnings reported in linux-next + also cleanup some comment errors noticed by Pekka Paalanen. Signed-off-by: Dave Airlie airl...@redhat.com --- drivers/gpu/vga/vgaarb.c | 11 - include/linux/vgaarb.h | 49

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-17 Thread Dave Airlie
A couple of years ago, any attempt to return anything else than 0 from drm poll resulted in an X server error. http://freedesktop.org/bugzilla/show_bug.cgi?id=1505. The fix mentioned in the bug was actually to return 0 from drm poll, and a comment about this is still present in drm.git. The

Re: [PATCH] Add modesetting pageflip ioctl and corresponding drm event

2009-08-17 Thread Dave Airlie
On Tue, Aug 18, 2009 at 9:54 AM, Kristian Høgsbergk...@bitplanet.net wrote: On Mon, Aug 17, 2009 at 7:14 PM, Dave Airlieairl...@gmail.com wrote: A couple of years ago, any attempt to return anything else than 0 from drm poll resulted in an X server error.

<    1   2   3   4   5   6   7   8   9   10   >