Re: Font selection with i915

2010-01-20 Thread Alan Stern
On Wed, 20 Jan 2010, Corbin Simpson wrote: > FYI the legacy non-KMS path (nomodeset) is deprecated in kernel and > was recently dropped from the X driver. You should be able to use > fbset to change the resolution of your fbcon though. I tried fbset, and it was distinctly unsuccessful. When I ra

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Luca Barbieri
> Also note that the delayed delete list is not in fence order but in > deletion-time order, which perhaps gives room for more optimizations. You are right. I think then that ttm_bo_delayed_delete may still need to be changed, because it stops when ttm_bo_cleanup_refs returns -EBUSY, which happens

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Luca Barbieri
When designing this, we should also keep in mind that some drivers (e.g. nouveau) have multiple FIFO channels, and thus we would like a buffer to be referenced for reading by multiple channels at once (and be destroyed only when all fences are expired, obviously). Also, hardware may support on-GPU

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Luca Barbieri
> We had to do a similar thing in the > Poulsbo driver and it turned out that we could save a significant amount of > CPU by using a delayed workqueue, collecting objects and destroying them > periodically. Yes, indeed, we don't really care about a fence expiring unless we want to use that buffer o

Re: Font selection with i915

2010-01-20 Thread Tomas M.
since 2.6.32 you can change the resolution with video=VGA-1:640x480 boot parameter and emulate the old behaviour. On 01/20/2010 05:39 PM, Alan Stern wrote: > On Wed, 20 Jan 2010, Corbin Simpson wrote: > > >> FYI the legacy non-KMS path (nomodeset) is deprecated in kernel and >> was recently

Re: [PATCH] drm/radeon/kms/200: fix bug in CS parser

2010-01-20 Thread Jerome Glisse
Both patch looks good. On Wed, Jan 20, 2010 at 6:00 PM, Alex Deucher wrote: > From 8097d73afefaf77132df11feb8b9bfa7e52032fe Mon Sep 17 00:00:00 2001 > From: Andrew Randrianasulu > Date: Wed, 20 Jan 2010 11:56:07 -0500 > Subject: [PATCH] drm/radeon/kms/200: fix bug in CS parser > > Add missing ve

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Luca Barbieri
Yes it's fine. I sent your patch to Dave with an expanded commit comment for merging. Here is a possible redesign of the mechanism inspired by this issue. It seems that what we are racing against is buffer eviction, due to delayed deletion buffers being still kept on the LRU list. I'm wondering i

[PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete (v3, final)

2010-01-20 Thread Luca Barbieri
Resending this with Thomas Hellstrom's signoff for merging into 2.6.33 ttm_bo_delayed_delete has a race condition, because after we do: kref_put(&nentry->list_kref, ttm_bo_release_list); we are not holding the list lock and not holding any reference to objects, and thus every bo in the list can b

Re: [patch] drm/ttm: remove unnecessary save_flags and ttm_flag_masked in ttm_bo_util.c

2010-01-20 Thread Yuan, Shengquan
On Wed, Jan 20, 2010 at 6:48 PM, Thomas Hellstrom wrote: > Yuan, > This looks like an old leftover from a previous cleanup. > > Although, when I look at the code, it seems save_flags should be removed > completely. > Can you take a look at that and respin the patch? Yes. One declaration wasn't rem

Re: [PATCH] drm: fix regression in fb blank handling

2010-01-20 Thread Zhenyu Wang
On 2010.01.20 13:14:23 +, James Simmons wrote: > > It's just adding the backlight api to the intel driver. In fact it gives > the user the ablity to control the brightness of the backlight which I see > is lacking in the intel driver. Wait, this regression has nothing to do with backlight c

[Bug 23992] Skybox corruption in Tremulous with KMS enabled.

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=23992 --- Comment #6 from Lukasz Krotowski 2010-01-20 15:23:26 PST --- (In reply to comment #5) > Commit you mentioned is "radeong: Fix EGL driver names." which can not fix > that. Maybe some earlier commit was real fixing one. Yup Rafał, my Engl

[Bug 23992] Skybox corruption in Tremulous with KMS enabled.

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=23992 --- Comment #5 from Rafał Miłecki 2010-01-20 15:17:37 PST --- (In reply to comment #4) > Seems fixed in mesa commit: c1334ce23550a8321023c0b8fa58ad266199afa1. I'll > watch it and close in couple of days if everything is ok. Commit you menti

[Bug 23992] Skybox corruption in Tremulous with KMS enabled.

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=23992 --- Comment #4 from Lukasz Krotowski 2010-01-20 15:10:28 PST --- Seems fixed in mesa commit: c1334ce23550a8321023c0b8fa58ad266199afa1. I'll watch it and close in couple of days if everything is ok. -- Configure bugmail: http://bugs.freede

[Bug 26145] New: [KMS] PM patches: inconsistency between clock set and read

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26145 Summary: [KMS] PM patches: inconsistency between clock set and read Product: DRI Version: XOrg CVS Platform: x86 (IA32) OS/Version: Linux (All) Status: NEW

[Bug 26103] Planeshift is unplayable, almost everything is gray and white

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26103 --- Comment #9 from darkbasic 2010-01-20 14:30:58 PST --- It's a texture compression related problem. If I disable it (Video.OpenGL.MesaForceS3TCEnable = no in data/config/r3dopengl.cfg) it works. Should I close the bug report? -- Config

[Bug 25597] SIGSEGV in _radeon_bo_unref

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25597 --- Comment #5 from Alex Deucher 2010-01-20 14:07:57 PST --- (In reply to comment #4) > I build drm 1st, so it should be enabled with the configure flag. I assume > that the other modules autotdetect it's presence no? Moreover, if not, it

[Bug 25597] SIGSEGV in _radeon_bo_unref

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25597 --- Comment #4 from David Ronis 2010-01-20 13:35:30 PST --- I build drm 1st, so it should be enabled with the configure flag. I assume that the other modules autotdetect it's presence no? Moreover, if not, it still should work, albeit wit

[Bug 25597] SIGSEGV in _radeon_bo_unref

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25597 --- Comment #3 from Alex Deucher 2010-01-20 13:29:21 PST --- (In reply to comment #2) > One more thing: I'm running with the ati driver with the ATI Radeon 9000 IGP > chip and kernel 2.6.32.3. I configure drm with > --enable-radeon-experi

[PATCH] drm/radeon/kms/r4xx: cleanup atom path

2010-01-20 Thread Alex Deucher
>From 0a08c5213b2f7e36841928f2e127387319ac363b Mon Sep 17 00:00:00 2001 From: Alex Deucher Date: Wed, 20 Jan 2010 16:22:53 -0500 Subject: [PATCH] drm/radeon/kms/r4xx: cleanup atom path most of radeon_legacy_atom_set_surface() is taken care of in atombios_set_base(), so remove the duplicate setup

[Bug 25597] SIGSEGV in _radeon_bo_unref

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=25597 --- Comment #2 from David Ronis 2010-01-20 13:11:53 PST --- One more thing: I'm running with the ati driver with the ATI Radeon 9000 IGP chip and kernel 2.6.32.3. I configure drm with --enable-radeon-experimental-api. Could this be an i

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Luca Barbieri wrote: > When designing this, we should also keep in mind that some drivers > (e.g. nouveau) have multiple FIFO channels, and thus we would like a > buffer to be referenced for reading by multiple channels at once (and > be destroyed only when all fences are expired, obviously). > Als

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Luca Barbieri wrote: >> Also note that the delayed delete list is not in fence order but in >> deletion-time order, which perhaps gives room for more optimizations. >> > You are right. > I think then that ttm_bo_delayed_delete may still need to be changed, > because it stops when ttm_bo_cleanu

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Luca Barbieri wrote: > Yes it's fine. I sent your patch to Dave with an expanded commit > comment for merging. > > Here is a possible redesign of the mechanism inspired by this issue. > It seems that what we are racing against is buffer eviction, due to > delayed deletion buffers being still kept o

Re: Postal 2 on RadeonProgram wiki page

2010-01-20 Thread Rafał Miłecki
2010/1/20 Corbin Simpson : > On Wed, Jan 20, 2010 at 2:00 AM, Mark Rosenstand wrote: >> Hey guys, >> >> I recently acquired a couple of R700 class cards and was sort of >> surprised how well 3D seemed to work, so I decided to try out one of the >> (very few) games I liked in the past, Postal 2. >>

Re: Font selection with i915

2010-01-20 Thread Corbin Simpson
FYI the legacy non-KMS path (nomodeset) is deprecated in kernel and was recently dropped from the X driver. You should be able to use fbset to change the resolution of your fbcon though. ~ C. On Tue, Jan 19, 2010 at 10:09 AM, Alan Stern wrote: > On Tue, 19 Jan 2010, James Simmons wrote: > >> >>

Re: Postal 2 on RadeonProgram wiki page

2010-01-20 Thread Corbin Simpson
Go for it. That page is really by users, for users, at this point. On Wed, Jan 20, 2010 at 2:00 AM, Mark Rosenstand wrote: > Hey guys, > > I recently acquired a couple of R700 class cards and was sort of > surprised how well 3D seemed to work, so I decided to try out one of the > (very few) games

[Bug 26061] rv280 in KMS mode triggers "Forbidden register 0x22D0"

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26061 --- Comment #2 from Alex Deucher 2010-01-20 09:00:50 PST --- Created an attachment (id=32744) --> (http://bugs.freedesktop.org/attachment.cgi?id=32744) patch for upstream good catch. patch for upstream. -- Configure bugmail: http://bugs

[PATCH] drm/radeon/kms/200: fix bug in CS parser

2010-01-20 Thread Alex Deucher
>From 8097d73afefaf77132df11feb8b9bfa7e52032fe Mon Sep 17 00:00:00 2001 From: Andrew Randrianasulu Date: Wed, 20 Jan 2010 11:56:07 -0500 Subject: [PATCH] drm/radeon/kms/200: fix bug in CS parser Add missing vertex shader regs for r200. fixed fdo bug 26061 agd5f: use official reg names Signed-o

[Bug 24159] [KMS] radeon module oops with stex3d and rv280

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=24159 --- Comment #4 from Alex Deucher 2010-01-20 08:41:58 PST --- Created an attachment (id=32742) --> (http://bugs.freedesktop.org/attachment.cgi?id=32742) patch for upstream good catch. I'm sending this upstream. -- Configure bugmail: http

[PATCH] drm/radeon/kms/r200: fix bug in CS parser

2010-01-20 Thread Alex Deucher
>From 95865cd65e02fa9ca31017c806a36ccc0f144939 Mon Sep 17 00:00:00 2001 From: Andrew Randrianasulu Date: Wed, 20 Jan 2010 11:36:30 -0500 Subject: [PATCH] drm/radeon/kms/r200: fix bug in CS parser The checks for CUBE and 3D textures were inverted. fixes fdo bug 24159 agd5f: added comments for cl

[Bug 26103] Planeshift is unplayable, almost everything is gray and white

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26103 --- Comment #8 from darkbasic 2010-01-20 07:25:33 PST --- Full pslaunch.sh log: http://pastebin.com/m2a21af5 In-game planeshift screenshot: http://darkbasic.homelinux.com/images/planeshift_ingame.jpeg -- Configure bugmail: http://bugs.fr

[Bug 26103] Planeshift is unplayable, almost everything is gray and white

2010-01-20 Thread bugzilla-daemon
http://bugs.freedesktop.org/show_bug.cgi?id=26103 --- Comment #7 from darkbasic 2010-01-20 06:50:51 PST --- I don't know if it is connected to the planeshift issue but this is nexuiz piece-o-cake first location (effects set to low): http://darkbasic.homelinux.com/images/nexuiz_afterglsl1.jp

Re: [PATCH] drm: fix regression in fb blank handling

2010-01-20 Thread James Simmons
> On 2010.01.19 16:17:56 +, James Simmons wrote: > > Sorry I meant the backlight power management state seperate from the > > encoder state. > > > > > drm_fb_helper_off() will find fb's crtc and attached encoders, then > > > call encoder_funcs->dpms() and turn off crtc, so for your DRM_MODE

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Thomas Hellstrom wrote: Yes, it looks correct. Although it seems a little unintuitive to enter the loop with the spinlock held, and exit it with the spinlock not held. I've attached yet another patch to have that fixed. Could you take a look at whether it seems OK with you and, in that case,

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Thomas Hellstrom wrote: Thomas Hellstrom wrote: Yes, it looks correct. Although it seems a little unintuitive to enter the loop with the spinlock held, and exit it with the spinlock not held. I've attached yet another patch to have that fixed. Could you take a look at whether it seems OK wi

Re: [PATCH] drm/ttm: Fix race condition in ttm_bo_delayed_delete

2010-01-20 Thread Thomas Hellstrom
Luca Barbieri wrote: >> Would nentry=list_first_entry(&entry->ddestroy, ) work? >> > Yes, it seems a bit less intuitive, but if that's the accepted > practice, let's do that instead. > > >> Here nentry may have been removed from the list by another process, which >> would trigger the u

Re: [patch] drm/ttm: remove unnecessary save_flags and ttm_flag_masked in ttm_bo_util.c

2010-01-20 Thread Thomas Hellstrom
Yuan, This looks like an old leftover from a previous cleanup. Although, when I look at the code, it seems save_flags should be removed completely. Can you take a look at that and respin the patch? Thanks, Thomas Yuan, Shengquan wrote: > commit d60326ac977a5e99047b44f9b313ff79cd3a14b4 > Autho

Postal 2 on RadeonProgram wiki page

2010-01-20 Thread Mark Rosenstand
Hey guys, I recently acquired a couple of R700 class cards and was sort of surprised how well 3D seemed to work, so I decided to try out one of the (very few) games I liked in the past, Postal 2. The menu works - including people fighting in 3D in the background - and the "Monday" splash shows up

RFC: xfree: dri2: libdrm as optional

2010-01-20 Thread Tiago Vignatti
Some drivers use DRI2 protocol but implement their own kernel rendering mananger. For these drivers, libdrm becomes useless. The only inconvenient right now to put libdrm optional to X server is concerning DRI2Authenticate. Such function uses drm_magic_t and drmAuthMagic symbols from libdrm. So I

Re: Font selection with i915

2010-01-20 Thread Alan Stern
On Tue, 19 Jan 2010, James Simmons wrote: > > > When Fedora 12 loads i915 during initramfs probing on my system, the > > driver automatically installs a tiny 160x64 font. How can I prevent it > > from doing this, or tell it to use a larger 80x25 font instead? > > > > Is this documented anywhere

Re: RFC: xfree: dri2: libdrm as optional

2010-01-20 Thread Oliver McFadden
On Tue, 2010-01-19 at 20:18 +0100, Vignatti Tiago (Nokia-D/Helsinki) wrote: > Some drivers use DRI2 protocol but implement their own kernel rendering > mananger. For these drivers, libdrm becomes useless. > > The only inconvenient right now to put libdrm optional to X server is > concerning DRI2Au