mbox series

[v2,0/2] virtio-gpu: coverity fixes

Message ID 20241111230040.68470-1-alex.bennee@linaro.org
Headers show
Series virtio-gpu: coverity fixes | expand

Message

Alex Bennée Nov. 11, 2024, 11 p.m. UTC 
v2,

Fixes after Dimitry's review.

Alex.

Alex Bennée (2):
  hw/display: factor out the scanout blob to fb conversion
  hw/display: check frame buffer can hold blob

 include/hw/virtio/virtio-gpu.h | 15 +++++++++
 hw/display/virtio-gpu-virgl.c  | 22 +------------
 hw/display/virtio-gpu.c        | 59 +++++++++++++++++++++-------------
 3 files changed, 52 insertions(+), 44 deletions(-)

Comments

Michael S. Tsirkin Nov. 14, 2024, 1:52 p.m. UTC | #1 
On Mon, Nov 11, 2024 at 11:00:38PM +0000, Alex Bennée wrote:
> v2,
> 
> Fixes after Dimitry's review.


You should CC Gerd.

> Alex.
> 
> Alex Bennée (2):
>   hw/display: factor out the scanout blob to fb conversion
>   hw/display: check frame buffer can hold blob
> 
>  include/hw/virtio/virtio-gpu.h | 15 +++++++++
>  hw/display/virtio-gpu-virgl.c  | 22 +------------
>  hw/display/virtio-gpu.c        | 59 +++++++++++++++++++++-------------
>  3 files changed, 52 insertions(+), 44 deletions(-)
> 
> -- 
> 2.39.5
Alex Bennée Nov. 18, 2024, 11:01 a.m. UTC | #2 
"Michael S. Tsirkin" <mst@redhat.com> writes:

> On Mon, Nov 11, 2024 at 11:00:38PM +0000, Alex Bennée wrote:
>> v2,
>> 
>> Fixes after Dimitry's review.
>
>
> You should CC Gerd.

Since 2f8cd5a9b6 (MAINTAINERS: drop virtio-gpu maintainership)
virtio-gpu is currently orphaned for maintainers. I've been reaching out
to various teams with GPU experience (including Dimitry) to see if we
can find a replacement maintainer. For now I'm happy to test and
upstream odd fixes.

>
>> Alex.
>> 
>> Alex Bennée (2):
>>   hw/display: factor out the scanout blob to fb conversion
>>   hw/display: check frame buffer can hold blob
>> 
>>  include/hw/virtio/virtio-gpu.h | 15 +++++++++
>>  hw/display/virtio-gpu-virgl.c  | 22 +------------
>>  hw/display/virtio-gpu.c        | 59 +++++++++++++++++++++-------------
>>  3 files changed, 52 insertions(+), 44 deletions(-)
>> 
>> -- 
>> 2.39.5