linux-kernel - Re: [GIT PULL] sh updates for v6.5

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>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 6 Jul 2023 07:01:37 -0700
From:   Guenter Roeck <linux@...ck-us.net>
To:     John Paul Adrian Glaubitz <glaubitz@...sik.fu-berlin.de>,
        Linus Torvalds <torvalds@...ux-foundation.org>
Cc:     Sergey Shtylyov <s.shtylyov@....ru>,
        Masahiro Yamada <masahiroy@...nel.org>,
        Artur Rojek <contact@...ur-rojek.eu>,
        Rich Felker <dalias@...c.org>,
        Yoshinori Sato <ysato@...rs.sourceforge.jp>,
        linux-sh@...r.kernel.org,
        linux-kernel <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL] sh updates for v6.5

On 7/5/23 14:43, John Paul Adrian Glaubitz wrote:
> Hi Linus!
> 
> I am being a little late this merge window since it took me a little longer
> to thoroughly review the changes which address important issues in the DMA
> and IRQ code in arch/sh.
> 
> The pull request includes a patch by me to fix a compiler warning in the J2
> probing code and a fix by Sergey Shtylyov to avoid using IRQ0 on SH3 and SH4
> targets. Masahiro Yamada made some clean-up in the build system to address
> reports by the 0day bot.
> 
> The most notable changes come from Artur Rojek who addressed a number of issues
> in the DMA code, in particular a fix for the DMA channel offset calculation that
> was introduced in in 7f47c7189b3e ("sh: dma: More legacy cpu dma chainsawing.")
> in 2012! Together with another change to correct the number of DMA channels for
> each SuperH SoC according to specification, Artur's series unbreaks the kernel
> on the SH7709 SoC allowing Linux to boot on the HP Jornada 680 handheld again.
> 
> Last but not least, Guenter Roeck sent in a patch to fix a build regression that
> was recently introduced in 99b619b37ae1 ("mips: provide unxlate_dev_mem_ptr() in
> asm/io.h").
> 
> The following changes since commit ac9a78681b921877518763ba0e89202254349d1b:
> 
>    Linux 6.4-rc1 (2023-05-07 13:34:35 -0700)
> 
> are available in the Git repository at:
> 
>    git://git.kernel.org/pub/scm/linux/kernel/git/glaubitz/sh-linux.git tags/sh-for-v6.5-tag1
> 
> for you to fetch changes up to 7497840d462c8f54c4888c22ab3726a8cde4b9a2:
> 
>    sh: Provide unxlate_dev_mem_ptr() in asm/io.h (2023-07-05 19:04:51 +0200)
> 
> Thanks for pulling!
> 
> Adrian
> 
> ----------------------------------------------------------------
> sh updates for v6.5
> 
> - sh: Provide unxlate_dev_mem_ptr() in asm/io.h
> - sh: dma: Correct the number of DMA channels for SH7709
> - sh: dma: Drop incorrect SH_DMAC_BASE1 definition for SH4
> - sh: dma: Fix DMA channel offset calculation
> - sh: Remove compiler flag duplication
> - sh: Refactor header include path addition
> - sh: Move build rule for cchips/hd6446x/ to arch/sh/Kbuild
> - sh: Fix -Wmissing-include-dirs warnings for various platforms
> - sh: Avoid using IRQ0 on SH3 and SH4
> - sh: j2: Use ioremap() to translate device tree address into kernel memory
> 
> ----------------------------------------------------------------
> Artur Rojek (3):
>        sh: dma: Fix DMA channel offset calculation
>        sh: dma: Drop incorrect SH_DMAC_BASE1 definition for SH4
>        sh: dma: Correct the number of DMA channels for SH7709
> 
> Guenter Roeck (1):
>        sh: Provide unxlate_dev_mem_ptr() in asm/io.h
> 

Perfect example why it is a bad idea to let build failures linger around.
The build failure fixed by this patch ...

> John Paul Adrian Glaubitz (1):
>        sh: j2: Use ioremap() to translate device tree address into kernel memory
> 
> Masahiro Yamada (4):
>        sh: Fix -Wmissing-include-dirs warnings for various platforms
>        sh: Move build rule for cchips/hd6446x/ to arch/sh/Kbuild
>        sh: Refactor header include path addition
>        sh: Remove compiler flag duplication
> 
> Sergey Shtylyov (1):
>        sh: Avoid using IRQ0 on SH3 and SH4
> 
... was hiding boot failures with all my qemu emulations caused by
this patch.

Guenter