On Fri, Dec 10, 2021 at 09:00:56AM +0000, Wang, Zhi A wrote: > 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. > Thanks for reviewing. Just to clarify, did you mean patch 7 (since that's the one you commented on)? And is it enough to send just that patch or do you want the entire series resent? Rikard
