On 2025/11/21 14:21, Honglei Huang wrote:
On 2025/11/21 11:39, Akihiko Odaki wrote:
On 2025/11/21 12:14, Honglei Huang wrote:
On 2025/11/21 10:56, Akihiko Odaki wrote:
On 2025/11/21 11:47, Honglei Huang wrote:
Add support for the USE_USERPTR blob flag in virtio-gpu to enable
user pointer mapping for blob resources. This allows guest
applications
to use user-allocated memory for GPU resources more efficiently.
Changes include:
- Add VIRTIO_GPU_BLOB_FLAG_USE_USERPTR flag definition
- Enhance blob resource creation to handle userptr flag properly
- Remove arbitrary nr_entries limit (16384) in mapping creation
- Add conditional handling for userptr vs regular blob mapping
I don't see the added conditional handling.
Sorry, the additional handing is replaced by the fixing of value check.
I will correct this commit message in the next version.
Not just the commit message, but it also questions the utility of
VIRTIO_GPU_BLOB_FLAG_USE_USERPTR and VIRTIO_GPU_F_RESOURCE_USERPTR.
Neither of them adds a new functionality. They should be dropped if
they are also replaced with the fix.
Yes totally agreed, it is my mistaken, I shouldn't mix the code for
fixing and the code for adding new features in one submission.
Actually this patch set are for another components upstream test, for
the sake of convenience, I have added both the fix and feature here,
that is a bad idea.
Will split the fix part into previous thread.
And for the check value fix thread, will send v4 as the final version.
Splitting fixes and features is a good idea, but that's not what I meant.
What I pointed out is that, it seems that one of the "features" you are
adding, namely VIRTIO_GPU_F_RESOURCE_USERPTR does nothing at at all. So
I'm wondering if you forgot to add a real implementation or the feature
is just no longer necessary.
