From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f73.google.com (mail-wm1-f73.google.com [209.85.128.73]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id EE8F3329C71 for ; Tue, 10 Feb 2026 13:26:50 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.73 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770730012; cv=none; b=Ya8h1I8OHqKeUAhcs9SQPv8ZMJMuulWKOnZvL4idrqSBYC0+fPOav0Uw4ehwjIQ3CSi8ajAYkaQWzN2TthR+Ub4tnpiGaXOJaCAFsIWuB1kAqWZAbLlOn2RPUVWNP3ZvX/NtR0a+QHGFH6TlkwRtXdaV6A+j7IVTRXxprg91fic= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1770730012; c=relaxed/simple; bh=vR75fwMbIvJ+FBIzC5dmd0aaO1PZ2nI36YQLTxKU3VM=; h=Date:In-Reply-To:Mime-Version:References:Message-ID:Subject:From: To:Cc:Content-Type; b=gBnGcLu30ONyOWnlaV6pXjFUp1OdzLFRZSFcS7KozBjscCpZKuWHKL+iYNe0meWJrhwFDjgmKS8JyIAR/loAjrkbe5ov6xb0BHhKo1Y4+HIb/XXn1n+o9IcD4occG3JFcahn3nFs42lq+7XEpD5va/7V1X7stXQYRk3uqZCl2mc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=4Yy7LRvb; arc=none smtp.client-ip=209.85.128.73 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--aliceryhl.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="4Yy7LRvb" Received: by mail-wm1-f73.google.com with SMTP id 5b1f17b1804b1-48336b182ebso15128725e9.3 for ; Tue, 10 Feb 2026 05:26:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1770730009; x=1771334809; darn=vger.kernel.org; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:from:to:cc:subject:date:message-id:reply-to; bh=FdusrkF1rdhhHjrXLuahuHjKwfdA8TDoSd3qb4xa01Y=; b=4Yy7LRvbbpOdQaRlYaxRvbMbpNgwX6X3KSsieZadL+91xmUV5qPfwukfI+Cn29xHVX 3XQY7i72cyvTpor4iArUN0VmDD7/dDDQUTfJ7tDwtMDSkvH2iah6aqlzRu4+Xewq5ShJ f7oqiNuPAW8hU97aCzGn+JVMCoLD8ohVjT1PNlEvvzNXA+13WB5C7NGkF3ZheJ8MgogB lqMlo5SmTEMCU/qTKmQEIz2pzIjIsVmVAsUUH5EeJIUJ72oWf84eAw0cvOKZrEx5PVd+ L5fK8I0kzc6FcFbgB41+68A9Fpyx9EkXG4W0mrXUZLeHUGxcy76yyoIGIEPY1SHHwWXI 8xBw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1770730009; x=1771334809; h=cc:to:from:subject:message-id:references:mime-version:in-reply-to :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=FdusrkF1rdhhHjrXLuahuHjKwfdA8TDoSd3qb4xa01Y=; b=mVsopzVmK96mzxDcA6Yii7+k/ZQFV1oihpGf/v5Gnr+a6QSMseYE/fW4A5jM4ruIG4 DMhyx+oZupmYL6ihKAbpDuIWdboDG9s4KN8AxbTKzTeChUAqY4Dgh5q1zoGDt8jtEW2g o68Ais87TbuNZnDteshesZXKWolMmelh/iGzecXfbrkcx9ZOWyApGmjAMVLe5m8RvOv2 5jqmm9WfbWgHonbb7V/a0MOgfSyLIyMDr5mKB+H8f8Sw0MNuL6rIgNWVsQLclPFGj0Bq seWAYACooI/qWKPcHYoORWQ7JO/ZcUTo/ngSAEKpjWgw/0AlLn9mMchRW5bTpBPmdeXe jtNQ== X-Forwarded-Encrypted: i=1; AJvYcCVBPQJu0kD0JNXz+UZExyq4EtmWfC9hSEHTkvCgtuhxrAApiSFb4zuMGhiUh+svaR4FBzgnGcuHVgIdwns=@vger.kernel.org X-Gm-Message-State: AOJu0YzsA8wHThqUFuyI+nam52A0db7V8QmHNvJbUDNz9mybjlLWT8p0 gtCNnpeQG8jog26OrpgjNM50Xz/oykWrUaYmax1S9/05VuHJVV4AdYDaKbCAhxBts0ThUKIYGmE 6bf+d6a+QF32dhqwPjA== X-Received: from wmjf25.prod.google.com ([2002:a7b:cd19:0:b0:480:4a03:7b5d]) (user=aliceryhl job=prod-delivery.src-stubby-dispatcher) by 2002:a05:600c:8b8a:b0:480:1dc6:269c with SMTP id 5b1f17b1804b1-48320231241mr195023655e9.37.1770730009431; Tue, 10 Feb 2026 05:26:49 -0800 (PST) Date: Tue, 10 Feb 2026 13:26:48 +0000 In-Reply-To: <20260210134913.33cb674f@fedora> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 References: <20260203081403.68733-4-phasta@kernel.org> <20260205095727.4c3e2941@fedora> <20260209155843.725dcfe1@fedora> <20260210101525.7fb85f25@fedora> <20260210134913.33cb674f@fedora> Message-ID: Subject: Re: [RFC PATCH 2/4] rust: sync: Add dma_fence abstractions From: Alice Ryhl To: Boris Brezillon Cc: "Christian =?utf-8?B?S8O2bmln?=" , Philipp Stanner , phasta@kernel.org, Danilo Krummrich , David Airlie , Simona Vetter , Gary Guo , Benno Lossin , Daniel Almeida , Joel Fernandes , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, rust-for-linux@vger.kernel.org Content-Type: text/plain; charset="utf-8" On Tue, Feb 10, 2026 at 01:49:13PM +0100, Boris Brezillon wrote: > On Tue, 10 Feb 2026 10:15:04 +0000 > Alice Ryhl wrote: > > > /// The owner of this value must ensure that this fence is signalled. > > struct MustBeSignalled<'fence> { ... } > > /// Proof value indicating that the fence has either already been > > /// signalled, or it will be. The lifetime ensures that you cannot mix > > /// up the proof value. > > struct WillBeSignalled<'fence> { ... } > > Sorry, I have more questions, unfortunately. Seems that > {Must,Will}BeSignalled are targeting specific fences (at least that's > what the doc and 'fence lifetime says), but in practice, the WorkItem > backing the scheduler can queue 0-N jobs (0 if no jobs have their deps > met, and N > 1 if more than one job is ready). Similarly, an IRQ > handler can signal 0-N fences (can be that the IRQ has nothing to do we > job completion, or, it can be that multiple jobs have completed). How > is this MustBeSignalled object going to be instantiated in practice if > it's done before the DmaFenceWorkItem::run() function is called? The {Must,Will}BeSignalled closure pair needs to wrap the piece of code that ensures a specific fence is signalled. If you have code that manages a collection of fences and invokes code for specific fences depending on outside conditions, then that's a different matter. After all, transfer_to_wq() has two components: 1. Logic to ensure any spawned workqueue job eventually gets to run. 2. Once the individual job runs, logic specific to the one fence ensures that this one fence gets signalled. And {Must,Will}BeSignalled exists to help model part (2.). But what you described with the IRQ callback falls into (1.) instead, which is outside the scope of {Must,Will}BeSignalled (or at least requires more complex APIs). Alice