netdev - RE: linux-next ftmac100 driver

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

Message-ID: <D557AF68929979498EDCFCBB34FFD4E82C62BC@ftcpcs78.faraday.com.tw>
Date:	Mon, 25 Apr 2011 11:26:13 +0800
From:	Ratbert Po-Yu Chuang(莊博宇) 
	<ratbert@...aday-tech.com>
To:	"Adam Jaremko" <adam.jaremko@...il.com>
Cc:	<ratbert.chuang@...il.com>, <netdev@...r.kernel.org>
Subject: RE: linux-next ftmac100 driver

Hi Adam,

CC to the netdev mainling list.

From: Adam Jaremko [mailto:adam.jaremko@...il.com] 
> Hello,
> 
> I want to first thank you for the upstream contributions as they greatly helped me in my porting effort.
> 
> I have been testing your ftmac100 driver on a Moxa UC-7112 and have been struggling with a kernel BUG "scheduling while atomic".
>
> I narrowed down the bug to the napi polling routine ftmac100_poll(...) when a physical change occurs, ie. RJ-45 cable removed.
> @ http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=blob;f=drivers/net/ftmac100.c;h=a31661948c420e0d467c7e8b7bb4ecae4c078de0;hb=HEAD#l932
>
> the call to mii_check_link(...) calls ftmac100_mdio_read(...) which may incur a call to usleep_range(...) causing the bug.
> @ http://git.kernel.org/?p=linux/kernel/git/next/linux-next.git;a=blob;f=drivers/net/ftmac100.c;h=a31661948c420e0d467c7e8b7bb4ecae4c078de0;hb=HEAD#l757
> I haven't got a fix but I wanted to inform as you may have more insight.

You can try to use udelay() to replace usleep_range(). I will also check this problem when I am free later.
If you solved this problem before me, please send a message (and keep the CC list).
It will be even better if you could submit a patch. :-)

Thanks,
Po-Yu Chuang

********************* Confidentiality Notice ************************
This electronic message and any attachments may contain
confidential and legally privileged information or
information which is otherwise protected from disclosure.
If you are not the intended recipient,please do not disclose
the contents, either in whole or in part, to anyone,and
immediately delete the message and any attachments from
your computer system and destroy all hard copies.
Thank you for your cooperation.
***********************************************************************

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html