[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-28 Thread Sedat Dilek
Hi Daniel, I have tested this upgraded patchset again with linux-next (next-20101126), they work fine. Can you next time label the complete series as "v2": [PATCH 0/3 v2] embed drm_gem_object into radeon_bo (don't ask me if git can create this automatically for you). Feel free to add: Test

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-27 Thread Sedat Dilek
Hi Daniel, I have tested this upgraded patchset again with linux-next (next-20101126), they work fine. Can you next time label the complete series as "v2": [PATCH 0/3 v2] embed drm_gem_object into radeon_bo (don't ask me if git can create this automatically for you). Feel free to add: Test

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-27 Thread Daniel Vetter
Hi Dave, As promised rebased on top of latest drm-next to resolve a conflict the pageflipping code. Tested on my agp rv570. Please review and consider merging for -next. Thanks, Daniel Daniel Vetter (3): drm/radeon: embed struct drm_gem_object drm/radeon: introduce gem_to_radeon_bo helper

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-27 Thread Daniel Vetter
Hi Dave, As promised rebased on top of latest drm-next to resolve a conflict the pageflipping code. Tested on my agp rv570. Please review and consider merging for -next. Thanks, Daniel Daniel Vetter (3): drm/radeon: embed struct drm_gem_object drm/radeon: introduce gem_to_radeon_bo helper

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Sedat Dilek
On Tue, Nov 16, 2010 at 8:37 PM, Sedat Dilek wrote: > On Tue, Nov 16, 2010 at 6:30 PM, Daniel Vetter wrote: >> On Tue, Nov 16, 2010 at 06:05:25PM +0100, Sedat Dilek wrote: >>> I have tested both patchsets from Daniel (see [1] and [2]) again on a >>> Radeon RV250 in a none-BKL-config and it looks

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Sedat Dilek
On Tue, Nov 16, 2010 at 6:30 PM, Daniel Vetter wrote: > On Tue, Nov 16, 2010 at 06:05:25PM +0100, Sedat Dilek wrote: >> I have tested both patchsets from Daniel (see [1] and [2]) again on a >> Radeon RV250 in a none-BKL-config and it looks like >> >> ? ? ? ?agd5f-pflip/0002-drm-radeon-kms-add-page

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Daniel Vetter
On Tue, Nov 16, 2010 at 06:05:25PM +0100, Sedat Dilek wrote: > I have tested both patchsets from Daniel (see [1] and [2]) again on a > Radeon RV250 in a none-BKL-config and it looks like > >agd5f-pflip/0002-drm-radeon-kms-add-pageflip-ioctl-support.patch > > is the culprit in combination

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Sedat Dilek
[ CCing Arnd Bergmann ] Hi, I have tested both patchsets from Daniel (see [1] and [2]) again on a Radeon RV250 in a none-BKL-config and it looks like agd5f-pflip/0002-drm-radeon-kms-add-pageflip-ioctl-support.patch is the culprit in combination with the below listed drm patches. I have

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Sedat Dilek
On Tue, Nov 16, 2010 at 6:30 PM, Daniel Vetter wrote: > On Tue, Nov 16, 2010 at 06:05:25PM +0100, Sedat Dilek wrote: >> I have tested both patchsets from Daniel (see [1] and [2]) again on a >> Radeon RV250 in a none-BKL-config and it looks like >> >>        agd5f-pflip/0002-drm-radeon-kms-add-page

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Daniel Vetter
On Tue, Nov 16, 2010 at 06:05:25PM +0100, Sedat Dilek wrote: > I have tested both patchsets from Daniel (see [1] and [2]) again on a > Radeon RV250 in a none-BKL-config and it looks like > >agd5f-pflip/0002-drm-radeon-kms-add-pageflip-ioctl-support.patch > > is the culprit in combination

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-16 Thread Sedat Dilek
[ CCing Arnd Bergmann ] Hi, I have tested both patchsets from Daniel (see [1] and [2]) again on a Radeon RV250 in a none-BKL-config and it looks like agd5f-pflip/0002-drm-radeon-kms-add-pageflip-ioctl-support.patch is the culprit in combination with the below listed drm patches. I have

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Thomas Hellstrom
On 11/15/2010 07:45 PM, Daniel Vetter wrote: > Hi Thomas, > > Thanks for your comments about ttm and vmwgfx. Some of my own ideas about > where this might all be heading below. > > On Mon, Nov 15, 2010 at 08:25:16AM +0100, Thomas Hellstrom wrote: > >> Hi, Daniel, >> >> My main concerns previous

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Daniel Vetter
Hi Thomas, Thanks for your comments about ttm and vmwgfx. Some of my own ideas about where this might all be heading below. On Mon, Nov 15, 2010 at 08:25:16AM +0100, Thomas Hellstrom wrote: > Hi, Daniel, > > My main concerns previously for embedding GEM objects as user-space > references for TTM

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Thomas Hellstrom
On 11/15/2010 07:45 PM, Daniel Vetter wrote: Hi Thomas, Thanks for your comments about ttm and vmwgfx. Some of my own ideas about where this might all be heading below. On Mon, Nov 15, 2010 at 08:25:16AM +0100, Thomas Hellstrom wrote: Hi, Daniel, My main concerns previously for embedding

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Sedat Dilek
Hi Daniel, I have tried this patchset on a RV250 with linux-next (next-20101115) in combination w/ patchset from "[PATCH 0/9] make struct drm_mm_node embeddable" [1]. glxgears works nice. 2nd test-case is Eric Anholt's OpenArena benchmark. The screen gets blank and system is unusable (cold start,

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Daniel Vetter
Hi Thomas, Thanks for your comments about ttm and vmwgfx. Some of my own ideas about where this might all be heading below. On Mon, Nov 15, 2010 at 08:25:16AM +0100, Thomas Hellstrom wrote: > Hi, Daniel, > > My main concerns previously for embedding GEM objects as user-space > references for TTM

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Thomas Hellstrom
Hi, Daniel, My main concerns previously for embedding GEM objects as user-space references for TTM has been twofold and implementation specific. 1) The locking has been using global mutexes where local spin- or RCU locks have been more appropriate. It looks like this has finally been / is fina

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-15 Thread Sedat Dilek
Hi Daniel, I have tried this patchset on a RV250 with linux-next (next-20101115) in combination w/ patchset from "[PATCH 0/9] make struct drm_mm_node embeddable" [1]. glxgears works nice. 2nd test-case is Eric Anholt's OpenArena benchmark. The screen gets blank and system is unusable (cold start,

Re: [PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-14 Thread Thomas Hellstrom
Hi, Daniel, My main concerns previously for embedding GEM objects as user-space references for TTM has been twofold and implementation specific. 1) The locking has been using global mutexes where local spin- or RCU locks have been more appropriate. It looks like this has finally been / is fi

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-13 Thread Daniel Vetter
Hi all, This patch series embeds drm_gem_object into struct radeon_bo and adjusts any references. I've decided against embedding the gem stuff into struct ttm_bo because - vmwgfx doesn't use gem and - ttm is used for implementing the memory management, whereas gem provides the userspace interfac

[PATCH 0/3] embed drm_gem_object into radeon_bo

2010-11-13 Thread Daniel Vetter
Hi all, This patch series embeds drm_gem_object into struct radeon_bo and adjusts any references. I've decided against embedding the gem stuff into struct ttm_bo because - vmwgfx doesn't use gem and - ttm is used for implementing the memory management, whereas gem provides the userspace interfac