Re: [PATCH v5 09/16] nvme-pci: implement dma-buf backed requests
Pavel Begunkov <[email protected]> Tue, 4 Aug 2026 09:41:47 +0100
| Newsgroups | dev.linux.lists.dm-devel,dev.linux.lists.nvdimm,org.freedesktop.lists.dri-devel,org.infradead.lists.linux-nvme,org.kernel.vger.ceph-devel,org.kernel.vger.io-uring,org.kernel.vger.linux-block,org.kernel.vger.linux-btrfs,org.kernel.vger.linux-fsdevel,org.kernel.vger.linux-kernel,org.kernel.vger.linux-media |
|---|---|
| Message-ID | <[email protected]> |
On 8/4/26 08:29, Anuj Gupta/Anuj Gupta wrote: > On 8/1/2026 9:16 PM, Pavel Begunkov wrote: ...>> + if (WARN_ON_ONCE(seg_shift < NVME_CTRL_PAGE_SHIFT)) >> + return ERR_PTR(-EFAULT); > > This should be 'goto err;' instead of a direct return. If this ever > triggers, it leaks the map and already established sgt mapping. I can change it as I'm going to respin the series anyway, but it doesn't really matter here as it's a warning and shouldn't happen. -- Pavel Begunkov