Hi, On 07/06/2011 11:24 AM, Tomasz Stanislawski wrote:
This patch add implicit conversion of single plane variant of ioctl to multiplane variant. The conversion is performed only in case if a driver implements only mplane api. The conversion is done by substituting SYS_IOCTL with a wrapper that converts single plane call to their mplane analogs. Function v4l2_fd_open was revised to work correctly with the wrapper. Signed-off-by: Tomasz Stanislawski<t.stanislaws@xxxxxxxxxxx> Signed-off-by: Kyungmin Park<kyungmin.park@xxxxxxxxxxx>
Thanks for the patch, I like the general idea, but I'm not completely happy with the implementation. I think overloading SYS_ioctl is not such a great idea, since this won't work for calls made by libv4lconvert, unless we export it from libv4l2 and use it in libv4lconvert too, which is quite ugly from an ABI pov. This is also problematic in the light of the upcoming plugin support (which just landed in v4l-utils git). Notice how that has replaced SYS_ioctl with dev_ops->ioctl, so that plugins can intercept ioctls. Actually the plugni support should make doing this more easy wrt libv4lconvert, since libv4lconvert now uses dev_ops->ioctl too. I think this can and should be handled in the following way, with a 2 patch patch-set: Patch1: Make the dev_ops member of v4l2_dev_info a struct rather then a pointer to a struct (and adjust v4l_plugin_init accordingly). Patch2: If one of the devices in question is detected the original dev_ops->ioctl should be saved in v4l2_dev_info and be replaced with the proposed wrapper, which then calls the saved original in cases where it now calls SYS_ioctl. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
