From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.9 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH,MAILING_LIST_MULTI, NICE_REPLY_A,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id C1FAEC43461 for ; Thu, 3 Sep 2020 14:58:49 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id A211620716 for ; Thu, 3 Sep 2020 14:58:49 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=kernel-dk.20150623.gappssmtp.com header.i=@kernel-dk.20150623.gappssmtp.com header.b="uDSAKNtW" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729029AbgICO6p (ORCPT ); Thu, 3 Sep 2020 10:58:45 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60918 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729112AbgICOHz (ORCPT ); Thu, 3 Sep 2020 10:07:55 -0400 Received: from mail-il1-x144.google.com (mail-il1-x144.google.com [IPv6:2607:f8b0:4864:20::144]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 75F87C06123F for ; Thu, 3 Sep 2020 07:07:43 -0700 (PDT) Received: by mail-il1-x144.google.com with SMTP id t13so2579091ile.9 for ; Thu, 03 Sep 2020 07:07:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20150623.gappssmtp.com; s=20150623; h=subject:to:cc:references:from:message-id:date:user-agent :mime-version:in-reply-to:content-language:content-transfer-encoding; bh=CB3FH1QTmBm5WlgHNeHzU8TKjErR+j9aqn+2gr2lPEw=; b=uDSAKNtWtRc8PWxUu0oPgJvfab/Bga9fBkkRGIIgQRqyCST1bhF02ygsIzz2i/xBxZ MtlnOt+1zNDQCKvvgR4PV1z/54LjcQdTc9McD3ScL4K5OLpJwGC8SyH8n6fyTLbal7WG afQbDqvOe5JaqD5krBY3LkRnjLDpMx/6sdhrjJk5YeQs+E0ng3kVa5Cg17SBTQuUpQHP qv/J723U4VdNybmZGzJJvUSpYqaOD4CJ5SFVOW47m8g0T/MoT8+IT/xA0Gw/A/TIlYkT lalpQ2Kb7ZwsepnWPSWsQbsGy6/Gx7hdM35VU+zkYrkkvtbUpR9yVmtJWW/WUHEZvuo9 IrVw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=CB3FH1QTmBm5WlgHNeHzU8TKjErR+j9aqn+2gr2lPEw=; b=DbJVI00lp0yAFJLu+FMl2g9Bjiwpetbgemn2M817ncSbbvQerWkXy2J9xDKPlPgY5u wj+c86ipCdMfJ6TWpbwXGwh/IkiLqveELQ3giP7Zl2kYicsiXJidfE8QdJ20Q+4amtG1 6iHbh/hcmVMSKjMv1mhtSi+ahTDBm9LBWciXoZcRPcTc3f/3H1imBsfdmiCRrmSYKbVQ T7JSqE1NPg+unU2pvktYVPU61nq+mUbp8V9JBJs8IChWMqmQfL2oW8q38/6aQrMwnBAC 1J3QiaTvaAYor6b8HkcUkC4Gqux1LoJ0qVaDgVPCl5rmLG3Gf9BSLWT13/3i7yrtW/F6 a6Zw== X-Gm-Message-State: AOAM532uyZuZvC2ti/HtAo6Js9wCEiA0AvTZl+BLMcs9NY9HZtQDbMXl 3IX1ucKrVbIUXD6hnWJTurGaH+sYKgsFqlVb X-Google-Smtp-Source: ABdhPJxXroKmKQ10gJDWIpNfRgglbkAEgQnZGGeTDbrG0AaTg20iBVmDDt2vx3NE6+TcJhc0uxLCJQ== X-Received: by 2002:a92:3402:: with SMTP id b2mr3253560ila.181.1599142062711; Thu, 03 Sep 2020 07:07:42 -0700 (PDT) Received: from [192.168.1.57] ([65.144.74.34]) by smtp.gmail.com with ESMTPSA id u18sm320881iln.78.2020.09.03.07.07.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 03 Sep 2020 07:07:42 -0700 (PDT) Subject: Re: [PATCH v3 1/4] fsstress: add IO_URING read and write operations To: Brian Foster , Zorro Lang Cc: fstests@vger.kernel.org, io-uring@vger.kernel.org References: <20200823063032.17297-1-zlang@redhat.com> <20200823063032.17297-2-zlang@redhat.com> <20200903124247.GA444163@bfoster> From: Jens Axboe Message-ID: Date: Thu, 3 Sep 2020 08:07:41 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.10.0 MIME-Version: 1.0 In-Reply-To: <20200903124247.GA444163@bfoster> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: fstests-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: fstests@vger.kernel.org On 9/3/20 6:42 AM, Brian Foster wrote: > On Sun, Aug 23, 2020 at 02:30:29PM +0800, Zorro Lang wrote: >> IO_URING is a new feature of curent linux kernel, add basic IO_URING >> read/write into fsstess to cover this kind of IO testing. >> >> Signed-off-by: Zorro Lang >> --- >> README | 4 +- >> configure.ac | 1 + >> include/builddefs.in | 1 + >> ltp/Makefile | 5 ++ >> ltp/fsstress.c | 139 ++++++++++++++++++++++++++++++++++++++++- >> m4/Makefile | 1 + >> m4/package_liburing.m4 | 4 ++ >> 7 files changed, 152 insertions(+), 3 deletions(-) >> create mode 100644 m4/package_liburing.m4 >> > ... >> diff --git a/ltp/fsstress.c b/ltp/fsstress.c >> index 709fdeec..7a0e278a 100644 >> --- a/ltp/fsstress.c >> +++ b/ltp/fsstress.c > ... >> @@ -2170,6 +2189,108 @@ do_aio_rw(int opno, long r, int flags) >> } >> #endif >> >> +#ifdef URING >> +void >> +do_uring_rw(int opno, long r, int flags) >> +{ >> + char *buf; >> + int e; >> + pathname_t f; >> + int fd; >> + size_t len; >> + int64_t lr; >> + off64_t off; >> + struct stat64 stb; >> + int v; >> + char st[1024]; >> + struct io_uring_sqe *sqe; >> + struct io_uring_cqe *cqe; >> + struct iovec iovec; >> + int iswrite = (flags & (O_WRONLY | O_RDWR)) ? 1 : 0; >> + >> + init_pathname(&f); >> + if (!get_fname(FT_REGFILE, r, &f, NULL, NULL, &v)) { >> + if (v) >> + printf("%d/%d: do_uring_rw - no filename\n", procid, opno); >> + goto uring_out3; >> + } >> + fd = open_path(&f, flags); >> + e = fd < 0 ? errno : 0; >> + check_cwd(); >> + if (fd < 0) { >> + if (v) >> + printf("%d/%d: do_uring_rw - open %s failed %d\n", >> + procid, opno, f.path, e); >> + goto uring_out3; >> + } >> + if (fstat64(fd, &stb) < 0) { >> + if (v) >> + printf("%d/%d: do_uring_rw - fstat64 %s failed %d\n", >> + procid, opno, f.path, errno); >> + goto uring_out2; >> + } >> + inode_info(st, sizeof(st), &stb, v); >> + if (!iswrite && stb.st_size == 0) { >> + if (v) >> + printf("%d/%d: do_uring_rw - %s%s zero size\n", procid, opno, >> + f.path, st); >> + goto uring_out2; >> + } >> + sqe = io_uring_get_sqe(&ring); >> + if (!sqe) { >> + if (v) >> + printf("%d/%d: do_uring_rw - io_uring_get_sqe failed\n", >> + procid, opno); >> + goto uring_out2; >> + } > > I'm not familiar with the io_uring bits, but do we have to do anything > to clean up this sqe object (or the cqe) before we return? The cqe/sqe resources are tied to the ring, so as long as io_uring_queue_exit() is called, then they are freed. And it is after the run, so looks fine to me. -- Jens Axboe