UbiquitousOS
/
OpenCloudOS-Kernel
mirror of https://gitee.com/OpenCloudOS/OpenCloudOS-Kernel.git
11
0
Fork 0
Code Issues Projects Releases Wiki Activity

WAN: flush tx_queue in hdlc_ppp to prevent panic on rmmod hw_driver. 

tx_queue is used as a temporary queue when not allowed to queue skb
directly to the hw device driver (which may sleep). Most paths flush
it before returning, but ppp_start() currently cannot. Make sure we
don't leave skbs pointing to a non-existent device.

Thanks to Michael Barkowski for reporting this problem.

Signed-off-by: Krzysztof Hałasa <khc@pm.waw.pl>
Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
Krzysztof Halasa 2010-04-14 14:09:52 +00:00 committed by David S. Miller
parent 334656f33c
commit 31f634a63d
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
drivers/net/wan/hdlc_ppp.c
View File

@ -628,9 +628,15 @@ static void ppp_stop(struct net_device *dev)
ppp_cp_event(dev, PID_LCP, STOP, 0, 0, 0, NULL); ppp_cp_event(dev, PID_LCP, STOP, 0, 0, 0, NULL);
} }
static void ppp_close(struct net_device *dev)
{
ppp_tx_flush();
}
static struct hdlc_proto proto = { static struct hdlc_proto proto = {
.start = ppp_start, .start = ppp_start,
.stop = ppp_stop, .stop = ppp_stop,
.close = ppp_close,
.type_trans = ppp_type_trans, .type_trans = ppp_type_trans,
.ioctl = ppp_ioctl, .ioctl = ppp_ioctl,
.netif_rx = ppp_rx, .netif_rx = ppp_rx,