On Wed, 2018-02-14 at 12:59 +0100, Hans Verkuil wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > This patch series fixes a number of bugs and culminates in the removal > of the set_fs(KERNEL_DS) call in v4l2-compat-ioctl32.c. > > This was tested with a VM running 3.16, the vivi driver (a poor substitute for > the much improved vivid driver that's available in later kernels, but it's the > best I had) since that emulates the more common V4L2 ioctls that need to pass > through v4l2-compat-ioctl32.c) and the 32-bit v4l2-compliance + 32-bit v4l2-ctl > utilities that together exercised the most common ioctls. > > Most of the v4l2-compat-ioctl32.c do cleanups and fix subtle issues that > v4l2-compliance complained about. The purpose is to 1) make it easy to > verify that the final patch didn't introduce errors by first eliminating > errors caused by other known bugs, and 2) keep the final patch at least > somewhat readable. > > While compiling the media drivers for 3.16 I also came across a bug > introduced in the 3.16 stable series that caused a compile error in the > adv7604 driver. That's fixed in the first patch. Call it a bonus patch :-) Thanks, I've queued up all of these. However, I rebased these on top of some earlier fixes to v4l2-compat-ioctl32.c which you incorporated into your backports. Ben. -- Ben Hutchings If the facts do not conform to your theory, they must be disposed of.
Attachment:
signature.asc
Description: This is a digitally signed message part