linux-kernel - Re: [PATCH net] mac8390: Fix mmio access size probe

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

Date:   Mon, 18 Mar 2019 16:59:01 -0700 (PDT)
From:   David Miller <davem@...emloft.net>
To:     fthain@...egraphics.com.au
Cc:     netdev@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH net] mac8390: Fix mmio access size probe

From: Finn Thain <fthain@...egraphics.com.au>
Date: Sat, 16 Mar 2019 14:21:19 +1100

> The bug that Stan reported is as follows. After a restart, a 16-bit NIC
> may be incorrectly identified as a 32-bit NIC and stop working.
> 
> mac8390 slot.E: Memory length resource not found, probing
> mac8390 slot.E: Farallon EtherMac II-C (type farallon)
> mac8390 slot.E: MAC 00:00:c5:30:c2:99, IRQ 61, 32 KB shared memory at 0xfeed0000, 32-bit access.
> 
> The bug never arises after a cold start and only intermittently after a
> warm start. (I didn't investigate why the bug is intermittent.)
> 
> It turns out that memcpy_toio() is deprecated and memcmp_withio() also
> has issues. Replacing these calls with mmio accessors fixes the problem.
> 
> Reported-and-tested-by: Stan Johnson <userm57@...oo.com>
> Fixes: 2964db0f5904 ("m68k: Mac DP8390 update")
> Signed-off-by: Finn Thain <fthain@...egraphics.com.au>

Applied and queued up for -stable, thanks.