[GIT PULL] parisc architecture fixes for 4.15-rc7

[GIT PULL] parisc architecture fixes for 4.15-rc7

[Helge Deller]

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(-)

Re: [GIT PULL] parisc architecture fixes for 4.15-rc7

[Kees Cook]

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

Re: [GIT PULL] parisc architecture fixes for 4.15-rc7

[Helge Deller]

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