Re: [PATCH] vdpa/pds: check virtqueue notify mapping
Xiong Weimin <[email protected]> Thu, 06 Aug 2026 08:58:06 +0800
| Newsgroups | dev.linux.lists.virtualization,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
On Tue, Aug 04, 2026, Brett Creeley wrote: > Nit, but why dev_warn() instead of dev_err()? It looks like this was > copied, but I wonder if the other spot should be a dev_err() instead of > a dev_warn too? > > Other than that, LGTM. > > Reviewed-by: Brett Creeley <[email protected]> Thanks for the review. Agreed — this is a hard failure on the device add path, and the rest of pds_vdpa_dev_add() already uses dev_err(). I'll send a standalone [PATCH v2] with that change and your Reviewed-by. The matching "Fail to map vq notify" in vp_vdpa.c is pre-existing; I can clean that up in a separate patch if you prefer. Thanks, Xiong