On Die, 2014-03-18 at 11:01 +0100, Daniel Vetter wrote: > On Tue, Mar 18, 2014 at 09:58:14AM +0900, Michel Dänzer wrote: > > From: Michel Dänzer <michel.daenzer@xxxxxxx> > > > > entry->size is the size of the node, not the size of the hole after it. > > So the code would actually find the hole which can satisfy the > > constraints and which is preceded by the smallest node, not the smallest > > hole satisfying the constraints. > > > > Reported-by: "Huang, FrankR" <FrankR.Huang@xxxxxxx> > > Signed-off-by: Michel Dänzer <michel.daenzer@xxxxxxx> > > But drm-next just gained my kerneldoc patch for drm_mm, so can you please > respin your patch and update the docs too? What kind of update are you thinking of? > While at it ... could you perhaps smash a bit of kerneldoc on top of > enum drm_mm_search_flags, I seem to have missed it. With that this is > > Reviewed-by: Daniel Vetter <daniel.vetter@xxxxxxxx> Thanks, but I'm afraid I'll have to pass on that. I'm just submitting a fix for a problem Frank stumbled upon. I don't have the time right now, nor the particular inclination to clean up the surrounding code. Meanwhile, I've submitted a less invasive v2 fix. BTW, do you think the fix would interact properly with coloring? -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel