Re: [patch] post superioctl inteface removal.

2007-10-19 Thread Thomas Hellström
Hi. Eric Anholt wrote: On Thu, 2007-10-18 at 07:55 +0800, Keith Packard wrote: On Wed, 2007-10-17 at 16:40 -0700, Eric Anholt wrote: Turn off CRTCs Unpin old framebuffer Allocate new framebuffer Copy from old to new We needn't copy on resize, leaving us with allocate new, unpin

Re: [patch] post superioctl inteface removal.

2007-10-19 Thread Thomas Hellström
Jerome Glisse wrote: Kristian Høgsberg wrote: On 10/18/07, Keith Whitwell [EMAIL PROTECTED] wrote: ... Doesn't Kristian changes to DRI interface (DRI2) already allow to clients to not care about front buffer. I mean if they all got private back buffer then they render into it. But

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Thomas Hellström
Hi, Eric. Eric Anholt wrote: On Wed, 2007-10-17 at 11:32 +0200, Thomas Hellström wrote: Dave Airlie wrote: DRM_BO_HINT_DONT_FENCE is implied, and use that instead of the set pin interface. We can perhaps rename it to drmBOSetStatus or something more suitable. This will get rid of

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Keith Whitwell
Thomas Hellström wrote: Hi, Eric. Eric Anholt wrote: ... Can you clarify the operation being done where you move scanout buffers before unpinning them? That seems contradictory to me -- how are you scanning out while the object is being moved, and how are you considering it pinned

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Jerome Glisse
Keith Whitwell wrote: Thomas Hellström wrote: Hi, Eric. Eric Anholt wrote: ... Can you clarify the operation being done where you move scanout buffers before unpinning them? That seems contradictory to me -- how are you scanning out while the object is being moved, and how

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Thomas Hellström
Keith Whitwell wrote: Jerome Glisse wrote: Keith Whitwell wrote: Thomas Hellström wrote: Hi, Eric. Eric Anholt wrote: ... Can you clarify the operation being done where you move scanout buffers before unpinning them? That seems contradictory to me -- how are you scanning

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Eric Anholt
On Thu, 2007-10-18 at 07:55 +0800, Keith Packard wrote: On Wed, 2007-10-17 at 16:40 -0700, Eric Anholt wrote: Turn off CRTCs Unpin old framebuffer Allocate new framebuffer Copy from old to new We needn't copy on resize, leaving us with allocate new, unpin old, pin new, free old.

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Kristian Høgsberg
On 10/18/07, Keith Whitwell [EMAIL PROTECTED] wrote: ... Doesn't Kristian changes to DRI interface (DRI2) already allow to clients to not care about front buffer. I mean if they all got private back buffer then they render into it. But i might have misunderstood this. Yes, of course.

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Dave Airlie
There is also the following (i don't think it was mentioned before in this thread): card with 8Mo of ram (who the hell have such hw ? :)) I've got 40 of them :( All of our desktops have integrated Intel ( i845g ) chips, and the BIOS has the option of stealing either 1MB or 8MB of

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Jerome Glisse
Kristian Høgsberg wrote: On 10/18/07, Keith Whitwell [EMAIL PROTECTED] wrote: ... Doesn't Kristian changes to DRI interface (DRI2) already allow to clients to not care about front buffer. I mean if they all got private back buffer then they render into it. But i might have misunderstood

Re: [patch] post superioctl inteface removal.

2007-10-18 Thread Daniel Kasak
On Fri, 2007-10-19 at 03:04 +0200, Jerome Glisse wrote: There is also the following (i don't think it was mentioned before in this thread): card with 8Mo of ram (who the hell have such hw ? :)) I've got 40 of them :( All of our desktops have integrated Intel ( i845g ) chips, and the BIOS

Re: [patch] post superioctl inteface removal.

2007-10-17 Thread Thomas Hellström
Dave Airlie wrote: DRM_BO_HINT_DONT_FENCE is implied, and use that instead of the set pin interface. We can perhaps rename it to drmBOSetStatus or something more suitable. This will get rid of the user-space unfenced list access (which I believe was the main motivation behind the set pin

Re: [patch] post superioctl inteface removal.

2007-10-16 Thread Dave Airlie
DRM_BO_HINT_DONT_FENCE is implied, and use that instead of the set pin interface. We can perhaps rename it to drmBOSetStatus or something more suitable. This will get rid of the user-space unfenced list access (which I believe was the main motivation behind the set pin interface?) while

Re: [patch] post superioctl inteface removal.

2007-10-09 Thread Thomas Hellström
Dave Airlie wrote: Hi, Once the 915 super ioctl is merged, the patch attached removes the unused interfaces left behind... Are any of these worth saving? Dave. Dave, As mentioned previously to Eric, I think we should keep the single buffer validate interface with the exception that

Re: [patch] post superioctl inteface removal.

2007-10-09 Thread Dave Airlie
Dave, As mentioned previously to Eric, I think we should keep the single buffer validate interface with the exception that the hint DRM_BO_HINT_DONT_FENCE is implied, and use that instead of the set pin interface. We can perhaps rename it to drmBOSetStatus or something more suitable. This

Re: [patch] post superioctl inteface removal.

2007-10-09 Thread Keith Whitwell
Dave Airlie wrote: Dave, As mentioned previously to Eric, I think we should keep the single buffer validate interface with the exception that the hint DRM_BO_HINT_DONT_FENCE is implied, and use that instead of the set pin interface. We can perhaps rename it to drmBOSetStatus or something more

[patch] post superioctl inteface removal.

2007-10-08 Thread Dave Airlie
Hi, Once the 915 super ioctl is merged, the patch attached removes the unused interfaces left behind... Are any of these worth saving? Dave. -- David Airlie, Software Engineer http://www.skynet.ie/~airlied / airlied at skynet.ie Linux kernel - DRI, VAX / pam_smb / ILUG From