hw/virtio: Constify VirtIOFeature

[PATCH 0/3] hw/virtio: Constify VirtIOFeature

Posted by Philippe Mathieu-Daudé 2 years, 11 months ago

Trivial patches to keep VirtIOFeature arrays read-only
(better safe than sorry).

Philippe Mathieu-Daudé (3):
  hw/virtio: Pass virtio_feature_get_config_size() a const argument
  virtio-blk: Constify VirtIOFeature feature_sizes[]
  virtio-net: Constify VirtIOFeature feature_sizes[]

 include/hw/virtio/virtio.h | 2 +-
 hw/block/virtio-blk.c      | 2 +-
 hw/net/virtio-net.c        | 2 +-
 hw/virtio/virtio.c         | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

-- 
2.26.3

Re: [PATCH 0/3] hw/virtio: Constify VirtIOFeature

Posted by Stefano Garzarella 2 years, 11 months ago

On Tue, May 11, 2021 at 12:41:54PM +0200, Philippe Mathieu-Daudé wrote:
>Trivial patches to keep VirtIOFeature arrays read-only
>(better safe than sorry).
>
>Philippe Mathieu-Daudé (3):
>  hw/virtio: Pass virtio_feature_get_config_size() a const argument
>  virtio-blk: Constify VirtIOFeature feature_sizes[]
>  virtio-net: Constify VirtIOFeature feature_sizes[]
>
> include/hw/virtio/virtio.h | 2 +-
> hw/block/virtio-blk.c      | 2 +-
> hw/net/virtio-net.c        | 2 +-
> hw/virtio/virtio.c         | 2 +-
> 4 files changed, 4 insertions(+), 4 deletions(-)
>

Reviewed-by: Stefano Garzarella <sgarzare@redhat.com>

Re: [PATCH 0/3] hw/virtio: Constify VirtIOFeature

Posted by Stefan Hajnoczi 2 years, 11 months ago

On Tue, May 11, 2021 at 12:41:54PM +0200, Philippe Mathieu-Daudé wrote:
> Trivial patches to keep VirtIOFeature arrays read-only
> (better safe than sorry).
> 
> Philippe Mathieu-Daudé (3):
>   hw/virtio: Pass virtio_feature_get_config_size() a const argument
>   virtio-blk: Constify VirtIOFeature feature_sizes[]
>   virtio-net: Constify VirtIOFeature feature_sizes[]
> 
>  include/hw/virtio/virtio.h | 2 +-
>  hw/block/virtio-blk.c      | 2 +-
>  hw/net/virtio-net.c        | 2 +-
>  hw/virtio/virtio.c         | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> -- 
> 2.26.3
> 
> 

Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>

Re: [PATCH 0/3] hw/virtio: Constify VirtIOFeature

Posted by Laurent Vivier 2 years, 11 months ago

Le 11/05/2021 à 12:41, Philippe Mathieu-Daudé a écrit :
> Trivial patches to keep VirtIOFeature arrays read-only
> (better safe than sorry).
> 
> Philippe Mathieu-Daudé (3):
>   hw/virtio: Pass virtio_feature_get_config_size() a const argument
>   virtio-blk: Constify VirtIOFeature feature_sizes[]
>   virtio-net: Constify VirtIOFeature feature_sizes[]
> 
>  include/hw/virtio/virtio.h | 2 +-
>  hw/block/virtio-blk.c      | 2 +-
>  hw/net/virtio-net.c        | 2 +-
>  hw/virtio/virtio.c         | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 

Series applied to my trivial-patches branch.

Thanks,
Laurent

Re: [PATCH 0/3] hw/virtio: Constify VirtIOFeature

Posted by Richard Henderson 2 years, 11 months ago

On 5/11/21 5:41 AM, Philippe Mathieu-Daudé wrote:
> Philippe Mathieu-Daudé (3):
>    hw/virtio: Pass virtio_feature_get_config_size() a const argument
>    virtio-blk: Constify VirtIOFeature feature_sizes[]
>    virtio-net: Constify VirtIOFeature feature_sizes[]

Reviewed-by: Richard Henderson <richard.henderson@linaro.org>

r~