<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 22, 2014 at 5:22 AM, Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On 22/02/14 03:33, Stéphane Marchesin wrote:<br>


> On Fri, Feb 21, 2014 at 7:04 PM, Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>>wrote:<br>
><br>
>> Implementation is a verbatim copy from the classic driver.<br>
>><br>
>> This introduces a driver dependency on libdrm-intel, as the winsys does not<br>
>> cache the aperture size of the device.<br>
>><br>
><br>
> Usually if you have to add calls into libdrm, you add them to the winsys,<br>
> and then add a new entry point to the winsys which you call... If you want<br>
> I can do it.<br>
><br>
> I've been ensuring that the only place calling into libdrm was the winsys<br>
> so far. Whether or not we remove the sw winsys, I think it's a healthy<br>
> thing to do.<br>
><br>
</div>Agreed, keeping OS/platform specifics within the winsys is the way to do it.<br>
<br>
Should I store the device aperture size within i915_winsys or is there a<br>
more suitable place for it ? The current implementation is essentially a<br>
vtable for the drm/sw callbacks.<br></blockquote><div><br></div><div>I would probably just add a function to the winsys to get the aperture size. Basically copy-paste that big chunk of code from <span style="font-family:arial,sans-serif;font-size:13px">case PIPE_CAP_VIDEO_MEMORY</span> into int iws->aperture_size(struct i915_winsys *iws);</div>

<div><br></div><div>Stéphane</div><div><br></div></div></div></div>