From patchwork Sun Apr 14 05:07:40 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Liu Yuan X-Patchwork-Id: 236402 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [208.118.235.17]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 543872C00A1 for ; Sun, 14 Apr 2013 15:08:15 +1000 (EST) Received: from localhost ([::1]:51664 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URFAj-0002kB-68 for incoming@patchwork.ozlabs.org; Sun, 14 Apr 2013 01:08:13 -0400 Received: from eggs.gnu.org ([208.118.235.92]:42980) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URFAS-0002jv-I3 for qemu-devel@nongnu.org; Sun, 14 Apr 2013 01:07:58 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1URFAR-0001r8-5b for qemu-devel@nongnu.org; Sun, 14 Apr 2013 01:07:56 -0400 Received: from mail-pb0-f46.google.com ([209.85.160.46]:50723) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1URFAQ-0001qy-S9 for qemu-devel@nongnu.org; Sun, 14 Apr 2013 01:07:55 -0400 Received: by mail-pb0-f46.google.com with SMTP id rp8so2019304pbb.19 for ; Sat, 13 Apr 2013 22:07:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=x-received:from:to:cc:subject:date:message-id:x-mailer:in-reply-to :references; bh=/1lt7HFvmDKGq+p4klo+leioVVWTfvz3bpFbcc8XvEs=; b=IyhG+eObbQP4UwIlWX/KhllQMhn0FFL1plx5J0z/IOTNyAzw8v5D8Y0NCEbkfMa+IZ YdlUId8izRqB3skxO0830XikP+RgZmjZYlc4UtEYvKQ9byq5EYI9s66aA7k4oqCWgQS9 zjWhfl8RTbYUJE9n82o6cAfBg/6col4ESCpViqM04LYhrJ4GHIGrP5xR4MfQuP9S+O2B pRyIS1TG1v0DZt0sC1qwdpKJvD7n0sZjafelUDFgy9bscIS5k31uNVMyKKrka18nrUvl lkk/TIdsOnSJq+NMk7ueSsAMD4vfEVF0f027mtlK/5CR6U8LSBFtUqFg5Dzn4ZV4DXBc PzLA== X-Received: by 10.66.120.49 with SMTP id kz17mr23505691pab.133.1365916074133; Sat, 13 Apr 2013 22:07:54 -0700 (PDT) Received: from localhost.localdomain ([114.244.4.112]) by mx.google.com with ESMTPS id ba10sm15024351pbd.21.2013.04.13.22.07.48 (version=TLSv1.1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Sat, 13 Apr 2013 22:07:52 -0700 (PDT) From: Liu Yuan To: sheepdog@lists.wpkg.org, qemu-devel@nongnu.org Date: Sun, 14 Apr 2013 13:07:40 +0800 Message-Id: <1365916060-11645-1-git-send-email-namei.unix@gmail.com> X-Mailer: git-send-email 1.7.9.5 In-Reply-To: <1365852442-28941-5-git-send-email-namei.unix@gmail.com> References: <1365852442-28941-5-git-send-email-namei.unix@gmail.com> X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x [fuzzy] X-Received-From: 209.85.160.46 Cc: Kevin Wolf , Paolo Bonzini , Stefan Hajnoczi , MORITA Kazutaka Subject: [Qemu-devel] [PATCH for QEMU v2] sheepdog: add discard/trim support for sheepdog X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org From: Liu Yuan The 'TRIM' command from VM that is to release underlying data storage for better thin-provision is already supported by the Sheepdog. This patch adds the TRIM support at QEMU part. For older Sheepdog that doesn't support, we return EIO to upper layer. Cc: MORITA Kazutaka Cc: Kevin Wolf Cc: Stefan Hajnoczi Cc: Paolo Bonzini Signed-off-by: Liu Yuan --- v2: - skip the object when it is not allocated block/sheepdog.c | 53 +++++++++++++++++++++++++++++++++++++++++++++++++++-- 1 file changed, 51 insertions(+), 2 deletions(-) diff --git a/block/sheepdog.c b/block/sheepdog.c index 987018e..0d1b047 100644 --- a/block/sheepdog.c +++ b/block/sheepdog.c @@ -34,6 +34,7 @@ #define SD_OP_GET_VDI_INFO 0x14 #define SD_OP_READ_VDIS 0x15 #define SD_OP_FLUSH_VDI 0x16 +#define SD_OP_DISCARD 0x17 #define SD_FLAG_CMD_WRITE 0x01 #define SD_FLAG_CMD_COW 0x02 @@ -269,6 +270,7 @@ enum AIOCBState { AIOCB_WRITE_UDATA, AIOCB_READ_UDATA, AIOCB_FLUSH_CACHE, + AIOCB_DISCARD, }; struct SheepdogAIOCB { @@ -656,7 +658,7 @@ static void coroutine_fn aio_read_response(void *opaque) int ret; AIOReq *aio_req = NULL; SheepdogAIOCB *acb; - unsigned long idx; + uint64_t idx; if (QLIST_EMPTY(&s->inflight_aio_head)) { goto out; @@ -727,6 +729,18 @@ static void coroutine_fn aio_read_response(void *opaque) rsp.result = SD_RES_SUCCESS; } break; + case AIOCB_DISCARD: + switch (rsp.result) { + case SD_RES_INVALID_PARMS: + error_report("you are running the old sheep that doesn't support " + "discard command.\n"); + break; + case SD_RES_SUCCESS: + idx = data_oid_to_idx(aio_req->oid); + s->inode.data_vdi_id[idx] = 0; + break; + } + break; } if (rsp.result != SD_RES_SUCCESS) { @@ -1016,6 +1030,9 @@ static int coroutine_fn add_aio_request(BDRVSheepdogState *s, AIOReq *aio_req, wlen = datalen; hdr.flags = SD_FLAG_CMD_WRITE | flags; break; + case AIOCB_DISCARD: + hdr.opcode = SD_OP_DISCARD; + break; } if (s->cache_flags) { @@ -1633,7 +1650,14 @@ static int coroutine_fn sd_co_rw_vector(void *p) flags = SD_FLAG_CMD_COW; } break; - default: + case AIOCB_DISCARD: + /* + * We discard the object only when the whole object is + * 1) allocated 2) trimmed. Otherwise, simply skip it. + */ + if (len != SD_DATA_OBJ_SIZE || inode->data_vdi_id[idx] == 0) { + goto done; + } break; } @@ -2071,6 +2095,28 @@ static int sd_load_vmstate(BlockDriverState *bs, uint8_t *data, } +static int sd_co_discard(BlockDriverState *bs, int64_t sector_num, + int nb_sectors) +{ + SheepdogAIOCB *acb; + QEMUIOVector dummy; + int ret; + + acb = sd_aio_setup(bs, &dummy, sector_num, nb_sectors); + acb->aiocb_type = AIOCB_DISCARD; + acb->aio_done_func = sd_finish_aiocb; + + ret = sd_co_rw_vector(acb); + if (ret <= 0) { + qemu_aio_release(acb); + return ret; + } + + qemu_coroutine_yield(); + + return acb->ret; +} + static QEMUOptionParameter sd_create_options[] = { { .name = BLOCK_OPT_SIZE, @@ -2103,6 +2149,7 @@ static BlockDriver bdrv_sheepdog = { .bdrv_co_readv = sd_co_readv, .bdrv_co_writev = sd_co_writev, .bdrv_co_flush_to_disk = sd_co_flush_to_disk, + .bdrv_co_discard = sd_co_discard, .bdrv_snapshot_create = sd_snapshot_create, .bdrv_snapshot_goto = sd_snapshot_goto, @@ -2128,6 +2175,7 @@ static BlockDriver bdrv_sheepdog_tcp = { .bdrv_co_readv = sd_co_readv, .bdrv_co_writev = sd_co_writev, .bdrv_co_flush_to_disk = sd_co_flush_to_disk, + .bdrv_co_discard = sd_co_discard, .bdrv_snapshot_create = sd_snapshot_create, .bdrv_snapshot_goto = sd_snapshot_goto, @@ -2153,6 +2201,7 @@ static BlockDriver bdrv_sheepdog_unix = { .bdrv_co_readv = sd_co_readv, .bdrv_co_writev = sd_co_writev, .bdrv_co_flush_to_disk = sd_co_flush_to_disk, + .bdrv_co_discard = sd_co_discard, .bdrv_snapshot_create = sd_snapshot_create, .bdrv_snapshot_goto = sd_snapshot_goto,