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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1E5ACC433EF for ; Sat, 30 Apr 2022 13:27:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1382733AbiD3Naj (ORCPT ); Sat, 30 Apr 2022 09:30:39 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:45880 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1382716AbiD3Nah (ORCPT ); Sat, 30 Apr 2022 09:30:37 -0400 Received: from mail-pj1-x1030.google.com (mail-pj1-x1030.google.com [IPv6:2607:f8b0:4864:20::1030]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 89A8935DC4 for ; Sat, 30 Apr 2022 06:27:15 -0700 (PDT) Received: by mail-pj1-x1030.google.com with SMTP id p6so9295374pjm.1 for ; Sat, 30 Apr 2022 06:27:15 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel-dk.20210112.gappssmtp.com; s=20210112; h=message-id:date:mime-version:user-agent:subject:content-language:to :cc:references:from:in-reply-to:content-transfer-encoding; bh=oCzWgWhhG7IWlDHkgweCLD8r25SvqJ41BjsLlfD+agc=; b=WLndbh79Pv5SNg8DK8nNF3XTxXb4916n15htOIRK+dLom00X4DHQK8aEbj1TwjiyGy uJqnz7des0BNwWrGA5r3wSLRJTg4PDz/bSMDEeTVcMc3nAzg7cjPtxPT6n7ELnkM5ZLC jU1tFwFkLOSNg+Ddk8RyBZ74EE5MmLS72gnghffxVEilAsLc8i9RmbB0Ih01mHBSlJA1 bF0SH75cbBJHrw27r1g9xWPDrEn/tvM3siGr+G4k0ErWtNrZ1cYp3Q5r1L3BYMlveYV4 GVwMNcPkd2ySCT8YT8FRH5beDo1i1r9xY05VF4MSGGE2NKa5Ol8QRMhbfoiygPb5ukK6 fUwg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=oCzWgWhhG7IWlDHkgweCLD8r25SvqJ41BjsLlfD+agc=; b=LZYyh/b6BuUkI+9zSSOJ9lFfsmi/cihlRk5LqDt3c92HJZpr+llBqsB4m47m67rtil 36hV0X33WhpfqufLTXarlIALiiQMYDrhFUcyY1/q1T/GGX13dLn9nLoF1+JzZzVW/cKO aafek44BQ0dt4VdWqNeXnkJthiGmU5dj7gWoox4NGvItYl7HFuFMBdXYMOEaR+wlanS6 oNe3xPj8fCb5FAdYCmBhGm3u+HENTMbkMYqx8irRWRk7Vd5s5DSiI39Ut//8iM+k9fCV 3q2I/NDR/KhNFbzyVNxWdECFqhs4f+DUfFte4I6a/8Po9KsjF/RUWAu5gNpQvLqPSRpm ULWQ== X-Gm-Message-State: AOAM533izm2iPJ4IvzPrYxCgPj0CrfnE8I4ZCnuqKAbYhKggKH3YlWkL Mhyg52jUdMeWNN97muaxzbxnMw== X-Google-Smtp-Source: ABdhPJyoZXoUeWNbVxt926XEaxMdZ9eiVvwyjxDOMovIluronLbzZA9Kp3RQvuXCK5E6r0FoMJ8UlA== X-Received: by 2002:a17:902:9005:b0:156:8a9d:ba49 with SMTP id a5-20020a170902900500b001568a9dba49mr3922744plp.42.1651325234978; Sat, 30 Apr 2022 06:27:14 -0700 (PDT) Received: from [192.168.1.100] ([198.8.77.157]) by smtp.gmail.com with ESMTPSA id v12-20020a17090331cc00b0015e8d4eb247sm1439220ple.145.2022.04.30.06.27.13 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 30 Apr 2022 06:27:14 -0700 (PDT) Message-ID: <015f58ed-09c1-cd27-064a-b6c0cc5580d2@kernel.dk> Date: Sat, 30 Apr 2022 07:27:13 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH 7/9] io-wq: implement fixed worker logic Content-Language: en-US To: Hao Xu , io-uring@vger.kernel.org Cc: Pavel Begunkov , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220429101858.90282-1-haoxu.linux@gmail.com> <20220429101858.90282-8-haoxu.linux@gmail.com> From: Jens Axboe In-Reply-To: <20220429101858.90282-8-haoxu.linux@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 4/29/22 4:18 AM, Hao Xu wrote: > @@ -1030,6 +1101,7 @@ static bool io_wq_work_match_item(struct io_wq_work *work, void *data) > static void io_wqe_enqueue(struct io_wqe *wqe, struct io_wq_work *work) > { > struct io_wqe_acct *acct = io_work_get_acct(wqe, work); > + struct io_wqe_acct *fixed_acct; > struct io_cb_cancel_data match; > unsigned work_flags = work->flags; > bool do_create; > @@ -1044,8 +1116,14 @@ static void io_wqe_enqueue(struct io_wqe *wqe, struct io_wq_work *work) > return; > } > > + fixed_acct = io_get_acct(wqe, !acct->index, true); > + if (fixed_acct->fixed_worker_registered && !io_wq_is_hashed(work)) { > + if (io_wqe_insert_private_work(wqe, work, fixed_acct)) > + return; > + } > + As per previous email, I was going to comment back saying "why don't we just always do hashed work on the non-fixed workers?" - but that's already what you are doing. Isn't this fine, does anything else need to get done here in terms of hashed work and fixed workers? If you need per-iowq serialization, then you don't get a fixed worker. -- Jens Axboe