From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:52401) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TkUEQ-0000A6-Er for qemu-devel@nongnu.org; Mon, 17 Dec 2012 01:34:56 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1TkU1E-00050Z-4s for qemu-devel@nongnu.org; Mon, 17 Dec 2012 01:21:14 -0500 Received: from mail-pb0-f45.google.com ([209.85.160.45]:58637) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1TkU1D-00050S-T1 for qemu-devel@nongnu.org; Mon, 17 Dec 2012 01:17:40 -0500 Received: by mail-pb0-f45.google.com with SMTP id mc8so3595477pbc.4 for ; Sun, 16 Dec 2012 22:17:39 -0800 (PST) From: Liu Yuan Date: Mon, 17 Dec 2012 14:17:26 +0800 Message-Id: <1355725047-5096-1-git-send-email-namei.unix@gmail.com> In-Reply-To: <1355565174-5121-1-git-send-email-namei.unix@gmail.com> References: <1355565174-5121-1-git-send-email-namei.unix@gmail.com> Subject: [Qemu-devel] [PATCH v2 1/2] sheepdog: don't update inode when create_and_write fails List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Kevin Wolf , MORITA Kazutaka From: Liu Yuan For the error case such as SD_RES_NO_SPACE, we shouldn't update the inode bitmap to avoid the scenario that the object is allocated but wasn't created at the server side. This will result in VM's IO error on the failed object. Cc: MORITA Kazutaka Cc: Kevin Wolf Signed-off-by: Liu Yuan --- block/sheepdog.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/block/sheepdog.c b/block/sheepdog.c index a48f58c..ef7bc81 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -714,10 +714,11 @@ static void coroutine_fn aio_read_response(void *opaque) * and max_dirty_data_idx are changed to include updated * index between them. */ - s->inode.data_vdi_id[idx] = s->inode.vdi_id; - s->max_dirty_data_idx = MAX(idx, s->max_dirty_data_idx); - s->min_dirty_data_idx = MIN(idx, s->min_dirty_data_idx); - + if (rsp.result == SD_RES_SUCCESS) { + s->inode.data_vdi_id[idx] = s->inode.vdi_id; + s->max_dirty_data_idx = MAX(idx, s->max_dirty_data_idx); + s->min_dirty_data_idx = MIN(idx, s->min_dirty_data_idx); + } /* * Some requests may be blocked because simultaneous * create requests are not allowed, so we search the -- 1.7.9.5