Re: [PATCH 0/9] drm/i915/gvt: Constify static structs

[Wang, Zhi A]

On 12/4/2021 12:55 PM, Rikard Falkeborn wrote:
> Constify a number of static structs that are never modified to allow the
> compiler to put them in read-only memory. In order to do this, constify a
> number of local variables and pointers in structs.
>
> This is most important for structs that contain function pointers, and
> the patches for those structs are placed first in the series.
>
> Rikard Falkeborn (9):
>    drm/i915/gvt: Constify intel_gvt_gtt_pte_ops
>    drm/i915/gvt: Constify intel_gvt_gtt_pte_ops
>    drm/i915/gvt: Constify intel_gvt_irq_ops
>    drm/i915/gvt: Constify intel_gvt_sched_policy_ops
>    drm/i915/gvt: Constify gvt_mmio_block
>    drm/i915/gvt: Constify cmd_interrupt_events
>    drm/i915/gvt: Constify formats
>    drm/i915/gvt: Constify gtt_type_table_entry
>    drm/i915/gvt: Constify vgpu_types
>
>   drivers/gpu/drm/i915/gvt/cmd_parser.c   |  2 +-
>   drivers/gpu/drm/i915/gvt/fb_decoder.c   | 24 ++++-----
>   drivers/gpu/drm/i915/gvt/gtt.c          | 68 ++++++++++++-------------
>   drivers/gpu/drm/i915/gvt/gtt.h          |  4 +-
>   drivers/gpu/drm/i915/gvt/gvt.h          |  2 +-
>   drivers/gpu/drm/i915/gvt/handlers.c     | 12 ++---
>   drivers/gpu/drm/i915/gvt/interrupt.c    | 10 ++--
>   drivers/gpu/drm/i915/gvt/interrupt.h    |  2 +-
>   drivers/gpu/drm/i915/gvt/sched_policy.c |  2 +-
>   drivers/gpu/drm/i915/gvt/scheduler.h    |  2 +-
>   drivers/gpu/drm/i915/gvt/vgpu.c         |  4 +-
>   11 files changed, 66 insertions(+), 66 deletions(-)
>
Thanks so much for the contribuition. You only need to refine the PATCH 
2 a little bit and re-send it.