<div dir="ltr">Hi all,<br><br>Accomplishments:<br>- Revised gernel drm ioctl decoding patchset according to the feedbacks I<br>get from ldv.<br>Summary of the changes:<br>Removed those redundant/unused code.<br>ifdef the whole bulk of drm parsers.<br>Used a function instead of a macro.<br>Changed the function to the traditional read-write fashion.<br>Whether struct drm_mode_get_connector needs to be mpersified still needs<br>more discussion.<br>A new version of patchset will be sent later.<br>- Wrote part of the remaining i915 drm decoder. But I then found some of<br>them are no-op ioctl (see /linux/driver/gpu/drm/i915/i915_drv.c line 2926).<br>Patrik once mentioned that he doesn't bother writing noop ioctls. I wonder<br>if they are worth decoding.<br>For example, DRM_IOCTL_I915_IRQ_EMIT is an IOWR command while it's a noop<br>ioctl and I can't find any use of it or docs that mention how it works. I<br>am now not sure how to decoding it on entering or exiting syscall. I may<br>take [1] and those old commits which drm_i915_irq_emit_t were still used<br>as references.<br>- Some manual tests for those mentioned above.<br><br>Priorities:<br>- Get the patchset ready. -v option is not yet added to drm ioctl parsers.<br>It's better if I use magic numbers in the tests (for now most of them are<br>numeric 1 or 0).<br>- Figure out how to exactly decode those noop ioctls (or if it's worth<br>decoding at all). Found [2] that might help.<br>- Test device-specific ioctls in the test suite. May be making them optional<br>is an alternative? I saw that in the test for btrfs $BTRFS_MOUNTPOINT is<br>provided to make tests run on real env instead of fault injection or<br>something else.<br><br>[1] <a href="https://anongit.freedesktop.org/git/drm-intel.git">https://anongit.freedesktop.org/git/drm-intel.git</a><br>[2] <a href="https://git.sphere.ly/mugenman1111/kernel_lge_g3/commit/07f4f8bf436e3f389a12cb792ad16275b9091b51?view=parallel">https://git.sphere.ly/mugenman1111/kernel_lge_g3/commit/07f4f8bf436e3f389a12cb792ad16275b9091b51?view=parallel</a><br><br>Thanks,<br>Zhibin Li<br></div>