From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755896Ab0G0IW3 (ORCPT ); Tue, 27 Jul 2010 04:22:29 -0400 Received: from hera.kernel.org ([140.211.167.34]:40984 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755417Ab0G0IW0 (ORCPT ); Tue, 27 Jul 2010 04:22:26 -0400 Message-ID: <4C4E970A.2030607@kernel.org> Date: Tue, 27 Jul 2010 10:21:30 +0200 From: Tejun Heo User-Agent: Mozilla/5.0 (X11; U; Linux i686 (x86_64); en-US; rv:1.9.2.4) Gecko/20100608 Thunderbird/3.1 MIME-Version: 1.0 To: "Michael S. Tsirkin" CC: Oleg Nesterov , Sridhar Samudrala , netdev , lkml , "kvm@vger.kernel.org" , Andrew Morton , Dmitri Vorobiev , Jiri Kosina , Thomas Gleixner , Ingo Molnar , Andi Kleen Subject: Re: [PATCH UPDATED 1/3] vhost: replace vhost_workqueue with per-vhost kthread References: <20100722155840.GA1743@redhat.com> <4C48B664.9000109@kernel.org> <20100724191447.GA4972@redhat.com> <4C4BEAA2.6040301@kernel.org> <20100726152510.GA26223@redhat.com> <4C4DAB14.5050809@kernel.org> <20100726155014.GA26412@redhat.com> <4C4DB247.9060709@kernel.org> <20100726162346.GD26412@redhat.com> <4C4DDC31.9070206@kernel.org> <20100726201907.GF27644@redhat.com> In-Reply-To: <20100726201907.GF27644@redhat.com> X-Enigmail-Version: 1.1.1 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.2.3 (hera.kernel.org [127.0.0.1]); Tue, 27 Jul 2010 08:21:31 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hello, On 07/26/2010 10:19 PM, Michael S. Tsirkin wrote: > Let's try to define what do we want to achieve then. Do you want > code that flushes workers not to block when workers are frozen? How > will we handle work submitted when worker is frozen? As I wrote earlier, it's not necessarily about correctness but rather avoiding unnecessary surprises and of course flushing can and should stall if the queue is frozen but let's not separate execution of a work and its completion with something which can take undeterminate amount of time. Thanks. -- tejun