On 12/12/2021 3:25 PM, Rikard Falkeborn worte: > 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 Hi Rikard, no worries then. I have already corrected them and queue them in the gvt-next branch, you can double check them if you like. They are going through a QA test cycle first and later start their journey to the upstream. All good. :) Thanks so much for the contribution again. Zhi.
