<div dir="ltr">I didn't mean to add libdrm, sorry, there is a separate change for that.  I'll double check and test the drivers that don't implement the virtual.  <div><br></div><div>Thanks for the quick reply!</div><div><br></div><div>-j</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 8, 2017 at 4:50 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 9 August 2017 at 09:42, Joe Kniss <<a href="mailto:djmk@chromium.org">djmk@chromium.org</a>> wrote:<br>
> Because all drivers currently use gem objects for framebuffer planes,<br>
> the virtual create_handle() is not required.  This change adds a<br>
> struct drm_gem_object *gems[4] field to drm_framebuffer and removes<br>
> create_handle() function pointer from drm_framebuffer_funcs.  The<br>
> corresponding *_create_handle() function is removed from each driver.<br>
><br>
> In many cases this change eliminates a struct *_framebuffer object,<br>
> as the only need for the derived struct is the addition of the gem<br>
> object pointer.<br>
<br>
</span>Why the libdrm in the tag? this isn't for libdrm.<br>
<br>
This will break drivers that don't current implement the virtual at all.<br>
<br>
I think vwmgfx.<br>
<br>
The current code checks if the virtual is there before callng it, this code<br>
just calls the gem code always.<br>
<span class="HOEnZb"><font color="#888888"><br>
Dave.<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><table cellspacing="0" cellpadding="0" style="text-align:center;color:rgb(80,0,80);font-family:'Times New Roman'"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top-style:solid;border-top-color:rgb(213,15,37);border-top-width:2px"><font size="1"><font face="garamond, serif">Dr. Joe Michael Kniss </font>|</font></td><td nowrap style="border-top-style:solid;border-top-color:rgb(51,105,232);border-top-width:2px"><font size="1"> <font face="garamond, serif"><font color="#0b5394">G</font><font color="#cc0000">o</font><font color="#bf9000">o</font><font color="#0b5394">g</font><font color="#38761d">l</font><font color="#cc0000">e</font><font color="#000000"> </font><font color="#444444">ChromeOS</font></font><font color="#000000"> </font>|</font></td><td nowrap style="border-top-style:solid;border-top-color:rgb(0,153,57);border-top-width:2px"><font size="1"><font face="garamond, serif"> <a href="mailto:djmk@google.com" target="_blank">djmk<font color="#1155cc">@google.com</font></a> </font> </font></td><td nowrap style="border-top-style:solid;border-top-color:rgb(238,178,17);border-top-width:2px"><font size="1"><a value="+16504298119" style="color:rgb(17,85,204)">| </a><font face="garamond, serif"><font color="#1155cc">1-801-898-7977</font><br></font></font></td></tr></tbody></table></div></div></div></div>
</div>