From: Sebastian Andrzej Siewior <bigeasy@linutronix.de>
To: Maarten Lankhorst <dev@lankhorst.se>
Cc: intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org
Subject: Re: [PATCH v4 0/7] drm/i915: All fixes to make i915 work well with PREEMPT_RT.
Date: Thu, 9 Jul 2026 13:02:00 +0200 [thread overview]
Message-ID: <20260709110200.2_5Eg8xF@linutronix.de> (raw)
In-Reply-To: <c4222f81-72b2-4eea-a900-da327f0df928@lankhorst.se>
On 2026-07-09 11:53:52 [+0200], Maarten Lankhorst wrote:
> Hello Sebastian,
Hi Maarten,
> The test results are available, mostly looks good except 1 spurious incomplete.
>
> I tried reproducing it, but was unable to. It seems that pre-existing incompletes
> on the specific IGT testcase already happened before the changes.
>
> Do you want to review the patches you didn't author, so I can move forward?
> Even if you are uncomfortable to review the signalers_lock changes, it should
> be possible to have the rest reviewed and merged upstream.
This looks good to me. The condition in 5/7 is the as in 3/7. Not sure
if this should be merged into a helper or if it should be passed from
the caller. But it seems the information of the calling context is lost.
Please move forward ;)
> Kind regards,
> ~Maarten Lankhorst
Sebastian
next prev parent reply other threads:[~2026-07-09 11:02 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-06 11:48 [PATCH v4 0/7] drm/i915: All fixes to make i915 work well with PREEMPT_RT Maarten Lankhorst
2026-07-06 11:48 ` [PATCH v4 1/7] drm/i915/gt: Use spin_lock_irq() instead of local_irq_disable() + spin_lock() Maarten Lankhorst
2026-07-06 12:03 ` sashiko-bot
2026-07-06 11:48 ` [PATCH v4 2/7] drm/i915: Drop the irqs_disabled() check Maarten Lankhorst
2026-07-06 11:48 ` [PATCH v4 3/7] drm/i915/guc: Consider also RCU depth in busy loop Maarten Lankhorst
2026-07-06 12:02 ` sashiko-bot
2026-07-06 11:48 ` [PATCH v4 4/7] drm/i915/gt: Fix selftests on PREEMPT_RT Maarten Lankhorst
2026-07-06 11:56 ` sashiko-bot
2026-07-09 10:36 ` Sebastian Andrzej Siewior
2026-07-09 11:03 ` Maarten Lankhorst
2026-07-09 12:23 ` Sebastian Andrzej Siewior
2026-07-06 11:48 ` [PATCH v4 5/7] drm/i915/gt: Set stop_timeout() correctly on PREEMPT-RT Maarten Lankhorst
2026-07-06 11:48 ` [PATCH v4 6/7] drm/i915: Use sleeping selftests for igt_atomic on PREEMPT_RT Maarten Lankhorst
2026-07-06 11:48 ` [PATCH v4 7/7] drm/i915/gt: Use signalers_lock to prevent starvation of irq_work Maarten Lankhorst
2026-07-06 12:04 ` sashiko-bot
2026-07-09 10:57 ` Sebastian Andrzej Siewior
2026-07-09 11:16 ` Maarten Lankhorst
2026-07-09 11:58 ` Sebastian Andrzej Siewior
2026-08-21 2:25 ` Shuangpeng Bai
2026-08-21 11:45 ` Maarten Lankhorst
2026-07-06 12:20 ` [PATCH v4 0/7] drm/i915: All fixes to make i915 work well with PREEMPT_RT Sebastian Andrzej Siewior
2026-07-06 12:51 ` Maarten Lankhorst
2026-07-09 9:53 ` Maarten Lankhorst
2026-07-09 10:01 ` Sebastian Andrzej Siewior
2026-07-09 11:02 ` Sebastian Andrzej Siewior [this message]
2026-07-09 11:36 ` Maarten Lankhorst
2026-07-06 14:49 ` ✗ i915.CI.BAT: failure for drm/i915: All fixes to make i915 work well with PREEMPT_RT. (rev4) Patchwork
2026-07-07 2:39 ` ✓ i915.CI.BAT: success for drm/i915: All fixes to make i915 work well with PREEMPT_RT. (rev5) Patchwork
2026-07-07 9:58 ` ✗ i915.CI.Full: failure " Patchwork
2026-07-08 15:54 ` Maarten Lankhorst
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260709110200.2_5Eg8xF@linutronix.de \
--to=bigeasy@linutronix.de \
--cc=dev@lankhorst.se \
--cc=dri-devel@lists.freedesktop.org \
--cc=intel-gfx@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.