[PATCH v2 4/5] DRI2: Expose API to set drawable swap limit.

Jesse Barnes jbarnes at virtuousgeek.org
Thu Oct 6 08:42:19 PDT 2011


On Thu, 15 Sep 2011 01:31:00 +0200
Mario Kleiner <mario.kleiner at tuebingen.mpg.de> wrote:

> On Sep 15, 2011, at 12:54 AM, Francisco Jerez wrote:
> 
> > Mario Kleiner <mario.kleiner at tuebingen.mpg.de> writes:
> >
> >> On Sep 14, 2011, at 6:02 PM, Keith Packard wrote:
> >>
> >>> On Wed, 14 Sep 2011 10:05:29 -0500, Jesse Barnes
> >>> <jbarnes at virtuousgeek.org> wrote:
> >>>
> >>>> Ah thanks Mario, I blame Keith. :p   I agree we should integrate  
> >>>> this
> >>>> patch as it would allow us to do more fun stuff with swapping...
> >>>
> >>> That patch changes a ton of stuff; would be nice to see it split  
> >>> into
> >>> smaller chunks that could be reviewed easily.
> >>>
> >>
> >> For reference, we're talking about this series, right?
> >>
> >> <http://www.mail-archive.com/xorg-devel@lists.x.org/msg14336.html>
> >>
> >> As far as i can see, they were already split up in chunks and  
> >> reviewed
> >> by Jesse, me and Franzisco Jerez - assuming they still apply  to the
> >> latest server version? At least 1/5, 4/5 and 5/5 looked simple   
> >> enough
> >> and 4/5 and 5/5, the swaplimit api, seem to be independent  from the
> >> rest of the series?
> >>
> > Note that my r-b only goes to 4/5, I had some objections to 5/5 and  
> > I'm
> > not sure if 1/5-3/5 still make sense.
> 
> I haven't checked if the rest still makes sense. 5/5 was about why a  
> driver who requests a dri2 swaplimit change should be called back to  
> confirm it is ok with the swaplimit change, which you said seems  
> totally redundant, right?
> 
> Ok, so we talk specifically about 4/5, which was reviewed by all of  
> us, i think non-controversial, and a simple addition of a dri2  
> swaplimit api.
> 
> Keith, what about that one for a start?
> 
> > BTW, is there any reason this is being discussed outside of the  
> > mailing
> > list?
> 
> No. It just started as a private conversation with Jesse and  
> "drifted" into this. cc'ing dri-devel, all that was said is in this  
> mail.

What's the latest here?  I still think we need the swap limit API...

-- 
Jesse Barnes, Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20111006/dc706677/attachment-0001.pgp>


More information about the dri-devel mailing list