From: Arnd Bergmann <arnd@kernel.org>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Arnd Bergmann <arnd@arndb.de>,
linux-media@vger.kernel.org, mchehab@kernel.org, hch@lst.de,
linux-kernel@vger.kernel.org
Subject: [PATCH v2 0/8] media: v4l2: simplify compat ioctl handling
Date: Fri, 30 Oct 2020 17:55:21 +0100 [thread overview]
Message-ID: <20201030165529.1255175-1-arnd@kernel.org> (raw)
From: Arnd Bergmann <arnd@arndb.de>
I have a series to remove all uses of compat_alloc_user_space() and
copy_in_user() from the kernel, this is the part of it that involves
the v4l2 compat code.
The resulting code is significantly shorter and arguably more readable,
but I have not done any testing beyond compilation on it, so at the
minimum this first needs to pass the test suite for both native and
compat users space.
The first version had a number of bugs that Hans Verkuil managed to
fix, I have now rebased my series on top of linux-5.10-rc1 and included
his bugfixes.
This series and the remaining changes for removing compat_alloc_user_space
are available for further testing in
https://git.kernel.org/pub/scm/linux/kernel/git/arnd/playground.git compat-alloc-user-space-4
Arnd
Arnd Bergmann (8):
media: v4l2: prepare compat-ioctl rework
media: v4l2: remove unneeded compat ioctl handlers
media: v4l2: move v4l2_ext_controls conversion
media: v4l2: move compat handling for v4l2_buffer
media: v4l2: allocate v4l2_clip objects early
media: v4l2: convert v4l2_format compat ioctls
media: v4l2: remaining compat handlers
media: v4l2: remove remaining compat_ioctl
drivers/media/common/saa7146/saa7146_video.c | 6 +-
drivers/media/pci/bt8xx/bttv-driver.c | 8 +-
drivers/media/pci/saa7134/saa7134-video.c | 19 +-
.../media/test-drivers/vivid/vivid-vid-cap.c | 18 +-
.../media/test-drivers/vivid/vivid-vid-out.c | 18 +-
drivers/media/v4l2-core/v4l2-compat-ioctl32.c | 1773 ++++++-----------
drivers/media/v4l2-core/v4l2-ioctl.c | 182 +-
include/media/v4l2-ioctl.h | 11 +
include/uapi/linux/videodev2.h | 2 +-
9 files changed, 760 insertions(+), 1277 deletions(-)
Cc: linux-media@vger.kernel.org
Cc: mchehab@kernel.org
Cc: hverkuil@xs4all.nl
Cc: hch@lst.de
Cc: linux-kernel@vger.kernel.org
--
2.27.0
next reply other threads:[~2020-10-30 16:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-30 16:55 Arnd Bergmann [this message]
2020-10-30 16:55 ` [PATCH v2 1/8] media: v4l2: prepare compat-ioctl rework Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 2/8] media: v4l2: remove unneeded compat ioctl handlers Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 3/8] media: v4l2: move v4l2_ext_controls conversion Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 4/8] media: v4l2: move compat handling for v4l2_buffer Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 5/8] media: v4l2: allocate v4l2_clip objects early Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 6/8] media: v4l2: convert v4l2_format compat ioctls Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 7/8] media: v4l2: remaining compat handlers Arnd Bergmann
2020-10-30 16:55 ` [PATCH v2 8/8] media: v4l2: remove remaining compat_ioctl Arnd Bergmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20201030165529.1255175-1-arnd@kernel.org \
--to=arnd@kernel.org \
--cc=arnd@arndb.de \
--cc=hch@lst.de \
--cc=hverkuil@xs4all.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox