Re: [PATCH v2] drm/i915/guc: Add synchronization on interrupt enable flag

2025-08-25 Thread Dong, Zhanjun
On 2025-08-25 6:00 a.m., Andi Shyti wrote: Hi Zhanjun, diff --git a/drivers/gpu/drm/i915/gt/intel_gt_irq.c b/drivers/gpu/drm/i915/gt/intel_gt_irq.c index 75e802e10be2..21804eec8320 100644 --- a/drivers/gpu/drm/i915/gt/intel_gt_irq.c +++ b/drivers/gpu/drm/i915/gt/intel_gt_irq.c @@ -20,7 +20,7 @

Re: [PATCH v2] drm/i915/guc: Add synchronization on interrupt enable flag

2025-08-25 Thread Andi Shyti
Hi Zhanjun, > diff --git a/drivers/gpu/drm/i915/gt/intel_gt_irq.c > b/drivers/gpu/drm/i915/gt/intel_gt_irq.c > index 75e802e10be2..21804eec8320 100644 > --- a/drivers/gpu/drm/i915/gt/intel_gt_irq.c > +++ b/drivers/gpu/drm/i915/gt/intel_gt_irq.c > @@ -20,7 +20,7 @@ > > static void guc_irq_handl

[PATCH v2] drm/i915/guc: Add synchronization on interrupt enable flag

2025-08-22 Thread Zhanjun Dong
Boolean flag access from interrupt context might have synchronous issueis on multiple processor platform, flags modified by one core might be read as an old value by another core. This issue on interrupt enable flag might causes interrupt misses or leakage. Change the interrupts.enable type to atom