linux-fsdevel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [git pull] vfs.git pile 6
@ 2017-07-06  9:11 Al Viro
  2017-07-06 23:32 ` Linus Torvalds
  0 siblings, 1 reply; 2+ messages in thread
From: Al Viro @ 2017-07-06  9:11 UTC (permalink / raw)
  To: Linus Torvalds; +Cc: linux-kernel, linux-fsdevel, David Airlie

[6 more piles tonight with several more left for tomorrow]

	DRM compat ioctl handling.  Kills double-copies in there and
tons of field-by-field copyin/copyout.  Several dead ioctls put to
rest, while we are at it - the native counterparts had been gone for
a decade, so we can bloody well fail early on the compat side.  No
point rearranging the 32bit structure into 64bit one (and back) only
to be told "piss off, I don't know that ioctl" by the native code...

	There's a trivial fixup patch tagged on the end of that branch;
I could fold it back, but one of the affected commits is early in the
queue and fixup does not affect gcc at all - it's dealing with sparse
misannotations only, so it's not a bisect hazard and I decided to leave
it at the branch tip.

The following changes since commit 2ea659a9ef488125eb46da6eb571de5eae5c43f6:

  Linux 4.12-rc1 (2017-05-13 13:19:49 -0700)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/viro/vfs.git work.drm

for you to fetch changes up to b87b786b1fa1aa1bba33da22c8bb6b3ec7b608d2:

  Fix trivial misannotations (2017-07-06 02:00:00 -0400)

----------------------------------------------------------------
Al Viro (29):
      new helper: drm_ioctl_kernel()
      drm_compat_ioctl(): prepare for conversions to drm_ioctl_kernel()
      switch compat_drm_addbufs() to drm_ioctl_kernel()
      switch compat_drm_version() to drm_ioctl_kernel()
      switch compat_drm_getunique() to drm_ioctl_kernel()
      compat_drm_setunique(): don't bother
      switch compat_drm_getmap() to drm_ioctl_kernel()
      switch compat_drm_infobufs() to drm_ioctl_kernel()
      switch compat_drm_getclient() to drm_ioctl_kernel()
      switch compat_drm_getstats() to drm_ioctl_kernel()
      switch compat_drm_addmap() to drm_ioctl_kernel()
      switch compat_drm_markbufs() to drm_ioctl_kernel()
      switch compat_drm_freebufs() to drm_ioctl_kernel()
      switch compat_drm_setsareactx() to drm_ioctl_kernel()
      switch compat_drm_getsareactx() to drm_ioctl_kernel()
      switch compat_drm_resctx() to drm_ioctl_kernel()
      switch compat_drm_dma() to drm_ioctl_kernel()
      compat_drm: switch AGP compat ioctls to drm_ioctl_kernel()
      compat_drm: switch sg ioctls
      switch compat_drm_update_draw()
      switch compat_drm_wait_vblank() to drm_ioctl_kernel()
      switch compat_drm_mode_addfb2() to drm_ioctl_kernel()
      switch compat_drm_rmmap() to drm_ioctl_kernel()
      switch compat_drm_mapbufs() to drm_ioctl_kernel()
      drm_compat_ioctl(): tidy up a bit
      drm compat: ia64 is not biarch
      radeon: take out dead compat ioctls
      mga: switch compat ioctls to drm_ioctl_kernel()
      Fix trivial misannotations

 drivers/gpu/drm/drm_bufs.c            | 116 +++---
 drivers/gpu/drm/drm_internal.h        |   3 +
 drivers/gpu/drm/drm_ioc32.c           | 750 +++++++++++++---------------------
 drivers/gpu/drm/drm_ioctl.c           |  48 ++-
 drivers/gpu/drm/drm_legacy.h          |   7 +
 drivers/gpu/drm/mga/mga_drv.h         |   2 +
 drivers/gpu/drm/mga/mga_ioc32.c       | 149 +++----
 drivers/gpu/drm/mga/mga_state.c       |   2 +-
 drivers/gpu/drm/radeon/Makefile       |   1 -
 drivers/gpu/drm/radeon/radeon_drv.c   |  18 +-
 drivers/gpu/drm/radeon/radeon_ioc32.c | 424 -------------------
 include/drm/drm_ioctl.h               |   1 +
 12 files changed, 476 insertions(+), 1045 deletions(-)
 delete mode 100644 drivers/gpu/drm/radeon/radeon_ioc32.c

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [git pull] vfs.git pile 6
  2017-07-06  9:11 [git pull] vfs.git pile 6 Al Viro
@ 2017-07-06 23:32 ` Linus Torvalds
  0 siblings, 0 replies; 2+ messages in thread
From: Linus Torvalds @ 2017-07-06 23:32 UTC (permalink / raw)
  To: Al Viro; +Cc: Linux Kernel Mailing List, linux-fsdevel, David Airlie

On Thu, Jul 6, 2017 at 2:11 AM, Al Viro <viro@zeniv.linux.org.uk> wrote:
> [6 more piles tonight with several more left for tomorrow]

Tiny side note (since I'm starting to look at this pile soon): it
would be nice if your subject line had the very short "this is what
it's about" thing to it too.

I don't mind the numbering at all, but it would be good if the subject
was something like

   [git pull] vfs.git pile 6: DRM compat ioctl handling

and then I'd have the summary for my initial explanation line:

  Pull DRM compat ioctl handling updates from Al Viro:
   "... full explanation goes here ..."

because as it is, I often end up in MakeShitUp(tm) mode.

Anyway, your current pull requests are fine too, and not really a problem.

I'd just thought I'd mention this small convenience when I get the
summary there. And I just like seeing it in my subject list, because
that is the overview of what I have pending when I do my usual mailbox
search criteria for "git pull".

                         Linus

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2017-07-06 23:32 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-07-06  9:11 [git pull] vfs.git pile 6 Al Viro
2017-07-06 23:32 ` Linus Torvalds

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).