From: Helge Deller <deller@gmx.de>
To: Linus Torvalds <torvalds@linux-foundation.org>,
linux-kernel@vger.kernel.org, linux-parisc@vger.kernel.org,
James Bottomley <James.Bottomley@HansenPartnership.com>,
John David Anglin <dave.anglin@bell.net>
Cc: Al Viro <viro@ZenIV.linux.org.uk>
Subject: [GIT PULL] parisc architecture fixes for 4.11-rc5
Date: Sat, 1 Apr 2017 21:10:31 +0200 [thread overview]
Message-ID: <20170401191031.GA18005@ls3530.fritz.box> (raw)
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
reply other threads:[~2017-04-01 19:10 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20170401191031.GA18005@ls3530.fritz.box \
--to=deller@gmx.de \
--cc=James.Bottomley@HansenPartnership.com \
--cc=dave.anglin@bell.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-parisc@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
--cc=viro@ZenIV.linux.org.uk \
/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
Powered by JetHome