From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:48938) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XgYum-0002iC-O7 for qemu-devel@nongnu.org; Tue, 21 Oct 2014 08:51:57 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XgYuh-0006sk-UU for qemu-devel@nongnu.org; Tue, 21 Oct 2014 08:51:52 -0400 Received: from mx1.redhat.com ([209.132.183.28]:54332) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XgYuh-0006s3-Du for qemu-devel@nongnu.org; Tue, 21 Oct 2014 08:51:47 -0400 Message-ID: <544656D6.8070908@redhat.com> Date: Tue, 21 Oct 2014 14:51:34 +0200 From: Paolo Bonzini MIME-Version: 1.0 References: <1413378926-52442-1-git-send-email-cornelia.huck@de.ibm.com> In-Reply-To: <1413378926-52442-1-git-send-email-cornelia.huck@de.ibm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 0/3] virtio-scsi-dataplane: graceful fail List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Cornelia Huck , famz@redhat.com Cc: qemu-devel@nongnu.org On 10/15/2014 03:15 PM, Cornelia Huck wrote: > Hi, > > these patches are practically a duplicate of the patches I did for > virtio-blk dataplane -- they don't convert to the error report > infrastructure, for example. The aim was to do the same changes > as for virtio-blk in order to avoid killing the whole guest if we > run e.g. out of file handles for the notifiers. > > Untested (better get your kittens to a safe place before trying out), > but a git branch is available at > > git://github.com/cohuck/qemu graceful-scsi-dataplane > > Cornelia Huck (3): > virtio-scsi: dataplane: print why starting failed > virtio-scsi: dataplane: fail setup gracefully > virtio-scsi: dataplane: stop trying on notifier error > > hw/scsi/virtio-scsi-dataplane.c | 92 +++++++++++++++++++++++++++++++++++------ > include/hw/virtio/virtio-scsi.h | 1 + > 2 files changed, 80 insertions(+), 13 deletions(-) > Thanks. I'll test and send a pull request. Paolo