Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
From:   Madalin-cristian Bucur <madalin.bucur@nxp.com>
To:     Yi Wang <wang.yi59@zte.com.cn>
CC:     "davem@davemloft.net" <davem@davemloft.net>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "xue.zhihong@zte.com.cn" <xue.zhihong@zte.com.cn>,
        "huang.junhua@zte.com.cn" <huang.junhua@zte.com.cn>,
        Junhua Huang <huangjunhua@zte.com.cn>
Subject: RE: [PATCH] fsl/fman: avoid sleeping in atomic context while adding
 an address
Thread-Topic: [PATCH] fsl/fman: avoid sleeping in atomic context while adding
 an address
Thread-Index: AQHUo/FzF7mNl5dSC0yYo3tIMbNfaaWfEBOg
Date:   Fri, 4 Jan 2019 12:52:56 +0000
Message-ID: <VI1PR04MB5567B65C73553524AA3774FFEC8E0@VI1PR04MB5567.eurprd04.prod.outlook.com>
References: <1546581001-38798-1-git-send-email-wang.yi59@zte.com.cn>
In-Reply-To: <1546581001-38798-1-git-send-email-wang.yi59@zte.com.cn>
Accept-Language: en-US
Content-Language: en-US
received-spf: None (protection.outlook.com: nxp.com does not designate
 permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: e8d3fe6f-4d31-4bdf-0cfc-08d672438ca3
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Jan 2019 12:52:56.6244
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB1614
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi,

this is a duplicate of this other patch addressing the issue:

	commit 0d9c9a238faf925823bde866182c663b6d734f2e
	Author: Scott Wood <oss@buserror.net>
	Date:   Thu Dec 27 18:29:09 2018 -0600

	    fsl/fman: Use GFP_ATOMIC in {memac,tgec}_add_hash_mac_address()

Thank you,
Madalin

> -----Original Message-----
> From: Yi Wang <wang.yi59@zte.com.cn>
> Sent: Friday, January 4, 2019 7:50 AM
> To: Madalin-cristian Bucur <madalin.bucur@nxp.com>
> Cc: davem@davemloft.net; netdev@vger.kernel.org; linux-
> kernel@vger.kernel.org; xue.zhihong@zte.com.cn; wang.yi59@zte.com.cn;
> huang.junhua@zte.com.cn; Junhua Huang <huangjunhua@zte.com.cn>
> Subject: [PATCH] fsl/fman: avoid sleeping in atomic context while adding
> an address
>=20
> From: Junhua Huang <huang.junhua@zte.com.cn>
>=20
> dev_set_rx_mode will call function pointer mac_dev->add_hash_mac_addr
> while holding spin_lock_bh. The function pointer points to
> memac_add_hash_mac_address when ethernet type is fman-memac,
> which will kmalloc use GFP_KERNEL flag.
>=20
> / # ifconfig eth2 192.168.1.168
> [  576.604544] BUG: sleeping function called from invalid context at
> mm/slab.h:393
> [  576.610587] in_atomic(): 1, irqs_disabled(): 0, pid: 2751, name:
> ifconfig
> [  576.616105] 2 locks held by ifconfig/2751:
> [  576.618916] #0:(rtnl_mutex)at: [<c00000000080ba60>]
> .rtnl_lock+0x1c/0x30
> [  576.625523] #1:(_xmit_ETHER)at: [<c0000000007f9430>]
> .dev_set_rx_mode+0x24/0x54
> [  576.632745] CPU: 5 PID: 2751 Comm: ifconfig Tainted: G   W 4.9.115-
> rt93-EMBSYS-@332 #3
> [  576.642942] Call Trace:
> [  576.644085] [c00000007499b440] [c000000000a09eb4]
> .dump_stack+0xe0/0x14c (unreliable)
> [  576.650642] [c00000007499b4d0] [c000000000076f3c]
> .___might_sleep+0x1ac/0x278
> [  576.656493] [c00000007499b560] [c0000000001aad6c]
> .kmem_cache_alloc+0x144/0x28c
> [  576.662518] [c00000007499b620] [c000000000634c18]
> .memac_add_hash_mac_address+0x100/0x194
> [  576.669416] [c00000007499b6b0] [c000000000630b54]
> .set_multi+0x1bc/0x20c
> [  576.674829] [c00000007499b760] [c00000000063718c]
> .dpaa_set_rx_mode+0x84/0x104
> [  576.680765] [c00000007499b7e0] [c0000000007f9394]
> .__dev_set_rx_mode+0x64/0xdc
> [  576.686701] [c00000007499b870] [c0000000007f943c]
> .dev_set_rx_mode+0x30/0x54
> [  576.692464] [c00000007499b8f0] [c0000000007f98f4]
> .__dev_change_flags+0x98/0x1c4
> [  576.698573] [c00000007499b980] [c0000000007f9a4c]
> .dev_change_flags+0x2c/0x80
> [  576.704429] [c00000007499ba10] [c0000000008cde28]
> .devinet_ioctl+0x624/0x8e0
> [  576.710191] [c00000007499bb00] [c0000000008d1678]
> .inet_ioctl+0x1f4/0x250
> [  576.715697] [c00000007499bb70] [c0000000007c89d8]
> .sock_do_ioctl+0x50/0xa8
> [  576.721284] [c00000007499bc00] [c0000000007c94e8]
> .sock_ioctl+0x2b8/0x39c
> [  576.726786] [c00000007499bca0] [c0000000001e2500]
> .do_vfs_ioctl+0xc8/0x8b4
> [  576.732373] [c00000007499bd90] [c0000000001e2d44] .SyS_ioctl+0x58/0xa4
> [  576.737612] [c00000007499be30] [c0000000000007a4]
> system_call+0x38/0x108
>=20
> Signed-off-by: Junhua Huang <huangjunhua@zte.com.cn>
> ---
>  drivers/net/ethernet/freescale/fman/fman_memac.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>=20
> diff --git a/drivers/net/ethernet/freescale/fman/fman_memac.c
> b/drivers/net/ethernet/freescale/fman/fman_memac.c
> index 71a5ded..21dd557 100644
> --- a/drivers/net/ethernet/freescale/fman/fman_memac.c
> +++ b/drivers/net/ethernet/freescale/fman/fman_memac.c
> @@ -923,7 +923,7 @@ int memac_add_hash_mac_address(struct fman_mac *memac=
,
> enet_addr_t *eth_addr)
>  	hash =3D get_mac_addr_hash_code(addr) & HASH_CTRL_ADDR_MASK;
>=20
>  	/* Create element to be added to the driver hash table */
> -	hash_entry =3D kmalloc(sizeof(*hash_entry), GFP_KERNEL);
> +	hash_entry =3D kmalloc(sizeof(*hash_entry), GFP_ATOMIC);
>  	if (!hash_entry)
>  		return -ENOMEM;
>  	hash_entry->addr =3D addr;
> --
> 2.15.2