Re: [PATCH] ethernet: s2io: fix setting mac address during resume

patchwork-bot+netdevbpf
2021-10-14T14:20:12Z

Hello:

This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Wed, 13 Oct 2021 16:35:49 +0200 you wrote:
> From: Arnd Bergmann <arnd@arndb.de>
>
> After recent cleanups, gcc started warning about a suspicious
> memcpy() call during the s2io_io_resume() function:
>
> In function '__dev_addr_set',
> inlined from 'eth_hw_addr_set' at include/linux/etherdevice.h:318:2,
> inlined from 's2io_set_mac_addr' at drivers/net/ethernet/neterion/s2io.c:5205:2,
> inlined from 's2io_io_resume' at drivers/net/ethernet/neterion/s2io.c:8569:7:
> arch/x86/include/asm/string_32.h:182:25: error: '__builtin_memcpy' accessing 6 bytes at offsets 0 and 2 overlaps 4 bytes at offset 2 [-Werror=restrict]
> 182 | #define memcpy(t, f, n) __builtin_memcpy(t, f, n)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> include/linux/netdevice.h:4648:9: note: in expansion of macro 'memcpy'
> 4648 | memcpy(dev->dev_addr, addr, len);
> | ^~~~~~
>
> [...]

Here is the summary with links:
- ethernet: s2io: fix setting mac address during resume
https://git.kernel.org/netdev/net/c/40507e7aada8

You are awesome, thank you!
--
Deet-doot-dot, I am a bot.