linux-kernel - Re: [PATCH] m68k: don't select ARCH_HAS_DMA_PREP

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]

Message-ID: <CAMuHMdVqCOwxG1ru-wvgDmtNfezgR92qBOVhtEJjaLrXAjaE+A@mail.gmail.com>
Date:   Tue, 9 Jul 2019 09:08:02 +0200
From:   Geert Uytterhoeven <geert@...ux-m68k.org>
To:     Christoph Hellwig <hch@....de>
Cc:     Guenter Roeck <linux@...ck-us.net>,
        linux-m68k <linux-m68k@...ts.linux-m68k.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Greg Ungerer <gerg@...ux-m68k.org>
Subject: Re: [PATCH] m68k: don't select ARCH_HAS_DMA_PREP_COHERENT for nommu
 or coldfire

Hi Christoph,

On Mon, Jul 8, 2019 at 11:23 PM Christoph Hellwig <hch@....de> wrote:
> On Mon, Jul 08, 2019 at 10:39:48PM +0200, Geert Uytterhoeven wrote:
> > On Mon, Jul 8, 2019 at 7:51 PM Christoph Hellwig <hch@....de> wrote:
> > >
> > > m68k only provides the dma_prep_coherent symbol when an mmu is enabled
> >
> > arch_dma_prep_coherent
> >
> > > and not on the coldfire platform.  Fix the Kconfig symbol selection
> > > up to match this.
> > >
> > > Fixes: 69878ef47562 ("m68k: Implement arch_dma_prep_coherent()")
> >
> > Do you know the SHA1 for the other commit, that causes the issue when
> > combined with the above?
>
> I think the culprit is:
>
> commit c30700db9eaabb35e0b123301df35a6846e6b6b4
> Author: Christoph Hellwig <hch@....de>
> Date:   Mon Jun 3 08:43:51 2019 +0200
>
>     dma-direct: provide generic support for uncached kernel segments
>
>
> Ad it turns out I can't just apply this fix to the dma-mapping tree
> because it doesn't have the m68k changes.  So either you'll have to
> queue it up, or I'll have to do secondary pull request to fix up
> the first one.  Maybe it is eiter if you just send it to Linus
> before I send the dma-mapping PR?

OK, will do, after a cycle in next.

I assume you just forgot to add your SoB, and I can add it?

Thanks!

Gr{oetje,eeting}s,

                        Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds