Free hw_crtc_state in verify_crtc_state after we are done using
this or else it's just a resource leak.
Fixes: 2745bdda2095 ("drm/i915: Stop clobbering old crtc state during state
check")
Signed-off-by: Suraj Kandpal
Reviewed-by: Ville Syrjälä
---
== Series Details ==
Series: More print message helper updates (rev2)
URL : https://patchwork.freedesktop.org/series/124853/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13732 -> Patchwork_124853v2
Summary
---
== Series Details ==
Series: More print message helper updates (rev2)
URL : https://patchwork.freedesktop.org/series/124853/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: More print message helper updates (rev2)
URL : https://patchwork.freedesktop.org/series/124853/
State : warning
== Summary ==
Error: dim checkpatch failed
e57654f9ec79 drm/i915/gt: More use of GT specific print helpers
-:163: CHECK:MACRO_ARG_REUSE: Macro argument
== Series Details ==
Series: drm/i915: Add bigjoiner force enable option to debugfs (rev2)
URL : https://patchwork.freedesktop.org/series/124730/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13731_full -> Patchwork_124730v2_full
== Series Details ==
Series: drm/i915/gt: Temporarily force MTL into uncached mode
URL : https://patchwork.freedesktop.org/series/124866/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13732 -> Patchwork_124866v1
Summary
> -Original Message-
> From: Ville Syrjälä
> Sent: Monday, October 9, 2023 6:08 PM
> To: Kandpal, Suraj
> Cc: Jani Nikula ; intel-gfx@lists.freedesktop.org
> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Free crtc_state in
> verify_crtc_state
>
> On Mon, Oct 09, 2023 at
== Series Details ==
Series: series starting with [1/4] drm/i915/dsb: Allocate command buffer from
local memory
URL : https://patchwork.freedesktop.org/series/124818/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13730_full -> Patchwork_124818v1_full
== Series Details ==
Series: More print message helper updates
URL : https://patchwork.freedesktop.org/series/124853/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13732 -> Patchwork_124853v1
Summary
---
**FAILURE**
== Series Details ==
Series: More print message helper updates
URL : https://patchwork.freedesktop.org/series/124853/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./arch/x86/include/asm/bitops.h:117:1:
== Series Details ==
Series: More print message helper updates
URL : https://patchwork.freedesktop.org/series/124853/
State : warning
== Summary ==
Error: dim checkpatch failed
9ac3d314aeb4 drm/i915/gt: More use of GT specific print helpers
-:163: CHECK:MACRO_ARG_REUSE: Macro argument reuse
== Series Details ==
Series: drm/i915/display: Free crtc_state in verify_crtc_state
URL : https://patchwork.freedesktop.org/series/124811/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13730_full -> Patchwork_124811v1_full
== Series Details ==
Series: drm/bridge_connector: implement OOB HPD handling (rev5)
URL : https://patchwork.freedesktop.org/series/120395/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13731 -> Patchwork_120395v5
Summary
== Series Details ==
Series: drm/bridge_connector: implement OOB HPD handling (rev5)
URL : https://patchwork.freedesktop.org/series/120395/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915: Define and use GuC and CTB TLB invalidation routines (rev4)
URL : https://patchwork.freedesktop.org/series/124744/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13731 -> Patchwork_124744v4
FIXME: CAT errors are cropping up on MTL. This removes them,
but the real root cause must still be diagnosed.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/i915/gt/intel_gt.c | 6 +-
drivers/gpu/drm/i915/gt/intel_lrc.c| 5 -
drivers/gpu/drm/i915/gt/uc/intel_guc.c | 5 -
== Series Details ==
Series: drm/i915: Define and use GuC and CTB TLB invalidation routines (rev4)
URL : https://patchwork.freedesktop.org/series/124744/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
== Series Details ==
Series: drm/i915: Define and use GuC and CTB TLB invalidation routines (rev4)
URL : https://patchwork.freedesktop.org/series/124744/
State : warning
== Summary ==
Error: dim checkpatch failed
36647bbe0ce7 drm/i915: Add GuC TLB Invalidation device info flags
07632c06461f
== Series Details ==
Series: drm/i915: Fix VLV color state readout
URL : https://patchwork.freedesktop.org/series/124824/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13731 -> Patchwork_124824v1
Summary
---
== Series Details ==
Series: drm/i915/lnl: Remove watchdog timers for PSR (rev2)
URL : https://patchwork.freedesktop.org/series/124715/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13731 -> Patchwork_124715v2
Summary
== Series Details ==
Series: drm/i915: Add bigjoiner force enable option to debugfs (rev2)
URL : https://patchwork.freedesktop.org/series/124730/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13731 -> Patchwork_124730v2
== Series Details ==
Series: series starting with [1/4] drm/i915/dsb: Allocate command buffer from
local memory
URL : https://patchwork.freedesktop.org/series/124818/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13730 -> Patchwork_124818v1
== Series Details ==
Series: series starting with [1/4] drm/i915/dsb: Allocate command buffer from
local memory
URL : https://patchwork.freedesktop.org/series/124818/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked
== Series Details ==
Series: drm/i915/display: Free crtc_state in verify_crtc_state
URL : https://patchwork.freedesktop.org/series/124811/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13730 -> Patchwork_124811v1
Summary
On Fri, Oct 06, 2023 at 04:37:17PM +0300, Imre Deak wrote:
> Use the connector's DSC DPCD capabilities in
> intel_dp_dsc_max_sink_compressed_bppx16().
>
> Signed-off-by: Imre Deak
Reviewed-by: Stanislav Lisovskiy
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 19 +++
> 1
On Fri, Oct 06, 2023 at 04:37:16PM +0300, Imre Deak wrote:
> Use the connector's DSC DPCD capabilities in intel_dp_supports_dsc().
>
> Signed-off-by: Imre Deak
Reviewed-by: Stanislav Lisovskiy
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 6 +++---
> 1 file changed, 3 insertions(+), 3
On 10/9/2023 13:02, Andi Shyti wrote:
Hi John,
...
if (intf_id >= INTEL_GSC_NUM_INTERFACES) {
- drm_warn_once(>i915->drm, "GSC irq: intf_id %d is out of
range", intf_id);
+ gt_warn_once(gt, "GSC irq: intf_id %d is out of range",
intf_id);
On Fri, Oct 06, 2023 at 04:37:15PM +0300, Imre Deak wrote:
> Use the connector's DSC DPCD capabilities in intel_dp_supports_fec().
>
> Signed-off-by: Imre Deak
Reviewed-by: Stanislav Lisovskiy
> ---
> drivers/gpu/drm/i915/display/intel_dp.c | 7 +--
> 1 file changed, 5 insertions(+), 2
On Fri, Oct 06, 2023 at 04:37:14PM +0300, Imre Deak wrote:
> Use the connector's DSC DPCD capabilities in intel_dp_dsc_compute_max_bpp()
> instead of the version stored in the encoder.
>
> Signed-off-by: Imre Deak
Reviewed-by: Stanislav Lisovskiy
> ---
>
Hi John,
...
> > > if (intf_id >= INTEL_GSC_NUM_INTERFACES) {
> > > - drm_warn_once(>i915->drm, "GSC irq: intf_id %d is out of
> > > range", intf_id);
> > > + gt_warn_once(gt, "GSC irq: intf_id %d is out of range",
> > > intf_id);
> > > return;
> > >
On 10/9/2023 12:54, Andi Shyti wrote:
Hi John,
...
--- a/drivers/gpu/drm/i915/i915_driver.c
+++ b/drivers/gpu/drm/i915/i915_driver.c
@@ -71,6 +71,7 @@
#include "gem/i915_gem_pm.h"
#include "gt/intel_gt.h"
#include "gt/intel_gt_pm.h"
+#include "gt/intel_gt_print.h"
#include
On 10/9/2023 12:50, Andi Shyti wrote:
Hi John,
...
if (intf_id >= INTEL_GSC_NUM_INTERFACES) {
- drm_warn_once(>i915->drm, "GSC irq: intf_id %d is out of
range", intf_id);
+ gt_warn_once(gt, "GSC irq: intf_id %d is out of range",
intf_id);
Hi John,
...
> --- a/drivers/gpu/drm/i915/i915_driver.c
> +++ b/drivers/gpu/drm/i915/i915_driver.c
> @@ -71,6 +71,7 @@
> #include "gem/i915_gem_pm.h"
> #include "gt/intel_gt.h"
> #include "gt/intel_gt_pm.h"
> +#include "gt/intel_gt_print.h"
> #include "gt/intel_rc6.h"
>
> #include
On 10/9/2023 12:43, Andi Shyti wrote:
Hi John,
From: Nirmoy Das
Add a function for ratelimitted debug print.
Cc: Maarten Lankhorst
Cc: Maxime Ripard
Cc: Thomas Zimmermann
Cc: David Airlie
Cc: Daniel Vetter
Reviewed-by: Matthew Auld
Reviewed-by: Andi Shyti
Signed-off-by: Nirmoy Das
Hi John,
...
> if (intf_id >= INTEL_GSC_NUM_INTERFACES) {
> - drm_warn_once(>i915->drm, "GSC irq: intf_id %d is out of
> range", intf_id);
> + gt_warn_once(gt, "GSC irq: intf_id %d is out of range",
> intf_id);
> return;
> }
>
> if
Hi John,
> > > From: Nirmoy Das
> > >
> > > Add a function for ratelimitted debug print.
> > >
> > > Cc: Maarten Lankhorst
> > > Cc: Maxime Ripard
> > > Cc: Thomas Zimmermann
> > > Cc: David Airlie
> > > Cc: Daniel Vetter
> > > Reviewed-by: Matthew Auld
> > > Reviewed-by: Andi Shyti
> >
Hi,
On 10/9/2023 7:10 AM, Borah, Chaitanya Kumar wrote:
Hello Rafael
Thanks for the report, I think that this is a lockdep assertion failing.
If that is correct, it should be straightforward to fix.
I'll take care of this early next week.
Thanks!
Thank you for your response. Please let us
On Mon, Oct 09, 2023 at 06:36:30PM +0300, Jani Nikula wrote:
> On Mon, 09 Oct 2023, Ville Syrjala wrote:
> > From: Ville Syrjälä
> >
> > VLV was missed when the color.get_config() hook was added.
> > Remedy that.
>
> Oops, how did I manage that?! Thanks for the fix,
vlv_color_funcs is a fairly
On 10/9/2023 01:56, Tvrtko Ursulin wrote:
On 06/10/2023 19:20, Jonathan Cavitt wrote:
From: Prathap Kumar Valsan
The GuC firmware had defined the interface for Translation Look-Aside
Buffer (TLB) invalidation. We should use this interface when
invalidating the engine and GuC TLBs.
Add
On 10/9/2023 09:52, Andi Shyti wrote:
Hi,
From: Nirmoy Das
Add a function for ratelimitted debug print.
Cc: Maarten Lankhorst
Cc: Maxime Ripard
Cc: Thomas Zimmermann
Cc: David Airlie
Cc: Daniel Vetter
Reviewed-by: Matthew Auld
Reviewed-by: Andi Shyti
Signed-off-by: Nirmoy Das
From: John Harrison
Update a bunch of GT related print messages in non-GT files to use the
GT specific helpers.
Signed-off-by: John Harrison
---
drivers/gpu/drm/i915/gt/uc/intel_gsc_uc.c | 8 +++-
drivers/gpu/drm/i915/i915_driver.c| 3 ++-
drivers/gpu/drm/i915/i915_perf.c
From: John Harrison
A bunch of print messages got missed in the update to using sub-system
specific helpers. So update those.
Signed-off-by: John Harrison
---
drivers/gpu/drm/i915/gt/intel_engine_cs.c | 29 +
drivers/gpu/drm/i915/gt/intel_gsc.c | 11
From: John Harrison
There was an update a while back to use sub-system specific print
helpers that implicitly add sub-system specific information to the
print. It seems a bunch of GT related messages got missed in that
update. So update them now.
Signed-off-by: John Harrison
John Harrison
Hi Janusz,
On 2023-10-09 at 14:28:01 +0200, Janusz Krzysztofik wrote:
> Recent improvements to the kernel kunit framework allow us to obtain a
> list of test cases provided by a kunit test module without actually
> running them. Use that feature to get a list of expected test cases
> before we
Implement the oob_hotplug_event() callback. Translate it to the HPD
notification sent to the HPD bridge in the chain.
Reviewed-by: Janne Grunau
Signed-off-by: Dmitry Baryshkov
---
drivers/gpu/drm/drm_bridge_connector.c | 31 +++---
1 file changed, 28 insertions(+), 3
From: Bjorn Andersson
In some implementations, such as the Qualcomm platforms, the display
driver has no way to query the current HPD state and as such it's
impossible to distinguish between disconnect and attention events.
Add a parameter to drm_connector_oob_hotplug_event() to pass the HPD
In some cases the bridge drivers would like to receive hotplug events
even in the case new status is equal to the old status. In the DP case
this is used to deliver "attention" messages to the DP host. Stop
filtering the events in the drm_bridge_connector_hpd_cb() and let
drivers decide whether
This is a resend, since the previous submission got no responses. The
patches have been reviewed/acked by several maintainers. Can we please
gain some attention and either get it merged or understand what should
be changed / improved. This series is required to delivere HPD events
from altmode
For the gt_tlb live selftest, when operating on the GSC engine,
increase the timeout from 10 ms to 200 ms because the GSC
engine is a bit slower than the rest.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/i915/gt/selftest_tlb.c | 11 +--
1 file changed, 9 insertions(+), 2
From: Prathap Kumar Valsan
The GuC firmware had defined the interface for Translation Look-Aside
Buffer (TLB) invalidation. We should use this interface when
invalidating the engine and GuC TLBs.
Add additional functionality to intel_gt_invalidate_tlb, invalidating
the GuC TLBs and falling back
It is not an error for GuC TLB invalidations to fail when the GT is
wedged or disabled, so do not process a wait failure as one in
guc_send_invalidate_tlb.
Signed-off-by: Fei Yang
Signed-off-by: Jonathan Cavitt
CC: John Harrison
---
.../gpu/drm/i915/gt/uc/intel_guc_submission.c | 18
In case of GT is suspended, don't allow submission of new TLB invalidation
request and cancel all pending requests. The TLB entries will be
invalidated either during GuC reload or on system resume.
Signed-off-by: Fei Yang
Signed-off-by: Jonathan Cavitt
CC: John Harrison
---
Enable GuC TLB invalidations for MTL. Though more platforms than just
MTL support GuC TLB invalidations, MTL is presently the only platform
that requires it for any purpose, so only enable it there for now to
minimize cross-platform impact.
Signed-off-by: Jonathan Cavitt
---
Implement GuC-based TLB invalidations and use them on MTL.
v2:
- Add missing supporting patches.
v3:
- Split suspend/resume changes and multi-gt support into separate
patches.
- Only perform GuC TLB invalidation functions when supported.
- Move intel_guc_is_enabled check function to usage
Add device info flags for if GuC TLB Invalidation is enabled.
Signed-off-by: Jonathan Cavitt
---
drivers/gpu/drm/i915/i915_drv.h | 3 ++-
drivers/gpu/drm/i915/intel_device_info.h | 1 +
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/i915/i915_drv.h
Add a helper function to the GuC CT buffer that reports the expected
time to process all outstanding requests. As of now, there is no
functionality to check number of requests in the buffer, so the helper
function just reports 2 seconds, or 1ms per request up to the maximum
number of requests the
Hi Janusz,
On 2023-10-09 at 14:28:00 +0200, Janusz Krzysztofik wrote:
> We are going to add support for reading a list of kunit test cases
> provided by a kunit test module prior to executing those test cases. That
- ^^
Two
Hi Janusz,
On 2023-10-09 at 14:27:59 +0200, Janusz Krzysztofik wrote:
> Planned changes require a couple of loops around kunit_result_free().
> Since we already have such loop, move it into a helper in preparation for
> future uses.
>
> Signed-off-by: Janusz Krzysztofik
Reviewed-by: Kamil
Hi Janusz,
On 2023-10-09 at 14:28:02 +0200, Janusz Krzysztofik wrote:
> IGT user interface allows to request execution of only those dynamic sub-
> subtests that match a user provided name pattern. If the user pattern
> doesn't match any names of test cases provided by a kunit test module used
>
Hi Janusz,
On 2023-10-09 at 14:27:58 +0200, Janusz Krzysztofik wrote:
> A workaround was implemented in IGT KTAP parser so it could accepted KTAP
> reports with missing top level KTAP version and test suite plan headers.
> Since kernel side commit c95e7c05c139 ("kunit: Report the count of test
>
Hi,
> From: Nirmoy Das
>
> Add a function for ratelimitted debug print.
>
> Cc: Maarten Lankhorst
> Cc: Maxime Ripard
> Cc: Thomas Zimmermann
> Cc: David Airlie
> Cc: Daniel Vetter
> Reviewed-by: Matthew Auld
> Reviewed-by: Andi Shyti
> Signed-off-by: Nirmoy Das
> Signed-off-by: Andi
On Sun, Oct 08, 2023 at 12:28:46AM +0900, Masahiro Yamada wrote:
> On Fri, Oct 6, 2023 at 9:35 PM Jani Nikula wrote:
> >
> > The kernel top level Makefile, and recently scripts/Makefile.extrawarn,
> > have included -Wall, and the disables -Wno-format-security and
> > $(call
On Sun, Oct 08, 2023 at 02:49:40PM -0700, Randy Dunlap wrote:
> Correct typo of "its".
> Add commas for clarity.
> Capitalize L3.
>
> Signed-off-by: Randy Dunlap
> Cc: Jani Nikula
> Cc: Joonas Lahtinen
> Cc: Rodrigo Vivi
> Cc: Tvrtko Ursulin
> Cc: intel-gfx@lists.freedesktop.org
> Cc:
On Mon, 09 Oct 2023, Ville Syrjala wrote:
> From: Ville Syrjälä
>
> VLV was missed when the color.get_config() hook was added.
> Remedy that.
Oops, how did I manage that?! Thanks for the fix,
Reviewed-by: Jani Nikula
> Not really sure what the final plan here was since a bunch of
> color
-Original Message-
From: Tvrtko Ursulin
Sent: Monday, October 9, 2023 1:57 AM
To: Cavitt, Jonathan ;
intel-gfx@lists.freedesktop.org
Cc: Dutt, Sudeep ; Chang, Yu bruce
; chris.p.wil...@linux.intel.com; Iddamsetty, Aravind
; Yang, Fei ; Shyti, Andi
; Harrison, John C ; Das,
Nirmoy ;
From: Ville Syrjälä
VLV was missed when the color.get_config() hook was added.
Remedy that.
Not really sure what the final plan here was since a bunch of
color related readout was left in intel_display.c anyway,
but that's for anothr day to figure out...
Cc: Jani Nikula
Fixes: 9af09dfcdfa1
On Mon, Oct 09, 2023 at 05:13:03PM +0300, Mika Kahola wrote:
> Currently we are not using watchdog timers for PSR/PSR2
> with Lunarlake. The patch removes the use of these timers
> from the driver code.
That still doesn't seem to say that the timers were removed
from the hardware. Or at least I
On Fri, Oct 06, 2023 at 04:37:09PM +0300, Imre Deak wrote:
> Check only the eDP or the DP specific DPCD revision depending on the
> sink type. Pass the corresponding revision to the function, which allows
> getting the DSC caps of a branch device (in an MST topology, which has
> its own DPCD and
== Series Details ==
Series: drm/i915: Define and use GuC and CTB TLB invalidation routines (rev3)
URL : https://patchwork.freedesktop.org/series/124744/
State : failure
== Summary ==
CI Bug Log - changes from CI_DRM_13728_full -> Patchwork_124744v3_full
Currently we are not using watchdog timers for PSR/PSR2
with Lunarlake. The patch removes the use of these timers
from the driver code.
BSpec: 69895
v2: Reword commit message (Ville)
Drop HPD mask from LNL (Ville)
Revise masking logic (Jouni)
Signed-off-by: Mika Kahola
---
On Fri, Oct 06, 2023 at 04:37:13PM +0300, Imre Deak wrote:
> Use the connector's DSC DPCD capabilities in i915_dsc_fec_support_show()
> instead of the version stored in the encoder. Atm the two are identical,
> but a follow-up patch will store the (MST) connector specific version
> in the
On Fri, Oct 06, 2023 at 04:37:27PM +0300, Imre Deak wrote:
> The previous patches converted all users of the DSC DPCD caps to look
> these up from the connector, so remove the version stored in intel_dp.
>
> A follow-up patchset will read out the MST connector specific
> capabilities in
On Fri, Oct 06, 2023 at 04:37:12PM +0300, Imre Deak wrote:
> Cahce the i915 specific device and connector pointers in
> i915_dsc_fec_support_show().
Typo cahce/cache. Would say more like "store", but doesn't matter.
Reviewed-by: Stanislav Lisovskiy
>
> Signed-off-by: Imre Deak
> ---
>
On Fri, Oct 06, 2023 at 04:37:11PM +0300, Imre Deak wrote:
> Similarly to eDP and SST-DP connectors read out the DSC capabilities for
> MST connectors as well. Atm these will match the root port's DSC caps
> and only used after a follow-up change enables the decompression for
> each stream
For validation purposes, it might be useful to be able to
force Bigjoiner mode, even if current dotclock/resolution
do not require that.
Lets add such to option to debugfs.
v2: - Apparently intel_dp_need_bigjoiner can't be used, when
debugfs entry is created so lets just check manually
Hi Tvrtko,
On 10/9/2023 2:54 PM, Tvrtko Ursulin wrote:
+static long must_wait_woken(struct wait_queue_entry *wq_entry, long
timeout)
+{
+ /*
+ * This is equivalent to wait_woken() with the exception that
+ * we do not wake up early if the kthread task has been completed.
+ *
From: Ville Syrjälä
With all the known issues sorted out we can start to use
DSB to load the LUTs.
Reviewed-by: Uma Shankar
Signed-off-by: Ville Syrjälä
---
drivers/gpu/drm/i915/display/intel_color.c | 3 ---
1 file changed, 3 deletions(-)
diff --git
From: Ville Syrjälä
In order to validate LUT programming more thoroughly let's
do a state check for all color management updates as well.
Not sure we really want this outside CI. It is rather heavy
and color management updates could become rather common
with all the HDR/etc. stuff happening.
From: Ville Syrjälä
The display engine does not snoop the caches so shoukd to mark
the DSB command buffer as I915_CACHE_NONE.
i915_gem_object_create_internal() always gives us I915_CACHE_LLC
on LLC platforms. And to make things 100% correct we should also
clflush at the end, if necessary.
Note
From: Ville Syrjälä
Using system memory for the DSB command buffer doesn't appear to work.
On DG2 it seems like the hardware internally replaces the actual memory
reads with zeroes, and so we end up executing a bunch of NOOPs instead
of whatever commands we put in the buffer. To determine that I
Quoting Rodrigo Vivi (2023-10-05 12:13:34-03:00)
>On Thu, Oct 05, 2023 at 03:05:31AM -0400, Kahola, Mika wrote:
>> > -Original Message-
>> > From: Vivi, Rodrigo
>> > Sent: Wednesday, October 4, 2023 3:56 PM
>> > To: Kahola, Mika
>> > Cc: intel-gfx@lists.freedesktop.org
>> > Subject: Re:
On 09/10/2023 13:12, Nirmoy Das wrote:
On 10/6/2023 8:20 PM, Jonathan Cavitt wrote:
From: Prathap Kumar Valsan
The GuC firmware had defined the interface for Translation Look-Aside
Buffer (TLB) invalidation. We should use this interface when
invalidating the engine and GuC TLBs.
Add
On 09/10/2023 12:40, Andi Shyti wrote:
Hi,
...
@@ -131,11 +132,23 @@ void intel_gt_invalidate_tlb_full(struct intel_gt *gt,
u32 seqno)
return;
with_intel_gt_pm_if_awake(gt, wakeref) {
+ struct intel_guc *guc = >uc.guc;
+
On Mon, 09 Oct 2023, "Kandpal, Suraj" wrote:
>> -Original Message-
>> From: Jani Nikula
>> Sent: Monday, October 9, 2023 4:07 PM
>> To: Kandpal, Suraj ; intel-gfx@lists.freedesktop.org
>> Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Free crtc_state in
>> verify_crtc_state
>>
>> On
IGT user interface allows to request execution of only those dynamic sub-
subtests that match a user provided name pattern. If the user pattern
doesn't match any names of test cases provided by a kunit test module used
with the subtest to be run then no results from any dynamic sub-subtests
will
Recent improvements to the kernel kunit framework allow us to obtain a
list of test cases provided by a kunit test module without actually
running them. Use that feature to get a list of expected test cases
before we enter a loop around igt_dynamic(). Once done, enter the
igt_dynamic() section
We are going to add support for reading a list of kunit test cases
provided by a kunit test module prior to executing those test cases. That
will be done by first loading kunit modules in list only mode, then
reading the list from /dev/kmsg with our KTAP parser. Since that parsing
will be
Planned changes require a couple of loops around kunit_result_free().
Since we already have such loop, move it into a helper in preparation for
future uses.
Signed-off-by: Janusz Krzysztofik
---
lib/igt_kmod.c | 20 ++--
1 file changed, 14 insertions(+), 6 deletions(-)
diff
There is a rule specified in Kunit Test Style and Nomenclature guidelines
[1] that states modules should be named after the test suite, followed by
_test. Of course, that rule applies only to modules that provide one test
suite per module.
As long as that rule is obeyed by authors of Kunit test
A workaround was implemented in IGT KTAP parser so it could accepted KTAP
reports with missing top level KTAP version and test suite plan headers.
Since kernel side commit c95e7c05c139 ("kunit: Report the count of test
suites in a module"), included in the mainline kernel since v6.6-rc1, has
fixed
Kunit test modules usually contain one test suite, named after the module
name with the trailing "_test" or "_kunit" suffix omitted. Since we
follow the same convention when we derive subtest names from module names,
there is a great chance that those two names match. Take this into
account when
There was an attempt to parse KTAP reports in the background while a kunit
test module is loading. However, since dynamic sub-subtests can be
executed only from the main thread, that attempt was not quite successful,
as IGT results from all executed kunit test cases were generated only
after
When igt_kunit() was converted to a helper and wrapped with a new function
promoted to take the name and role of the library API, related
documentation was left unchanged and still placed in front the demoted
function. Update that documentation and move it to where it now belongs.
Signed-off-by:
Function fcntl(..., F_GETFL, ...) that returns file status flags may also
return a negative error code. Handle that error instead of blindly using
the returned value as flags.
Signed-off-by: Janusz Krzysztofik
Reviewed-by: Kamil Konieczny
---
lib/igt_kmod.c | 4 ++--
1 file changed, 2
Use a more verbose variant of igt_fail() when failing a dynamic sub-
subtest on kernel taint. Also, print a debug message on string
duplication failure.
Signed-off-by: Janusz Krzysztofik
Reviewed-by: Kamil Konieczny
---
lib/igt_kmod.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
v2: Add a new patch that provides all results cleanup helper,
- split out changes in handling of modprobe errors and kernel taints from
"Fetch a list of test cases in advance" to separate patches (Kamil),
- prepare for KTAP parsing after modprobe completed in a separate
patch,
- drop
== Series Details ==
Series: Refactor i915 HDCP for XE (rev6)
URL : https://patchwork.freedesktop.org/series/123955/
State : success
== Summary ==
CI Bug Log - changes from CI_DRM_13728 -> Patchwork_123955v6
Summary
---
**SUCCESS**
On Mon, Oct 09, 2023 at 10:58:22AM +, Kandpal, Suraj wrote:
>
>
> > -Original Message-
> > From: Jani Nikula
> > Sent: Monday, October 9, 2023 4:07 PM
> > To: Kandpal, Suraj ;
> > intel-gfx@lists.freedesktop.org
> > Subject: Re: [Intel-gfx] [PATCH] drm/i915/display: Free crtc_state
Hi Kees,
On Fri, Oct 06, 2023 at 01:17:45PM -0700, Kees Cook wrote:
> Prepare for the coming implementation by GCC and Clang of the __counted_by
> attribute. Flexible array members annotated with __counted_by can have
> their accesses bounds-checked at run-time via CONFIG_UBSAN_BOUNDS (for
>
== Series Details ==
Series: Refactor i915 HDCP for XE (rev6)
URL : https://patchwork.freedesktop.org/series/123955/
State : warning
== Summary ==
Error: dim sparse failed
Sparse version: v0.6.2
Fast mode used, each commit won't be checked separately.
+./arch/x86/include/asm/bitops.h:117:1:
1 - 100 of 126 matches
Mail list logo