From: Matthew Wilcox <willy@infradead.org>
To: Markus Elfring <Markus.Elfring@web.de>
Cc: "Ekansh Gupta" <ekansh.gupta@oss.qualcomm.com>,
dri-devel@lists.freedesktop.org, iommu@lists.linux.dev,
linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
linaro-mm-sig@lists.linaro.org,
"Christian König" <christian.koenig@amd.com>,
"David Airlie" <airlied@gmail.com>,
"Jörg Rödel" <joro@8bytes.org>,
"Jonathan Corbet" <corbet@lwn.net>,
"Maarten Lankhorst" <maarten.lankhorst@linux.intel.com>,
"Maxime Ripard" <mripard@kernel.org>,
"Oded Gabbay" <ogabbay@kernel.org>,
"Robin Murphy" <robin.murphy@arm.com>,
"Shuah Khan" <skhan@linuxfoundation.org>,
"Simona Vetter" <simona@ffwll.ch>,
"Sumit Semwal" <sumit.semwal@linaro.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
"Bharath Kumar" <quic_bkumar@quicinc.com>,
"Bjorn Andersson" <andersson@kernel.org>,
"Chenna Kesava Raju" <quic_chennak@quicinc.com>,
"Dmitry Baryshkov" <dmitry.baryshkov@oss.qualcomm.com>,
"Konrad Dybcio" <konradybcio@kernel.org>,
"Rob Clark" <robin.clark@oss.qualcomm.com>,
"Srinivas Kandagatla" <srini@kernel.org>,
"Will Deacon" <will@kernel.org>
Subject: Re: [PATCH 09/15] accel/qda: Add DMA-backed GEM objects and memory manager integration
Date: Tue, 19 May 2026 13:28:35 +0100 [thread overview]
Message-ID: <agxXc8ttEzBFOlE2@casper.infradead.org> (raw)
In-Reply-To: <5e0d72fa-929a-4905-9066-6648892bef4a@web.de>
Feel free to ignore everything Markus says.
On Tue, May 19, 2026 at 02:14:34PM +0200, Markus Elfring wrote:
> …
> > Assisted-by: Claude:claude-4-6-sonnet
> …
>
> Did such an information source gather the knowledge to benefit more
> from the application of scope-based resource management?
>
>
> …
> > +++ b/drivers/accel/qda/qda_drv.c
> …
> > @@ -32,6 +33,18 @@ static void qda_postclose(struct drm_device *dev, struct drm_file *file)
> > {
> …
> > + if (refcount_dec_and_test(&iommu_dev->refcount)) {
> > + spin_lock_irqsave(&iommu_dev->lock, flags);
> > + iommu_dev->assigned_pid = 0;
> > + iommu_dev->assigned_file_priv = NULL;
> > + spin_unlock_irqrestore(&iommu_dev->lock, flags);
> > + }
> …
>
> Under which circumstances would you become interested to apply a statement
> like “guard(spinlock_irqsave)(&iommu_dev->lock);”?
> https://elixir.bootlin.com/linux/v7.1-rc4/source/include/linux/spinlock.h#L619-L622
>
> Regards,
> Markus
>
next prev parent reply other threads:[~2026-05-19 12:28 UTC|newest]
Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-05-19 6:15 [PATCH 00/15] accel/qda: Qualcomm DSP Accelerator driver Ekansh Gupta via B4 Relay
2026-05-19 6:15 ` [PATCH 01/15] MAINTAINERS: Add entry for Qualcomm DSP Accelerator (QDA) driver Ekansh Gupta via B4 Relay
2026-05-19 6:15 ` [PATCH 02/15] accel/qda: Add QDA driver documentation Ekansh Gupta via B4 Relay
2026-05-20 14:12 ` Dmitry Baryshkov
2026-05-20 15:47 ` Tomeu Vizoso
2026-06-03 5:22 ` Ekansh Gupta
2026-06-03 8:54 ` Tomeu Vizoso
2026-06-03 5:19 ` Ekansh Gupta
2026-05-19 6:15 ` [PATCH 03/15] accel/qda: Add initial QDA DRM accelerator driver Ekansh Gupta via B4 Relay
2026-05-20 14:18 ` Dmitry Baryshkov
2026-06-03 5:26 ` Ekansh Gupta
2026-05-19 6:15 ` [PATCH 04/15] accel/qda: Add compute bus for QDA context banks Ekansh Gupta via B4 Relay
2026-05-20 14:19 ` Dmitry Baryshkov
2026-06-03 5:28 ` Ekansh Gupta
2026-06-03 13:25 ` Dmitry Baryshkov
2026-05-19 6:15 ` [PATCH 05/15] iommu: Add QDA compute context bank bus to iommu_buses Ekansh Gupta via B4 Relay
2026-05-20 14:19 ` Dmitry Baryshkov
2026-05-19 6:15 ` [PATCH 06/15] accel/qda: Create compute context bank devices on QDA compute bus Ekansh Gupta via B4 Relay
2026-05-20 14:23 ` Dmitry Baryshkov
2026-06-03 5:39 ` Ekansh Gupta
2026-06-03 13:26 ` Dmitry Baryshkov
2026-05-19 6:15 ` [PATCH 07/15] accel/qda: Add memory manager for CB devices Ekansh Gupta via B4 Relay
2026-05-20 14:26 ` Dmitry Baryshkov
2026-06-03 5:44 ` Ekansh Gupta
2026-05-20 14:27 ` Dmitry Baryshkov
2026-06-03 5:46 ` Ekansh Gupta
2026-05-19 6:15 ` [PATCH 08/15] accel/qda: Add QUERY IOCTL and QDA UAPI header Ekansh Gupta via B4 Relay
2026-05-20 14:29 ` Dmitry Baryshkov
2026-06-03 5:51 ` Ekansh Gupta
2026-06-03 13:43 ` Dmitry Baryshkov
2026-05-19 6:15 ` [PATCH 09/15] accel/qda: Add DMA-backed GEM objects and memory manager integration Ekansh Gupta via B4 Relay
2026-05-19 12:14 ` Markus Elfring
2026-05-19 12:28 ` Matthew Wilcox [this message]
2026-05-19 12:32 ` Markus Elfring
2026-05-19 6:16 ` [PATCH 10/15] accel/qda: Add GEM_CREATE and GEM_MMAP_OFFSET IOCTLs Ekansh Gupta via B4 Relay
2026-05-19 6:16 ` [PATCH 11/15] accel/qda: Add PRIME DMA-BUF import support Ekansh Gupta via B4 Relay
2026-05-19 6:55 ` Christian König
2026-06-03 6:11 ` Ekansh Gupta
2026-06-03 13:40 ` Christian König
2026-06-08 5:14 ` Ekansh Gupta
2026-05-19 6:16 ` [PATCH 12/15] accel/qda: Add FastRPC invocation support Ekansh Gupta via B4 Relay
2026-05-20 13:56 ` Dmitry Baryshkov
2026-06-04 5:09 ` Ekansh Gupta
2026-06-07 21:14 ` Dmitry Baryshkov
2026-06-10 9:38 ` Ekansh Gupta
2026-06-12 8:21 ` Dmitry Baryshkov
2026-05-19 6:16 ` [PATCH 13/15] accel/qda: Add DSP process creation and release Ekansh Gupta via B4 Relay
2026-05-20 14:00 ` Dmitry Baryshkov
2026-06-04 5:17 ` Ekansh Gupta
2026-06-07 21:16 ` Dmitry Baryshkov
2026-05-19 6:16 ` [PATCH 14/15] accel/qda: Add remote memory mapping to DSP address space Ekansh Gupta via B4 Relay
2026-05-19 6:16 ` [PATCH 15/15] accel/qda: Add remote memory unmap from " Ekansh Gupta via B4 Relay
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=agxXc8ttEzBFOlE2@casper.infradead.org \
--to=willy@infradead.org \
--cc=Markus.Elfring@web.de \
--cc=airlied@gmail.com \
--cc=andersson@kernel.org \
--cc=christian.koenig@amd.com \
--cc=corbet@lwn.net \
--cc=dmitry.baryshkov@oss.qualcomm.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=ekansh.gupta@oss.qualcomm.com \
--cc=iommu@lists.linux.dev \
--cc=joro@8bytes.org \
--cc=konradybcio@kernel.org \
--cc=linaro-mm-sig@lists.linaro.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=maarten.lankhorst@linux.intel.com \
--cc=mripard@kernel.org \
--cc=ogabbay@kernel.org \
--cc=quic_bkumar@quicinc.com \
--cc=quic_chennak@quicinc.com \
--cc=robin.clark@oss.qualcomm.com \
--cc=robin.murphy@arm.com \
--cc=simona@ffwll.ch \
--cc=skhan@linuxfoundation.org \
--cc=srini@kernel.org \
--cc=sumit.semwal@linaro.org \
--cc=tzimmermann@suse.de \
--cc=will@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox