From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paolo Bonzini Subject: Re: [PATCH] tcm_vhost: Add hotplug/hotunplug support Date: Wed, 06 Mar 2013 18:41:47 +0100 Message-ID: <51377FDB.1080605@redhat.com> References: <1362475027-12018-1-git-send-email-asias@redhat.com> <20130305121125.GA823@redhat.com> <20130306012811.GA20372@hj.localdomain> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org, "Michael S. Tsirkin" , virtualization@lists.linux-foundation.org, target-devel@vger.kernel.org, Stefan Hajnoczi To: Asias He Return-path: In-Reply-To: <20130306012811.GA20372@hj.localdomain> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org List-Id: kvm.vger.kernel.org Il 06/03/2013 02:28, Asias He ha scritto: >> > This can queue up quite a bit of memory if the handler thread >> > is delayed, no? Can we limit the # of outstanding events? >> > Will guest recover from a missed event? > Hmm, good point. Will limit the number. The size of 'struct > tcm_vhost_evt' is around 20 bytes. So if we limit it to 128, it is ~2.5K > of memory. > > Paolo, if we limit the number of outstanding events and set > vs->vs_events_dropped, the guest will recover from a missed event, right? Yes. At least it should (it doesn't yet). Paolo