[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH v4 net] xen-netback: disable rogue vif in kthread context
On Wed, Apr 02, 2014 at 10:05:40AM +0100, Ian Campbell wrote: > On Tue, 2014-04-01 at 16:27 -0400, David Miller wrote: > > From: Wei Liu <wei.liu2@xxxxxxxxxx> > > Date: Tue, 1 Apr 2014 12:46:12 +0100 > > > > > When netback discovers frontend is sending malformed packet it will > > > disables the interface which serves that frontend. > > > > > > However disabling a network interface involving taking a mutex which > > > cannot be done in softirq context, so we need to defer this process to > > > kthread context. > > > > > > This patch does the following: > > > 1. introduce a flag to indicate the interface is disabled. > > > 2. check that flag in TX path, don't do any work if it's true. > > > 3. check that flag in RX path, turn off that interface if it's true. > > > > > > The reason to disable it in RX path is because RX uses kthread. After > > > this change the behavior of netback is still consistent -- it won't do > > > any TX work for a rogue frontend, and the interface will be eventually > > > turned off. > > > > > > Also change a "continue" to "break" after xenvif_fatal_tx_err, as it > > > doesn't make sense to continue processing packets if frontend is rogue. > > > > > > This is a fix for XSA-90. > > > > > > Reported-by: Török Edwin <edwin@xxxxxxxxxx> > > > Signed-off-by: Wei Liu <wei.liu2@xxxxxxxxxx> > > > > Applied. > > Thanks. > > Wei -- can you advise as to which stable trees this should be queued > for? Looks like 3.12 onwards according to > http://xenbits.xen.org/xsa/advisory-90.html which says the issue arose > between 3.11 and 3.12-rc1? > Correct. Wei. > Cheers, > Ian. > _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx http://lists.xen.org/xen-devel
|
Lists.xenproject.org is hosted with RackSpace, monitoring our |