linux-kernel - [GIT PULL] parisc architecture fixes for 4.11-rc5

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [day] [month] [year] [list]

Message-ID: <20170401191031.GA18005@ls3530.fritz.box>
Date:   Sat, 1 Apr 2017 21:10:31 +0200
From:   Helge Deller <deller@....de>
To:     Linus Torvalds <torvalds@...ux-foundation.org>,
        linux-kernel@...r.kernel.org, linux-parisc@...r.kernel.org,
        James Bottomley <James.Bottomley@...senPartnership.com>,
        John David Anglin <dave.anglin@...l.net>
Cc:     Al Viro <viro@...IV.linux.org.uk>
Subject: [GIT PULL] parisc architecture fixes for 4.11-rc5

Hi Linus,

Please pull three fixes for the parisc architecture for v4.11-rc5 from
  git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git parisc-4.11-3

Al Viro reported that - in case of read faults - our copy_from_user()
implementation may claim to have copied more bytes than it actually did. In
order to fix this bug and because of the way how gcc optimizes register usage
for inline assembly in C code, we had to replace our pa_memcpy() function with
a pure assembler implementation.

While fixing the memcpy bug we noticed some other issues with our get_user()
and put_user() functions, e.g. nested faults may return wrong data. This is now
fixed by a common fixup handler for get_user/put_user in the exception handler
which additionally makes generated code smaller and faster.

The third patch is a trivial one-line fix for a patch which went in during
4.11-rc and which avoids stalled CPU warnings after power shutdown (for parisc
machines which can't plug power off themselves).

Due to the rewrite of pa_memcpy() into assembly this patch got bigger than what
I wanted to have sent at this stage.

Those patches have been running in production during the last few days on our
debian build servers without any further issues.

Thanks,
Helge

----------------------------------------------------------------
Helge Deller (3):
      parisc: Fix access fault handling in pa_memcpy()
      parisc: Clean up fixup routines for get_user()/put_user()
      parisc: Avoid stalled CPU warnings after system shutdown

 arch/parisc/include/asm/uaccess.h |  59 ++---
 arch/parisc/kernel/parisc_ksyms.c |  10 -
 arch/parisc/kernel/process.c      |   2 +
 arch/parisc/lib/Makefile          |   2 +-
 arch/parisc/lib/fixup.S           |  98 --------
 arch/parisc/lib/lusercopy.S       | 318 ++++++++++++++++++++++++++
 arch/parisc/lib/memcpy.c          | 461 +-------------------------------------
 arch/parisc/mm/fault.c            |  17 ++
 8 files changed, 375 insertions(+), 592 deletions(-)
 delete mode 100644 arch/parisc/lib/fixup.S