On Mi, 2015-09-02 at 12:30 +0300, Mike Rapoport wrote: > Update snprintf format in virtgpu_fence.c and virtgpu_debugfs.c to fix the > following compilation warnings: > > C [M] drivers/gpu/drm/virtio/virtgpu_fence.o > drivers/gpu/drm/virtio/virtgpu_fence.c: In function > âvirtio_timeline_value_strâ : > drivers/gpu/drm/virtio/virtgpu_fence.c:64:2: warning: format â%luâ > expects argument of type âlong unsigned intâ, but argument 4 has type > âlong long intâ [-Wformat=] > snprintf(str, size, "%lu", atomic64_read(&fence->drv->last_seq)); > ^ > CC [M] drivers/gpu/drm/virtio/virtgpu_debugfs.o > drivers/gpu/drm/virtio/virtgpu_debugfs.c: In function > âvirtio_gpu_debugfs_irq_infoâ: > drivers/gpu/drm/virtio/virtgpu_debugfs.c:39:6: warning: format â%ldâ > expects argument of type âlong intâ, but argument 3 has type âlong long > intâ [-Wformat=] > vgdev->fence_drv.sync_seq);
Doesn't fly. I assume this removes the warnings for 32bit builds, but with the patch applied I get warnings on 64bit builds ... cheers, Gerd