* [GIT PULL] parisc architecture fixes for 4.15-rc7
@ 2018-01-07 16:21 Helge Deller
2018-01-09 23:03 ` Kees Cook
0 siblings, 1 reply; 3+ messages in thread
From: Helge Deller @ 2018-01-07 16:21 UTC (permalink / raw)
To: Linus Torvalds, linux-kernel, linux-parisc, James Bottomley,
John David Anglin
Cc: Richard Henderson
Hi Linus,
please pull a few fixes for the parisc architecture from:
git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git parisc-4.15-3
The fixes are:
- Many small fixes to show the real physical addresses of devices instead of
hashed addresses.
- One important fix to unbreak 32-bit SMP support: We forgot to 16-byte align
the spinlocks in the assembler code.
- Qemu support: The host will get a chance to sleep when the parisc guest is
idle. We use the same mechanism as the power architecture by overlaying the
"or %r10,%r10,%r10" instruction which is simply a nop on real hardware.
Thanks,
Helge
----------------------------------------------------------------
Helge Deller (6):
parisc: Show unhashed hardware inventory
parisc: Show initial kernel memory layout unhashed
parisc: Show unhashed HPA of Dino chip
parisc: Show unhashed EISA EEPROM address
parisc: Fix alignment of pa_tlb_lock in assembly on 32-bit SMP kernel
parisc: qemu idle sleep support
arch/parisc/include/asm/ldcw.h | 2 ++
arch/parisc/kernel/drivers.c | 2 +-
arch/parisc/kernel/entry.S | 13 +++++++++++--
arch/parisc/kernel/pacache.S | 9 +++++++--
arch/parisc/kernel/process.c | 39 +++++++++++++++++++++++++++++++++++++++
arch/parisc/mm/init.c | 10 +++++-----
drivers/parisc/dino.c | 10 +++++-----
drivers/parisc/eisa_eeprom.c | 2 +-
8 files changed, 71 insertions(+), 16 deletions(-)
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [GIT PULL] parisc architecture fixes for 4.15-rc7
2018-01-07 16:21 [GIT PULL] parisc architecture fixes for 4.15-rc7 Helge Deller
@ 2018-01-09 23:03 ` Kees Cook
2018-01-10 13:44 ` Helge Deller
0 siblings, 1 reply; 3+ messages in thread
From: Kees Cook @ 2018-01-09 23:03 UTC (permalink / raw)
To: Helge Deller, Linus Torvalds
Cc: LKML, linux-parisc, James Bottomley, John David Anglin,
Richard Henderson, Laura Abbott
On Sun, Jan 7, 2018 at 8:21 AM, Helge Deller <deller@gmx.de> wrote:
> Hi Linus,
>
> please pull a few fixes for the parisc architecture from:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git parisc-4.15-3
>
> The fixes are:
> - Many small fixes to show the real physical addresses of devices instead of
> hashed addresses.
>
> - One important fix to unbreak 32-bit SMP support: We forgot to 16-byte align
> the spinlocks in the assembler code.
>
> - Qemu support: The host will get a chance to sleep when the parisc guest is
> idle. We use the same mechanism as the power architecture by overlaying the
> "or %r10,%r10,%r10" instruction which is simply a nop on real hardware.
>
> Thanks,
> Helge
>
> ----------------------------------------------------------------
> Helge Deller (6):
> parisc: Show unhashed hardware inventory
> parisc: Show unhashed EISA EEPROM address
These looks like a physical addresses, yes.
> parisc: Show initial kernel memory layout unhashed
> parisc: Show unhashed HPA of Dino chip
But these are virtual addresses and should be just removed, not
switched to %px. They're not physical addresses that I can see, e.g.
"dino_dev" is clearly being used as an in-memory structure, and the
kernel layout literally says "virtual kernel memory layout".
Especially the kernel memory layout going to dmesg should be removed.
That kind of thing has been entirely removed from other architectures
(or will be soon[1]). See commit adb1fe9ae2ee ("mm/page_alloc: Remove
kernel address exposure in free_reserved_area()"). (Also note that
hiding behind CONFIG_DEBUG_KERNEL is not sufficient: most distros
build with that config.)
-Kees
[1] https://www.spinics.net/lists/arm-kernel/msg624390.html
> parisc: Fix alignment of pa_tlb_lock in assembly on 32-bit SMP kernel
> parisc: qemu idle sleep support
>
> arch/parisc/include/asm/ldcw.h | 2 ++
> arch/parisc/kernel/drivers.c | 2 +-
> arch/parisc/kernel/entry.S | 13 +++++++++++--
> arch/parisc/kernel/pacache.S | 9 +++++++--
> arch/parisc/kernel/process.c | 39 +++++++++++++++++++++++++++++++++++++++
> arch/parisc/mm/init.c | 10 +++++-----
> drivers/parisc/dino.c | 10 +++++-----
> drivers/parisc/eisa_eeprom.c | 2 +-
> 8 files changed, 71 insertions(+), 16 deletions(-)
--
Kees Cook
Pixel Security
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [GIT PULL] parisc architecture fixes for 4.15-rc7
2018-01-09 23:03 ` Kees Cook
@ 2018-01-10 13:44 ` Helge Deller
0 siblings, 0 replies; 3+ messages in thread
From: Helge Deller @ 2018-01-10 13:44 UTC (permalink / raw)
To: Kees Cook, Linus Torvalds
Cc: LKML, linux-parisc, James Bottomley, John David Anglin,
Richard Henderson, Laura Abbott
On 10.01.2018 00:03, Kees Cook wrote:
> On Sun, Jan 7, 2018 at 8:21 AM, Helge Deller <deller@gmx.de> wrote:
>> Hi Linus,
>>
>> please pull a few fixes for the parisc architecture from:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/deller/parisc-linux.git parisc-4.15-3
>>
>> The fixes are:
>> - Many small fixes to show the real physical addresses of devices instead of
>> hashed addresses.
>>
>> - One important fix to unbreak 32-bit SMP support: We forgot to 16-byte align
>> the spinlocks in the assembler code.
>>
>> - Qemu support: The host will get a chance to sleep when the parisc guest is
>> idle. We use the same mechanism as the power architecture by overlaying the
>> "or %r10,%r10,%r10" instruction which is simply a nop on real hardware.
>>
>> Thanks,
>> Helge
>>
>> ----------------------------------------------------------------
>> Helge Deller (6):
>> parisc: Show unhashed hardware inventory
>> parisc: Show unhashed EISA EEPROM address
>
> These looks like a physical addresses, yes.
>
>> parisc: Show initial kernel memory layout unhashed
>> parisc: Show unhashed HPA of Dino chip
>
> But these are virtual addresses and should be just removed, not
> switched to %px. They're not physical addresses that I can see, e.g.
> "dino_dev" is clearly being used as an in-memory structure, and the
> kernel layout literally says "virtual kernel memory layout".
Agreed, some of those should go.
I'll clean that up in upcoming patches.
Thanks!
Helge
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-01-10 13:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-07 16:21 [GIT PULL] parisc architecture fixes for 4.15-rc7 Helge Deller
2018-01-09 23:03 ` Kees Cook
2018-01-10 13:44 ` Helge Deller
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).