From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on gnuweeb.org X-Spam-Level: X-Spam-Status: No, score=-0.0 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,NICE_REPLY_A, SPF_HELO_PASS,SPF_SOFTFAIL autolearn=ham autolearn_force=no version=3.4.6 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6064DC433EF for ; Sun, 19 Jun 2022 12:15:58 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S235412AbiFSMP5 (ORCPT ); Sun, 19 Jun 2022 08:15:57 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52700 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234180AbiFSMP5 (ORCPT ); Sun, 19 Jun 2022 08:15:57 -0400 Received: from mail-pl1-x62b.google.com (mail-pl1-x62b.google.com [IPv6:2607:f8b0:4864:20::62b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id E9F28BC2C for ; Sun, 19 Jun 2022 05:15:55 -0700 (PDT) Received: by mail-pl1-x62b.google.com with SMTP id r1so7485746plo.10 for ; Sun, 19 Jun 2022 05:15:55 -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=YgxjGZdQaBeR/oHVwc4ZoktGlSsQsePTQFdkY3TGNFk=; b=3h3+GdZiq9Dn5Rl1yIAyUCB+WgpDyfifjIoktg3XLUajRSOnXZdaeXMv2PZKySpXnD z/k27RJWRrIT0Oc6v+m9VrCOXFn3GLqzog+AVdA/v78Hctn+AbbsB+ZhVB6eWpkVOwim 69snhOVgZpjOa0RAGxC6WfU34f/5bQpu8vSoLXEM8iRc8zVOJhXgxyfA0Lk/+GOPZyGZ 0VtZYWdKempD8qUEi9yX569x61l6H7KgHoOyocESoh2covq/ELTVGZLWYNNChNXCpLWF LJDP1m7Ez3IJIJFwcJA0iWdDZWKIMtnnSCgsuv9ZF4331mgbBaG/EJg2hpWSgu+1CacX RB+A== 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=YgxjGZdQaBeR/oHVwc4ZoktGlSsQsePTQFdkY3TGNFk=; b=vYTYu81NZda2/B+t9InAU8x568af7Lj4mOIl5toB4dYYdEWarF2IjEKI0zk2XEieIb DrOW1wd2SNNbRHEqo5xR/nJoffUWFsAU8VwfyNE0Cd9yHuE2EG0o2okVdxkbkx4Z3qS0 O3gVPLN8sdq7/SGKkR37EcBvma3gD0ogIhMl6tCHEAcn7UCKsFTTsM/ZqA2f/fqlriZh 3bX/Rvk4RIGvlAN72TEVNEtGJM1wRrSKmZE6WRlmbVyklYPbgcdTtTjBJSwxZmOB5yWu o9LSMpd6+xdu9LDEEkTY9zigdtiSRbUMv6prwXSGU99aizK+O+jzCPvH3EoigsZXQVcQ pS5w== X-Gm-Message-State: AJIora8zewum99o0VNMeTEvJknwxt7w+wAHA1CGml8wW4P+BO/LkG6ww Xf14FfFad3ENuEjgCX53KsfC6mwTcmGUKA== X-Google-Smtp-Source: AGRyM1uy7vtZ5i/DkpPuBxPDlP7ssn2CNQxLuM/Xq0K/zRtdxjGDNbIUiXbKavfcY000JYUXdAyNfg== X-Received: by 2002:a17:903:2290:b0:167:59ad:52fb with SMTP id b16-20020a170903229000b0016759ad52fbmr18083526plh.78.1655640955253; Sun, 19 Jun 2022 05:15:55 -0700 (PDT) Received: from [192.168.1.100] ([198.8.77.157]) by smtp.gmail.com with ESMTPSA id y6-20020a056a00180600b0051c147eee7bsm7034135pfa.110.2022.06.19.05.15.54 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sun, 19 Jun 2022 05:15:54 -0700 (PDT) Message-ID: Date: Sun, 19 Jun 2022 06:15:53 -0600 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux aarch64; rv:91.0) Gecko/20100101 Thunderbird/91.10.0 Subject: Re: [PATCH 3/3] io_uring: add sync cancelation API through io_uring_register() Content-Language: en-US To: Ammar Faizi Cc: Pavel Begunkov , Carter Li , io-uring Mailing List References: <20220619020715.1327556-1-axboe@kernel.dk> <20220619020715.1327556-4-axboe@kernel.dk> <786492be-f732-59d0-ccef-e7be6a101002@gnuweeb.org> From: Jens Axboe In-Reply-To: <786492be-f732-59d0-ccef-e7be6a101002@gnuweeb.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: io-uring@vger.kernel.org On 6/19/22 5:16 AM, Ammar Faizi wrote: > Hi Jens, > > On 6/19/22 9:07 AM, Jens Axboe wrote: >> + cd.seq = atomic_inc_return(&ctx->cancel_seq), > > Not sure if it's intentional, but the comma after atomic_inc return() > looks unique. While that's valid syntax, I think you want to use a > semicolon to end it consistently. Oops. Not intentional, that line was copied from the struct init at the top of that function. I'll fix it up, thanks. -- Jens Axboe