Re: [PATCH] vdpa/pds: check virtqueue notify mapping

Xiong Weimin <[email protected]>
Newsgroups gmane.linux.kernel.virtualization,gmane.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
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.