On Fri, Jul 05, 2013 at 10:36:46AM +0200, Bart Van Assche wrote: > Wake up work->done waiters even if the TIF_NEED_RESCHED task flag > has been set. This patch fixes a regression introduced in commit > d550dda (kernel v3.4). > > Signed-off-by: Bart Van Assche <bvanassche@xxxxxxx> > Reference: https://bugzilla.kernel.org/show_bug.cgi?id=60505 > Cc: Michael S. Tsirkin <mst@xxxxxxxxxx> > Cc: Asias He <asias@xxxxxxxxxx> > Cc: Nadav Har'El <nyh@xxxxxxxxxxxxxxxxxxx> > Cc: Abel Gordon <abelg@xxxxxxxxxx> > Cc: <stable@xxxxxxxxxxxxxxx> # v3.4+ I just posted a patch fixing a bug in this function. [PATCHv3] vhost-net: fix use-after-free in vhost_net_flush could you please try with this patch applied? > --- > drivers/vhost/vhost.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/drivers/vhost/vhost.c b/drivers/vhost/vhost.c > index 60aa5ad..cd544ae 100644 > --- a/drivers/vhost/vhost.c > +++ b/drivers/vhost/vhost.c > @@ -227,8 +227,16 @@ static int vhost_worker(void *data) > if (work) { > __set_current_state(TASK_RUNNING); > work->fn(work); > - if (need_resched()) > + if (need_resched()) { > + spin_lock_irq(&dev->work_lock); > + work->done_seq = seq; > + if (work->flushing) > + wake_up_all(&work->done); > + spin_unlock_irq(&dev->work_lock); > + > + work = NULL; > schedule(); > + } > } else > schedule(); > > -- > 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
