<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Apr 28, 2017 at 6:06 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 28 April 2017 at 13:55, Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>> wrote:<br>
> On Friday, 2017-04-28 13:14:20 +0200, Philipp Zabel wrote:<br>
>> To restart interrupted system calls, use drmIoctl.<br>
>><br>
>> Suggested-by: Emil Velikov <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>><br>
>> Signed-off-by: Philipp Zabel <<a href="mailto:p.zabel@pengutronix.de">p.zabel@pengutronix.de</a>><br>
><br>
> Reviewed-by: Eric Engestrom <<a href="mailto:eric.engestrom@imgtec.com">eric.engestrom@imgtec.com</a>><br>
><br>
> A quick grep shows 8 other `ioctl()`; do we want to fix them as well?<br>
><br>
> src/gallium/auxiliary/hud/hud_<wbr>nic.c:128:   if (ioctl(sockfd, SIOCGIWRATE, &req) == -1) {<br>
> src/gallium/auxiliary/hud/hud_<wbr>nic.c:163:   if (ioctl(sockfd, SIOCGIWSTATS, &req) == -1) {<br>
</span>Not DRM specific, so no.<br>
<span class=""><br>
> src/gallium/winsys/i915/drm/<wbr>i915_drm_winsys.c:27:   ret = ioctl(fd, DRM_IOCTL_I915_GETPARAM, &gp, sizeof(gp));<br>
> src/gallium/winsys/radeon/drm/<wbr>radeon_drm_bo.c:1305:            if (ioctl(ws->fd, DRM_IOCTL_GEM_FLINK, &flink)) {<br>
<br>
<br>
> src/intel/vulkan/anv_gem.c:<wbr>207:      ret = ioctl(device->fd, DRM_IOCTL_I915_GEM_SET_TILING, &set_tiling);<br>
> src/intel/vulkan/anv_gem.c:<wbr>257:      ret = ioctl(fd, DRM_IOCTL_I915_GEM_SET_TILING, &set_tiling);<br>
> src/intel/vulkan/anv_gem.c:39:      ret = ioctl(fd, request, arg);<br>
> src/mesa/drivers/dri/i965/brw_<wbr>bufmgr.c:994:      ret = ioctl(bufmgr->fd, DRM_IOCTL_I915_GEM_SET_TILING, &set_tiling);<br>
><br>
</span>These four are fine as-is. No idea about the i915g/radeon winsys ones.<br></blockquote><div><br></div><div>The one in i915/winsys (it's a getparam) should probably be fixed, but it's unlikely that you would get interrupted while getting a chip id. I can do that separately. Thanks for the heads up!</div><div><br></div><div>Stéphane</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
-Emil<br>
</font></span><div class="HOEnZb"><div class="h5">______________________________<wbr>_________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</div></div></blockquote></div><br></div></div>