From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:42846) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WspRa-0002fJ-PG for qemu-devel@nongnu.org; Fri, 06 Jun 2014 04:24:18 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WspRT-0005A2-AX for qemu-devel@nongnu.org; Fri, 06 Jun 2014 04:24:10 -0400 Received: from oxygen.pond.sub.org ([144.76.244.19]:56278) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WspRT-00059m-4S for qemu-devel@nongnu.org; Fri, 06 Jun 2014 04:24:03 -0400 Received: from blackfin.pond.sub.org (p5B328A66.dip0.t-ipconnect.de [91.50.138.102]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by oxygen.pond.sub.org (Postfix) with ESMTPSA id E51F324DC3 for ; Fri, 6 Jun 2014 10:24:01 +0200 (CEST) From: Markus Armbruster References: <1401970536-18019-1-git-send-email-armbru@redhat.com> <53916F8B.9050809@redhat.com> Date: Fri, 06 Jun 2014 10:24:01 +0200 In-Reply-To: <53916F8B.9050809@redhat.com> (Paolo Bonzini's message of "Fri, 06 Jun 2014 09:36:43 +0200") Message-ID: <87vbsev42m.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [PATCH 0/3] virtio-blk: Suppress error action on r/w beyond end List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com, uobergfe@redhat.com Paolo Bonzini writes: > Il 05/06/2014 14:15, Markus Armbruster ha scritto: >> When a device model's I/O operation fails, we execute the error >> action. This lets layers above QEMU implement thin provisioning, or >> attempt to correct errors before they reach the guest. But when the >> I/O operation fails because its invalid, reporting the error to the >> guest is the only sensible action. >> >> This short series does exactly that for virtio-blk. I intend to do >> the same for IDE and SCSI. > > Actually SCSI already does it (see check_lba_range in > hw/scsi/scsi-disk.c). Thanks for thinking about it though! Your hint saved me some digging. Thanks!